getters mocked as simple function

Hello,
I've encountered problem while mocking some angular material module (MatAutocompleteModule in my case). In my component that implements autocomplete, I have to check if panel is opened, and if so I close it. MaAutocompleteTrigger directive has a panelOpen property that returns panel's opened status, if we look at source code, we can see that this property is a getter.
Now in my spec, this.autocompleteTrigger.panelOpen is always evaluated to true, because if I log panelOpen property, I can see that it has been mocked as a function. And so else branch is never covered. I neither can do this.autocompleteTrigger.panelOpen === true, because then if branch is never covered.
The only workaround I found is:
const trigger: MatAutocompleteTrigger = fixture.debugElement.query(By.css('input').injector.get(MatAutocompleteTrigger);
Object.defineProperty(trigger, 'panelOpen', {
  get: () => true, // or false according to the case
  configurable: true,
});

Any thoughts about it?

There's the fix for your issue: https://github.com/ike18t/ng-mocks/pull/81


Prep work for supporting Clearlinux

This PR contains a number of preparatory PRs for getting ccloudvm ready to work with Clearlinux.  It

Adds support for .xz compressed backing images.
Adds support for specifying a BIOS file when booting an instance.  This is needed for EFI boot.
Outputs an ssh connection string when the --debug flag is passed to the create command.  This allows users to ssh into an inside which has failed to create properly due to a corrupted cloud-init file.
Fix the erroneous Downloading messages which were output even when ccloudvm was using a cached file rather than a backing image.

Need to update deps.go

Coverage decreased (-1.2%) to 67.463% when pulling 86bcd0dbfb00ca072ea07c986ade992ac77e021c on markdryan:bios into 21b87f5f38823de6b079ba1ad2cbd3d3fa437042 on intel:master.

@rbradford Should be good to go now.


New Exploit

I added a new Hapara Exploit, that uses data: links, to create a Google Iframe, and Hapara doesn't check those, so it is an actual way to bypass Hapara Focus Sessions. (Hope that this helps you out!)

I do want to note that this is just a base, you can edit it any way you want, I just made a kind of starter for it, considering I found an exploit.

It may not work, I got another tho, you can use this: data:text/html;base64,PHNjcmlwdD4KdmFyIHdpbj13aW5kb3cub3BlbigiaHR0cHM6Ly9kcml2ZS5nb29nbGUuY29tIikKICAgIHdpbi5kb2N1bWVudC53cml0ZShgCiAgICA8c3R5bGU+CiAgICAgICAgKnttYXJnaW46MDtwYWRkaW5nOjA7Ym9yZGVyOm5vbmU7fQogICAgICAgIGlmcmFtZXt3aWR0aDoxMDB2dztoZWlnaHQ6MTAwdmh9CiAgICA8L3N0eWxlPgogICAgPGlmcmFtZSBpZD0iZnJhbWUiPjwvaWZyYW1lPgogICAgPHNjcmlwdD4KICAgIHZhciBmcmFtZT1kb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiZnJhbWUiKTsKICAgIGZyYW1lLnNldEF0dHJpYnV0ZSgic3JjIixwcm9tcHQoIkluc2VydCBsaW5rIHRvIGVtYmVkIDoiLCJpbnNlcnQgbGluayB0byB3ZWJzaXRlIikpCiAgICA8XC9zY3JpcHQ+CmApCiAgICB3aW5kb3cuY2xvc2UoKSAgICAKPC9zY3JpcHQ+ (Make sure to include https:// in the link.

if it says "site name Refused to connect, try using a web proxy. Sometimes the links won't open handled Iframes, and it won't work.

So how did i use this?


Take too long to load

When I press on doujin it take about 5 to 7 seconds to load

this problem does not depend on the application, manga is loaded differently at different times of the day, it depends on the site. try using VPN, sometimes it helps


Doc updates related to Flows no longer throwing

See #4610

Do not merge until https://github.com/apollographql/docs/pull/337 is merged


Use updated siphasher to avoid duplicate code and undefined behavior

This PR removes the implementation of siphash from the swc_common crate in favor of the siphasher crate which is already used transitively by dependencies.
This is done to avoid some undefined behavior in the previous implementation where a pointer that only had read access to one byte would be used to read 8 bytes.  It also lets the project use another crate with a more extensive implementation that can get bugfixes on its own.
This PR also indirectly updates the existing transitive dependency which fixes some other undefined behavior.

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

Every use of the previous siphash implementation that wrote more than a single byte caused UB.  The UB was in the load_int_le macro which was used in every code path that wrote to the hasher.  This code was in the previous implementation and the get_unchecked produces a pointer that is only valid for the byte at the passed index.
        ptr::copy_nonoverlapping(
            $buf.get_unchecked($i),
            &mut data as *mut _ as *mut u8,
            mem::size_of::<$int_ty>(),
        );

The siphasher crate received an update earlier today to remove the get_unchecked in favor of buf.as_ptr().offset($i) which produces a pointer valid for the whole slice and offsets it.


Fix Perf Test Regression

Reverting the changes from this PR updating actor code: update master w/ release-1.13 branch changes resulted in the perf tests passing again, specifically the TestActorReminderRegistrationPerformance test.
The changes involve reverting the: json serialization and api level.

/ok-to-perf

/ok-to-perf

/ok-to-perf

/ok-to-perf

/ok-to-perf

/ok-to-perf

/ok-to-perf

Just for grins I'll re-run the perf tests

/ok-to-perf

/ok-to-perf

/ok-to-perf

/ok-to-perf

Looks like TestBulkPubsubPublishGrpcPerformance might be flaky? I will re-run the perf test since actor reminder registration one is passing now

/ok-to-perf

Maybe that unit test for placement membership is flaky?

/ok-to-perf

/ok-to-perf

@cicoyle you can use /ok-to-perf actor_reminder to run just the test you're debugging, avoiding running for 3 hours which burns credit and puts out CO2 for no reason :)

/ok-to-perf actor_reminder

/ok-to-perf actor_reminder

/ok-to-perf actor_reminder

Looks like there is a panic with controller-gen that was recently introduced. This PR confirms a perf regression of ~5 for actor reminder registration - need to look into this still

/ok-to-perf

/ok-to-perf

This was already merged in, so going to close this.


Added old Jetty 7 and Jetty 8

we prefer format:
jetty/8/spoon.me
jetty/7/spoon.me
so files are always named spoon.me

Ok...will fix that.

Fixed layout

merged. Thx!


Wrong message being displayed

You have done a good job with the library. Thank you!
But I have a problem/question with the code in lines 54-56 of the MPU9250 module.
The code which you have written is:
		name = self.__whoAmI()
		if not `(name[0] == 113 | name[0] == 115 ):`
			print ("The name is wrong {0}".format(name))

So the returning value from sensor is 0x71 which is value equals 113.
If the code is the (name[0] == 113) is True; the (name[0] == 115) is False so the full condition (name[0] == 113 | name[0] == 115 ): is False. if not  gives the True value and as a results the print line is executed and in my opinion the line should not be executed (because the sensor has proper value read by __whoAmI function.
So my question is it is some mistake or I don't understand something properly?
Thank You for work, help and explanation.

Thanks for the compliment. I could not get to the issues last week as I busy with something else. I will soon try to create blogs for easy understanding and use-cases.
In your statement below
If the code is the (name[0] == 113) is True; the (name[0] == 115) is False so the full condition (name[0] == 113 | name[0] == 115 ): is False
"OR" condition will give true if even one of them is true, so even if one of the conditions name[0] == 113 or name[0] ==115 is true, it doesn't matter what the other statement is, it will just be true. The outside negation will make it false and hence the debug statements won't be printed.
table for OR

Input 1
Input 2
Output

0
0
0

0
1
1

1
0
1

1
1
1

Thanks. I agree with You and of course You are right.
I don't know why - but I had to change the OR operator from | to the Python or
So the checking name line looks:
(name[0] == 113 or name[0] == 115 )
And everything works as with the intention.
Probably this is some Python specific thing with bitwise and logical operator but I don't know how to track the difference.
BTW, thanks. You can close this topic.

Sure.


PHP 8.1 support

Hey,
As mentioned in other commands, last september PHP 8.1 compatibility has been added via:
https://github.com/predis/predis/pull/712
However there is still no release of the package in the 1.x stream. Since PHP 8.1 has been released a while ago when can we expect the actual tag?
Update
I was a bit too soon and it seems it is still not compatible:
phpfpm_1   | [Wed Dec 08 14:36:30.364115 2021] [php:notice] [pid 19] [client <IP_ADDRESS>:47686] PHP Deprecated:  Return type of Predis\\Session\\Handler::open($save_path, $session_id) should either be compatible with SessionHandlerInterface::open(string $path, string $name): bool, or the #[\\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /app/vendor/predis/predis/src/Session/Handler.php on line 68, referer:

Best

Yes there is, the fixes are included in v1.1.9.
Can you submit a fix the session handler, if you're currently experiencing that on v1.1.9?

Yes there is, the fixes are included in v1.1.9.
Can you submit a fix the session handler, if you're currently experiencing that on v1.1.9?

Will see if i can do that. But it seems it lacks test support in that case as it passes for 8.1

@tillkruss im reviewing the issue but there is already an issue with PHPUnit which has a different interface definition for 9.x which is not compatible with the existing implementation.
Since there is a 5.3 support range available i have no idea which way you guys want to go with this?
The first thing that will popup is:
root@1e6fae7cc728:/app# ./vendor/bin/phpunit
PHPUnit 9.4.4 by Sebastian Bergmann and contributors.

Class "PHPUnit_Framework_Constraint" not found
root@1e6fae7cc728:/app#

I'm not sure what you're asking. Were you able to find a PHP 8.1 related issue?
We apply a bunch of patches to PHPUnit: https://github.com/predis/predis/tree/v1.1/tests I would not try to run those local 😉

I'm not sure what you're asking. Were you able to find a PHP 8.1 related issue?
We apply a bunch of patches to PHPUnit: https://github.com/predis/predis/tree/v1.1/tests I would not try to run those local 😉

Yeah the issue is there on the session handler but i thought it would be the way to go to run the tests on 8.1 to review if all is ok after the change.


[heatmap] numerous improvements

flexibility as to how to sort X and Y axis (alpha/value, desc/asc)
option to show a legend
fixed margins, maximize real estate
allowed users to define bounds

Coverage decreased (-0.002%) to 69.117% when pulling 2feaa42616f491da1e0c77bf6ec5d91053931993 on mistercrunch:heatmap into 8223729e1ebee9fa0a1116c736037f750e8b2578 on apache:master.

Coverage decreased (-0.007%) to 69.112% when pulling 2774a5de6098c6bbd9588aa9c3652a89bf4f6798 on mistercrunch:heatmap into 8223729e1ebee9fa0a1116c736037f750e8b2578 on apache:master.

LGTM!


Allows it to function normally when in safe mode.

What feature or improvement do you think would benefit Files?
I was using this till I realized its linked to windows hypervisor and essentially is useless when that visor is down such as when in safe mode.
My desire is to have an alternative to windows explorer.
Requirements
I have no idea. I am not a coder.
Files Version
<IP_ADDRESS>
Windows Version
Windows 11 Pro
Comments
I don't know if this can be done. Since it really is a valuable importance though not sure I will even use the application if I can't use it when the system is having issues. I keep an eye on it though.

That expected, the whole point of safe mode is to remove all unnecessary programs/drivers.


make endpoints name to const defines in endpoint_gen.go

declare const for endpoint name. so you don't have to remenber the key string for endpoint when creating middleware
the sample of endpoint_gen.go before
for _, m := range mdw[“CheckOTP”] {
	eps.CheckOTPEndpoint = m(eps.CheckOTPEndpoint)
}

after
// Endpoints Name inside Service
const (
	EpCheckOTPName      = "CheckOTP"
	EpCheckPasswordName = "CheckPassword"
)

for _, m := range mdw[EpCheckOTPName] {
	eps.CheckOTPEndpoint = m(eps.CheckOTPEndpoint)
}

Coverage decreased (-0.1%) to 25.051% when pulling 5ef8f8daf793a8b646456591dfc82cdcdb1037cb on billyplus:master into 69ae4c313796525ae8defec3edf24640b726938a on kujtimiihoxha:master.


DotSpatial.Data.Rasters.GdalExtension locks image file on disk.

This issue was imported from CodePlex
mudnug wrote 2011-10-31 at 22:05
When I remove layer with tif image from map I want to use that image in another aplication. But it can not be used because "it is used by another program".
************** Exception Text **************System.OutOfMemoryException: Out of memoryw at System.Drawing.Bitmap.LockBits(Rectangle rect, ImageLockMode flags, PixelFormat format, BitmapData bitmapData) at System.Drawing.Bitmap.LockBits(Rectangle rect, ImageLockMode
flags, PixelFormat format) at DotSpatial.Data.Rasters.GdalExtension.GdalImage.CopyValuesToBitmap() in C:\DotSpatial_29eb17f63168\DotSpatial.Data.Rasters.GdalExtension\DotSpatial.Data.Rasters.GdalExtension\GdalImage.cs:line 953 at DotSpatial.Data.Rasters.GdalExtension.GdalImage.ReadRgb()
in C:\DotSpatial_29eb17f63168\DotSpatial.Data.Rasters.GdalExtension\DotSpatial.Data.Rasters.GdalExtension\GdalImage.cs:line 671 at DotSpatial.Data.Rasters.GdalExtension.GdalImage.Open() in C:\DotSpatial_29eb17f63168\DotSpatial.Data.Rasters.GdalExtension\DotSpatial.Data.Rasters.GdalExtension\GdalImage.cs:line
226 at DotSpatial.Data.ImageData.Open(String fileName) in C:\DotSpatial_29eb17f63168\DotSpatial.Data\DotSpatial.Data\ImageData.cs:line 163 at DotSpatial.Data.Rasters.GdalExtension.GdalImageProvider.OpenFile(String fileName) in C:\DotSpatial_29eb17f63168\DotSpatial.Data.Rasters.GdalExtension\DotSpatial.Data.Rasters.GdalExtension\GdalImageProvider.cs:line
256 at DotSpatial.Data.Rasters.GdalExtension.GdalImageProvider.DotSpatial.Data.IDataProvider.Open(String fileName) in C:\DotSpatial_29eb17f63168\DotSpatial.Data.Rasters.GdalExtension\DotSpatial.Data.Rasters.GdalExtension\GdalImageProvider.cs:line 164 at DotSpatial.Data.DataManager.OpenFile(String
fileName, Boolean inRam, IProgressHandler progressHandler) in C:\DotSpatial_29eb17f63168\DotSpatial.Data\DotSpatial.Data\DataManager.cs:line 408
mudnug wrote 2012-01-13 at 03:06
Setting all the Access.GA_Update requests to Access.GA_ReadOnly didn't solve this problem. I'm not sure how to determine what is causing the lock.
mudnug wrote 2012-01-17 at 22:44
It looks like there will be a lock, but it can be released by calling Close()
mudnug wrote 2012-01-19 at 00:04
I might recommend replacing the Open method in GdalRasterProvider with the following. I don't have a good way of testing this for side effects, however.
    public IRaster Open(string fileName)
    {
        IRaster result = null;
        try
        {
            using (Dataset dataset = Gdal.Open(fileName, Access.GA_ReadOnly))
            {
                if (dataset != null)
                {
                    // Single band rasters are easy, just return the band as the raster.
                    //if (dataset.RasterCount == 1)
                    //{
                    result = GetBand(fileName, dataset, dataset.GetRasterBand(1));
                    //}
                    //else if (dataset.RasterCount > 1)
                    //{
                    //    // Open the first band as a raster and ignore other bands
                    //    // TODO: make a more complicated raster structure with individual bands.
                    //    result = GetBand(fileName, dataset, dataset.GetRasterBand(1));
                    //}
                }
            }
        }
        catch (Exception ex)
        {
            throw new GdalException(ex.ToString());
        }

        return result;
    }

Not actual


rerender with conda smithy 3.3.0

Checklist

[X] Used a fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@ocefpaf @jakirkham @jhamman @scopatz check the windows failures?  also note that cmake-feedstock now skips builds for VC<14: https://github.com/conda-forge/cmake-feedstock/blob/master/recipe/meta.yaml#L14

Azure windows failures are OK ATM


AMBARI-24466 Add README.md for Ambari project

What changes were proposed in this pull request?
Added a readme.md for Apache Ambari
How was this patch tested?
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Ambari Contributing Guide before opening a pull request.

@u39kun @Jetly-Jaimin - Please review

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3570/
Test FAILed.
Test FAILured.

@shavi71 im closing this, im extended your PR (see: #2390)


Add default command options

Adds the following:

Default command options, so it is no longer necessary to specify them in a plugin
Support for 3 new properties

auto_help and help_text
help_text - Text returned when help <command> is used
auto_help - Automatically return the help text if command errors in execution

aliases
Now you can have a command with multiple aliases, for example, maybe you want the "list" command to be also accessible under "commands". Simply do (in the plugin):
command.opts = {
  aliases: ["commands"]
};

Bugged, please use next PR


Colab notebook: Where are weights?

Thanks for sharing the notebook.  Could it perhaps be documented a bit more to make it a bit easier for new users (like me) to make it work without crashing?  I can't figure out how to fill in the missing information about where to find the weights.
Below is a log of my run...
!nvidia-smi -L
GPU 0: Tesla P100-PCIE-16GB (UUID: GPU-5218d88a-592a-b7c2-d10c-ff61031ab247)

Mount your drive
Mounted at /content/drive

Install Word2Wave, import necessary packages
Cloning into 'word2wave'...
remote: Enumerating objects: 349, done.
remote: Counting objects: 100% (349/349), done.
remote: Compressing objects: 100% (311/311), done.
remote: Total 349 (delta 185), reused 81 (delta 33), pack-reused 0
Receiving objects: 100% (349/349), 1.10 MiB | 5.21 MiB/s, done.
Resolving deltas: 100% (185/185), done.
/content/word2wave
Requirement already satisfied: matplotlib>=2.2.4 in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 1)) (3.2.2)
Requirement already satisfied: numpy>=1.16.3 in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 2)) (1.19.5)
Collecting librosa==0.6.3
  Downloading librosa-0.6.3.tar.gz (1.6 MB)
     |████████████████████████████████| 1.6 MB 5.0 MB/s
Collecting pescador>=2.0.1
  Downloading pescador-2.1.0.tar.gz (20 kB)
Requirement already satisfied: torch>=1.1.0 in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 5)) (1.10.0+cu111)
Requirement already satisfied: tqdm>=4.32.1 in /usr/local/lib/python3.7/dist-packages (from -r requirements.txt (line 6)) (4.62.3)
Collecting numba==0.49.0
  Downloading numba-0.49.0-cp37-cp37m-manylinux2014_x86_64.whl (3.6 MB)
     |████████████████████████████████| 3.6 MB 36.1 MB/s
Collecting torchaudio==0.8.1
  Downloading torchaudio-0.8.1-cp37-cp37m-manylinux1_x86_64.whl (1.9 MB)
     |████████████████████████████████| 1.9 MB 64.3 MB/s
Requirement already satisfied: audioread>=2.0.0 in /usr/local/lib/python3.7/dist-packages (from librosa==0.6.3->-r requirements.txt (line 3)) (2.1.9)
Requirement already satisfied: scipy>=1.0.0 in /usr/local/lib/python3.7/dist-packages (from librosa==0.6.3->-r requirements.txt (line 3)) (1.4.1)
Requirement already satisfied: scikit-learn!=0.19.0,>=0.14.0 in /usr/local/lib/python3.7/dist-packages (from librosa==0.6.3->-r requirements.txt (line 3)) (1.0.1)
Requirement already satisfied: joblib>=0.12 in /usr/local/lib/python3.7/dist-packages (from librosa==0.6.3->-r requirements.txt (line 3)) (1.1.0)
Requirement already satisfied: decorator>=3.0.0 in /usr/local/lib/python3.7/dist-packages (from librosa==0.6.3->-r requirements.txt (line 3)) (4.4.2)
Requirement already satisfied: six>=1.3 in /usr/local/lib/python3.7/dist-packages (from librosa==0.6.3->-r requirements.txt (line 3)) (1.15.0)
Requirement already satisfied: resampy>=0.2.0 in /usr/local/lib/python3.7/dist-packages (from librosa==0.6.3->-r requirements.txt (line 3)) (0.2.2)
Requirement already satisfied: setuptools in /usr/local/lib/python3.7/dist-packages (from numba==0.49.0->-r requirements.txt (line 7)) (57.4.0)
Collecting llvmlite<=0.33.0.dev0,>=0.31.0.dev0
  Downloading llvmlite-0.32.1-cp37-cp37m-manylinux1_x86_64.whl (20.2 MB)
     |████████████████████████████████| 20.2 MB 1.3 MB/s
Collecting torch>=1.1.0
  Downloading torch-1.8.1-cp37-cp37m-manylinux1_x86_64.whl (804.1 MB)
     |████████████████████████████████| 804.1 MB 2.6 kB/s
Requirement already satisfied: typing-extensions in /usr/local/lib/python3.7/dist-packages (from torch>=1.1.0->-r requirements.txt (line 5)) (<IP_ADDRESS>)
Requirement already satisfied: python-dateutil>=2.1 in /usr/local/lib/python3.7/dist-packages (from matplotlib>=2.2.4->-r requirements.txt (line 1)) (2.8.2)
Requirement already satisfied: pyparsing!=2.0.4,!=2.1.2,!=2.1.6,>=2.0.1 in /usr/local/lib/python3.7/dist-packages (from matplotlib>=2.2.4->-r requirements.txt (line 1)) (3.0.6)
Requirement already satisfied: kiwisolver>=1.0.1 in /usr/local/lib/python3.7/dist-packages (from matplotlib>=2.2.4->-r requirements.txt (line 1)) (1.3.2)
Requirement already satisfied: cycler>=0.10 in /usr/local/lib/python3.7/dist-packages (from matplotlib>=2.2.4->-r requirements.txt (line 1)) (0.11.0)
Requirement already satisfied: pyzmq>=15.0 in /usr/local/lib/python3.7/dist-packages (from pescador>=2.0.1->-r requirements.txt (line 4)) (22.3.0)
Requirement already satisfied: threadpoolctl>=2.0.0 in /usr/local/lib/python3.7/dist-packages (from scikit-learn!=0.19.0,>=0.14.0->librosa==0.6.3->-r requirements.txt (line 3)) (3.0.0)
Building wheels for collected packages: librosa, pescador
  Building wheel for librosa (setup.py) ... done
  Created wheel for librosa: filename=librosa-0.6.3-py3-none-any.whl size=1573336 sha256=fff7ac07e9d03aa008fcf3f1f369f8acffdb27082d75ffead993dfeb62fb468d
  Stored in directory: /root/.cache/pip/wheels/de/c1/94/619fb8b04ee1f567115662d26650677ecf79bc7d8e462d21f8
  Building wheel for pescador (setup.py) ... done
  Created wheel for pescador: filename=pescador-2.1.0-py3-none-any.whl size=21104 sha256=4a7aaeaff3c65a1913ee3bad1cbd83c1c6e541790056b38a2848a31f5568f4e9
  Stored in directory: /root/.cache/pip/wheels/f0/e3/c6/32d30d5eb5292dac352d2fca4ebf393aa94e09b9b8b4b0f341
Successfully built librosa pescador
Installing collected packages: llvmlite, numba, torch, torchaudio, pescador, librosa
  Attempting uninstall: llvmlite
    Found existing installation: llvmlite 0.34.0
    Uninstalling llvmlite-0.34.0:
      Successfully uninstalled llvmlite-0.34.0
  Attempting uninstall: numba
    Found existing installation: numba 0.51.2
    Uninstalling numba-0.51.2:
      Successfully uninstalled numba-0.51.2
  Attempting uninstall: torch
    Found existing installation: torch 1.10.0+cu111
    Uninstalling torch-1.10.0+cu111:
      Successfully uninstalled torch-1.10.0+cu111
  Attempting uninstall: torchaudio
    Found existing installation: torchaudio 0.10.0+cu111
    Uninstalling torchaudio-0.10.0+cu111:
      Successfully uninstalled torchaudio-0.10.0+cu111
  Attempting uninstall: librosa
    Found existing installation: librosa 0.8.1
    Uninstalling librosa-0.8.1:
      Successfully uninstalled librosa-0.8.1
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
torchvision 0.11.1+cu111 requires torch==1.10.0, but you have torch 1.8.1 which is incompatible.
torchtext 0.11.0 requires torch==1.10.0, but you have torch 1.8.1 which is incompatible.
kapre 0.3.6 requires librosa>=0.7.2, but you have librosa 0.6.3 which is incompatible.
Successfully installed librosa-0.6.3 llvmlite-0.32.1 numba-0.49.0 pescador-2.1.0 torch-1.8.1 torchaudio-0.8.1
/usr/local/lib/python3.7/dist-packages/librosa/util/decorators.py:9: NumbaDeprecationWarning: An import was requested from a module that has moved location.
Import requested from: 'numba.decorators', please update to use 'numba.core.decorators' or pin to Numba version 0.48.0. This alias will not be present in Numba version 0.50.0.
  from numba.decorators import jit as optional_jit
/usr/local/lib/python3.7/dist-packages/librosa/util/decorators.py:9: NumbaDeprecationWarning: An import was requested from a module that has moved location.
Import of 'jit' requested from: 'numba.decorators', please update to use 'numba.core.decorators' or pin to Numba version 0.48.0. This alias will not be present in Numba version 0.50.0.
  from numba.decorators import jit as optional_jit

But then part that says...
Copy the pre-trained WaveGAN and COALA weights from drive
drive_path:  "/content/drive/<path/to/word2wave_files/>"

...it's not clear from the notebook what to enter in this string.
I see further up in the output that it seems to have installed into /content/word2wave, but when I type that into the prompt, I get
cp: cannot stat '/content/word2wavewavegan': No such file or directory
cp: cannot stat '/content/word2wavecoala': No such file or directory
mv: cannot stat '/content/word2wave/coala/coala/': No such file or directory

Looking around on Drive, I see...
!ls /content/drive

MyDrive  Shareddrives

If I just ignore the above errors and try to run the notebook with no changes, then when it comes to the part for generating with "firework", I see:
NameError                                 Traceback (most recent call last)
<ipython-input-9-cf00c4445550> in <module>()
      9 id2tag = json.load(open('/content/word2wave/coala/id2token_top_1000.json', 'rb'))
     10
---> 11 check_text_input(text)

<ipython-input-7-0cff9a5913d2> in check_text_input(text)
     28
     29 def check_text_input(text):
---> 30   _, words_in_dict, words_not_in_dict = word2wave.tokenize_text(text)
     31   if not words_in_dict:
     32       raise Exception("All the words in the text prompt are out-of-vocabulary, please try with another prompt")

NameError: name 'word2wave' is not defined

One other issue:
Generate audio from text
Clicking on "laughing" produces:
NameError: name 'widgets' is not defined

This happens because you're using widgets before you import it (which happens in the next cell below).  So the line
import ipywidgets as widgets

should be moved upward.

Hi Scott, thank you for reporting this!
I have updated the notebook and all the above issues should now be solved.
A summary of the changes:

The notebook was previously written assuming users would first train their own WaveGAN models. I realise this would be an unnecessary barrier to playing around with the notebook straight away, so a few of the models I trained can now be found in a pre-release. The notebook now has code that automatically downloads the weights from there and imports them to Colab upon running the cell, without any further input from the user. Custom WaveGAN models can still be imported via Google Drive if the weights are uploaded to a folder and the path copied into the text box in the cell:

drive_path = "" #@param {type:"string"}
if drive_path:
  !cp -r -v {drive_path} "/content/word2wave/"
else:
  !mkdir "/content/word2wave/wavegan/"
  %cd /content/word2wave/wavegan/
  !wget https://github.com/ilaria-manco/word2wave/releases/download/0.1/gan_fs_loop_3.tar
  !wget https://github.com/ilaria-manco/word2wave/releases/download/0.1/gan_fs_loop_4.tar
  !wget https://github.com/ilaria-manco/word2wave/releases/download/0.1/gan_drum.tar
  %cd /content/word2wave/

Changed check_text_input(text) to take an instance of the Word2Wave class as an additional argument. This solves the name 'word2wave' is not defined error
Moved import ipywidgets as widgets to the previous cell

I will add some further documentation to make it more intuitive as soon as I have time, but hopefully the above changes help for now. Let me know if there are any other issues.

Ilaria, thanks for your reply! I ended up really busy this week and haven't yet had a chance to try out your revised notebook.  Based on the changes you describe, I'll leave it up to you as to whether you want to keep this Issue open until I get a chance to try the notebook or close this Issue right now anyway.  Thanks!

I'll close it for now, but please feel free to open a new one if you notice any other issue. Thank you!


Make permissions dispatcher work again

This will generate PostEditFragmentPermissionsDispatcher
see https://github.com/hotchemi/PermissionsDispatcher/issues/270#issuecomment-279283213

Thanks for initiating this, I'll need to do add some more commits on top of this to work with Realm 2.x.

This is blocked by #112

As I comment before, Realm v1.2.0 won't fix this issue. I haven't tested v2.x except v2.3.1. Since I'm not familiar with Realm, I let you decide which version you guys use. Feel free to use/ditch my commit!
By the way, thanks for reporting detail issue. That helps us a lot. Also, thanks for picking our library for handling Runtime Permissions. We feel really good when apps like quill uses our library.

Fixed in #112 and #119. Thanks for the help @shiraji!


Add documentation of how halo_num_satellites and halo_num_centrals is computed

@yymao says he went looking for this in the populate_mock method and hit a dead end.

PR #521 should resolve this issue, @yymao.
Tagging @nickhand who had also requested that a tutorial on the HOD mock-making algorithm be included in the documentation.
Feedback on the new sections addressing this documentation would be great, but I understand the two of you may have moved on and that this tutorial is rather involved. If you don't have time to read the tutorial but could comment on its "discoverability", that would also be helpful. Regardless, thanks for pointing out the necessity of this section of the docs.

This tutorial looks great -- it's exactly what I had in mind for this section. Thanks for adding it!
I do have a somewhat related issue that I was wondering about while reading through the tutorial about setting random seeds when populating mocks. I've opened up #551


Tooltip on scatter plot raises error if x axis is timeseries

I have a dataset where x axis is a date (e.g. '2018-01-02') and y axis a value.
I display the scatter plot and to have x axis display properly I set options:
axis: { x: { type: 'timeseries'},..
X axis shows now correct.
When I hover above dots, expecting to see the aggregation value I get error:
VM541:1 Uncaught TypeError: Cannot read property '' of undefined.
After some debugging I see that the error comes from
return formatter(scatterData.t[series][x][y]);
on params.tooltip.format, since x contains now "Tue Jan 02 2018 00:00:00 GMT+0100 (CET)" rather than the original "2018-01-02" that scatterData.t was initialised with.
I don't think I'm doing anything wrong, so wonder if anyone can help me to determine if this is a bug.
Thanks

I assume this is using the C3 renderers?

yes, sorry, forgot to mention that.


fast_float extensions and conversion crashes

Hi,
I'm doing an image operation in extended linear p3 color space and converting the result into sRGB using the fast float extensions.  I'm getting some very odd crashes doing this that don't appear to happen when the fast float extensions are not enabled.
I've been able to stop the crashes by clamping the floating point values from 0 - 1, but that's not really appropriate for all of the image processing I'm doing.
Is it possible there are some range or safety checks that are being bypassed in the fast float extensions?

Hello and thanks for reporting.
Do you have any crash dump or reproducible sample?
I would be glad to solve the issue but not sure where to search.

Found a possible cause, which is NAN and de normal values in the input data. After commit b423949ac9abcfd4009606856eba39217b3bd9b2 it should not segfault anymore.

Thank you for being so proactive! I have not had time yet to try and pull the data/repro case out of my test harness into a simpler test case, so I just mentioned it in case it rang any bells, which it sounds like it might have.  I will work on integrating the new code and seeing if the crashes are still happening later this week.

My test designed to induce the crash is now passing after the latest github commit.  Great catch!!!
I will close this issue for now and report back if we see any other instabilities.


Allow categorising expenses

The transaction pane from solid-panes allows adding annotations to an imported bank statement
I personally want to categorise my expenses in a way that's partially generic and partially personal.
I will need to create these categories and apply them in some way that can generalise in various ways.
For instance in-shop payments depend on the shop type. Monthly direct debits depends on the
beneficiary to know if it's for instance in the "insurance" category.
The use of an annotation store makes sense.

I added an expenseCategory field now but it's still tied in to my personal data, so I didn't push it to github. I need to disentangle that. I could at least put the mapping from iDEAL-recipient / MCC code / incassantId to expenseCategory into an RDF doc on my pod. But I think I can do better. -> https://github.com/solid/money-pane/issues/13

Continued in https://solidos.solidcommunity.net/Team/2021/money-pane-project/money pane/state.ttl#Iss1610015100120

Continued in https://solidos.solidcommunity.net/Team/2021/money-pane-project/money pane/state.ttl#Iss1610015100120


Assistance on installing on CentOS 7

Hi there,
Have you (or anyone reading) successfully managed to install elastix on CentOS 7? I've been unable to find an up to date enough library that provides GLIBCXX_3.4.21 and I'm trying to avoid doing a full gcc upgrade and compile of elastix if possible.

We have not done so ourselves.
Where you able to do a compilation yourself in the meantime?

Hey! Sorry for the delay in responding. Yes, I compiled it myself. If you'd like the rpm I'm happy to provide it if it would help anyone.


[BUG] python use python -m grpc_tools.proto gen code get ModuleNotFoundError: No module named 'buf'

Description

one proto file gen go and python code. go run success, but python run get
ModuleNotFoundError: No module named 'buf'
Steps to Reproduce

First step to reproduce the bug
proto

syntax = "proto3";

package my.package;

import "google/protobuf/timestamp.proto";
import "buf/validate/validate.proto";

message Transaction {
  uint64 id = 1 [(buf.validate.field).uint64.gt = 999];
  google.protobuf.Timestamp purchase_date = 2;
  google.protobuf.Timestamp delivery_date = 3;

  string price = 4 [(buf.validate.field).cel = {
    id: "transaction.price",
    message: "price must be positive and include a valid currency symbol ($ or £)",
    expression: "(this.startswith('$') or this.startswith('£')) and float(this[1:]) > 0"
  }];

  option (buf.validate.message).cel = {
    id: "transaction.delivery_date",
    message: "delivery date must be after purchase date",
    expression: "this.delivery_date > this.purchase_date"
  };
}

Second step to reproduce the bug

go use protoc to gen code and run sucess.
python use
python3 -m grpc_tools.protoc

to gen code and run falied

Third step to reproduce the bug
... additional steps as needed

Expected Behavior
python run success.

Actual Behavior
ModuleNotFoundError: No module named 'buf'

Screenshots/Logs

Environment

Operating System:
Version:
Compiler/Toolchain:
Protobuf Compiler & Version:
Protoc-gen-validate Version:
Protovalidate Version:

Possible Solution

Additional Context

Closing as a duplicate of #146 . Please see the fix in that thread.

@dobet you will need to compile the proto source files with protoc. The source files are contained within this repo: https://github.com/bufbuild/protovalidate/tree/main/proto . Once downloaded you can use protoc to generated the python files:
python3 -m grpc_tools.protoc \
  -I . \
  -I path/to/validate/ \
  --python_out=":./gen"

Ensure the runtime matches the intended python version.


Spooky Weather

More lightning & less clear skies

Note to self: add a config to turn this off later. But for now, looks good.


⚠ Bofrost.lu has degraded performance

In 7217864, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1849 ms

Resolved: Bofrost.lu performance has improved in af922dd after 16 minutes.


[#38] [Feature] As a user, I can visit the application in different languages

#38

What happened 👀

[x] Add packages for i18n and react dependencies
[x] Translate the LoginForm
[x] Adjust tests accordingly (to map text from translations)
[x] Update tests to match changes
[x] Add tests for locals & language switch
[x] Fix broken tests... :/

Insight 📝

In order to use the useTranslation() hook, I had to switch from a class component to a function component.
⚠ An error message displayed does NOT reload when switching language. I will investigate and check if extracting the error message in a sub-component helps solve this detail. If not, I won't spend much time on it, this seems minor IMO.

Proof Of Work 📹

Design
Usage

Closing as I would start a new Repository if I take this IC again.


Add In-Page HorizontalTabView with first iteration of new design

Borrowing from the same idea which allows the vertical tabs flyout to exist, we now add the horizontal multitasking control directly onto ModernShellPage.
In the future, this change will require us to have the horizontal tab view be disabled automatically when Panes are in use.
This PR updates the design of the tabs based on user feedback. We didn't like how the titlebar and sidebar clashed so the old acrylic titlebar is gone in favor of a different drag region and content now extended directly upwards. This change, pictured below enhances how pronounced tabs are in light/dark mode while retaining our undocked design. The selected tab now has a better looking drop shadow which increases ever so slightly on hover to invite the user to manipulate the tab further.

@duke7553 Great work! LGTM

This looks great! One question though:
here the TabStrip was moved from MainPage to ModernShellPage, wasn't it better to move the Sidebar from ModernShellPage to MainPage instead? So there where no issues with the dual pane view.


Demonstrate Apollo + React

Two main aims here:

Demonstrate querying the ATLAS GraphQL endpoint:

Show how the Apollo extension for VS Code can help with syntax highlighting, autocomplete, etc:

Auto complete fields:

Validate fields:

Explore GraphQL type definitions:

Browse schema from within VS Code:

@sampjvv I don't want to get in the way of any of your forward progress on your WIP branch but if you were to do the following:
git fetch && git checkout task/apollo
cd webviews
pnpm i
pnpm exec vite

...you should be able to see a working example.
I've based this off of #2 to get the vite devserver working again.
I can help land these changes on your latest branch if you want to push it to GitHub, but otherwise I just wanted to leave some breadcrumbs for you :-)

Closed since this is just to demonstrate things.

nevermind, I got it going! thanks:)

nevermind, I got it going! thanks:)

🔥


[WIP] feat(GUI): add electron-native file-picker component

We add a file-picker written with Rendition/React. It is activated with
the ETCHER_EXPERIMENTAL_FILE_PICKER environment variable. Further
customisation can be done with the ETCHER_FILE_BROWSER_CONSTRAIN_FOLDER
variable that takes a path and allows one to constrain the file-picker to
a folder.
Depends: https://github.com/resin-io/etcher/pull/2276
Related: https://github.com/resin-io/etcher/issues/2238
Related: https://github.com/resin-io/etcher/issues/2285
Change-Type: patch
Changelog-Entry: Add electron-native file-picker component.

Filters will be in a follow-up PR.

@Shou does this support multiple mountpoints (i.e. if the source plugged in has multiple partitions)?


Cant able to locate SentryCapacitor Class

Environment
How do you use Sentry?
Sentry SaaS (sentry.io)
Steps to Reproduce
Just install @sentry/capacitor latest version
Expected Result
"@sentry/capacitor": "^0.2.0",
installed

What you thought would happen.
Actual Result
It should able to detect the Sentry Class
What actually happened. Maybe a screenshot/recording? Maybe some logs?
[]Nothing happens

@phoenix503 Did you import the class? import io.sentry.capacitor.SentryCapacitor;

Yes , i have imported the class @jennmueng
FYI: Currently i am using Capacitor v2 not v3

Closing due to inactivity, please let us know if the problem still exists.


Remove thanos_ prefix from metric

All other metrics don't have a thanos_ prefix anymore.

Closed in favor of #48


fsync() and fdatasync() check if the FD is valid

Define also the syncfs() function.
Add tests (make test).

Hi @kjn, thanks for your nice project! It's used by the build server of the Fedora project to optimize builds.
When nosync.so was installed on the build server, the Python test suite started to fail (when building the Python package). There are tests on fsync(fd) and fdatasync(fd) which require these functions to fail if the file descriptor is invalid.
I propose to call fstat() to check if the file descriptor is invalid. On Linux, you can simply call dup() which does not require I/O, but dup() doesn't fail on invalid file descriptor on FreeBSD. Comment from the Python source code (see bpo-32849):
/* dup() is faster than fstat(): fstat() can require input/output operations,
   whereas dup() doesn't. There is a low risk of EMFILE/ENFILE at Python
   startup. Problem: dup() doesn't check if the file descriptor is valid on
   some platforms.

   bpo-30225: On macOS Tiger, when stdout is redirected to a pipe and the other
   side of the pipe is closed, dup(1) succeed, whereas fstat(1, &st) fails with
   EBADF. FreeBSD has similar issue (bpo-32849).

   Only use dup() on platforms where dup() is enough to detect invalid FD in
   corner cases: on Linux and Windows (bpo-32849). */

I also took the opportunity to add syncfs() function and to add tests.

Fedora CI issue: https://pagure.io/fedora-ci/general/issue/238
Here are my two manual tests to debug the issue (before I discovered that nosync.so was installed and preloaded).
Python test:
import os
with open(__file__) as fp:
    fd = fp.fileno()
try:
    os.fsync(fd)
except OSError as exc:
    print(f"fsync(<invalid FD>): {exc!r} => OK")
else:
    print(f"fsync(<invalid FD>): success => BUG")
try:
    os.fdatasync(fd)
except OSError as exc:
    print(f"fdatasync(<invalid FD>): {exc!r} => OK")
else:
    print(f"fdatasync(<invalid FD>): success => BUG")

C test:
// Build it with:
//
// $ gcc fsync_invalid_fd.c -o fsync_invalid_fd
//
// Expected result:
//
// $ ./fsync_invalid_fd
// fsync(<invalid FD>): error 9 => OK
// fsync(<invalid FD>): error 9 => OK

#include <errno.h>
#include <fcntl.h>
#include <stdio.h>
#include <sys/stat.h>
#include <sys/types.h>
#include <unistd.h>

int main(void)
{
    int fd = open("/etc/issue", O_RDONLY);
    close(fd);

    int res = fsync(fd);
    int exitcode = 0;
    if (res == 0) {
        printf("fsync(<invalid FD>): success => BUG!\n");
        exitcode = 1;
    }
    else {
        printf("fsync(<invalid FD>): error %i => OK\n", errno);
    }

    res = fdatasync(fd);
    if (res == 0) {
        printf("fsync(<invalid FD>): success => BUG!\n");
        exitcode = 1;
    }
    else {
        printf("fsync(<invalid FD>): error %i => OK\n", errno);
    }

    return exitcode;
}


Add clarity to java sdk config descriptions around endpoints

Related to #4611
Clarifies that the OTLP endpoint refers to the address of a collector

Thank you @jaydeluca

@svrnm thank you! it has been a goal of mine to become a member but I wasn't sure at what point it was appropriate to request sponsorship


clear hdash file cache and rerun from scratch?

Seems there are some file error alerts persisting even after they've been corrected by centers.  For example, on https://hdash.website-us-east-1.linodeobjects.com/HTA10.html there are 398 validation errors.  There are multiple "links connect" errors referring to ID=JP-TRANS-2.
Looking at one of those errors “HTA10_0000_06191 references parent ID=JP-TRANS-2, but no such ID exists. [Error occurred while processing file: syn51703772 of type MassSpectrometryLevel1]." when I freshly download syn51703772 from Synapse I can't find any references JP-TRANS-2.
We thought perhaps hdash is caching files somewhere locally and this cache might need to be cleared before hdash could be run from scratch?

I don't yet have an answer on this, but I also downloaded syn51703772, and I also couldn't find JP-TRANS-2.  So, at least we see the same thing so far :-)

Ok... I have no confirmed that hdash has a meta_cache table, and I see 5 records for syn51703772.  So, could be a caching issue.

Hi Alex, this is actually not a caching issue.
Rather, we have two files:  syn39282351
and syn51703772
Both files use the same primary HTAN IDs, e.g. both files claim to be annotating HTA10_0000_06037.  This is not allowed, and it is causing an error in the validation message, e.g.
HTA10_0000_06037 references parent ID=JP_Desc_3, but no such ID exists. [Error occurred while processing file: syn51703772 of type MassSpectrometryLevel1].
because HTA10_0000_06037, hdash cannot distinguish between the two files.  But, if you look at syn39282351, you will see that HTA10_0000_06037 does reference parent ID=JP_Desc_3.
To fix this issue, I think we have to fix the meta files themselves.

This might be opening a can of worms, but I now added a duplicate Primary ID check.
See:  https://hdash.website-us-east-1.linodeobjects.com/HTA10.html
Good news is that we now have the root problem:
Primary ID HTA10_0000_06191 has already been defined in OtherAssay. [Error occurred while processing file: syn51703772 of type MassSpectrometryLevel1].
Bad news is that Stanford has lots of duplicate primary IDs.

@alexeliotlash I added this suggestion, you can see here:  https://hdash.website-us-east-1.linodeobjects.com/HTA10.html
are we good to close this issue now?

Looks good.  You can close the issue.  Thanks.

Closing!! :-)


Fix for new Fixer.IO Endpoint and API key

1.- Modified Fixer.IO endpoint to use "http://data.fixer.io/api/" instead of old "http://api.fixer.io/"
2.- Added "apiKey" parameter to Rate and Convert functions to receive new Fixer.IO API key

Hi Ryan, how are you?
Sure, I can re-do the changes (although I used Visual Studio 2017 so it's
very odd it made that change you describe), just give me a chance because
is a busy day today...
The parameterized license key is because the free license only allows 1000
hits a month, so if you're providing the key yourself it will be a bit
complicated, also this brings flexibility to the end user I think.
I'll msg you again when I have the changes.
Have a good day!
2018-07-03 10:43 GMT+02:00 Ryan Morrin<EMAIL_ADDRESS>
Hi there, thanks for the contribution! 👍
It looks like your editor has converted all the whitespace from tabs to
spaces. This makes it difficult to review your actual changes. Would you be
able to re-do them and keep spaces for consistency?
I also wonder if it's worth accessing the API key through
ConfigurationManager
https://msdn.microsoft.com/en-us/library/system.configuration.configurationmanager(v=vs.110).aspx
or similar? This will mean we only have to provide the key once and avoids
having to supply it as a parameter to every method.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rmorrin/FixerSharp/pull/9#issuecomment-402060301, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMqffjmK7i4UXvbh5wuBBRKpmIA-Xff1ks5uCy6ggaJpZM4UZO6t
.

--
Mario Alejandro Puglisi Vieira
C.I.:20.303.953
Estudiante de Ingeniería Informática
Universidad Católica Andrés Bello

Hi Ryan,
If you go to my pull request and check the option "Hide whitespace changes"
over "Diff settings" you can see all the changes I did without me having to
redo them
Tell me if this solves it for you
2018-07-03 11:11 GMT+02:00 Mario Puglisi Vieira <
<EMAIL_ADDRESS>
Hi Ryan, how are you?
Sure, I can re-do the changes (although I used Visual Studio 2017 so it's
very odd it made that change you describe), just give me a chance because
is a busy day today...
The parameterized license key is because the free license only allows 1000
hits a month, so if you're providing the key yourself it will be a bit
complicated, also this brings flexibility to the end user I think.
I'll msg you again when I have the changes.
Have a good day!
2018-07-03 10:43 GMT+02:00 Ryan Morrin<EMAIL_ADDRESS>
Hi there, thanks for the contribution! 👍
It looks like your editor has converted all the whitespace from tabs to
spaces. This makes it difficult to review your actual changes. Would you be
able to re-do them and keep spaces for consistency?
I also wonder if it's worth accessing the API key through
ConfigurationManager
https://msdn.microsoft.com/en-us/library/system.configuration.configurationmanager(v=vs.110).aspx
or similar? This will mean we only have to provide the key once and avoids
having to supply it as a parameter to every method.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rmorrin/FixerSharp/pull/9#issuecomment-402060301,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMqffjmK7i4UXvbh5wuBBRKpmIA-Xff1ks5uCy6ggaJpZM4UZO6t
.

--
Mario Alejandro Puglisi Vieira
C.I.:20.303.953
Estudiante de Ingeniería Informática
Universidad Católica Andrés Bello

--
Mario Alejandro Puglisi Vieira
C.I.:20.303.953
Estudiante de Ingeniería Informática
Universidad Católica Andrés Bello

Hi mate,
Yes this will show only the changes but I'd rather not introduce them in the first place if that makes sense? The rest of the project already uses spaces (as I believe this is the default configuration in VS2017) so opinions aside, I'd at least like to keep it consistent across all the files. Visual Studio should have the option to convert the files back for you: Edit > Advanced > Untabify Selected Lines.
Regarding the API key, I agree that we should allow the user to provide this but I think it would be more convenient if we set this in one place, rather than having it as a required parameter for every operation (this also introduces breaking changes, which I would like to avoid). I am currently investigating options of providing this via configuration parameters in the consuming application, but .NET Standard makes things a bit more interesting here.

After some consideration, I've come up with a method of providing the API key which shouldn't be too intrusive. You can see my implementation on the new-api-compatibility branch. The main difference is that the API key is only set once (e.g. in application startup) and this gets used for all subsequent requests. I think this should be okay as the API key should not be something which will change frequently (if at all) and it helps keeps compatibility with existing code:
// One-time call to SetApiKey
Fixer.SetApiKey("xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx")

// Existing calls should work in the same way as before
var converted = Fixer.Convert(Symbols.GBP, Symbols.USD, 100);

Since you've raised the initial PR here, I'm happy for you to incorporate this change or something similar into your branch if you'd like? Otherwise if you're happy (or I don't hear back!) then I can merge new-api-compatibility and get a new version published to NuGet as the library is broken in its current state.


feature/cors

PR:

Allows Cross-Origin requests to support gui

Checklist

[x] Default values for headers and methods
[x] Reduce required cor flags to two
[x] Add tests

Also we need an e2e test here. We need to run Trow with CORS on, send an OPTIONS request and see the headers come back. And also verify that they're not on non-options requests or when CORS isn't on.


Connection closing optimization

(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
1.After the (proxy -> broker) connection is closed abnormally, there is no reconnection mechanism. The client needs to re-establish the connection. (client -> proxy -> broker)
2.The active event is triggered after the connection between proxy and broker is established. The connection channel in the construction may be null. Calling write() throws an NPE.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below.
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: gaoran10:x: mingyifeimingyifei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


warn on unmount

Adds a runtime assertion to warn if the root node is being unmounted. Should help catch cases like https://github.com/yoshuawuyts/choo/issues/484. Perhaps this should be caught at the nanomorph layer, although: perhaps not. Anyway, hope this helps. Thanks!

👍 I can confirm this throws in case of the example provided in #484. Thanks.

v5.4.0 :tada:


chore: push release images to dockerhub

If I understand correctly, a user could run the Docker Hub image with docker run acornlabs/otto8-ai/otto8. Is that right?

docker run acornlabs/otto8 - afaik docker hub still doesn't support nesting the image under a second namespace like that.

they dont allow dashes, but i can get otto8.

I changed it to otto8


One time/limited use pcd argument.

Relate with #28
Motivation
We can limited number proof user can create. This is useful for avoid spam and DOS attack
Research
We can add RLN circuit to our circuit or think about another solution.

RLN is a good one, we could also add an "expiration time" similar to how Semaphore implementes "fingerprint duration"

Yes, this is the first solution I think about. We should add this feature to Anon Aadhaar v2.

This mechanism is not relevant since the new implementation in v1.0.0.
Replaced by signature timestamp as TOTP in https://github.com/anon-aadhaar/anon-aadhaar/pull/143


MB-15175 - Modify weight calculations

Jira ticket for this change
Summary
This PR should accomplish two changes to our weight calculations:

Include adjusted weights in weight ticket net weight calculations
Include PPM estimated weights in shipment estimated weight calculations

Things to look out for:

Do the new calculations make sense?
Do all usages of the functions in shipmentWeights.js make sense? Pages like MoveTaskOrder.jsx and EditPPMNetWeight.jsx will have different values in their weight calculations now for certain scenarios.
Does it make sense to export all of the functions in shipmentWeights.js? I left them public because I think their unit tests are valuable and there may be use cases to import them elsewhere in the future.

Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

You can run the front end tests with make client_test and validate that they pass.

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Messages

:book:
:link: MB-15175

  Generated by :no_entry_sign: dangerJS against bd008244ce3cc725399d2e39ff49f1a3a4ba0cfc


Wait for broker to come up before connecting via NATS

Signed-off-by: Ashish Tiwari<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

@Revolyssup Please see this failed lint check - https://github.com/meshery/meshsync/actions/runs/3146783300/jobs/5115617898#step:4:25

Thank you, @acald-creator. I'm resyncing with master now. Hopefully, the new lint checks will use the workflow updates from #157.


Trying to create a Eclipse project.

Required Info

Camera Model
R200, F200 or SR300

Firmware Version

Operating System & Version
Ubuntu

Kernel Version (Linux Only)
16.04

Build System
VS2013, XCode or Makefile

Hi all... I'm trying to create a project using Eclipse. I'm not having much luck. I've configured my project to point to the proper include directory, proper library directory, included the librealsense library. Yet, I'm getting LOTs of build errors in Eclipse. See pictures.   I was hoping someone might be able to help me isolate why I'd be getting these.
I've included various screen shots to help show you how I've configured as well as various errors I'm seeing and getting.  I have no doubt it's operator error.

Update.... Got it figured out that in the Miscellaneous settings for GCC C++ Compiler. I had to set -std-c++11 manually under the "Other Flags" section.

So, getting past the build errors, I'm still having a linker issue. Even though the librealsense lib is in /usr/local/lib and I've set that path and the librealsense library in Eclipse, I'm getting a linker error saying it can't find the library.
Any ideas why?

Found out i'm supposed to use realsense not librealsense. Compiling and linking now.

Hi @RickBlacker
Hope you managed to figure out Eclipse issues, we currently do not provide out-of-the-box solution for Eclipse developers as we do with Qt. You are welcomed to publish the steps once you succeed, it can be useful to the community.
Another way to support Eclipse developers would be through the use of CMake (we are still working on it)
Closing the issue for now, feel free to reopen and update on your findings.


Better Detection of Daemon Errors

Currently the UI does not recognise when the daemon is in Safe Mode, and still allows normal interaction. This needs to be exposed by the 0MQ API and then rendered as completely blocking in the client. (It's not helpful for the client to have any functionality in this case; displayed data will be incorrect.)

We should probably try to detect any other anomalous conditions as well.

Blocking added in Safe Mode. the api status method is the only method still allowed; a safeMode key has been added here also. If other anomalous conditions come up we can handle them as they come.


correct_inheritence_of_vendor_when_expanding

The following changes are proposed:

Fix vendor inheritance in presets. Without this change, vendor fields are inherited only when the child vendor is undefined.

@AdamBartosCodasip We recently made some refactors and changes to the expanding of presets, could you re-assess this PR to make sure, 1) this is still necessary, and 2) whether this still does what you expect?

This is not needed anymore. Thanks.

This is not needed anymore. Thanks.


SkyWalking log clear

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Hello, may I ask if SkyWalking log cleanup is configured? Automatic cleanup discovery does not work.

We don't have this if someone wants to contribute, welcome.

ok, got it~

Hello，does SkyWalking have a configuration to turn off log printing? Now print a lot of logs and fill up the memory, including the agent.

Our document has included that.

Thanks~

Duplicate with #2633


[PROBLEM] 233. Number of Digit One

Difficulty
Hard
Problem Description
Given an integer n, count the total number of digit 1 appearing in all non-negative integers less than or equal to n.
Example 1:

Input: n = 13
Output: 6

Example 2:

Input: n = 0
Output: 0

Constraints:
0 <= n <= 109
Link
https://leetcode.com/problems/number-of-digit-one/

Can you please assign this issue to me @godkingjay?


Double-backslash is collapsed into one

OS version: [e.g. Win 10, Mac Catalina] Fedora Linux
Editor: [e.g. vscode, intelij, sublime] VS Code
Editor version:	1.57.1
Programming language: [e.g. js, ts, c++]	C
TabNine extension version: 3.4.12
Engine version (could be located within Tabnine Hub): 3.5.14-32

Issue Details:
When completing an AI suggestion with double backslash such as:
int sawEsc = input->data[0] == '\\';

the backslashes are reduced to one. I assume this is due to some improper escape sequence handling somewhere.
gz#3121

Hi, thanks for pointing this out to us! It was added to our backlog, and we will fix it.

is there any work around to this issue ?


Speaker diarization ValueError

I run simple test on small audio with 2 speakers.
from pyAudioAnalysis import audioSegmentation as AS AS.speaker_diarization(filename='data/test_2.wav', n_speakers=2, plot_res=True)
It give me error
ValueError: Found array with 0 feature(s) (shape=(40, 0)) while a minimum of 1 is required.
Can anyone help me with this? Also I don't know requirements to audio. Maybe problem is in this.

I got a ValueError: n_components cannot be larger than min(n_features, n_classes - 1). I resolved it by adding lda_dim=0 to the speaker_diarization() call.


question: w3c credentials v2

Do you support "https://www.w3.org/2018/credentials/v2"?

We are following the WG for https://www.w3.org/TR/vc-data-model-2.0/
As far as what we have seen up to now they are loosen the specs to accept different VC format as JWT, CBOR and SD-JWT and anoncreds with BBS+ signature. So yes we can say atht we support

I realized that my question was stupid when I looked at the release of the version 2... I asked because I wanted to know what I need to follow to be compatible with altme.
I would be happy about a review/tuning of our credentials that you offered:
https://github.com/ASCS-eV/credentials


New version: glzr-io.glazewm version 2.1.1

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.5 schema?

Resolves #136300

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


Added Depth Map to Realsense Camera

The realsense.py file now contains methods to get the depth image and depth map of the device using the pyrealsense2 wrapper calls.

A few things we need to address:
Depth-color field of view differences
Some Realsense cameras have a bigger depth FoV than color FoV. Luckily it seems to be pretty easy to align depth and color in that case (see align-depth2color.py). I think we should do that because it's what most users will want. After aligning the depth with color images, the intrinsics also become the same, so then we could remove self._depth_intrinsics_matrix.
Depth map and image datatypes
The realsense SDK seems to return depth values as rs.format.z16 which become np.uint16 through np.asanyarray(). This is also what the methods in the current PR seem to return. However in camera interface we promise/expect 32-bit floats from get_depth_map() and uint8 from get_depth_image(), so this PR should be made consistent with that. We should also try to do the distance -> grayscale conversion ("normalization") similar to the ZED SDK for the depth image.
Testing
We should add the tests from test_hw.py to this file. However we can't use manual_test_stereo_rgbd_camera() because we the realsenses only have a single color camera.
Additionally, maybe we should add datatype checks to the manual_test functions?

For reference, this is what the depth images from the ZED look like:

there you go, the remarks have been addressed now :)

The current depth images maps have large black regions (missing values), do we want to fill these in/do some smoothing like ZED? Does the realsense SDK have this functionality?

This is a screenshot from the Realsense viewer with hole filling enabled. I'm not sure if I like it, but it's closer to the default for the ZEDs.

I cleaned up some code and enabled several filters for the depth images/maps. Below is an example of what they look like right now. For me this PR is ready to merge.


[Release 1v1 [RANK] Canyon of the Dead v1] Player is unable to see terrain at all map borders (bottom)

Player is unable to see terrain at all map borders (bottom)

Please explain.

Fixed with https://github.com/xezon/GeneralsRankedMaps/commit/3763eb1c1c1fa7771faa4e4f90c9833947a8b621


Fix links to example CLI inputs

Broken links on:

https://neurobagel.org/dictionaries/

also, move tip box containing examples further up. Actually, the contents should probably be moved out of the tip box in to its own section called "Example data dictionary" or something. This section could come before or after the "Phenotypic attributes" section

https://neurobagel.org/graph_data/

take a look at https://github.com/marketplace/actions/check-links-with-linkcheck, also @Remi-Gau might have some advice on checking for dead links in the docs, I know that's one of the CI jobs for BIDS.

Yup there are a couple of tools out there to check links in MD documents.
I can set it up if you want.

here is an example: https://github.com/bids-standard/bids-website/blob/gh-pages/.github/workflows/check_md_links.yml
and the associated config:
https://github.com/bids-standard/bids-website/blob/gh-pages/md_link_check_config.json

Thanks @Remi-Gau!

@alyssadai

also, move tip box containing examples further up. Actually, the contents should probably be moved out of the tip box in to its own section called "Example data dictionary" or something. This section could come before or after the "Phenotypic attributes" section

I think that text is too small for section, how about we move it above phenotypic attributes and put it in an info box instead of a tip box?

Hey @rmanaem, sure, we can leave it in an info box.
The goal would be to make the examples super easy to find on the page, so -

how about we move it above phenotypic attributes

yes, I think it should be in the Overview section, maybe under the introductory sentence?


Feature: Bootstrap pkgin on NetBSD

Installing pkgin should be as simple as /usr/sbin/pkg_add pkgin.
        let privilege_provider =
            utilities::get_privilege_provider(&contexts).unwrap_or_else(|| "sudo".to_string());
        vec![Step {
            atom: Box::new(Exec {
                command: String::from("pkg_add"),
                arguments: vec![
                    String::from("pkgin"),
                ],
                privileged: true,
                privilege_provider: privilege_provider.clone(),
                ..Default::default()
            }),
            initializers: vec![],
            finalizers: vec![],
        }]

Yea, this could be done in the bootstrap method. The only reason I've never gotten around to it was because of some of the configuration before that.
# From https://www.netbsd.org/docs/pkgsrc/using.html
# PATH="/usr/pkg/sbin:/usr/pkg/bin:$PATH"
# PKG_PATH="https://cdn.NetBSD.org/pub/pkgsrc/packages"
# PKG_PATH="$PKG_PATH/OPSYS/ARCH/VERSIONS/All/"
# export PATH PKG_PATH
# pkg_add pkgin

That is how I've always done it when setting up a new NetBSD system.

If you want to implement the bootstrap method for the package provider, I can help you with that. Otherwise, I can do it.

Thanks @martintc, if you could take the lead, that'd be great.

I think to allow setting environment variables during bootstrap, with #515 , we should be able to accomplish it.


Luggage restrictions

This information, and much more, can be specified in the notes section for a shuttle, which was added in e09fe30.

Look at your comment, this was already fixed.


fixed failed unit test because of new handleOutgoingMessage() in IOFSwitchManager

Fixed one failed unit test (see #548)

After fixing that particular test, all unit tests are now running fine when building with maven.

@oxisto Thanks for following up on this!


Use plugin in Obsidian from mobile devices

Problem
Thank you very much for this great plugin. I like the amount of formulas and functions that the Excel format brings, but I was wondering if it could also be made compatible with the Obsidian android app, currently it does not even allow you to load the plugin
Solution
Plugin compatibility on Obsidian Android
Additional details

I agree. This is the deciding factor that would change everything colossally and I could use this plugin for counts within my notes

We support the Android app in this version but sadly maybe it can not adapt well to mobile devices because the adaptive function is not supported by univer at the initial stage. Hoping this plugin can help u better ur life.


Use sync icons from Material to reduce toolbox clutter

In effort to reduce toolbox cluttering, here a change to use Material Icons font's icons for Sync plugin.
See screenshot:

Same can be done for other common plugins like bookmark, drawtools...

I like the idea of cleaning up the toolbox.  This was never merged.  Does anyone think its a good idea to do similar with other things?  I could probably take a crack at it.

It's not faction biased, it's just reusing the visual common sense to report disabled (gray), error (red), or ok (green) sync status.
Others icons without live status indication, like bookmarks icon, should keep the default toolbox color (yellow).

Same can be done for other common plugins like bookmark, drawtools...

Imagine there are 3 plugins with icons. Could we avoid triple including the same stylesheet in the head?

I suggest not import heavy third-party Google icons, which slows down the IITC start, but to import only the necessary Google Material icons in svg format, as I started here:
https://github.com/IITC-CE/ingress-intel-total-conversion/commit/59cc3126bd85f5c9e298e94517384b34c8458d65

@modos189
Really heavy?
They say that it's only 42KB for 900+ icons.

Imagine there are 3 plugins with icons. Could we avoid triple including the same stylesheet in the head?

First, if it's exactly the same URL, the cached content will be reused, not triple downloaded.
But, agreed, it will be triple evaluated, which is bad.
We can mitigate this by assign an id to check if it's already there:
if ( $( "#material-icons-stylesheet" ).length == 0 ) {
  // add Google Material icons.
  $("head").append(
    '<link id="material-icons-stylesheet" rel="stylesheet" href="https://fonts.googleapis.com/icon?family=Material+Icons">'
  );
}


"Setting up carbon_home" image is misleading

Description:
Change the image in https://is.docs.wso2.com/en/latest/setup/installing-as-a-windows-service/#setting-up-carbon_home into

If the variable name is set to CARBON_HOME instead of carbon_home,
java.lang.ClassNotFoundException: org.wso2.carbon.bootstrap.Bootstrap error will be getting when starting the wrapper in console mode.
Affected Product Version:
IS: WSO2-IS- 5.10.0
OS: Windows 10

Closing this as it has been fixed with https://github.com/wso2/docs-is/pull/2165


CCM LBaaS: PortRangeMin/PortRangeMax and ICMP types

/kind bug
What happened:
Looks like there is a typo and PortRangeMin PortRangeMax were mixed up: https://github.com/kubernetes/cloud-provider-openstack/blob/80d14660d6d7756a7d3f2f832c76cc3e3ab8d583/pkg/cloudprovider/providers/openstack/openstack_loadbalancer.go#L1446..L1447
Besides gophercloud doesn't allow to set 0 (ICMP type 0, Echo Reply)[https://www.iana.org/assignments/icmp-parameters/icmp-parameters.xhtml#icmp-parameters-types]
What you expected to happen:
gophercloud lib should be fixed and PortRangeMin PortRangeMax have to be set correspondingly.
How to reproduce it (as minimally and precisely as possible):
n/a
Anything else we need to know?:
https://github.com/gophercloud/gophercloud/issues/1775
Environment:

openstack-cloud-controller-manager version: 1.16 master

ICMP security groups were introduced in https://github.com/kubernetes/kubernetes/pull/31921 and it looks like a typo. However it is not clear why icmpv6 has different min/max types rather icmpv4.
/cc @lingxiankong @grahamhayes

/remove-lifecycle rotten

/remove-lifecycle stale


How to add CommandItem?

In my case,
sending get  attribute single will get will get "incorrect data" RR,
I compared this with other Python projects, and the packets sent out from them were successful. The differences between the two are shown in the two pictures below. In this case, there is one less CommandItemNull in the CommandItem. Could you guide me on how to add and test this?

Perhaps the problem might lie in the incorrect calculation of length. As shown in the picture below, the correct value should be 24.

That length 20 is probably right.
Because length should not include command id and length self.
Command Id (0x006F): 2 Bytes
Length: 2 Bytes
So, length be 20 subtracted 4 bytes from total data 24 bytes.
I pushed a new commit e5d42d51f78ddb74d58e7d8ab2a520d48ec76b55
It commit mean you can add another CommandItem before exchange SendRRData.
https://github.com/gisellevonbingen/Giselle.Net.EtherNetIP/blob/e5d42d51f78ddb74d58e7d8ab2a520d48ec76b55/Giselle.Net.EtherNetIP/ENIP/ENIPCodec.cs#L147-L149
Try upper code like to below and test.
public DataProcessor GetAttribute(Stream stream, AttributePath path) => this.ExchangeSendRRData(stream,
	this.CIPCodec.HandleGetAttribute, // This is response callback
	// Array of CommandItem for send
   new CommandItemNull(), this.CIPCodec.CreateGetAttribute(path));

I'm not currently in a situation where I can test, sorry.

else.. can you send screenshot showing length when success?

Success! Additionally, the link to the program I used for packet comparison is as follows:
https://github.com/xl7dev/ICSecurity/blob/master/ethernetip.py

And now continue debug，Forward Open is failed, error package:

the OK package:

The image provided above is incorrect; it's the one for successful return. The correct one (python successful transmission) is shown below.

Fixed an issue where ForwardOpen SendRRData wasn't passing the Originator's UDP port number right now. 45eca2e321fb01bd2d9c5152dd2d00eb179377b2
Try pull and retry.
If that doesn't work, I'll do a deep analysis of the data, but it will take some time.

public ForwardOpenResult ForwardOpen(Stream stream, ForwardOpenOptions options) => this.ExchangeSendRRData(stream,
response => this.CIPCodec.HandleForwardOpen(response, options),
new CommandItemNull(), this.CIPCodec.CreateForwardOpen(options));
-> cannot add new CommandItemNull() after new commit, and the CommandItemNull is necessary.

Oops, i miss it.
In this code,
https://github.com/gisellevonbingen/Giselle.Net.EtherNetIP/blob/45eca2e321fb01bd2d9c5152dd2d00eb179377b2/Giselle.Net.EtherNetIP/CIP/CIPCodec.cs#L54-L62
Try edit like this.
public IEnumerable<CommandItem> CreateForwardOpen(ForwardOpenOptions options)
{
	// Add this line
	yield return new CommandItemNull();

	var udRequest = new CommandItemUnconnectedDataRequest();

Same error, CS1660, cannot convert Lambda to CommandItem

Oops, i forgot it.
In this code,
https://github.com/gisellevonbingen/Giselle.Net.EtherNetIP/blob/45eca2e321fb01bd2d9c5152dd2d00eb179377b2/Giselle.Net.EtherNetIP/CIP/CIPCodec.cs#L54-L62
Try edit like this.
public IEnumerable<CommandItem> CreateForwardOpen(ForwardOpenOptions options)
{
	// Add this line
	yield return new CommandItemNull();

	var udRequest = new CommandItemUnconnectedDataRequest();

'yield return new CommandItemNull();' is will instead of it.
Remove 'new CommandItemNull(), ' in pictur

Currently everything is OK, but in my case, ConfigData must be added to ForwardOpen. Below is the Python implementation code:
path = struct.pack(">I",0x34040000) + struct.pack("I",0) +
struct.pack(">I",0x00002004) + struct.pack("B",0x24) + struct.pack("B",configinst) +
struct.pack("B",0x2c) + struct.pack("B",outputinst) + struct.pack("B",0x2c) +
struct.pack("B",inputinst)
plen = int(len(path) / 2)
if configData != None:
# 0x80 = simple data segment, with length in words => max 512 bytes of data
if len(configData) > 512:
return 1
path += struct.pack("BB", 0x80, int(len(configData)/2) )
path += configData
plen = int(len(path) / 2)
how should I correspondingly add it to project?
P.S. The Configuration Assembly ID should be set in this location of the CreateForwardOpen method:
.....
var connectionPath = new EPath
{
EPathSegmentLogical.FromClassID(options.ClassID),
EPathSegmentLogical.FromInstanceID(0x66)    // Configuration Assembly ID is 0x66
};
......

I know python code created what data, but I don't understand why it works.
Can you take a screenshot that from wireshark?

My BALLUFF EtherNet/IP Scanner for IO-Link (referred to by BALLUFF as EtherNet/IP IO-LINK Master by EIP) needs to go through a configuration process. Only after this, the TO/OT UDP data will carry information, otherwise, its content is empty. However, other communications are currently functioning normally. If I first connect it to a PLC (which has an EDS that can be viewed), and then go through the PLC's configuration process, the data can be normally obtained on the C# program side. I plan to capture packets on the PLC equipment, but I currently don't have a port mirroring switch, so you'll need to wait for me for a bit.

The InstanceID of the Assembly you use for ForwardOpen is 0x64, 0x65?

yes, just like following code:
// Reserved
reqProcessor.WriteByte(0x30);
reqProcessor.WriteByte(0x30);
reqProcessor.WriteByte(0x30);
        reqProcessor.WriteUInt(options.O_T_Assembly.RequestPacketRate);
        reqProcessor.WriteUShort(options.O_T_Assembly.Flags);
        reqProcessor.WriteUInt(options.T_O_Assembly.RequestPacketRate);
        reqProcessor.WriteUShort(options.T_O_Assembly.Flags);

        // Transport Type/Trigger
        reqProcessor.WriteByte(0x01);

        var connectionPath = new EPath
        {
            EPathSegmentLogical.FromClassID(options.ClassID),
            EPathSegmentLogical.FromInstanceID(0x66)    // kenghua: Configuration Assembly ID
        };

        if (options.O_T_Assembly.ConnectionType != ConnectionType.Null)
        {
            connectionPath.Add(EPathSegmentLogical.FromConnectionPointID(options.O_T_Assembly.InstanceID));
        }

        if (options.T_O_Assembly.ConnectionType != ConnectionType.Null)
        {
            connectionPath.Add(EPathSegmentLogical.FromConnectionPointID(options.T_O_Assembly.InstanceID));
        }

And all setting and data will define in EDS that PLC will be automatically configured, so there is no need for manual setup like we do.

I extended EPathSegmentLogical.cs by adding SimpleDataSegment = 0x80, and handling the value as byte[] (I added byte[] ValueByteArray). I also modified the WriteValue method to write byte[] values. Everything went well, and I successfully configured my EIP Scanner through the Configuration Assembly. Although the specification mentioned this part, it was quite vague, and few devices use this for configuration (usually, Explicit Messaging is used for configuration). In any case, I'm extremely grateful to gisellevonbingen; you are truly amazing bro!

Already complete? Wow.. you genius..
I been had push a new commit 818c125fb6f0a7ebfeedfc2f59412a62db3b209c just right now.
The code changes were tested on my device(ControlTechniques Drive), and I confirmed that compatibility was maintained.
You just need to add the code as below.
var connectionPath = new EPath
{
	EPathSegmentLogical.FromClassID(options.ClassID),
	EPathSegmentLogical.FromInstanceID(0x01)
};

if (options.O_T_Assembly.ConnectionType != ConnectionType.Null)
{
	connectionPath.Add(EPathSegmentLogical.FromConnectionPointID(options.O_T_Assembly.InstanceID));
}

if (options.T_O_Assembly.ConnectionType != ConnectionType.Null)
{
	connectionPath.Add(EPathSegmentLogical.FromConnectionPointID(options.T_O_Assembly.InstanceID));
}

// Add this line
connectionPath.Add(new EPathSegmentData(KnownEPathDataSegmentType.Simple, new byte[] { your config data }));

connectionPath.Write(reqProcessor);

"Simple Data Segment" in Connection Path type is 0x80

Yes it is, in byte.

in this picture, upper 3 bits mean segment type. => 0b100
and remaining 5 bits are data type. => 0b00000
They can be merge to 0x80.

Wait.. i seen typo...

Now i think it should work. c72f7313453d28eec745b0c2d620aa6bc485e331

please fix bug:
public void WriteValue(DataProcessor processor) method:
processor.WriteByte((byte)payload.Length); ---->
processor.WriteByte((byte)(payload.Length / 2));    // kenghua: the first byte represents the data length, calculated in words. 2bytes = 1word.
AND...
Do you have any suggestions for establishing connections with multiple machines? (All connections cannot bind same local port 2222)

How length does pass when length is odd?
ceiling? flooring?
Once treated as a ceil. 7a835554eac55bf0e672fe36f575c6a759b4785c
I have plan for port change.
I'll test it a bit more and let you know.

Can change PC port with T_O_UDPPort
https://github.com/gisellevonbingen/Giselle.Net.EtherNetIP/blob/34acf84e0376dfc0849f9ac66c6a51def6ab7688/Giselle.Net.EtherNetIP.Test/ImplicitMessaging.cs#L90-L110
When change O_T_UDPPort to 1234
var openOptions = new ForwardOpenOptions();
openOptions.T_O_UDPPort = 1234; // Support alternate port, Default is 2222

A few questions are listed below for discussion, and I suggest you can commit back to the main branch, thank you:

This is the correct way, payload.Length don't need add 1, and the specifications require providing the length in Word, so there should not be definitions with an odd number of byte lengths:
public void WriteValue(DataProcessor processor)
{
var payload = this.Payload;
processor.WriteByte((byte)(payload.Length / 2));
processor.WriteBytes(payload);
// Pad to words
processor.WriteBytes(new byte[payload.Length % 2]);
}

In ENIPCodec.cs, I suggest adding CommandItemNull according to our previous discussion, because when I capture PLC communication packets, they do the same. Could you test the compatibility of doing this on your equipment? If I don't add it on my side, the device will directly indicate a format error and refuse to OpenForward:
public DataProcessor GetAttribute(Stream stream, AttributePath path) => this.ExchangeSendRRData(stream,
this.CIPCodec.HandleGetAttribute,
new CommandItemNull(), this.CIPCodec.CreateGetAttribute(path));
 public byte SetAttribute(Stream stream, AttributePath path, byte[] values) => this.ExchangeSendRRData(stream,
     this.CIPCodec.HandleSetAttribute,
     new CommandItemNull(), this.CIPCodec.CreateSetAttribute(path, values));

 public ForwardOpenResult ForwardOpen(Stream stream, ForwardOpenOptions options) => this.ExchangeSendRRData(stream,
     response => this.CIPCodec.HandleForwardOpen(response, options),
     this.CIPCodec.CreateForwardOpen(options));

 public ForwardCloseResult ForwardClose(Stream stream, ForwardCloseOptions options) => this.ExchangeSendRRData(stream,
     response => this.CIPCodec.HandleForwardClose(response, options),
     new CommandItemNull(), this.CIPCodec.CreateForwardClose(options));

The issue with IPV6 exists on .net core, and I have already upgraded to use .net core. Along the line-map of Microsoft's .Net, I also suggest you do the same. I noticed that you have specifically implemented IPV4 enforcement for communication, but there are still some places that have been missed, leading to the capture of IPV6 internal IP addresses. The following areas need to be modified (I apologize, I kind of forgot what changes I've made), just like:

options.LocalAddress = ((IPEndPoint)this.TcpClient.Client.LocalEndPoint).Address; -> options.LocalAddress = ((IPEndPoint)this.TcpClient.Client.LocalEndPoint).Address.MapToIPv4();
search all "TcpClient(" and "UDPClient{"，then add: new TcpClient(AddressFamily.InterNetwork) and new UdpClient(AddressFamily.InterNetwork)

This is to prevent users from accidentally entering odd-length arrays.
If sure to simple data's data type is word, It might be better to declare it as ushort[] instead of byte[].

OK i will add it if it works on my test.

my device doesn't support IPv6, so maybe i can't test it.
umm.. i will do that for now.

For users, byte[] will be more convenient, so I hold off for now.

Looks good work on my device, It added from 940f8dd9337d6297d1c98ea5f9ced51ee5f6e6af

I think can use IPv6 at PC internally.
So... added map LocalEndPoint to IPv4 just before IPAddress be passed to device in here. 8c4232ca961839ce4e59ae306456f30e91c97a51
https://github.com/gisellevonbingen/Giselle.Net.EtherNetIP/blob/940f8dd9337d6297d1c98ea5f9ced51ee5f6e6af/Giselle.Net.EtherNetIP/CIP/CIPCodec.cs#L118-L126


Onboarding Contextual Tutorial Rendering

Task/Issue URL: https://app.asana.com/0/72649045549333/1207701578629995/f
CC: @SabrinaTardio
Description:
This PR changes the rendering of the onboarding contextual messages based on the cohorts.
For cohort ma show the Old onboarding overlays
For cohort mb show the new Dax Contextual Dialogs.
I created a ContextualOnboardingPresenter and a ContextualDaxDialogsFactory.
The ContextualDaxDialogsFactory responsibility is to create specific contextual dialogs based on the BrowsingSpec. In this PR, only one type is created. In the next PRs different types will be returned.
The presenter is in charge of:

Perform the DaxDialog segue and show the messages as full-screen overlay if the cohort is ma
Ask the ContextualDaxDialogsFactory for the correct dialog, add it as a child and render it as part of the Tab screen.

Video
https://github.com/duckduckgo/iOS/assets/1089358/3bfb3955-cb8d-4454-b541-eb5ba0ad29d9
Steps to test this PR:
SCENARIO 1 - Old Onboarding
Prerequisites: Delete DuckDuckGo App from Simulator.

DuckDuckGo Scheme -> Run -> Arguments -> Environment Variables -> VARIANT -> ma
Add temporary newInstallCompletion(self) at line 141 in DefaultVariantManager
Launch the App.
Complete the Onboarding Intro.
The New Tab Page should show the old Dax Dialog.
Follow the instructions and input a website (e.g. “facebook.com”) into the address bar.
Expected Result: An overlay informing the user about the trackers blocked should appear on top of the screen.
Dismiss the overlay.
Tap the fire Button.
Expected Result: The action sheet to clear the data should appear on the screen along with an overlay informing the user about the fire button.

SCENARIO 2 - New Onboarding
Prerequisites: Delete DuckDuckGo App from Simulator.

DuckDuckGo Scheme -> Run -> Arguments -> Environment Variables -> VARIANT -> mb
Add temporary newInstallCompletion(self) at line 141 in DefaultVariantManager
Launch the App
Complete the Onboarding Intro.
The New Tab Page should show the old Dax Dialog.
Follow the instructions and input a website (e.g. “facebook.com”) into the address bar.
Expected Result: The Web page should shift down and the Dax Contextual Dialog should appear on the screen.
Tap the action button of the Dax Contextual Dialog.
Expected Result: The Dax Contextual Dialog should disappear and the web page shift up.
Tap the fire Button.
Expected Result: The action sheet to clear the data should appear on screen but it should not show any overlay.

Definition of Done (Internal Only):

[ ] Does this PR satisfy our Definition of Done?

Copy Testing:

[ ] Use of correct apostrophes in new copy, ie ’ rather than ’

Orientation Testing:

[ ] Portrait
[ ] Landscape

Device Testing:

[ ] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[ ] iPhone 14 Pro
[ ] iPad

OS Testing:

[ ] iOS 15
[ ] iOS 16
[ ] iOS 17

Theme Testing:

[ ] Light theme
[ ] Dark theme

—
Internal references:
Software Engineering Expectations
Technical Design Template

This works as advertised! :)
However, in the onboarding flow on iPad mini in landscape Dax is cut off on the welcome view and the bottom of the dax dialog is lost on the browser comparison.
I've asked on the ship review for Figma updates showing different screen sizes and orientations so that we know what scale to expect, but if you don't get any feedback by the time you're back on it, please just do your best to get it looking reasonable :)

This is now rebased on top of main contextual feature branch and contains the fixes for iPad mini in landscape


RTMP source not works , 2. udp multicast output <IP_ADDRESS>:2226

Describe the feature

when i try to give RTMP source always gives invalid body size..
how can i get udp multicast output e.g <IP_ADDRESS>:2226

This is not a feature request. Please follow instructions in order to ask questions or fill bug reports.


[flash_ctrl] flash_ctrl_rand_ops test fails due to incorrect modeling

Hi,
I noticed that the rand_ops test is failing in the OS due to incorrect modeling of bank erase in the flash_ctrl scoreboard.
In the spec it states that whenever a bank erase is initiated to the info partition, ALL of the 3 info partitions should be erased, link below:
https://docs.opentitan.org/hw/ip/flash_ctrl/doc/index.html#:~:text=If info partition is selected%2C all data in the data partition is erased AND all data in the info partitions (including all info types) is also erased.
However, the scoreboard is only modeling this operation to delete only the data and info0 partition:
https://github.com/lowRISC/opentitan/blob/bf25d8b4160c6a888933d3a224bac79851537a31/hw/ip/flash_ctrl/dv/env/flash_ctrl_scoreboard.sv#L373-L383
In addition there is a printing that should be also corrected, since it's assuming that only the data partition can be deleted. Note that it states "scb_flash_data" which is the data partition scoreboard model and it should allow selection of multiple partitions.
https://github.com/lowRISC/opentitan/blob/bf25d8b4160c6a888933d3a224bac79851537a31/hw/ip/flash_ctrl/dv/env/flash_ctrl_scoreboard.sv#L672-L673

@jdonjdon has this been fixed in the mean time?

@jdonjdon has this been fixed in the mean time?

Yes in here(#14442)

Cool, thanks for confirming!


[BUG] GPU plans are sometimes not generated unnecessarily because of a non-UTC timezone

Describe the bug
The planner sometimes fails to generate a GPU plan because the timezone being set to a non-UTC. However, this does not seem always necessary.
Steps/Code to reproduce bug
Seq(("a", "a"),("b", "b")).toDF("k", "v").createOrReplaceTempView("tmp")
spark.sql("""select k, first(v) col2 from tmp group by k;""").show()

...

24/08/21 11:49:01 WARN GpuOverrides:
!Exec <SortAggregateExec> cannot run on GPU because not all expressions can be replaced
  @Expression <AttributeReference> k#7 could run on GPU
  @Expression <AggregateExpression> first(v#8, false) could run on GPU
    @Expression <First> first(v#8)() could run on GPU
      @Expression <AttributeReference> v#8 could run on GPU
  @Expression <AttributeReference> first(v#8)()#91 could run on GPU
  @Expression <Alias> toprettystring(k#7, Some(America/Los_Angeles)) AS toprettystring(k)#97 could run on GPU
    !Expression <ToPrettyString> toprettystring(k#7, Some(America/Los_Angeles)) cannot run on GPU because class org.apache.spark.sql.catalyst.expressions.ToPrettyString is not supported with timezone settings: (JVM: America/Los_Angeles, session: America/Los_Angeles). Set both of the timezones to UTC to enable class org.apache.spark.sql.catalyst.expressions.ToPrettyString support
      @Expression <AttributeReference> k#7 could run on GPU
  @Expression <Alias> toprettystring(first(v#8)()#91, Some(America/Los_Angeles)) AS toprettystring(col2)#98 could run on GPU
    !Expression <ToPrettyString> toprettystring(first(v#8)()#91, Some(America/Los_Angeles)) cannot run on GPU because class org.apache.spark.sql.catalyst.expressions.ToPrettyString is not supported with timezone settings: (JVM: America/Los_Angeles, session: America/Los_Angeles). Set both of the timezones to UTC to enable class org.apache.spark.sql.catalyst.expressions.ToPrettyString support
      @Expression <AttributeReference> first(v#8)()#91 could run on GPU

The query above fails to produce a GPU plan because of the expression toprettystring(first(v#8)()#91, Some(America/Los_Angeles)).
Expected behavior
The query should be able to run on GPU when it should not be impacted by the timezone setting.

This impacts customers not running in UTC.


Added disconnect timeout for client.

The interface looks good like this. It may take a while before I am able to test the code. But I see you added a unit test to test the functionality.

I do have one remark, but maybe this is related to other code as well:
Why use size_t as time and use milliseconds as base?
Why not use 'boost::posix_time::time_duration' and then the user can call:
disconnect(boost::posix_time::milliseconds(1000))

or:
disconnect(boost::posix_time::seconds(1))

That way there is no unclarity about the interface: the user can specify the time duration in milliseconds or seconds and does not have to lookup what time unit the library expects.

I agree with you. I updated the type.
And added some more fix and tests.

Hello redboltz, you closed this issue, but I was thinking it may be a good idea to give a default value for the timeout argument. I guess many users don't want to bother thinking this timeout or do not know what a good value for the timeout would be. So I would set it to a timeout of 5 seconds for example (or whatever you think is a good value), to give the indication as the developer of the library you think this is a reasonable value.

I'd like to keep the original behavior. If disconnect() is called with no argument, that means no timeout. So I implemented the following overload.
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/client.hpp#L315
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/client.hpp#L341
I think that appropriate timeout value is different each environment. Users have a change to think about the timeout and their environment. I think that it is not so bad.

Ah yes, that sounds good :)


Test Conversion Script

Run the script against graphics and spotcheck individual graphics.

This has been done....awesome @esomo22e


windows binaries

Hi,
I'm having trouble building mxnet on windows and would like to try the pre-built binaries referred to on:
http://newdocs.readthedocs.io/en/latest/build.html#installing-pre-built-packages-on-windows
but there don't seem to be any available at the link given. Does anyone know if these are still provided/maintained?
Thanks very much!

You can pip install mxnet on windows now

Hi,
Thank you for your reply. I would like to use mxnet from my own c++ code rather than through python. Is the pip installation for python only?
Steve.

I need the scala bindings on windows. The windows build instructions require Visual Studio 2013 which I can't find for download. Could you at least update the build instructions for W10?
Thanks!

Welcome to the c++ api club. If you want to use c++ api you need to build the library yourself because some include files are generated by the build process. These are unfortunately not distributed with binaries.
I made a short list of steps (with the help of a webpage) and could build the library until 0.95 but since 0.11 the build breaks for VS2013/15/17 on Win10 64 bit  #(8155). So if anyone has it working please share it. I am stuck for a month now.

For those using c++ API, there is nor real docs and limited examples. So if you have how-to questions I might be able to help you. It would be good to share knowledge.

@apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage.
For general "how-to" questions, our user forum (and Chinese version) is a good place to get help.