Bean class is not public

I'm happy to find this project because it will help for my use case and I'm thankful for that project.
I launch the base code at it prints correctly all the dependencies of my application :
import com.headstartech.beansgraph.BeansGraphProducer;
import com.headstartech.beansgraph.ConsoleReporter;
import com.headstartech.beansgraph.EnableBeansGraph;
import org.springframework.stereotype.Component;

@EnableBeansGraph
@Component
public class ConfigureBeanGraphReporters implements BeansGraphConfigurer {

    @Override
    public void configureReporters(BeansGraphProducer producer) {
        ConsoleReporter.forSource(producer).build();
    }
}

However, i want to have my own Reporter, therefore I create a class that implements BeansGraphListener and I want to implement my logic in onBeanGraphResult:
public class BeanGraphChecker implements BeansGraphListener {
    @Override
    public void onBeanGraphResult(ApplicationContext applicationContext, BeansGraphResult result) {
        checkBeanDependencies(applicationContext, result);
    }

    private void checkBeanDependencies(ApplicationContext applicationContext, BeansGraphResult result) {
        UnmodifiableDirectedGraph<Bean, DefaultEdge> graph =  result.getDependencyGraph();
        // Implement my logic on the graph here
    }
}

However, that code does not compile because the Bean class is package protected and not public.
I tried to hack by moving the class to the com.headstartech.beansgraph; package, but I get a runtime error this time.
How am I suppose to use/extend the library when I cannot accessing the Bean class ?
I could fork the project to change the source code but currently I only need that little change.
The Bean class was public and was changed to package since commit d72147, what was the reason to change the visibility at the time ?
Thanks for reading my problem :)

The Bean class was public and was changed to package since commit d72147, what was the reason to change the visibility at the time ?

Was probably overthinking the design and messed up (probably was the setClassName method and mutability that caused that brainfart)


undefined index error after upgrading to nova 2.5.0

i am not sure why but i think there is a compatibility  issue with nova 2.5.0, everything is fine until nova 2.4.0 version.

Exactly the same in my environment

It's definitely in the right place.
To fix this temporarily in your project, you can just add this to each NovaDependencyContainer:
->required(function () {
  return true;
})

@nemrutco @Tarpsvo @snk-lab Yea they added requiredCallback to Fields.php. A simple return false in the NovaDependencyContainer class constructor would fix it.
public function __construct($fields, $attribute = null, $resolveCallback = null)
{
    parent::__construct('', $attribute, $resolveCallback);

    $this->withMeta(['fields' => $fields]);
    $this->withMeta(['dependencies' => []]);

    $this->requiredCallback = function() { return false; };
}

But I'm not sure if this is just a workaround or an actual fix. They seem to be mixing a lot now in Fields.php.

I don't really like the solution of adding a useless requiredCallback (even though it is a fix) and I'm hoping they merge my PR with the !empty($this->attribute) check.
Giving the NovaDependencyContainer an actual attribute should also be a fix, though I haven't tested it and the attribute itself would also be equally useless.

I don't really like the solution of adding a useless requiredCallback (even though it is a fix) and I'm hoping they merge my PR with the !empty($this->attribute) check.
Giving the NovaDependencyContainer an actual attribute should also be a fix, though I haven't tested it and the attribute itself would also be equally useless.

Follow updates at laravel/nova#684.

your link gets 404, that's why i asked are you sure you posted in right place.

@nemrutco Yea laravel/nova isn't public as far as I know.
@Tarpsvo A PR doesn't help much if nova expects a key to be present in an array, where some PR for this package obviously didn't wrote anything compatible to novas behaviour prior to 2.5.0.
The default behaviour should apparently always be.
return [$this->attribute => [..]];

Although it would nice to check the index first, I think they made this choice to escalate deliberately.
Should be fixed in 1.2.7, please verify and if so close this issue.

The problem could be avoided by including the following.
->required(function () {   return false; })

@wize-wiz

@snk-lab composer update epartment/nova-dependency-container should accomplish the same thing without requiring anything in your code.

Thank you very much. Updated to remove unnecessary modifications.

@snk-lab composer update epartment/nova-dependency-container should accomplish the same thing without requiring anything in your code.
Problem solved. Thanks

Please close this issue if resolved.


fix(doc): The port change from 8080 to 8888

Right now what the readme says doesn't work, you need to make this change to make it work.

Thanks for the contribution, @SistachReinoso!


Failed to build DAIN

When I'm trying to build DAIN with PaddleGAN/applications/DAIN/pwcnet/correlation_op/make.sh, I get this:
/opt/conda/envs/python35-paddle120-env/lib/python3.7/site-packages/paddle/include
/opt/conda/envs/python35-paddle120-env/lib/python3.7/site-packages/paddle/libs
cc: error trying to exec 'cc1plus': execvp: No such file or directory
g++: error: correlation_op.cu.o: No such file or directory
I run this sh right after I cloned PaddleGAN, is there anything else I need to do before that?

correlation api has been added to Paddle, you can use https://paddlegan.bj.bcebos.com/whls/paddlepaddle_gpu-0.0.0-cp37-cp37m-linux_x86_64.whl this whl for python3.7.
paddle.fluid.contrib.correlation


Book errata

Book really helpful, thank you. I found some small issues:

Page 18: need semicolon on create user line
Likewise on the grant line
Also need GRANT USAGE, CREATE ON SCHEMA public TO your_user; (see https://stackoverflow.com/questions/74110708/postgres-15-permission-denied-for-schema-public#answer-74111630)
Page 28: don't copy the file in migration/versions
Page 28: directory diagram doesn't include env.py
Page 30: "It is tempting to put our functionality in models/serializers since that’s easiest, but writing the functionality in serializers is typically better" - should that be "writing the functionality in services is typically better"?
Page 31/32: the call to account_management_services.create_account doesn't pass the password which that function requires.
Page 35 - footnote URL 404s (https://ultimatedjango.com/learn-django/lessons/create-stripe-processing-code)
Page 38 - you use the database name flask_for_startups burt earlier it was your-database-name. Might be best to use flask_for_startups from the beginning.
Page 52, account_management_services.py  listing is missing imports for AccountValidator and custom_errors
Code in Kindle really hard to read
Page 62 account_management_views.py: The imports are wrong - some needed are missing (eg, marshmallow) and some included that are not needed (eg, db, User, Account)
Page 73: I had created more than one user, so the user query failed. Maybe .first() instead of .one()?
Page 83, list item 4: missing "db" in "we have a clean by deleting"
Page 86: not important, but you may want to make the test mail domain be example.com which is designed for that use.
Page 87: semicolon on psql line, and also schema permissions as above
Page 96: contact email incorrect<EMAIL_ADDRESS>

Thank you very much Keith! I really appreciate the feedback and the kind words.
I agree with each point, and have updated the book. If you go to the original email from Gumroad, you should be able to see the updated book.
I currently don't have a solution for the kindle issue, but I will continue investigating.
Thank you once again!


Add widgets for reparenting widgets

Wrap widgets you want to reparent in a Mimicable widget and assign the
Mimicable widget a global key. Then, given the same global key to a Mimic
widget to make it appear elsewhere in the view hierarchy.

LGTM


Show table name in different defs warning

Different defs CHANGE messages are not particularly helpful identifying the concerned table:

-- schemaType: INDEX
-- CHANGE: Different defs:
--    PRIMARY KEY (domain, user_eid)
--    PRIMARY KEY (user_eid, domain)

This change add the table name to the output to help resolving the issue:

-- schemaType: INDEX
-- CHANGE: Different defs on users_domains:
--    PRIMARY KEY (domain, user_eid)
--    PRIMARY KEY (user_eid, domain)

Really great tool, by the way. It's saving me so many hours! If you happen to be in Berlin, drop me a message I'll buy a beer ;)

Thank you!  I love to hear that it saves people time.


The upstream project does not compile with ffmpeg >= 5

I opened a separate issue as otherwise the information may be lost as migration PRs are constantely be closed and re-opened.
Apparently there is not a lot interest for this upstream, see https://github.com/openscenegraph/OpenSceneGraph/issues/1111. If there is no interest in fixing this, probably the easiest way forward is just to disable the ffmpeg plugin, as for example Ubuntu since 22.10 handled this: https://launchpad.net/ubuntu/+source/openscenegraph/3.6.5+dfsg1-7ubuntu1 .
Actually it seems that the only user of openscenegraph in conda-forge is actually using ffmpeg https://github.com/search?q=repo%3AGepetto%2Fgepetto-viewer ffmpeg&type=code . @jcarpent @ymontmarin  it is possible somehow to use gepetto-viewer without ffmpeg? Otherwise I am afraid that to go forward with the ffmpeg 5.0 migration, the only way is to leave behind this package. Note that there is a similar situation even using apt for Ubuntu >= 22.10, as also there the only ffmpeg available is 5.

Hi @traversaro , I will dig into this. During this time, this package can be left behind from the global ffmpeg migration.

Just saw that openscenegraph is legacy and has been replaced by VulkanSceneGraph... So it doesn't sound like this will be tackled soon or at all, which puts us in quite a bit of pain, until everyone has moved on to VSG...

@h-vetinari This is a subject I raised with the core team of the HPP and gepetto stack and this is still in discussion, but it will not move quickly...
I think the HPP stack is doomed to use old assimp and ffmpeg while still on OSG

I think the HPP stack is doomed to use old assimp and ffmpeg while still on OSG

There's only 2-3 packages missing in the assimp/boost migrations now, but yeah, getting the whole stack sounds like it won't happen. Is there a way to use gepetto without ffmpeg, or is that just completely broken?

Sadly, we need ffmpeg for gepetto

I think the HPP stack is doomed to use old assimp and ffmpeg while still on OSG

There's only 2-3 packages missing in the assimp/boost migrations now (stuck on conda-forge/gepetto-viewer-feedstock#38), but yeah, getting the whole stack sounds like it won't happen. Is there a way to use gepetto without ffmpeg, or is that just completely broken?

Adding a variant that has no ffmpeg dependency

Sadly, we need ffmpeg for gepetto

Are you sure? I was looking a bit on update on this issue triggered by @jslee02 comment, and it seems that in Ubuntu 24.04 apt packages the ffmpeg plugin of osg was disabled (see https://git.launchpad.net/ubuntu/+source/openscenegraph/commit/?h=ubuntu/noble&id=a453c15270e96863f6c174c2ad1f6bf08e917111), and in indeed I can't find it in the apt osg package in Ubuntu 24.04 :
traversaro@IITBMP014LW012:~$ dpkg -L libopenscenegraph161 | grep ffmpeg
traversaro@IITBMP014LW012:~$

Furthermore, looking at gepetto-viewer, I may be wrong but it seems that it is using ffmpeg, but just invoking directly the ffmpeg command (see https://github.com/search?q=repo%3AGepetto%2Fgepetto-viewer ffmpeg&type=code), not via the osg ffmpeg plugin based on AudioStream or ImageStream (see https://github.com/search?q=repo%3AGepetto%2Fgepetto-viewer+ImageStream&type=code).
@ymontmarin @jcarpent  As far as you know is anyone using gepetto-viewer on Ubuntu 24.04 with apt dependencies? If that works fine, probably it is an indication that the ffmpeg plugin of openscenegraph is not used, and so we can drop its compilation here as well? Just asking so we can streamline the maintenance of this package, if we can just have a single package instead of incompatible -noffmpeg and a -ffmpeg4 variants it would be much easier to manage the feedstock.

I also opened a similar issue on the gepetto-viewer issue tracker: https://github.com/Gepetto/gepetto-viewer/issues/226, as it may be of interest also for packagers of different ecosystem (nix, apt, aur).

@traversaro I take the point !

I study in this PR: https://github.com/conda-forge/gepetto-viewer-feedstock/pull/48 how we can get rid of ffmpeg

I study in this PR: conda-forge/gepetto-viewer-feedstock#48 how we can get rid of ffmpeg

Perfect! I will open a PR instead here to drop the ffmpeg dependency.

Sadly, we need ffmpeg for gepetto

@traversaro I am very sorry for this confusion, I was convinced the dependency was deeper and do not know from where I had this belief.

No problem at all, it is easy to get lost with huge dependency trees, thanks for all the efforts in the maintenance of gepetto-related conda packages!

Should we close this issue now then ?

It will be closed automatically once https://github.com/conda-forge/openscenegraph-feedstock/pull/42 is merged.


python310Packages.adafruit-platformdetect: 3.40.2 -> 3.40.3

Changelog: https://github.com/adafruit/Adafruit_Python_PlatformDetect/releases/tag/3.40.3
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.05 Release Notes (or backporting 22.11 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 217561 run on x86_64-linux 1

2 packages built:

python310Packages.adafruit-platformdetect
python311Packages.adafruit-platformdetect


Refactor and optimize IsInputAssociatedWithPubkey

Remove the extra GetTransaction call by passing height back from IsInputAssociatedWithPubkey function and using it in CheckOutpoint. Also remove GetTransaction call in IsInputAssociatedWithPubkey in favor of GetUTXOCoins which is more efficient and does the same thing. The only difference is that GetUTXOCoins will fail if the collateral has moved whereas GetTransaction will not. I hope that is a safe assumption as if collateral is moved I would like to think that this function should fail anyway (seems to be only called when a masternode broadcast has occured)

Just another step to remove as many GetTransaction calls as possible (I know we still cannot remove the txindex dependency entirely but we are getting there :+1: )

I like the idea, I would however go even further https://github.com/dashpay/dash/pull/1783

Agreed. Good thinking.


Web view background images layout issue

We've stumbled upon an interesting rendering issue with the library. Adding a background cover photo to a div causes the contents to have a zero height.
Ideally, we want a background cover image to fill the entire screen with some buttons/text along the bottom. Think of a signup/register page with a pretty background image. This was working fine for us before upgrading to 1.0.3. We didn't change any CSS or HTML markup during the upgrade.
See the following two screenshots for a demonstration of the issue. The first is the page viewed from the Android's Chrome browser, the second is the same page viewed from the Turbolinks demo app.
Background Image in Chrome

Background Image in Turbolinks Demo App

To replicate, the changes are simple. Replace demoapp/server/lib/turbolinks_demo/views/index.erb with the following:

Hello from Turbolinks! This demo app will help you get acquainted with the framework.

Open a new activity by following a regular link.

See the loading indicator and test Turbolinks’ preview cache with this slow-loading page.

Trigger an HTTP 404 to see how Turbolinks handles error responses.

Is this an issue with the framework or is Android doing something funky with our CSS? Of note, commenting out all of the pull-to-refresh code fixes the display issue.

Confirmed. I snapshot back to 1.0.2, and see the same results. 1.0.3 doesn't show the background image, 1.0.2 is OK.
This certainly could be related to nesting the WebView inside a SwipeRefreshLayout. It's part of the support lib which was new for 1.0.3, and though I don't have details yet, it wouldn't surprise me if a support lib component affected rendering of webview.
@joemasilotti Were you able to test commenting out specifically just the SwipeRefreshLayout stuff, or did you snapshot back all the way to 1.0.2? Just curious because 1.0.3 also had a number of other changes. Were you able to find any workaround for now?

Whoa, sorry for the late reply - didn't see this until now.
Commenting out SwipeRefreshLayout, and only that, fixed the issue entirely.

We have the same problem. Seems that it has something to do with the document.height. See the .header-home container on https://www.ticketswap.com. It has a min-height of 100%, but looks like this:

@joemasilotti Did you find a solution?
1.0.2 works, 1.0.3/4 contain this bug.

@frankrrr The only reliable solution we found is to remove pull-to-refresh. This forces us to fork and maintain our own copy of the repo, which is less than idea.

Hi @lateplate!
Thanks for this great project. Any news about merging the @kassim 's PR?

Earlier this week I confirmed that #60 fixes the issue for us. 👍

@risinglf @joemasilotti @kassim Thanks. Dropped some minor feedback in the PR. Once I can do a quick smoke test and those changes are made, we should be good to merge.

Merged the fix.


Add note

when you're not connected, add the possibility to put a note (osm' note)

A note on what?

In osm, you can add a note (see picture). You don't need to have an account, only question or knowledge :)

Ok, not so complicated but it touches several things in MapContrib. I put the hard label to indicates it's not trivial in that way.

The UI aspect of that issue is related to the issue #136.


Fix vcxproj path to node_modules

Summary:
Currently, manually linking the react-native-slider to a react-native-windows project requires additional modifications to the vcxproj file located in the node_modules while other projects such as Picker do not.
Test Plan:
Perform the manual linking steps on Windows described in the README. Should work on a brand new react-native project created through these steps.

@rectified95 @jonthysell @acoates-ms can you take a look? I forget where we landed with the node_modules path in the project


⚠ Samsung has degraded performance

In b23195f, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2246 ms

Resolved: Samsung performance has improved in fb53fa6.


SPIFFS: failed to read addr 00006a00, size 00000100, err 257 (IDFGH-2483)

Environment

Module or chip used: ESP32-WROOM-32
IDF version :  v4.0-dev-1443-g39f090a4f 2nd stage bootloader
Build System: Make
Compiler version : xtensa-esp32-elf-
Operating System: Ubuntu 18.04
Power Supply: USB

Problem Description:
Frequent errors occur when writing data to SPIFFS or reading data from SPIFFS. Error messages are as follows:

ErrorError 2:

Error 3:

I would like to ask, how to deal with this type of problem, and how to ensure that using SPIFFS to read and write data is almost 100% successful. If you do not use SPIFFS, but directly operate on Flash, will the stability be improved?

Is there progress on this issue?

@GitOfLance Thanks for reporting and sorry for the late reply. We will look into. Thanks.

Hi @GitOfLance, this is very likely the same issue as https://github.com/espressif/esp-idf/issues/4769. The fix for it should be merged shortly. In the meantime you can apply the change mentioned in https://github.com/espressif/esp-idf/issues/4769#issuecomment-586583027 locally.

Hi, @Alvin1Zhang @igrr
It looks like this is a Flash read / write error caused by this bug. Thank you very much for your answers。

+1
W (198897) wifi:alloc eb len=24 type=3 fail, heap:0

W (198897) wifi:m f null

Fix mentioned in https://github.com/espressif/esp-idf/issues/4769 is available on GitHub, feel free to reopen.

Fix mentioned in https://github.com/espressif/esp-idf/issues/4769 is available on GitHub, feel free to reopen.


[FEATURE] Implement Dynamic Contest Date Display with Time Zone Conversion Link

Goals:
Implement a feature that dynamically displays contest dates adjusted for the user's time zone and provides a clickable link for detailed time zone conversion using the provided startTimeUnix for each Card.
Implementation Steps:

Time Zone Calculation:

Retrieve the user's time zone using browser capabilities or user settings.
Use the startTimeUnix value for each Card to calculate the contest date and time in the user's time zone.

Display Adjusted Date:

Present the converted contest date and time on the Card component.
Example: A contest starting at startTimeUnix will display as "August 24 at 7:00 PM" in the user's local time.

Clickable Time for Conversion:

Make the displayed contest time clickable.
Clicking the time will directly link users to timeanddate.com's conversion page for the specific date and time.

Example Usage:
For a contest starting at startTimeUnix, the Card would dynamically display: "August 25 at 7:00 PM" in the user's local time zone. Users can click the time to be taken directly to timeanddate.com for detailed time zone conversion.
By enhancing the contest date display and enabling direct time zone conversion, this feature will provide users with a seamless experience and accurate time information.
Feel free to contribute your insights and ideas to help implement this user-friendly feature.

I installed this repository on my local machine. But, I can't figure out why the contest cards aren't showing while I am in
localhost: ....

@pranshugupta54 i am familiar with  REACT JS is there anything for me to contribute to your project?

Hey @hafeeluddeen, our frontend is on React and you can work on any listed issue or even try something new.
Join us on Discord for any help.

Hello, @pranshugupta54 If this issue is not assigned to anyone then I would like to contribute on this, please assign this to me

Hi @snehalkadwe! Thanks for wanting to contribute. To begin, try setting it up on your computer. If you run into any issues or have questions, feel free to join us on Discord. We're here to assist!

Hey @pranshugupta54 ! Can we discuss on this?

@Chakit22 sure!

Created a PR for this @pranshugupta54 !

Updated the PR @pranshugupta5. Works fine now.

Can we close this @pranshugupta54 . It is implemented.


Thread sanitizer reporting non-existing test files

Description
The thread sanitizer (Mac12.x-AppleClang-dbg-TSan) build in RogueResearch19 has been consistently reporting 5 tests not being run due to required files being missing:
https://open.cdash.org/viewTest.php?onlynotrun&buildid=7534737
Steps to Reproduce

Set up a machine using Clang's thread sanitizer (https://clang.llvm.org/docs/ThreadSanitizer.html)
Clone ITK
Configure and build with the BULD_TESTING flag to ON.

Expected behavior
The expected output files should exist/the dependent tests should not report that the required files are missing.
Actual behavior
The following 5 tests report that the corresponding required input files are missing:
itkAdditiveGaussianNoiseImageFilterTestPSNR
itkJPEG2000Test03
itkSaltAndPepperNoiseImageFilterTestPSNR
itkShotNoiseImageFilterTestPSNR
itkSpeckleNoiseImageFilterTestPSNR

Reproducibility
100%.
Versions
ITK master.
Environment
macOS
Apple Clang
TSan
But could probably be reproduced on any platform.
Additional Information
All of the blamed tests depend on a previous test being run before and writing an output that is used/required by them, e.g.
https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/Filtering/ImageNoise/test/CMakeLists.txt#L25
itk_add_test(NAME itkAdditiveGaussianNoiseImageFilterTest
  COMMAND ITKImageNoiseTestDriver
  itkAdditiveGaussianNoiseImageFilterTest
   DATA{${ITK_DATA_ROOT}/Input/cthead1.png}
   ${ITK_TEST_OUTPUT_DIR}/itkAdditiveGaussianNoiseImageFilterTest.png
   29.85 0)
itk_add_test(NAME itkAdditiveGaussianNoiseImageFilterTestPSNR
  COMMAND ITKImageNoiseTestDriver
  itkPeakSignalToNoiseRatioCalculatorTest
   DATA{${ITK_DATA_ROOT}/Input/cthead1.png}
   ${ITK_TEST_OUTPUT_DIR}/itkAdditiveGaussianNoiseImageFilterTest.png
   20 1)
set_tests_properties(itkAdditiveGaussianNoiseImageFilterTestPSNR
  PROPERTIES
   DEPENDS itkAdditiveGaussianNoiseImageFilterTest
   REQUIRED_FILES ${ITK_TEST_OUTPUT_DIR}/itkAdditiveGaussianNoiseImageFilterTest.png)

So the test itkAdditiveGaussianNoiseImageFilterTestPSNR uses itkPeakSignalToNoiseRatioCalculatorTest.cxx, which requires an input image and a noisy version of it to compute the PSNR. The noisy version is generated by another test (e.g. itkAdditiveGaussianNoiseImageFilterTest through itkAdditiveGaussianNoiseImageFilterTest.cxx), and thus the CMake set_tests_properties command is used to signal that itkAdditiveGaussianNoiseImageFilterTestPSNR depends on itkAdditiveGaussianNoiseImageFilterTest, requiring its output file ${ITK_TEST_OUTPUT_DIR}/itkAdditiveGaussianNoiseImageFilterTest.png.
In the ImageNoise module all of the tests correspond to those run through itkPeakSignalToNoiseRatioCalculatorTest.cxx.

I think tomorrow's cdash should have a working TSan submission from Rogue19.

OK, the build I ran by hand worked, now "only" 283 tests failing:
https://open.cdash.org/viewTest.php?onlyfailed&buildid=7550267
This patch should fix some:
https://github.com/InsightSoftwareConsortium/ITK/pull/2853

@seanm I had a look at the last TSan builds and it looks like the issue I reported has been fixed since a few days/you made the proper setup of the build.
I'd say the issue is fixed. Do you agree if I close?

Agreed.


Allow abstract classes in protocol

Currently all classes specified in a protocol need to be concrete, due to a check in com.cinnober.msgcodec.ProtocolDictionaryBuilder.GroupMeta.GroupMeta(Class<?>)
It would be nice if msgcodec would allow an abstract class in the protocol if it is also annotated with @Dynamic. Or maybe it would automatically become @Dynamic if it is abstract.

Message classes may now be abstract, regardless if dynamic or not. The Dynamic annotation still need to be explicit.
Decoding abstract message classes will result in a DecodeException to be thrown.
Note that due to a schema upgrade a sub class may be flattened to its super group class, and if that is abstract a DecodeException is thrown.
For example the message classes Request and Response in ProtocolUpgradeExample.java could have been abstract, but the client would fail decoding GetUserRsp as Response in that case.


How to use ComfyUI in Automatic 1111 (in Colab)

Hello!
I just have installed ComfyUI in Automatic 1111 (in Google Colab environment). And I saw the "ComfyUI tab" and "ComfyUI option" in "txt2img" Tab.
However, I cannot see any ComfyUI workflow in any "ComfyUI Tab" and "ComfyUI option" after generating image.
Please help (and if there is any tutorial reference).
Thanks!

Hello, thank you for trying out the extension.
In order to run on collab, you need to go into the settings and enable the reverse proxy.

Please advise on the attached png if it is correct because I have set the reverse proxy (as shown), but ComfyUI still do not show workflow after generating image.

Please advise on the attached png if it is correct because I have set the reverse proxy (as shown), but ComfyUI still do not show workflow after generating image.

Thanks for your advise. Surely, I have ComfyUI interface in txt2img / img2img tabs and ComfyUI tab. I am loading Automatic 1111 in google colab with MacOS. Please see below attached PNG files for the ComfyUI tabs mentioned above (the status after I generated a image in img2img). Thanks.

I see some misbihaving UI in the tab, there should be either nothing or ComfyUI. If I remember correctly, I think there was a compatibility issues with the mov2mov extension because it was duplicating elemids.
Can you try disabling it?
If it still doesn't work, are there any errors in the console?

Aha, found it: #176

Closing this as not planned because it seems to be related to the mov2mov extension issue. If you have another issue unrelated to the extension conflict with mov2mov, you can reopen this issue or open another one.


Adds custom psql toxic for injecting failure into postgres connections.

See the Toxiproxy docs for details on custom toxics.
Injects failure into a postgres connection, the failure can be triggered after a predefined number of SQL statements have been sent to the server.
Attributes:

failure_type - set to ConnectionFailure or SyntaxError. ConnectionFailure will terminate the connection, possibly
causing retries depending on client implementation**. SyntaxError will inject a bad SQL statement into the connection
causing a soft failure.
search_text - regular expression used to match SQL statements
fail_on - number of statements matching search_text to trigger failure on
recover_after - number of statements matching search_text to trigger recovery after

** The golang psql client implementation will automatically retry on this type of failu

@elliotwms-f3 this has everything from the standard Toxiproxy  + the custom one as well, so only need to run one thing


[shaders] rlights.h -- RLIGHTS_IMPLEMENTATION not working as intended

You cannot include rlights.h in multiple source files because the full declaration of lightsCount is in the header section, causing a symbol collision on link.

@rdominick Oh! Good catch! Solved! :)


pow wrong???

Building with Clang 9.0.1 on Manjaro Linux, I'm getting the result that
sprout::pow(8, 4) is equal to 4095 instead of 4096. Am I dumb, or is there a problem here?
Ok, just tried it with gcc 9.2.0 and it gives the right answer. (4096). Spooky.

@a-day-old-bagel I looked at the code here and noticed that sprout::pow returns a floating point value when integral typed arguments are passed, so maybe in your case it returns 4095.999999999... and then you or your compiler casts that to an integral type, so you get 4095 ?

Well switching the compiler from gcc to clang was the literal only thing I had to change to reproduce the problem, so I don't think it was any cast of mine. I decided not to use this library, since no matter who's fault it is, if I'm getting different results between compilers (all else remaining constant) from a math library, that's not a good place to be.
4096 has a perfect representation in floating point, too, so if for some reason the function is returning 4095.9999999..., that's something that it should not be doing, right?

@OlivierSohn Well switching the compiler from gcc to clang was the literal only thing I had to change to reproduce the problem, so I don't think it was any cast of mine. I decided not to use this library, since no matter who's fault it is, if I'm getting different results between compilers (all else remaining constant) from a math library, that's not a good place to be.
4096 has a perfect representation in floating point, too, so if for some reason the function is returning 4095.9999999..., that's something that it should not be doing, right?

clang 9.0.1 on Arch Linux, installed from the LLVM github releases fails to reproduce the reported issue. If it is still a problem for you and you wish to investigate further I would like to help.

@thinkoid Let me see if I can dig up that old system I had back then and reproduce the problem. If you don't hear back from me after a while we can probably just close this issue.


Allow expanding rows in occlusion mode

pretty straightforward, I can't find a reason that this won't work. But ultimately if someone is experiencing problems they can just turn expanded rows off or occlusion off
see https://github.com/offirgolan/ember-light-table/issues/542

This is the only thing stopping me from using occlusion

I was able to fix this issue using custom row component extending Row when occlusion is enabled in the table.
Added the code to render columns (as in It-row.hbs) inside below code.


feat(automation): new world doc pr creation

DEX-1021
This PR create a command to generate a new-world-docs PR that'll update the .mdx files for the CLI documentation.
How to test?
go run ./cmd/docs --app_data-path docs/app_data/cli/commands --target new

It will output the files in at docs/app_data/cli/commands
To do(?)

[x] Transform format Example string to separates descriptions and snippets
[ ] Handle runCommand attribute in the .mdx file

@dylantientcheu @clemfromspace
I'm not sure about how we should handle the runInCommand property for each example 🤔
Do you have any idea? Should we hardcode a list of commands that allow this feature in the CLI codebase?

For now we can edit the submitted PR each time, i'd prefer not to bloat the cli code.

I think we should run the example directly, so we will have to edit the example to make them match what indices / things we have in the doc. That way, no editing is necessary and it's more coherent maybe. WDYT?

@clemfromspace I agree, let's remove all Examples formatting for now and do it in new-world-doc 👍
Same for the WebCLI I think? You're suggesting doing outside of this repo if I understand correctly?

@loicsay I saw that you actually removed the example splitting, this was not what I wanted to say 😓
Sorry about the confusion, I wanted to say that we should directly run the examples in the webCLI (for the available commands), but that we should edit those examples, directly in the CLI, to match the indices names & co that are present in the docs. This should be done maybe in a second step, along with adding a availableInWebCLI annotation on the commands.

Summary of our discussion with @clemfromspace:

We'll add annotation (runInWebCli) to commands with web CLI enabled in new-world-docs
Because there's differences between CLI examples and the ones run with the web CLI, we'll standardize all examples with the new-world-docs/webCLI (in a future PR)
Examples with output or pipe symbols will automatically be non-runnable


Serialize local vars early on

No need to have actual references floating around. This also helps us avoid tampering with the underlying objects by mistake later on.
The "proper" way to do this would be to somehow yank var serialization out of the serializer to be able to use it standalone, but this would be a huge undertaking and possible source of bugs due to how complicated the serializer is. Which is doubly problematic given how integral it is. So what I opted for instead is a minimally invasive approach.
WIP: still getting this right and will probably add another test.

Closing, I think I found a better way.


Fixed an inheritance issue where enxrypted fields would fail parsing …

…db data to the correct type

Where can i see the test results to know i did everything correctly?
having issues with running travis myself..

Do you have some more details on what failed or what your stacktrace has been? Or is this change only required to let EncryptedJSONField work?


make_dvcyaml: Include metrics section also if live.summary.

Found debugging https://github.com/iterative/example-repos-dev/pull/191#issuecomment-1492252216

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (8730da0) 95.70% compared to head (60a2b22) 95.70%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #512   +/-   ##
=======================================
  Coverage   95.70%   95.70%
=======================================
  Files          41       41
  Lines        2582     2587    +5
  Branches      220      220
=======================================
+ Hits         2471     2476    +5
  Misses         66       66
  Partials       45       45

Impacted Files
Coverage Δ

src/dvclive/dvc.py
93.10% <100.00%> (ø)

tests/test_dvc.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Unexpected behavior in sf run apex test

Hello, I use a custom GitHub action to execute apex tests in my CI/CD pipeline that wraps the sf/cli and I've just made an update that uncovered some unexpected behavior for me and I wanted to ask if it's intended or not. And, potentially improve my understanding or put eyes on it.
In most asynchronous commands on the sf-cli I'm expecting to enqueue the operation through the command and then be able to reach out to check on the status using the job-id. For example, with a package install request the install is queued and then I am provided an id to let me check the status of the job. It will tell if it's in progress, completed, failed, etc. Providing a wait param will instead poll and wait for me until the job is complete or the wait times out.
I've had to update my action because we found that the --wait parameter on this command does not throw a failure when the waiting period times out. It provides an exit 0 and no indication that it timed out so it provides a false positive when using only the wait parameter.
Okay, to account for that we can just run without a wait and call out to get the current status of the test run. We've done that previously when creating a 2GP package version through a local polling system until the status is complete. However, I found that the sf apex get test command will sit and wait until the tests results are back to return a result. I've only been able to get it to return a complete pass or fail, never an in progress status. I've tried to catch an in-progress status with several different size packages and it's waited until the test completes every time. Curiously, this approach also looks to be significantly faster than running sf apex run test with a wait included.
Running the same set of tests using a test-suite, we received test results in 5-8 minutes using sf apex run test with no wait and immediately requesting sf apex get test. Compared to 18-35 minutes when running sf apex run tests with a sufficient wait time.
There's no blocker present here but the behavior is surprising and appears to contrast other behavior within the CLI in my experience.
Is this intentional?
Is this performance difference real or am I missing something here?

I've personally observed the described behavior from @salesforce/cli/2.48.6 darwin-arm64 node-v20.15.0 up to latest.

Hey @Cerelius 👋 Lot to unpack here. Going to do my best to summarize it.

You are saying that the sf package install will exit with a 0 when the timeout is reached? If so, that is a bug and we will fix it.
We are aware that some of the sf apex commands operate slightly different than our other commands (sync vs async)

We have plans to revamp this plugin soon to bring it more inline with our best practices

I am not seeing any significant differences in execution times with and without the --wait flag with Suites. Here's what I tried:

I started with https://github.com/iowillhoit/dreamhouse-lots-o-tests
I created 150 (1..150) slow tests with source scripts/generate-slow-tests.sh
Created a scratch org
Deployed the generated tests
I went into Developer Console to create a new Suite
Ran sf apex run test --suite-names MySuite --wait 30 several times and noted the execution time
Then ran sf apex run test --suite-names MySuite followed by sf apex get test -i THD_ID
Compared the times (no really difference)
I also compared times without using a test suite (same results)

We will need more details on that one and ideally a repository that allows us to easily reproduce the issue. This might be best to create a new Issue if you are able to get us those details.
Thanks!


Failed jobs are removed instead of being unchecked

Found this bug in <IP_ADDRESS>. If error occurs, after some timeout the job continues, but failed job is being deleted instead of being unchecked.

Thanks for reporting, it's fixed in the next build.

In v<IP_ADDRESS> it's still here. Hope next test will solve it )


[Suggestion]  A Config option to be able to change the max radius of the Paradox

Describe the feature you'd like to have added
Pretty much what the title says, it would be nice to be able to modify how large, or small, the Paradox can grow to.  Currently, at least personally speaking, the size of the Paradox seems a bit lackluster even at the max radius.
Additional information
No response


Clarify documentation page

I'm working on some dev-focused docs for this sample and figured some edits and clarifications could be helpful here.
Related: https://github.com/trilitech/tezos-developer-docs/pull/392

Hi @timothymcmackin , thaks for the edits!


Clicking on a notification element should take you to the correct place in the dialog

Right now, clicking on a notification will only change dialog. It should also scroll back and somehow highlight the position in the dialog.

Closing this in favour of #330.


Update configure-exchange-server-for-hybrid-modern-authentication.md

I believe the EvoSTS object created by the HCW at one time just named the object 'EvoSTS' so the powershell command to look it up may have worked at one point. That is no longer the case. The object is made with a name of 'EvoSts - GUID' so an 'eq' doesn't work anymore.
I changed this to 'like' with a wildcard on the end and then output to a table to show the full name and the enabled status.

That's not the one I meant to paste in, here's the actual example

I also noticed, when moving on to the next step in the doc, that the enablement is done by this command
Set-AuthServer -Identity EvoSTS -IsDefaultAuthorizationEndpoint $true
Which isn't going to work via a copy and paste since it's no longer named just EvoSTS. This will probably require a bigger edit to reflect the change that appears to have happened in the HCW.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@kelleyvice-msft Please merge this Pr. Thanks.

@kelleyvice-msft bump?

@kelleyvice-msft one more bump?

@kelleyvice-msft Just checking on this again

@dstrome  -- Another item requiring review from Exchange perspective. Thanks.


:zap: feat(bundlers): Add in memory cache to bundlers and ts-plugin

Description
Add in memory cache to all bundlers and lsp
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #498

PR #499   👈

This comment was auto-generated by Graphite.


jamil- issues

ls -h does not show the files size, it only shows the directories within the folder, it is basically as doing ls, but ls-hl does work and shows the files size.

After typing the command 'mkdir cookbook' then typing 'for i in {25..45..1};do echo $i; mkdir $i; done' the subdirectories are not created within 'cookbook' but in the home directory.

the 'cp icecream.xyz ../favorites/icecream.xyz' does not work because the favorites folder does not exists within the home directory, Trying to creates the folder 'favorites' without '../' does work because one do not need '..' since we are currently in the home directory.

the command 'mv icecream.xyz ../favorites/icecream.xyz' works unless it is written without '../' because i'm still in my home directory.

the command 'grep --after-context=8 "guacamole" salads.txt > avocado_love.txt' does not work because the file 'salads.txt' does not exist im the home directory.


🐛 [bug] - Create alerts exercise seems to be incomplete

📝 Description
Step 6 causes an alert, but there are no steps to see the alert and/or verify it's fine. I couldn't see the alert everywhere, FWIW.
🚶 Steps to reproduce
Execute up to step 6.
🧙♀ Suggested solution
Instruct the user to go to the proper place to view the alert. Maybe, remove the dded file.
Also note that there are no instructions to exit from rsh here:
https://github.com/rht-labs/tech-exercise/blame/main/docs/4-return-of-the-monitoring/2-create-alerts.md#L80
Which may be confusing.

yeah we should add that the alert firing is visible on OpenShift UI main page..on it :)


Fixing panic issue in Validate when properties are nil

What this PR does / why we need it: Avoiding panics during validation
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #2586
Special notes for your reviewer:
If applicable:

[ ] documentation
[x] unit tests
[x] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

@jackfrancis I've made the changes. As for the references, I am not sure if adding multiple comments is a good idea. I thought of pointing to this as a reference for other readers , https://golang.org/doc/faq#nil_error. Thoughts?

@shrutir25 @amanohar @weinong how does this PR align with #3197 ?

/lgtm


Add servo_power_meter

Import a power measurment tool from github.com/hikalium/hikalium.com.

 I don't mind if you make it part of mainline - but I'll have to send a couple patches to fix some known issues.


escape url in Htmlbuilder with minifiedAjax

Summary of problem or feature request
when i use htmlbuilder with function minifiedAjax it shows url as escaped with \/ slashs as showing below:
here is the result of $html->scripts();
$(function(){window.LaravelDataTables=window.LaravelDataTables||{};window.LaravelDataTables["dataTableBuilder"]=$("#dataTableBuilder").DataTable({"serverSide":true,"processing":true,"ajax":"http:\/\/jpaemirates.mie\/users","columns":[{"name":"id","data":"id","title":"Id","orderable":true,"searchable":true},{"name":"first_name","data":"first_name","title":"First Name","orderable":true,"searchable":true},{"name":"email","data":"email","title":"Email","orderable":true,"searchable":true},{"name":"created_at","data":"created_at","title":"Created At","orderable":true,"searchable":true},{"name":"updated_at","data":"updated_at","title":"Updated At","orderable":true,"searchable":true}]});});
here is my controller code:
public function index(Builder $builder) {
if (request()->ajax()) {
return DataTables::of(User::all())->make(true);
}
    $html = $builder->minifiedAjax()->columns([
                ['data' => 'id', 'data'=>'id', 'title' => 'Id'],
                ['data' => 'first_name', 'title' => 'First Name'],
                ['data' => 'email', 'title' => 'Email'],
                ['data' => 'created_at', 'title' => 'Created At'],
                ['data' => 'updated_at', 'title' => 'Updated At'],
            ]);

    return view('tableList', compact('html'));
}

here is blade template:
@extends('layouts.app')
@section('content')
{!! $html->table() !!}
@endsection
@push('scripts')

{!! $html->scripts() !!}
@endpush

Use raw columns, see https://yajrabox.com/docs/laravel-datatables/master/raw-columns.

The problem is when the service parsing Ajax url on builder script it's escaping the url its show like this http:// .....etc
Did you got my point?

Oic, didn't noticed you were referring to the actual URL :)
Maybe try specifying the url?
$builder->minifiedAjax(route('route-name'))


Bug: Inputs stuck in gamepad mode

Bug report from user RedX2104 on ESOUI;

When using Gamepad mode, When selecting anything it wont let me Use the A button to jump or the B button to use a Skill.
It stays like this until i bring up the Menu again and close it with the same button i assigned it to.
Edit: It seems to be that way only in Toggle mode. When not exiting the menu thorugh the map etc

Fixed at ef1e62e4 in main


#Some Important Operation Using Bit Manipulation

Description
Description
Description
getBit()
setBit()
clearBit()
using bitmanipulation
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign

/assign

/assign


build error

hi,
when i tried to build the cmocka library i got the following error message
make -DCMAKE_INSTALL_PREFIX=/home/mkarthig/repo/mkarthig/5601/sw/y/build/netconfdevel/ -DCMAKE_BUILD_TYPE=Debug -DUNIT_TESTING=ON ..
-- Check if the system is big endian
-- Searching 16 bit integer
CMake Error at /usr/local/share/cmake-3.7/Modules/TestBigEndian.cmake:41 (message):
no suitable type found
Call Stack (most recent call first):
ConfigureChecks.cmake:142 (test_big_endian)
CMakeLists.txt:43 (include)
-- Configuring incomplete, errors occurred!
See also "/home/mkarthig/repo/mkarthig/5601/sw/y/build/netconfdevel/cmocka-master/build/CMakeFiles/CMakeOutput.log".
See also "/home/mkarthig/repo/mkarthig/5601/sw/y/build/netconfdevel/cmocka-master/build/CMakeFiles/CMakeError.log".

hi,
can you post the logs from
/home/mkarthig/repo/mkarthig/5601/sw/y/build/netconfdevel/cmocka-master/build/CMakeFiles/CMakeOutput.log
and
/home/mkarthig/repo/mkarthig/5601/sw/y/build/netconfdevel/cmocka-master/build/CMakeFiles/CMakeError.log
to get more information on the issue?

I'd recommend to move this to the upstream repository for more information. But questions are always welcome.


System.TypeLoadException: Could not load type 'Microsoft.Practices.EnterpriseLibrary.Common.Configuration.Manageability.ConfigurationSectionManageabilityProviderAttribute'

When installing Swashbuckle via Nuget into our products pre-existing WebAPI project, the following exception occurs when trying to access our API.
System.TypeLoadException: Could not load type 'Microsoft.Practices.EnterpriseLibrary.Common.Configuration.Manageability.ConfigurationSectionManageabilityProviderAttribute' from assembly 'Microsoft.Practices.EnterpriseLibrary.Common, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
Our project makes use of Enterprise Library 6. Is there some compatibility issues here with Swashbuckle?

I have the same error raised after installing the swashbuckle. @shorttylad did you solve it?


Example of cross compiling module

Is there an example of building a node module where cross compilation of a binary library is performed?
I am trying to build https://github.com/sandeepmistry/node-bluetooth-hci-socket, but gyp seems to be calling the host compiler.

At least one recipe at meta-nodejs-contrib is cross-compiling native dependencies:
https://github.com/imyller/meta-nodejs-contrib/blob/master/recipes-extended/node-red/node-red_0.14.6.bb
It has dependency to node-serialport, which gets cross-compiled as native addon.

Also, I'd like to remind that cross-compiling npm is available only as oe_runnpm when you have inherited the npm-base class. That is analogous to oe_runmake available from base class in Yocto, where calling plain make in the recipe skips all cross-compilation magic.
If you run npm command as-is, you're very likely calling host machine npm or the target npm with host machine node binary - and cross-compilation does not take place.
Helper classes for Node.js recipes such as npm-install or npm-install-global use oe_runnpm internally. This is the case with node-red recipe example I gave before.

@ringtailsoftware I hope these help.
https://github.com/soonsebii/meta-nodejs-contrib/blob/master/recipes-extended/noble/noble_1.7.0.bb


Frame [odom] does not exist

I'm new to rviz and ros2, after I run ros2 launch go2_bringup go2.launch.py rviz:=True, In the Global Status > Fix Frame, I get the following errors:
Frame [odom] does not exist
How can I solve it? Thank you.

@phamthaihoangtung switch from odom to base_link, does rviz now show any part of the robot?

@judy33a it only shows base_link and head_lower and head_upper

@phamthaihoangtung this happened to me as well, it is related to the network connection so no worries since it shows some parts in the base_link,
what i did was that i connected both my laptop and the robot to the same wireless network (while keeping the ethernet) (i already had configured a lot of stuff, so im not quite sure what would be different between my case and yours)


calcium and antic

Calcium: Exact real and complex numbers in C

Source URL: https://github.com/fredrik-johansson/calcium
License: LGPL-2.1

and its required dependency
Antic: Algebraic Number Theory In C

Source URL: https://github.com/flintlib/antic
License: LGPL-2.1

I would package them myself, but in a related package (arb), I see mentioned of the sage team: https://github.com/NixOS/nixpkgs/blob/677ed08a50931e38382dbef01cba08a8f7eac8f6/pkgs/development/libraries/arb/default.nix#L38.

Fixed by #209644 and #209645


Unable to determine what CMake generator to use.

Unable to determine what CMake generator to use. Please install or configure a preferred generator, or update settings.json or your Kit configuration.

Could I get some information about what platform you are using? What CMake version do you have installed?

This issue is being closed because it appears to have been abandoned. If you require further assistance, please let us know and we can reopen it.


inserting duplicate record with a unique constraint causes crecto to hang

This might be an issue with crystal-db or crystal-pg, I can replicate without crecto

yep, created an issue there https://github.com/crystal-lang/crystal-db/issues/46


Issue with schema

Hi there, I've been searching for a bit of time now, a way to have this code work:
package broken

import (
	"testing"

	"github.com/cayleygraph/cayley"
	"github.com/cayleygraph/cayley/quad"
	"github.com/cayleygraph/cayley/schema"
	"github.com/stretchr/testify/assert"
)

func TestBroken(t *testing.T) {
	store, err := cayley.NewMemoryGraph()
	assert.NoError(t, err)
	store.AddQuad(quad.Quad{quad.IRI("dialect-source-file.raml#vocabularies/validation"), quad.IRI("http://www.w3.org/1999/02/22-rdf-syntax-ns#type"), quad.IRI("http://raml.org/vocabularies/meta#External"), nil})
	store.AddQuad(quad.Quad{quad.IRI("dialect-source-file.raml#vocabularies/validation"), quad.IRI("http://raml.org/vocabularies/meta#name"), quad.String("validation"), nil})
	store.AddQuad(quad.Quad{quad.IRI("dialect-source-file.raml#vocabularies/validation"), quad.IRI("http://raml.org/vocabularies/meta#uri"), quad.String("../vocabulary/validation.raml"), nil})

	schema.RegisterType(quad.IRI("http://raml.org/vocabularies/meta#External"), DialectExternal{})

	var dst DialectExternal
	assert.NoError(t, schema.LoadTo(nil, store, &dst, quad.IRI("dialect-source-file.raml#vocabularies/validation")))
	assert.Equal(t, "validation", dst.Name)
}

type DialectExternal struct {
	// rdfType struct{} `quad:"@type > http://raml.org/vocabularies/meta#External"`
	// ID      quad.IRI `quad:"@id"`
	Name    string   `quad:"http://raml.org/vocabularies/meta#name,optional"`
	URI     string   `quad:"http://raml.org/vocabularies/meta#uri,optional"`
}

This is using version baf955d0b2d19f260fc9701659754742b58628f7
Running go test on that fails.
Anyone has insights ?  I might have tripped over https://godoc.org/github.com/cayleygraph/cayley/schema#LoadTo where I read

Predicate IRIs in RDF can have a long namespaces, but they can be written in short form. They will be expanded automatically if namespace prefix is registered within QuadStore or globally via "voc" package. There is also a special predicate name "@type" which is mapped to "rdf:type" IRI.

Is it possible that this is not true ? That somehow Cayley expects the short form to be in the database ?

See discussions in https://discourse.cayley.io/t/difficulty-using-the-schema-package/1107/14 also.

You are right about the short form - schema lib naively assumes that all IRIs match exactly on insert and in struct tags. However, the @type predicate is mapped to a short form, and resolution to full form is not done correctly, I guess.
It's time to do this properly. It will require all quad inserts to resolve to full form, and all queries to resolve their arguments to full form as well. For now, the best place for it is QuadStore.ValueOf and QuadStore.ApplyDeltas.
The downside is that users sometimes prefer to use short form, and now they will see a full form as an output of queries. Always printing short form will lead to the same confusion - written values might be in full form, while short form is returned as output.

What I was thinking is create a Schema strict and have a DefaultSchema like
http.DefaultClient, and be able to configure the iriType in that reader. It
would also allow the types to be registered within that Schema instance, as
well as have its own reference to the QuadStore. Would simplify the
arguments to LoadTo.
The top level LoadTo could still use the DefaultSchema or create a new one
on the fly, with the current defaults.
What do you think? I wouldn't want to change the semantics for everyone or
force the Data to be always full either. If it's this way, it's probably
because some use the short form in their data.
Le jeu. 22 févr. 2018 03 h 39, Denys Smirnov<EMAIL_ADDRESS>a
écrit :

You are right about the short form - schema lib naively assumes that all
IRIs match exactly on insert and in struct tags. However, the @type
predicate is mapped to a short form, and resolution to full form is not
done correctly, I guess.
It's time to do this properly. It will require all quad inserts to resolve
to full form, and all queries to resolve their arguments to full form as
well. For now, the best place for it is QuadStore.ValueOf and
QuadStore.ApplyDeltas.
The downside is that users sometimes prefer to use short form, and now
they will see a full form as an output of queries. Always printing short
form will lead to the same confusion - written values might be in full
form, while short form is returned as output.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cayleygraph/cayley/issues/694#issuecomment-367605660,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFs8MWBnJS233Htq4bQ8GPxni9JERu2ks5tXSeygaJpZM4SOSYP
.

@abourget I think it's the best option for this particular case.
Note that graphql endpoint will still fail the same way (it also uses @type shorthand).

@abourget Feel free to reopen if it does not cover your use case

Wow great! Thanks!
On Sat, Mar 3, 2018, 3:14 PM Denys Smirnov<EMAIL_ADDRESS>wrote:

@abourget https://github.com/abourget Feel free to reopen if it does
not cover your use case
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cayleygraph/cayley/issues/694#issuecomment-370176433,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFs8Ko1AIzyELkx7R9TzJoIuXibXv0cks5tavmZgaJpZM4SOSYP
.


Is main all-in-one-dbless.yaml something missed, no route matched with those values.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
when we are deploy latest all-in-one-dbless.yaml, we has some problems in our aws eks 1.26 env.

kong service account seem to has not list endpoint permission
E0518 09:15:26.960856       1 reflector.go:140] pkg/mod/k8s.io/client-go@v0.26.3/tools/cache/reflector.go:169: Failed to watch *v1.Endpoints: failed to list *v1.Endpoints: endpoints is forbidden: User “system:serviceaccount:kong:kong-serviceaccount” cannot list resource “endpoints” in API group “” at the cluster scope
miss CONTROLLER_INGRESS_CLASS
miss CONTROLLER_INGRESS_CLASS will cause no route matched with those values if we already apply related ingress.

Expected Behavior
expect apply all-in-one-dbless.yaml anything fine.
Steps To Reproduce
`kubectl apply -f https://bit.ly/k4k8s`

Kong Ingress Controller version
2.9.3

Kubernetes version
1.26

Anything else?
No response

https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/v2.9.3/deploy/single/all-in-one-dbless.yaml

Thanks for your reply.
We will try to use tagged version.


id.json missing in docker context

Hi I am seeing this issue when running on wsl 2 via docker. I checked the docker image via vscode and don't see the id.json file in /app/id.json
#mango-explorer solana-keygen new --force --outfile /app/id.json
Generating a new keypair
For added security, enter a BIP39 passphrase
NOTE! This passphrase improves security of the recovery seed phrase NOT the
keypair file itself, which is stored as insecure plain text
BIP39 Passphrase (empty for none):
Error: Unable to write /app/id.json: Is a directory (os error 21)

It's quite right for the id.json file not to exist in the docker container - it's to be specified on the docker command line (otherwise everyone using the docker container would be using the same Solana and Mango accounts!)
The usual cause for this particular error is running the docker command before running the touch command to create the id.json file. (Docker is problematic here - if the file passed as a 'volume' doesn't already exist, docker creates that volume as a directory.)
So, to fix this:

Check and see if the path you're specifying on your computer for the id.json file (maybe ~/mango-explorer/id.json) has been created as a directory. If it has, and it's empty(!), delete it.
Then run:
touch ~/mango-explorer/id.json
chmod 600 ~/mango-explorer/id.json

(Just like in the quickstart.)
Then you should be able to run the keygen command that was failing.
Many thanks.


append is buggy with a bit of logic in tables

What happened?
Still working on a query builder to convert transformation steps into SQL via PRQL
We have an issue with append that doesn't seem to work well with basic tables
I'm more than happy to help on debugging this
PRQL input
from `albums`
select { `album_id`, `title` }
sort {+`album_id`}
take 2
append (
  from `albums`
  select { `album_id`, `title` }
  sort {-`album_id`}
  take 2
)

SQL output
fails

Expected SQL output
WITH table_0 AS (
  SELECT
    album_id,
    title
  FROM
    albums
  ORDER BY
    album_id DESC
  LIMIT
    2
)
SELECT
  *
FROM
  (
    SELECT
      album_id,
      title
    FROM
      albums
    ORDER BY
      album_id
    LIMIT
      2
  ) AS table_1
UNION
ALL
SELECT
  *
FROM
  table_0

MVCE confirmation

[X] Minimal example
[X] New issue

Anything else?
No response

It may be the same bug as https://github.com/PRQL/prql/issues/4063. I'll start digging

Interesting serendipity (of timing). Just this morning, I was looking at this and #5055 which has a similar pattern. Rewriting the original query as follows works as expected in the current Playground.
I wonder if the fix from @lukapeshke will address #5055 as well...
# ==== from #4947 - rewritten
let a = (
from `albums`
select { `album_id`, `title` }
sort {+`album_id`}
take 2
)

let b = (
  from `albums`
  select { `album_id`, `title` }
  sort {-`album_id`}
  take 2
)
from a
append b


Dialog or session state not persisted in console emulator

Summary: When using the command line emulator, dialogs are not followed through the waterfall + session data is not kept. Instead with every interaction the user does seems to be treated as new interaction.
Tested With:

Botbuilder v3.0
Microsoft Bot Framework Channel Emulator v3 (console, Mac OS 10.10.5)
Microsoft Bot Framework Channel Emulator v3 (GUI, Windows 10)
Node 4.4.7 on both OS

Steps to Reproduce:

Take code from here: https://gist.github.com/DirkSonguer/b0dc43e4054cfeb2229d6ca655f70a26
Run node with test code
Start Bot Framework Emulator
Say "hi" (anything will do)
Say "Dirk" (anything will do)

Expected Behaviour:
This can be verified with the GUI emulator in Windows 10.
In Emulator:

User: "hi"
Bot: "Hi! What is your name?"
User: "Dirk"
Bot: "Hello Dirk"

In node console:
1: undefined
2
5
6
7[object Object]
1: Dirk
3
4

Actual Behaviour:
This happens when the same code is run in the console emulator under Mac OS.
In Emulator:

User: "hi"
Bot: "Hi! What is your name?"
User: "Dirk"
Bot: "Hi! What is your name?"

In node console:
1: undefined
2
5
1: undefined
2
5

Regardless of input, the emulator / app will always put out 1,2,5, cycling "Hi! What is your name?".
See screenshot from Windows 10 and Mac OS here: https://postimg.org/image/ru01yqfdz/

This still happens with Botbuilder 3.0.1. The output in console from the app is more verbose now:
Input in console emulator:

Send message to bot:
hi
User1 said:
hi
Bot1 said:
Hi! What is your name?
Dirk
User1 said:
Dirk
Bot1 said:
Hi! What is your name?
Dirk
User1 said:
Dirk
Bot1 said:
Hi! What is your name?

Output in node console:
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
session.beginDialog(/)
/ - waterfall() step 0 of 2
1: undefined
2
/ - session.beginDialog(/profile)
./profile - waterfall() step 0 of 2
5
./profile - session.beginDialog(BotBuilder:Prompts)
..Prompts.text - session.send()
..Prompts.text - session.sendBatch() sending 1 messages
ChatConnector: sending message.
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
session.beginDialog(/)
/ - waterfall() step 0 of 2
1: undefined
2
/ - session.beginDialog(/profile)
./profile - waterfall() step 0 of 2
5
./profile - session.beginDialog(BotBuilder:Prompts)
..Prompts.text - session.send()
..Prompts.text - session.sendBatch() sending 1 messages
ChatConnector: sending message.
WARN: ChatConnector: receive - emulator running without security enabled.
ChatConnector: message received.
session.beginDialog(/)
/ - waterfall() step 0 of 2
1: undefined
2
/ - session.beginDialog(/profile)
./profile - waterfall() step 0 of 2
5
./profile - session.beginDialog(BotBuilder:Prompts)
..Prompts.text - session.send()
..Prompts.text - session.sendBatch() sending 1 messages
ChatConnector: sending message.

Just tested with the console version under Windows 10 - it has the same behaviour as with Mac OS: 1,2,5 in a loop, not recognising dialogs and session data.

related to #730. We should have a fix with a new build of console emulator.

I can confirm that this is fixed in v3.0.6043.30574, good work, thank you!


Layer groups manipulation

Quoting these comments

I guess the kind of UI we'd want is being able to select multiple layers then right-click -> group selection, we'll need a context menu in your tree widget.
This context menu will be useful later for other commands.

@brichet :

Yes, the context menu is a good option to create groups from layers.
We probably also want drag and drop to move the layers individually.

Context menu: #48
Drag and Drop: #90


document how to use xmldiff with GIT_EXTERNAL_DIFF

For a git repository that tracks xml files, it would be great to have an easy way to use git to see what the differences are.
A section in the documentation with a recipe about how to apply xmldiff to specific files in a git repository when viewing changes would be very useful, even (especially?) if the documentation is simple.
I tried the simple:
in .gitattributes:
*.xml diff=xml

and in .git/config:
[diff "xml"]
      command = xmldiff

but that produced this error:
usage: xmldiff [-h] [-v] [-f {diff,xml,old}] [-w] [-p] [-F F] [--unique-attributes [UNIQUE_ATTRIBUTES]] [--ratio-mode {accurate,faster,fast}] [--fast-match] file1 file2
xmldiff: error: unrecognized arguments: 00065406b95a72188e41ab1165bbd8d7aa58562f 100644 assignments/pgp-parameters/pgp-parameters.xml<PHONE_NUMBER>000000000000000000000000000000 100644
fatal: external diff died, stopping at assignments/pgp-parameters/pgp-parameters.xml

I think the answer has to do with the GIT_EXTERNAL_DIFF  command as described in git(1), but i haven't sorted it out yet.
Maybe a different cli entry point that knows how to handle GIT_EXTERNAL_DIFF would be good?

For that to be useful you need a formatter that outputs a human-readable diff that is better and more easy to read than a text diff. If someone wants to contribute that, then making a special git entry point should be relatively easy.


Can't render deployment template when i wanna enable GPU flag

These are error messages:
Failed to get upgrade info: template: ollama/templates/deployment.yaml:56:89: executing "ollama/templates/deployment.yaml" at <.Values.ollama.gpu.nbrGpu>: nbrGpu is not a method but has arguments

Hello @thinking-and-coding,
This error will be fix in the next release today, the old variable "nbrGpu" has been deleted.
Thanks you!


Mastodon Icon Doesn't Render for sites other than mastodon.social

I've noticed that there is a function for rendering the icon for mastodon here: https://github.com/denoland/deno_blog/blob/571a875d774c4043fb0c196134dfac01a2209746/components.tsx#L426
However, when I add it to the links as shown in the readme, for example:
 links: [
    { title: "Email", url<EMAIL_ADDRESS>},
    { title: "GitHub", url: "https://github.com/denobot" },
    { title: "Twitter", url: "https://twitter.com/denobot" },
    { title: "Mastodon", url: "https://somethingotherthanmastodon.social/@denobot"},
  ],

It's not going to render.
This is a problem, in general, and for me personally because mastodon is a decentralized network and isn't owned by mastodon.social, and the icon will not render for a different url. This is because of the code snippet:
    const icon = socialAppIcons.get(
    url.hostname.replace(/^www\./, ""),);

In the source here: https://github.com/denoland/deno_blog/blob/571a875d774c4043fb0c196134dfac01a2209746/components.tsx#L79
Which uses a regex to get the name of the social media site from the url to pass to the socialAppIcons function.
I can understand some possible reasons we want to use the url here, but couldn't we just use the title instead?  I would issue a pull request but I wanted to understand the mainters' intention here before starting one.

@macbraughton you can use this link as a workaround:
https://mastodon.social/<EMAIL_ADDRESS>or you can copy IconMastodon into your main.tsx file:
 { title: "Mastodon", url: "https://somethingotherthanmastodon.social/@denobot", icon: IconMastodon },

but couldn't we just use the title instead?

I think exporting the icon would be the best solution, since title can specific to the language eg "мастодон"


backend/drm: drop unused arg from get_possible_crtcs

Constify drmModeConnector while at it.

Good catch, thanks!


Installed package hides tabs on content nodes

We noticed after installing the package and adding an experimental property (wrapping the text editor) that tabs have started disappearing from the content nodes.
Without the package installed:

With the package installed:

In some cases, the contents from the hidden tabs are not visible (but not all tabs are gone); while in others all tabs are gone but the content is displayed.
Also, tabs inherited by composition are still there.
Umbraco version: 10.3.2
DotNet version: net7.0

Hi Robert,
Sorry to hear that your first experience with the package is somewhat problematic.
The 'removal' of tabs should only happen if there are no properties on the tab after any 'restricted' ones have been removed, as you can see on this bit of code:
https://github.com/LottePitcher/umbraco-admin-only-property/blob/a22f9711541e20af27cb1f73ee42b82042948589/src/AdminOnlyProperty/AdminOnlyPropertySendingContentNotificationHandler.cs#L60-L63
I have this working on a couple of sites and have not experienced the disappearing of tabs that you describe.
We have a 'admin only' textarea in a composition, and that works as expected.
You used the word 'inheritance' in your post, which makes me realise I've not tested anything with actual inheritance, but as I say I have it working in a composition ok.
If there is an easy way for you to share with me your configuration (perhaps uSync config files?) I could try and see what's happening here.
Lotte

Thanks @GijsD-iD for the feedback. I've not tested with groups (confession, haven't used them for ages), but I'll see if I can replicate what you report as yes it sounds a sensible thing to support if feasible

I have the exact same issue: Tabs where all properties are under group(s).
https://i.imgur.com/NDuH19n.png
Umbraco version: 11.2.0
DotNet version: net7.0

Thanks @martinbrud for your feedback.
I have replicated this issue and it has been fixed in commit bc965b1.
The fix will be coming in the next release, which I should be able to get done next week. I'll update this issue as soon as it is.

version 1.1.2 has been released which fixes this


Update Site settings “info”

Automatically generated by Netlify CMS

👷 Deploy Preview for devops-toolbox processing.

Name
Link

🔨 Latest commit
35c9ee52db4e1d983a2c6bdc120cd1bdde1d44bb

🔍 Latest deploy log
https://app.netlify.com/sites/devops-toolbox/deploys/63c5e64a60db6500093714f4


{CI} Extension regression test pipeline

Description

This PR implements CLI Extension regression test pipeline

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

CI


Extend error response

add ErrorReason in proto file.

enum ErrorReason {
  USER_NOT_FOUND = 0;
}

return in service

return nil, ErrUserNotFound("not found")

assert error

if IsUserNotFound(err) {
	fmt.Println("user not found")
}

@qazwsxedckll  Why was the directory name _examples  changed back?

@qazwsxedckll Tracking the go.sum file of examples may causing a lot unnecessary changes.

👋🏻 @cupen

@cupen  It is ignored by go build and I cannot get any lint from vscode even if I add go.work. he is right about remove example dependencies from the main module by adding go.mod to every example. maybe there were many errors before go.work came out. so add underscore to ignore them make sense. now we can just run go work init && go work use -r .  to add all examples.

Hi @rogeralsing   @qazwsxedckll  To my point of view, these examples are not a part of protoactor-go , but rather some usecases for demonstration purposes. So the examples and protactor-go can be treated as two separate projects, it might be enough to just place a go.work in the examples directory for linter.
So, for avoid the lots of unnecessary changes with go.sum of examples, is there a better way  than .gitignore?
btw: These examples also could be the first choice to check our locally modification, so the replace xxx => ./.. in go.mod of examples is useful. And also it's could be used with go.work.

@cupen First, go.work is not recommanded to be commited to the repository. Run go work init && go work use -r . in the root directory is the way to go. When using go.work, it is alway using the local modifications, no matter the verson chosed in go.mod. When not using go.work, examples are using the version specified in go.mod. I find it is extremely useful. So replace is not neccessay here. You can try it out first.

@qazwsxedckll Yes, that's true.  I mistakenly thought you wanted to put a go.work file in the root path for resolve the linter problems, so I suggested use the examples as an alternative.


feat(opentelemetry-exporter-jaeger): http sender

Which problem is this PR solving?
Enables the usage of jaeger-client-node HTTPSender in order to send spans over http.
The behavior implemented complies with the following  jaeger-client-node documentation statement:
By default, the client sends traces via UDP to the agent at localhost:6832.
Use JAEGER_AGENT_HOST and JAEGER_AGENT_PORT to send UDP traces to a different host:port.
If JAEGER_ENDPOINT is set, the client sends traces to the endpoint via HTTP, making the JAEGER_AGENT_HOST and JAEGER_AGENT_PORT unused.
If JAEGER_ENDPOINT is secured, HTTP basic authentication can be performed by setting the JAEGER_USER and JAEGER_PASSWORD environment variables.

Extracted from: https://github.com/jaegertracing/jaeger-client-node#environment-variables
Short description of the changes
1. packages/opentelemetry-exporter-jaeger/src/jaeger.ts:

Populates the config fields endpoint, username and password at JaegerExporter contruction . code
Checks if config.endpoint is setten and instanciates this._sender as  a HTTPSender and reffers original documentation in commentaries. code

2. packages/opentelemetry-exporter-jaeger/src/types.ts:

Adds the properties endpoint, username and password to ExporterConfig interface . code
Declares HTTPSender as a jaeger type. code

Codecov Report

Merging #965 into master will decrease coverage by 17.40%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #965       +/-   ##
===========================================
- Coverage   94.75%   77.34%   -17.41%
===========================================
  Files         248       74      -174
  Lines       11288     1832     -9456
  Branches     1079      309      -770
===========================================
- Hits        10696     1417     -9279
+ Misses        592      415      -177

Impacted Files
Coverage Δ

packages/opentelemetry-core/test/utils/url.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...ckages/opentelemetry-core/test/platform/id.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...pentelemetry-core/src/platform/node/performance.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...pentelemetry-core/test/internal/validators.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...ntelemetry-core/test/trace/NoRecordingSpan.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...telemetry-core/test/platform/hex-to-base64.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...elemetry-core/test/trace/spancontext-utils.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...lemetry-core/test/trace/ProbabilitySampler.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...try-core/test/trace/fixtures/test-package/index.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...ry-api/test/noop-implementations/noop-span.test.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 175 more

@open-telemetry/javascript-approvers this is ready for review

Any release date for this?

Any release date for this?

Sorry for the delay in response. We will include this change in 0.8.3 release. Let us know if you're blocked on this, we can cut the release sometime this week.

Any release date for this?

Sorry for the delay in response. We will include this change in 0.8.3 release. Let us know if you're blocked on this, we can cut the release sometime this week.

Not a blocker. I was looking for a standard way to do this across node and java services. Java auto instrumentation sends it directly to the collector. I couldn't find a way to make use of the agent there, whereas nodejs agent would only send to the agent.

@dyladan please take a look when you get a chance.
@leonardodalcin Please, resolve the conflicts.


custom_jvp + cond on tangent side = assertion failure

Description
from @bunelr
import jax
import jax.numpy as jnp

def mask_fun(arr, choice):
  out = (1 - choice) * arr.sum() +  choice * (1 - arr.sum())
  return out

def switch_fun(arr, choice):
  choice = jnp.floor(choice).astype(jnp.int32)
  out = jax.lax.switch(choice, [lambda x: x.sum(), lambda x: 1 - x.sum()], arr)
  return out

test_arr = jnp.arange(3.)
test_val = 0.

print(mask_fun(test_arr, test_val))     # 3.0, all good
print(switch_fun(test_arr, test_val))   # 3.0, all good

print(jax.grad(mask_fun)(test_arr, test_val))   # -> 1. everywhere -> good
print(jax.grad(switch_fun)(test_arr, test_val)) # -> 1. everywhere -> good

def good_switchfun_jvp(primals, tangents):
  arr, choice = primals
  arr_dot, choice_dot = tangents
  return switch_fun(arr, choice), mask_fun(arr_dot, choice)

def bad_switchfun_jvp(primals, tangents):
  arr, choice = primals
  arr_dot, choice_dot = tangents
  # tangent_out is different, goes through the switch implementation.
  return switch_fun(arr, choice), switch_fun(arr_dot, choice)

good_custom_switchfun = jax.custom_jvp(switch_fun)
good_custom_switchfun.defjvp(good_switchfun_jvp)
print(jax.grad(good_custom_switchfun)(test_arr, test_val)) # 1. everywhere, good

bad_custom_switchfun = jax.custom_jvp(switch_fun)
bad_custom_switchfun.defjvp(bad_switchfun_jvp)
print(jax.grad(bad_custom_switchfun)(test_arr, test_val)) # ASSERTION ERROR

This variant of the repro works:
import jax
import jax.numpy as jnp
jax.config.update('jax_enable_checks', True)

def mask_fun(arr, choice):
  out = (1 - choice) * arr.sum() +  choice * (1 - arr.sum())
  return out

def switch_fun(arr, choice):
  choice = jnp.floor(choice).astype(jnp.int32)
  out = jax.lax.switch(choice, [lambda x: x.sum(), lambda x: 1 - x.sum()], arr)
  return out

def switch_fun_linear(arr, choice):
  choice = jnp.floor(choice).astype(jnp.int32)
  from jax._src.lax.control_flow.conditionals import _cond
  out = _cond(choice, lambda x: 1 - x.sum(), lambda x: x.sum(), arr,
              linear=(True,))
  return out

test_arr = jnp.arange(3.)
test_val = 0.

print(mask_fun(test_arr, test_val))     # 3.0, all good
print(switch_fun(test_arr, test_val))   # 3.0, all good

print(jax.grad(mask_fun)(test_arr, test_val))   # -> 1. everywhere -> good
print(jax.grad(switch_fun)(test_arr, test_val)) # -> 1. everywhere -> good

def good_switchfun_jvp(primals, tangents):
  arr, choice = primals
  arr_dot, choice_dot = tangents
  return switch_fun(arr, choice), mask_fun(arr_dot, choice)

def bad_switchfun_jvp(primals, tangents):
  arr, choice = primals
  arr_dot, choice_dot = tangents
  # tangent_out is different, goes through the switch implementation.
  return switch_fun(arr, choice), switch_fun_linear(arr_dot, choice)

good_custom_switchfun = jax.custom_jvp(switch_fun)
good_custom_switchfun.defjvp(good_switchfun_jvp)
print(jax.grad(good_custom_switchfun)(test_arr, test_val)) # 1. everywhere, good

bad_custom_switchfun = jax.custom_jvp(switch_fun)
bad_custom_switchfun.defjvp(bad_switchfun_jvp)
print(jax.grad(bad_custom_switchfun)(test_arr, test_val)) # works now!

for control flow primitives like switch or cond, jax internally tracks in which inputs functions are guaranteed to be linear when the function is produced from jax's jvp autodiff pass. that linearity information is then useful when transposing.
originally (with scan, the first differentiable control flow operation) i think we only meant for that linearity information to be used in error-checking. but i think for cond we may have accidentally made it more load-bearing.
when using custom_jvp, rather than jax's internal jvp of switch, this linearity information isn't added by the jvp pass, and so downstream transposition fails
the above version of the repro works because i've made a switch_fun_linear variant of the switch_fun function to apply to the arr_dot values, which are linear inputs, and in switch_fun_linear i'm setting the linearity info explicitly. (i don't think the option is plumbed through to switch, so i used an internal cond.) this isn't the fix! manually annotating linearity information shouldn't be necessary. but i just wanted to test the hypothesis that somehow switch not getting linearity information was the problem


GEODE-6028: Remove checks that are not necessary.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

@mcmellawatt Bruce also had the same question with an earlier similar PR. I generally add .diff at the end of the PR hyperlink (https://github.com/apache/geode/pull/2827.diff) and copy the page and in intelliJ I do the "apply  patch from clipboard".  Doing that you will be able easily see why the checks are unnecessary. It helps to look at PRs when it is tough to analyze it from the github webpage.


add missing alt text to some  tags

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against 144d74de98589912c57fc4c8d9d48e7c825294ad


How to disable drag drog when click on element in SortableElement?

I render select tag in SortableElement, I can not catch on change event of select tag.
Can I disable drag drop event only on child element in SortableElement?
Thank you for your help!

Have you tried using shouldCancelStart property?

As mentioned by @zfeed, try using shouldCancelStart

Same issue. Please show how use shouldCancelStart prop.

@pvtran1995 please update, had your problem been solved?

how to use it please ? no documentation found


Panels getting reloaded arbitrarily in the background

I'm using the Eole theme which is built using foo_spider_monkey_panel. I've been trying to debug an issue for a long time now that manifests as seemingly random reloading of all loaded panels. To the best of my knowledge, this is not being triggered by the panels' code itself (traced every call to `window.Reload()'). I'm wondering if it might have to do with my particular component config (see below) that somehow triggers this behaviour.
I know this isn't a whole lot to go on, but can you think of any corner-cases where panels can spontaneously get reloaded?
 If you are willing to work on diagnosing this issue, I can create a special SMP build for you that will trace all panel reloads (i.e. reloads triggered by fb2k as well). That way we can understand what's causing it.

Sure thing! Let me know how I can help.

Sorry for the lack of activity, I will try to send you a debug build this weekend.

Sorry, clicked on the wrong button. No worries, there's no rush.

Uhm... So... I've finally got around to working on this component again...
Do you still need help? >_<

It's been a while since I saw this bug since I changed my usage habits a bit since I opened this issue, but a quick look at the console seems to suggest that it does keep happening. If the debug DLL is still on the table, I can try to debug the issue when I get the time.

Ok, on it!

Can you try the following version? (based on v1.3.1)
https://ci.appveyor.com/api/buildjobs/fm7tg66nnyy2icmi/artifacts/_result%2FWin32_Release%2Ffoo_spider_monkey_panel.fb2k-component
It should log some info into console.
Please attach console output once you experience the same issue again.

Will give it a shot, thanks!

Closing the issue, since there is no activity here.
Fell free to reopen if needed.


'int' object is not subscriptable when trying to create autoscale profile for recurrence

This is autogenerated. Please review and update as needed.
Describe the bug
When trying to create a recurrence profile using --recurrence week sat sun or any combination of days throws the below error.  It does not happen when creating a profile with a start date and end date, seems to be confined to recurrence.
Command Name
az monitor autoscale profile create
Errors:
'int' object is not subscriptable
Traceback (most recent call last):
Lib\site-packages\knack\cli.py, ln 231, in invoke
cli\core\commands\__init__.py, ln 657, in execute
cli\core\commands\__init__.py, ln 720, in _run_jobs_serially
cli\core\commands\__init__.py, ln 712, in _run_job
...
cli\core\commands\__init__.py, ln 691, in _run_job
cli\core\commands\__init__.py, ln 328, in __call__
azure\cli\core\__init__.py, ln 807, in default_command_handler
command_modules\monitor\operations\autoscale_settings.py, ln 248, in autoscale_profile_create
command_modules\monitor\operations\autoscale_settings.py, ln 215, in _create_recurring_profile
cli\command_modules\monitor\_autoscale_util.py, ln 591, in build_autoscale_profile
TypeError: 'int' object is not subscriptable

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az monitor autoscale profile create --autoscale-name {} --resource-group {} --name {} --count {} --timezone {} --recurrence {} {} {} {}

Expected Behavior
For it to create a recurrence scale setting
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.22.1

Extensions:
subscription 0.1.4
webapp 0.3.1

Additional Context

monitor

Hi @chigeroo ,Maybe you can try the updated version. I did not find the same problem in the following versions.
azure-cli: 2.27.0
logic: 0.1.4
This is the command I used:az monitor autoscale profile create -g rg --autoscale-name myvmss -n test1 --copy-rules default --min-count 1 --count 2 --max-count 2 --recurrence week sat --timezone "Pacific Standard Time"


Loosen version dependency of requests

PyGorse currently depends on requests~=2.14.0, but that is somewhat old and should probably be changed to requests>=2.14.0 to reduce the chances of conflicts

Thank you for your feedback. It has been fixed in #4.


BLE-MESH Sensor Client Example Bug (IDFGH-3927)

Environment

Development Kit:      ESP32-DevKitC
Kit version (for WroverKit/PicoKit/DevKitC): v4
Module or chip used:  ESP32-WROOM-32D
IDF version (run git describe --tags to find it): v4.2-dev-1660-g7d7521367
Build System:        Make
Compiler version (run xtensa-esp32-elf-gcc --version to find it): ( esp-2020r1) 8.2.0
// 1.22.0-80-g6c4433a
Operating System:     Linux
Power Supply:         USB

Problem Description
Found Bug in the BLE-MESH example of the Sensor Client Model . The problem appears when 3 (or more) mpids and values are sent from the server to the client. When trying to parse 3 values (or more), the code does not parse good property ids or anything due to a problem in the iteration through the "for loop".
The code provided in the master branch, updates the "data" pointer as follows:
data += length;
This causes to move more than 3 bytes after the first iteration causing missing data and wrong parsing. Instead that line should be:
data += mpid_len + data_len + 1;
This way, the data pointer moves 3 bytes per iteration while the count "length" is updated.
Expected Behavior
Correct Parsing
Actual Behavior
Wrong iteration!
Steps to reproduce

Modify Sensor Server example to send more than 2 values with their property ids.
Send those values as a Status Message to a Sensor Client Model and try to parse them.

It happens exactly in line 569. Please review if this error occur in other branches or examples. Thank you

Thanks for reporting, we will look into.

Hi @MrCurio
The code in the example is:
if (data_len != ESP_BLE_MESH_SENSOR_DATA_ZERO_LEN) {
    ESP_LOG_BUFFER_HEX("Sensor Data", data + mpid_len, data_len + 1);
    length += mpid_len + data_len + 1;
} else {
    length += mpid_len;
}
data += length;

Code above data += length will check if the data length is ZERO and update the lengthvalue.
Thanks.

Thank you!
Should i close the issue?

Hi @MrCurio
Yes, you can close this issue. I have submitted a MR to fix this.
Thanks.

Can any one please help me how to get datas in sensor mesh in group and what and things to do.
Also after getting the indoor and outdoor values in sensor mesh ,How can i change change the data to desired datas.
Advance Thanks for reply.


Click should always go to notification page...

Please consider reverting (or fixing up) #32 (click when there are no notifications goes to GitHub home page).
Bad use case:

I get an email notification that there's a new issue/comment.
Hit Ctrl-Shift-G (which I bind to GitHub Notifier).
But I'm taken to the GitHub page .... 'cause GHN hasn't yet figured out that there's a new notification.

This is a really bad UX.
Possible fixes:

Wholly revert #32.
Go to https://github.com/notifications?all=1 if there seems to be no new notifications.
Add a dynamic check as to whether there are new notifications.

@Narno Thoughts?

And obligatory XKCD:

Hello @smblott-github, thanks for your feedback.
OK, what is happening : GHN fetchs notifications asynchronously from the GH API while respecting the interval allowed between each call.
If you receive an e-mail about a new notification and clicks on the icon (or use a keybord shortcut) between 2 calls, the GHN counter is not yest up-to-date.
About your possible fixes proposals:

Wholly revert #32.

 :-) (I think @sindresorhus has the last word)

Go to https://github.com/notifications?all=1 if there seems to be no new notifications.

Not sure it would be usable for most of user, maybe as an option?

Add a dynamic check as to whether there are new notifications.I like this idea, but I don't know what will be the impacts on performances and risk of API spam.

Maybe the best solution is working on the desktop notification feature? #25 :-)

I would revert #32. I do not like this behaviour.

Getting the notification count is fast. Why not just fetch the count when the user clicks the button before taking them to GitHub. That way we'll be sure to do the correct thing.

Getting the notification count is fast. Why not just fetch the count when the user clicks the button before taking them to GitHub.

That seems better than the status quo.
However, I'd still argue that GHN is not a bookmark for the github home page; it's for notifications.  So this would be a better alternative target.
(Somebody argued that that might be confusing for users.  No more confusing, I'd say, than dumping them in the home page -- which for at least some users is completely useless.)

Another proposal: Do something like https://github.com/Narno/Facebook-Notifier/blob/master/README.md (see the screenshot of the options' page).

Ok, let's revert to only going to the notifications page (don't see the point of going to ?all=1). In hinsight, that makes the most sense. If you want to go to the front page, just use a bookmark.
Sorry @Narno. Not interested in adding an option for this.

No problem, I understand.
Now I'm waiting that some guys come here after the revert, to create issues : "wtf, why did you delete this funk*ng smart feature?!" :bowtie:

+1, it was confusing to all of a sudden be taken to GH home page.

It has already been decided. Just waiting for someone to do the actual work.

@smblott-github Help me out; How do you bind GitHub notifier to a key?

@rojepp Go to the extensions page (chrome://extensions/), at the bottom on the right, click keyboard shortcuts, then follow your nose.


chore: release v0.3.5 (next v0.0.12)

#588  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite


Wrong interpretation of named dollar quotes (when generating inserts with quotes) - POSTGRES

Description
dBeaver misinterpret named dollar quotes and do not find where sql command is ending.
Here is simple example. I am using dollar quotes for generating inserts.

Command will pass, however dBeaver do not recognize the end of this command properly. So it happens that it will execute all the commands below this line! It also corresponds with the wrong coloration of command, when using dollar quotes together with normal single quotes. However, sometimes its necessary to use it like this.
When following some prepared commands in scripts one by one, it is very dangerous - when every line passes without error, in some cases you will even do not found out that it was execute everything.
Here is the example. I want to run just first command (cursor on first line and press 'execute SQL Statement'). However it do also the second command. So old_table will be dropped without even notice.

DBeaver Version
Community edition 22.1.5.
Operating System
Windows 10
Database and driver
PostgresSQL 14.5
postgres-jdbc 42.2.23
Try it also on PostgresSQL 9.5 - same problem.
Steps to reproduce

connect to posgtres database in dBeaver
create new empty script
prepare some sample table and data, for example:

create table old_table (id int4, column1 text, name text, description text);
insert into old_table values (1, 'blabla', 'some_name', 'this is sample row');

Copy following commands to you script

select $a$insert into my_table values (nextval('seq_s1'), '$a$ || column1 || $b$','$b$ || name || $c$','$c$ || description || $d$', NULL);$d$ ii from old_table; --first command
drop table old_table; --second command

Try to run only first command- click on first command and press Execute SQL statement.
--> It will run the first command but ALSO the second, so you table will be dropped.

Additional context
No response

Thank you for bug report.
As a workaround you can highlight the whole query and press Execute SQL statement.

Similar case SELECT $$'Test'' Test$$ AS text1, 'test' AS text2;:

This is the same bug I reported here.

@philosaur thank you! I'm closing this issue then as a duplicate of #22652
@dentsu-sh @Dorregaray1 you can subscribe and vote for the mentioned issue with 👍 under the issue description to promote it to a higher priority.


PR #20 broke CI

#20 has broken the CI loop.
@hfp

It seems the PR was not complete, with my Hotfix 3ef7d1d the problem is reduced to some ICX issue.

Perhaps revert the commit (and fix up) until green?

The problem is the ICX compiler (and only ICX) and then having undefined symbols when linking.
@hfp: any update on this?

@hfp can you look at this? It's still broken and reverting the patch (and hotfix) will break compatibility with libxsmm.

In fact that bug seems to be in the Makefile.Inc of the main libxsmm project. Even libxsmm itself built with icx and samples using OpenMP don't work.

BTW: some folks are running into similar issues, but the entry is not really helping
https://community.intel.com/t5/Intel-oneAPI-Data-Parallel-C/undefined-symbol-tid-addr-bid-addr/m-p/1266855
@hfp: please follow up on what needs to be fixed.

@alheinecke tests still red. :(

nothing to worried about.... see #23


Several modified cyclodextrin hosts have incorrect atom name

Several of the CD hosts have an incorrect atom name; one of the carbon atoms has a name more appropriate for a hydrogen atom. This will likely cause issues for some workflows and needs correcting.

Resolved by #42