docker-for-win: support PowerShell

Ref #148.

@phil294:
Concerning my idea of a docker-x11docker-image: Yes, I know, it would not help much: a simple batch script could achieve the same thing. However, there is no such batch script right now and I did not know there were any plans on doing so.

Since x11docker is itself a single bash script, adapting it to a batch script would imply rewriting almost the full project. Furthermore, x11docker relies on several tools that are normally found on Linux/Bash contexts, so I don't think it is a matter of syntax only.

@phil294:
With no batch script available and x11docker containerized, Windows users could be spared to install msys2 or cygwin. Feel free to ignore this paragraph, I should have opened a seperate issue.

This is a specific requirement, and we might be able to work around it. It is currently possible to execute bash from PowerShell, and it seems that the integration will improve in upcoming versions: https://www.youtube.com/watch?v=lwhMThePdIo.
So, it might be possible to use x11docker from PowerShell with VcXsrv, without requiring neither cygwin nor msys2. Cygwin would still be required in order to use --xwin.
I just tried the following:
PS C:\Users\username> bash
username@host:/mnt/c/Users/username$ export PATH=$PATH:/mnt/c/Program\ Files/Docker/Docker/resources/bin/
username@host:/mnt/c/Users/username$ cd /mnt/c/Program\ Files/Docker/Docker/resources/bin/
username@host:/mnt/c/Program\ Files/Docker/Docker/resources/bin/$ ln -s docker.exe docker
username@host:/mnt/c/Program\ Files/Docker/Docker/resources/bin/$ cd /mnt/c/Users/username
username@host:/mnt/c/Users/username$ curl -fsSL https://raw.githubusercontent.com/mviereck/x11docker/master/x11docker | bash -s -- --clipboard --gpu x11docker/check
mkdir: cannot create directory ‘/c’: Permission denied

x11docker ERROR: Could not create cache folder
  /c/Users/eine/x11docker/cache

  Type 'x11docker --help' for usage information
  Debug options: '--verbose' (full log) or '--debug' (log excerpt).
  Logfile will be:
  Please report issues at https://github.com/mviereck/x11docker

I did a test in Powershell, too. It works here.
However, a different cache folder is created, the right one is /mnt/c/Users/.... I am not sure why /mnt is missing in your case.
x11docker itself checks uname -r. If it contains Microsoft, it configures itself for WSL.
PS C:\Users\User\Downloads> bash -c 'uname -r'
4.4.0-17134-Microsoft

Can you check bash -c 'uname -r' in Powershell, please?

Can you check bash -c 'uname -r' in Powershell, please?

With 'Windows PowerShell' or 'Windows PowerShell ISE':
PS C:\Users\username> bash -c 'uname -r'
4.4.0-17134-Microsoft

With 'Windows PowerShell (x86)' or 'Windows PowerShell ISE (x86)':
PS C:\Users\username> bash -c 'uname -r'
3.0.6-338.x86_64

With 'Debian GNU/Linux':
username@DESKTOP-E1ER43H:~$ uname -r
4.4.0-17134-Microsoft

Thanks.
So x11docker's check fails in 'Windows PowerShell ISE (x86)'. Maybe it should check PATH instead (or afterwards) for something like /Windows/System32.
I would expect that x11docker will already work on your system in 'Windows PowerShell' or 'Windows PowerShell ISE'. I did my own test run in just 'Powershell'.

The error reported in the first comment corresponds to 'Windows PowerShell'. I get the same result with either of them.

There will be several changes for x11docker on MS Windows (#160).
I'll look at this ticket again after the changes are done.

I've installed a new Windows developer VM (WinDev1905Eval) and run x11docker with runx (#165) in PowerShell:
bash /usr/bin/runx -- /usr/bin/x11docker -e xterm

It succeeds, I still cannot reproduce the issue.

PS C:\> bash
eine@DESKTOP-E1ER43H:/mnt/c$ /mnt/d/data-dev/github/runx/runx -- /mnt/d/data-dev/github/x11docker/x11docker -e xterm
runx note: MSYS2 does not provide xauth to create a cookie.
  Fallback: Enabling discouraged option --no-auth.

runx ERROR: No X server found for MSYS2.
  Please install X server VcXsrv:
      https://sourceforge.net/projects/vcxsrv

Anyway, I'll focus on #165.

The issue here might be related to the failing WSL runs in #165.
WSL is somehow not detected properly.
Do you have cygcheck.exe available in WSL? That would confuse x11docker.
I've changed the order of the Cygwin/MSYS2/WSL check, maybe that fixes this issue as well as the WSL failures in #165.
Previously WSL was checked first, now at last. The new check:
  # Check for MS Windows subsystem
  command -v cygcheck.exe >/dev/null && {
    cygcheck.exe -V | rmcr | grep -q "(cygwin)"   && Winsubsystem="CYGWIN"
    cygcheck.exe -V | rmcr | grep -q "(msys)"     && Winsubsystem="MSYS2"
  }
  uname -r | grep -q "Microsoft"                  && Winsubsystem="WSL"

Do you have cygcheck.exe available in WSL? That would confuse x11docker.

Yes, it is available in WSL:
~$ cyg
cygcheck.exe               cygpath.exe                cygwin-console-helper.exe
~$ which cygcheck.exe
/mnt/c/msys64/usr/bin/cygcheck.exe

I've changed the order of the Cygwin/MSYS2/WSL check, maybe that fixes this issue as well as the WSL failures in #165.

It works now. Both when launching WSL or when running bash from PowerShell. Some probably fixable messages are shown:
# runx -- x11docker -e xterm
...
mkdir: cannot create directory ‘/run/user/1000’: Permission denied
chown: cannot access '/run/user/1000': No such file or directory
chmod: cannot access '/run/user/1000': No such file or directory
...

It works now. Both when launching WSL or when running bash from PowerShell.

Great!

Some probably fixable messages are shown:

I'll look into this. Do you get an output in WSL or PowerShell for:
echo $XDG_RUNTIME_DIR

?

I'll look into this. Do you get an output in WSL or PowerShell for:
echo $XDG_RUNTIME_DIR

?

No. It's empty/unset. No matter if I check it in WSL, Cygwin or MSYS.

The mkdir ... errors are fixed. Thanks for your tests!


Census block groups should highlight when selected

Describe the bug
According to the latest spec, we should have a 5px highlighted region around a census block group when a user selects it. This is not currently happening.
To Reproduce
Steps to reproduce the behavior:

Open the map, click a block group
The popup appears, but the block group is not highlighted

Expected behavior
The region should highlight on click
Screenshots

Desktop (please complete the following information):
All
Smartphone (please complete the following information):
N/A

Spec here:


Before function doesn't allow console to print output

I have a before function that makes an API call, and if the test times out, then it doesn't print anything, which isn't useful when I'm trying to see what the server logs.

Can you please provide some code snippet or fiddle example @ilanbiala ?
thx

The code inside the before function used to not work, and debugging it was extremely difficult because I would not get any logging from the server.
before(function(done) {
    this.timeout(3000);

    agent
      .post('/auth/mock')
      .send({
    		firstName: 'Test',
    		lastName: 'User',
    		displayName: 'Test User',
    		email<EMAIL_ADDRESS>    		provider: 'local',
    		password: 'testuser',
    		isAdmin: true
    	})
      .expect(200)
      .end(function(err, res) {
        if (err) {
          return done(err);
        }

        res.body.should.have.property('success', true);

        user = res.body.user;

        done();
      });
  });

Dunno... if it times out, it times out.  How is Mocha supposed to know what to print?
My suggestion would be to set a timeout on your HTTP request itself.  Is this workable?

@boneskull it didn't time out, it threw an error, but it wouldn't print out anything from anywhere else.

@ilanbiala What error? In the first post you actually mentioned 'when it times out'..

@dasilvacontin I changed a lot of my code to get it working now, so I can't reproduce it and tell you whether it said it timed out or that an error occurred. I'll provide any more information if I encounter this issue again.

Oh, so you can't reproduce the error anymore, @ilanbiala?

The error was reproducible when I had certain code to test, but that code has changed. I'll post any more info here if I hit that wall again.

If the problem is that Mocha quits before you log anything, why don't make the server log into a file?

@ilanbiala Closing for now. If you're able to reproduce the issue and provide a small test case, feel free to re-open! :)


Cloudformation

:fire: WIP :fire: This is going to be broken for a while.
This is a rewrite of the boto3 management backend with troposphere and CloudFormation.
Right now it deploys and seems to work for simple test projects. :+1:
All kinds of stuff is broken, including:

lambda versioning
a lot of the tests
there's a hard limit from AWS on template size (with current Zappa this is 6 deep, max)

I'll probably look into some chunking ideas and see if the size limitation is simple to work around.
Here's the resulting monstrosity (6 deep):

Closing for now - goes out of date quick!


Refactor byte code emission for BrOnNoProperty family of cases

Preparatory to coming changes making it easier for the JIT to renumber temp lifetimes effectively, do the following:

Separate "with" cases from other kinds of cases involving dynamic scopes;
Eliminate bail-on-implicit-call checks from the non-with cases
Eliminate the IsES6UnscopablesSupported switch, which has never had any real testing utility and has long stopped representing a shippable state of the engine

@dotnet-bot test Windows 8


Use of webpack.config in another project - Stuck while building

I find your webpack config fitting my needs to a project and  I have therefore tried to use your webpack config in a project of mine: https://github.com/Thisen/angular-apollo-stack/
The change I have made has been to change 'src' folder to 'client' in the config.
The problem is the build gets stuck:

Is there a obvious reason to this? Thanks in advance.

I don't see anything wrong at all in there. That is the desired output.

It's stuck and doesn't finish. And it doesn't hash the files.

Looks like the development output... you get hashes in the filenames and emitted files for the production build. I think you just have the dev server running there, maybe?


NOISSUE - Remove Debug function

Signed-off-by: Ivan Milošević<EMAIL_ADDRESS>What does this do?
Removes log function form Debug package
Which issue(s) does this PR fix/relate to?
No issue

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6ebc758). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #699   +/-   ##
=========================================
  Coverage          ?   86.53%
=========================================
  Files             ?       64
  Lines             ?     3848
  Branches          ?        0
=========================================
  Hits              ?     3330
  Misses            ?      361
  Partials          ?      157

Continue to review full report at Codecov.

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


Add developer guide for eigen in coding/using_eigen.rst

@ziggyman you need to add this page to the table of contents before it can build properly.


Fix/716 tables tab; COUNTRY=mongolia

The below commit adds a Tables Tab in the LeftPanel Tabs Summary. The Tables Tab follows the implementation of the DataDrawer component. Some refactoring have been made in order to facilitate the changes. Some of those are:

Deleted DataDrawer component as it wasn't needed anymore.
Rearranged  a little bit all four tab panels components (ChartsPanel, AnalysisPanel, LayersPanel, TablesPanel).
The layers and tables are being filtered and feeded inside each individual component directly from the parent, which filters the menuList accordingly.
utils file which was in the LayersPanel component has been moved to LeftPanelTabs component because the menuList, which was exported is now being processed in the LeftPanelTabs component.
Moved Chart component to Common folder.

Practically when a country has tables the Tables Tab with Tables Panel will be added in the LeftPanelMenu and will display all the available tables in the dropdown menu.
This touches quite a few components so let me know if anything isn't clear.  Cc @ericboucher @wadhwamatic .
Below I attach some screenshots.
Screenshot/video of feature:

I am adding also a screenshot below with all four tabs in place. If we have a specific country that has also layers,charts,analysis and tables.

great work @HarryMytilinaios!
I temporarily added a table config for Cambodia and a chart config for Mongolia to see how charts / tables work together. It's working for Mongolia (national level only because the admin codes aren't there for admin1 and admin2).
I couldn't get tables to show up for Cambodia. I might need a reminder on how this is setup. I thought it just required adding the rawTables reference in src/config/country/index.ts, creating tables.json, and adding tables to prism.json. What else am I missing? cc @ericboucher

@wadhwamatic I think this is now fixed. The problem was that the menuList that we are used to populate both layers and tables was expeting a categories key inside the prism.json which will include both layers and tables. There was a typo in the prism.json of cambodia and the tables instead of being a child of categories was at the same level as categories as siblings.

Thanks for catching the issue and resolving it @HarryMytilinaios. Everything looks good from my side now. I removed the test configs for Cambodia and Mongolia. This is GTG from my side


Name/Username of RemoteJID

Is there a way I can retrieve the name of the RemoteJID?
I wanted to see the registered name or even just the username of the person who sent me a message.
The Python version of this i think have one and it also just listens to the web socket of whatsapp that's why I am wondering where I can get the name or even just the username?

@SchulteMK it seems to be that when i printed the contact on the for loop inside the updateContact func i can get the contacts and if I am not mistaken the notify string in the Contact struct is the name of the whatsapp user if I haven't yet registered their number on my phone?
Will be closing this issue after I just need some more enlightenment. I'm also sorry for not seeing that, I'm new to go and this is a rush project that was given to me.
I so much appreciate your help


Scale the ruler to use simulation units

I'm not actually sure what units they are in, whatever the Scale is in.  I tested it mathematically by manually calculating the distance between two notes using their positions and the ruler value seems to be approximately equal to the calculated value.

Perfect! We can have @mkinsey get the filter working later


Hugging face FIXED error KeyError: 'allocated_bytes.all.current'

So I was looking at huggingface VQGAN_CLIP, and it gave me an error. I clone the repo and make one by myself, this was the error...

I also opened this Issue https://github.com/EleutherAI/vqgan-clip/issues/14
I solved it by changing some thing in app.py, you just need to delete this in line 309         #res = nvidia_smi.nvmlDeviceGetUtilizationRates(handle)         #print(f'gpu: {res.gpu}%, gpu-mem: {res.memory}%')
and also this in line 198 #torch.cuda.memory_summary(device=None, abbreviated=False)

I don’t understand, you removed lines that were commented out and it made the code work?

@StellaAthena you can see mine working in here https://huggingface.co/spaces/BernardoOlisan/vqganclip without the lines of cuda and nvidia

 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 4 committers have signed the CLA.:white_check_mark: StellaAthena:x: LouisCastricato:x: aman-tiwari:x: DanJbkLouisCastricato 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.


define version for openapi spec independent from project

Rebased, and merged.
Thanks for your contribution !

@johnaohara may you rebase against the master? then we can see green CI


Bump Guava from 32.1.3 to 33.3.1

:mag: Description
Issue References 🔗

This pull request fixes #
Describe Your Solution 🔧

Bump guava from 32.1.3 (Oct 11, 2023) to 33.3.1 (Sep 23, 2024 ) : https://github.com/google/guava/releases/tag/v33.3.1

Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[ ] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (c6212d0) to head (39abf1b).
Report is 1 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6747   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         684     684
  Lines       42281   42281
  Branches     5768    5768
======================================
  Misses      42281   42281

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix crash when changing node type from PopupMenu to ItemList

This is fixing the crash from #76673
Changing the type of a node from PopupMenu to ItemList will crash if the PopupMenu has an item with a 'checktable_type' other than "No". This PR adds a check for when the array of items is empty, just like for the other properties.
Note that this instead prints an "out of bounds" error message in the output, just like for the other properties, so this doesn't fix the core of the issue, just the crash.

It's consistent with other checks, but the method is not very useful. It can't be really exposed, because the enum it returns is not exposed.
I agree that in this case it's better to add ERR_FAIL_INDEX_V directly in the _get() case.

In _get(), I need to set the r_ret value, and return true.
} else if (property == "checkable") {
	r_ret = get_item_checkable_type(item_index);
	return true;
}

I'm not sure to see an elegant way to handle both and trigger the error message from the macro ERR_FAIL_INDEX_V without using a function.
Something like this :
} else if (property == "checkable") {
	r_ret = item_index < items.size() ? items[item_index].checkable_type : Item::CHECKABLE_TYPE_NONE;
	ERR_FAIL_INDEX_V(item_index, items.size(), true);
	return true;
}

would double the 'return true', as one is done in the macro.
Am I missing something?

Hmm, technically r_ret does not need to be modified if you return false. It means the property was not found. But I guess it's not a very safe behavior 🤔
You don't need to use ERR_FAIL_INDEX, you can copy its condition outside and use ERR_FAIL instead.
if (item_index < 0 || item_index >= items.size()) {
    r_ret = Item::CHECKABLE_TYPE_NONE;
    ERR_FAIL_V(true);
}

Thanks, and congrats on your first merged Godot PR!

Cherry-picked for 4.0.4.


itk: 4.13.0 -> 4.13.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from itk.
meta.description for itk is: '"Insight Segmentation and Registration Toolkit"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/bin/itkTestDriver had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 4.13.1 with grep in /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1
directory tree listing: https://gist.github.com/407742efdf0305ccd060014152788969
du listing: https://gist.github.com/4370ede54352f2dd9c2f145c5ce43241

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/a4e2562ecdba00b8e60ae4e54572dd767e2cdcdb.tar.gz -p itk
ls -la /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1

cc @viric for testing.

Success on x86_64-linux (full log)
Attempted: itk
Partial log (click to expand)
shrinking /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/lib/libITKMesh-4.13.so.1
shrinking /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/lib/libitkgdcmDICT-4.13.so.1
shrinking /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/lib/libITKIOTransformHDF5-4.13.so.1
shrinking /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/lib/libITKIOTransformMINC-4.13.so.1
shrinking /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/lib/libitkgdcmcharls-4.13.so.1
shrinking /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/lib/libITKPolynomials-4.13.so.1
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/lib  /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1/bin
patching script interpreter paths in /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1
checking for references to /build in /nix/store/imlg0jvyhkz4chvhi82bkm1hzb7d8pbc-itk-4.13.1...

Success on aarch64-linux (full log)
Attempted: itk
Partial log (click to expand)
shrinking /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1/lib/libitkv3p_netlib-4.13.so.1
shrinking /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1/lib/libitknetlib-4.13.so.1
shrinking /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1/lib/libitkvcl-4.13.so.1
shrinking /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1/lib/libitksys-4.13.so.1
shrinking /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1/lib/libitkdouble-conversion-4.13.so.1
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1/lib  /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1/bin
patching script interpreter paths in /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1
head: write error: Broken pipe
checking for references to /build in /nix/store/gzd18pkzkrmwqkhyx75y3m74skviiv3x-itk-4.13.1...

Success on x86_64-darwin (full log)
Attempted: itk
Partial log (click to expand)
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKIOMRC-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKVtkGlue-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKznz-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKIOGE-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKIOMINC-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKniftiio-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKIOTransformBase-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKIOStimulate-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKIONRRD-4.13.1.dylib: fixing dylib
/nix/store/69gmcs8jxga6rjd7fars75zfkly7hn6w-itk-4.13.1/lib/libITKIOIPL-4.13.1.dylib: fixing dylib


WIP: Chart testing

Description
Same as #221, but from a branch within this repo, hoping that we can use repository secrets
How Has This Been Tested?

Checklist:

[ ] I have updated the relevant Helm chart(s) version(s)
[ ] I have updated the relevant component changelog(s)
[X] This change does not affect any particular component (e.g. it's CI or docs change)

Hey, this PR still doesn't solve what we want with #215, but I'm opening this early just to make sure I'm going in the right direction.
So far:

CI Builds a KinD cluster
Installs our charts
Succeed if the chart is installed successfully or fails if not.

Missing:

Hit Coralogix API and check if logs/traces/metrics were ingested.

While implementing this, I found a problem with FluentD/FluentBit (See #221). Their containers fail if we pass an invalid Coralogix API Key, at the same time, Github Actions do not provide secrets to PRs coming from forks, which means that FluentD/FluentBit will always fail from external contributions.
I'm not really sure how to workaround this problem 🤔


Apply explicit string casts for strtolower() and urlencode() functions

(cherry picked from commit ebb7c6c3f780feb9c3e69caf3dbafcc4cea75f4d)
While we were migrating our application to PHP 8.1, some of our phpunit tests failed, as they somehow managed to get "null" inside symfony internals, where now strings are the only allowed argument types (resulting in deprecation warnings for now, but we configured our test suites to fail on those as well).
This happend for cases of strtolower() and urlencode() functions. Thus, we simply got our application (test suites) working again by applying explicit string casts to those function calls via simple search-and-replace statements throught all source files. I am aware that this results in possibly many non-required casts, but we wanted to be sure and thus applied the patch to all uses of these functions.
Thought this might be useful for you as well, thus I made this pull request.

Seems a bit of a heavy solution which negates most of the advantages of having type checks: sometimes the nulls shouldn't be getting in there in the first place and the issue may be on the inputs: with the overall casting, we would miss the warnings/errors when on that.
I've been using the latest release on a medium usage website and only spotted one deprecation warning at:
https://github.com/FriendsOfSymfony1/symfony1/blob/2c5a4d625d1853cb207dfbb73817505831874371/lib/validator/sfValidatorError.class.php#L99
where $value sometimes is null when using sfValidatorSchemaCompare but I haven't manage to isolate the issue into a trivial trigger, so haven't go around submitting a fix.
Anyway, that's the only place that I've found a deprecation warning so far. Personally I wouldn't apply a blanket casting.

I wouldn't add it to every method as well. As discussed in https://github.com/FriendsOfSymfony1/symfony1/pull/283, we are about to move to a higher platform minimum version (7.4 as of latest discussions) and therefore a future change might bring typehints to the method signatures, bringing type errors back to who is responsible for calling them correctly.

I am using the latest version of symfony1 under 8.2 without any deprecation warnings. It is very strange that so many errors were found. If a method does not cause an error, it should not be changed.


Regression tests run out of memory in CI

We're not sure if this is a bug in Fuzzit, but our regression tests seem to be failing:
2020/03/20 13:09:35 Running regression...
FUZZER: INFO: Seed:<PHONE_NUMBER>
FUZZER: INFO: 65536 Extra Counters
FUZZER: ./fuzzer: Running 521 inputs 1 time(s) each.
FUZZER: Running: corpus/0022ae0ec320596f561aa52c8b47f17e04db2a00
FUZZER: Executed corpus/0022ae0ec320596f561aa52c8b47f17e04db2a00 in 0 ms
FUZZER: Running: corpus/00e9813b71fa586ae6890dcce51fd8acca6a3448
FUZZER: Executed corpus/00e9813b71fa586ae6890dcce51fd8acca6a3448 in 0 ms
FUZZER: Running: corpus/015f14bc2402f59075cb46c913e08570421bd9ae
FUZZER: ==18== ERROR: libFuzzer: fuzz target overwrites it's const input
FUZZER: SUMMARY: libFuzzer: out-of-memory

Full log here:
https://github.com/caddyserver/caddy/runs/521935588
(And related discussion, toward the end of https://github.com/caddyserver/caddy/pull/3152)
Is that something in our code that is causing out-of-memory? I thought that the fuzzer itself shouldn't run out of memory, but maybe I'm reading this wrong. Thanks!

I was running into this as well, and for a little while assumed the fuzzer was just running out of memory (as the SUMMARY suggests), but was completely neglecting the line above:
FUZZER: ==18== ERROR: libFuzzer: fuzz target overwrites it's const input

In my case, the input bytes were being modified, which libFuzzer isn't happy about (though go-fuzz is fine with). Might be worth looking into.


Error when running SD build script

after
cd docker/
./00create-docker.sh

building the docker image works correctly
[+] Building 3.5s (11/11) FINISHED                                                                                docker:desktop-linux
 => [internal] load build definition from Dockerfile                                                                                0.1s
 => => transferring dockerfile: 893B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ubuntu:22.04                                                                                3.1s
 => [auth] library/ubuntu:pull token for registry-1.docker.io                                                                                0.0s
 => [1/5] FROM docker.io/library/ubuntu:22.04@sha256:6042500cf4b44023ea1894effe7890666b0c5c7871ed83a97c36c76ae560bb9b                                    0.0s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 33B                                                                                0.0s
 => CACHED [2/5] RUN sed -i 's|deb http://us.archive.ubuntu.com/ubuntu/|deb mirror://mirrors.ubuntu.com/mirrors.txt|g' /etc/apt/sources.list &&     apt  0.0s
 => CACHED [3/5] RUN useradd --uid 1000 --create-home --shell /bin/bash -G sudo,root rohan     && echo '%sudo ALL=(ALL) NOPASSWD:ALL' >> /etc/sudoers    0.0s
 => CACHED [4/5] WORKDIR /home/rohan                                                                                0.0s
 => CACHED [5/5] COPY br-build.sh .                                                                                0.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:a0852c9ce789295e0368ab1674c0e3c427bcc815c38f7dcf533b8fcf48b6c5eb                                                             0.0s
 => => naming to docker.io/library/br-build:latest          0.0s

while in docker dir (of this cloned repo)
docker run --ipc=host --rm -it -v $PWD/:/home/$USER/work br-build /home/$USER/br-build.sh
Downloading ...
Extracting ...
tar: buildroot-2023.02.4.tar.gz: Cannot open: No such file or directory
tar: Error is not recoverable: exiting now
ln: failed to create symbolic link './sdcard.img': Permission denied
/home/rohan/br-build.sh: 18: cd: can't cd to buildroot-2023.02.4
Building ...
make: *** No rule to make target 'licheepi_nano_defconfig'.  Stop.
make: *** No targets specified and no makefile found.  Stop.

Updating url to https solves the problem.
ll'l let you updated  the fix and then close this issue once updated.

I couldn't reproduce the problem in my environment, but I fixed the URL in the script.
Thanks!


[Feature Request]: Support accessModes ReadWriteMany

What feature would you like to have in the provider?
I've just tested to change pvc.yaml in example to use ReadWriteMany instead of ReadWriteOnce:
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: my-sbs-pvc
  namespace: awesome
spec:
  accessModes:
    - ReadWriteMany
  resources:
    requests:
      storage: 100Gi
  storageClassName: exoscale-sbs

I end up with an error to start the pod defined in deployment.yaml:
MountVolume.MountDevice failed for volume "pvc-ce3b02e3-40be-4423-b50b-2f17909e2300" : rpc error: code = InvalidArgument desc = volume ch-gva-2/cc1b1bb9-55dc-4643-bd50-a9de091b74a9 capability not supported

It seems it supports only ReadWriteOnce.
Is it planed in the future?
Thanks

Thanks for your feature request,
Currently, it's not possible to use ReadWriteMany, since it's not possible to attach an Exoscale Block Storage volume to multiple Instances.
You can stay aware of our future announcements, if we plan to support this in the future.
For sure, we will update the CSI if such a change is added to our Block Storage.

@llambiel I've seen you closed the issue as completed. Is the feature already implemented? If not, I guess the issue should stay open as it is a feature request and the feature is not there. Furthermore it is not a n'exhibèrent feature request for a k8s CSI.

If not, I guess the issue should stay open as it is a feature request and the feature is not there.

Fair enough

Furthermore it is not a crazy feature request for a k8s CSI.

Indeed it's not. However we can't really say the same for the block storage service side, on which the CSI ReadWriteMany depends.

@thomas-chauvet the current goal of the CSI is to cover the current scope of Exoscale's block storage offering, which is RWO.
An RWX storage system is a significantly different beast, and certainly something we would want to provide in the long run. However we haven't started anything in that direction. When we do, CSI integration would obviously come with it.
As such there is not a lot of value in keeping this feature request open within the CSI. It's an Exoscale feature request, and we don't really have a process for tracking them across our public github repos.
I will close this as "not planned", which doesn't mean we're never going to provide it, but would better indicate that we are not actively working on such an offering.
Thank you for your understanding.


Web GUI open questions

I'm working on a web GUI over here (early and non-functional)
There are a bunch questions that should be addressed rather sooner than later:
Root privilige handling.
I'd rather not run the whole thing as root. Maybe better to have a separate process that can execute pre-defined stuff with root priviliges.
The multi-user problem
@danmons knows the project in and out. Do we have to ensure that only one user is doing stuff at the same time?
CSS
I can do some basic stuff, but it's not going to look cream-of-the-crop. Help very much appreciated
Generating web GUI options from TUI
I have a very basic proof-of-concept of this working, but it's really going to need some specific begin-end comment markers and separators (e.g. between tool name and description) to work well.
Some little technical details
Will ask Dan directly

Picture for any interested parties

May I suggest that this is moved to Discussions with issues left for found problems with RetroNAS. It may be better suited there since this is not particularly an issue with the project more of a technical discussion with a new feature set.
https://github.com/danmons/retronas/discussions
I also did this recently with the TCPser thread before i realised discussions were open for this project, I ended up closing my (non)issue and moved it to discussions

Ah right, that's a thing that exists! Will do


Allow disabling clean

Original author<EMAIL_ADDRESS>(March 20, 2013 11:01:23)
Is it possible to add confirmation to the "clean" command?
Adding an "--force-drop-database" option will skip the confirmation.
This doesn't have a security impact but it will "avoid" some problems.
Thanks.
Original issue: http://code.google.com/p/flyway/issues/detail?id=455

Implemented.

Thanks! Definitely going to use this config option in production.


Property 'root' does not exist on type 'SetupContext<Record<string, any>>'.Vetur(2339)

Version
3.0.0-beta.4
Steps to reproduce

Create a app with typescript support
try to access context.root in setup method of component by giving context as second argument
Error will happen due to there is no root Here

What is expected?
No error when call root in context
What is actually happening?
A error when call root in context

There is no root property on SetupContext. This is by design.

import { getCurrentInstance } from "vue";

setup() {
  const instance = getCurrentInstance();
  instance.proxy. // == context.root
}

@johnsoncodehk Object is possibly 'null'. How to solve this?

@johnsoncodehk Object is possibly 'null'. How to solve this?

Vuejs and TypaScript ?

@johnsoncodehk Object is possibly 'null'. How to solve this?

Vuejs and TypeScript ?

Yes~

Put ! in end of value to force it.
Example :
let a = my_var!
Le ven. 4 juin 2021 à 02:18, ZeHung @.***> a écrit :

@johnsoncodehk https://github.com/johnsoncodehk Object is possibly
'null'. How to solve this?
Vuejs and TypeScript ?
Yes~
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vuejs/vue-next/issues/1056#issuecomment-854286633,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASXMK5GM7V33ETSAJ2TKDWTTRASWFANCNFSM4MRG2UEQ
.


Mark cupyx.jit.rawkernel as experimental

Marks cupyx.jit.rawkernel as experimental because its interface may change in the future.

Resolved conflict.

Jenkins, test this please

Jenkins CI test (for commit e1840ef2d1e5c91e264f283b9b464787349b24dc, target branch master) succeeded!


issue/6631-add-dic_path_library-core (#6634)

Pull request opened by the merge tool on behalf of #6634

Merged into branches iso6 in 7bef45c50833602a23f471cee4a5f9839db64314


Update generator

The generator hangs on the Rollup func being named the same as the Rollup namespace.
This fixes that and generate a new set of apis based on the updated spec.

Btw I have this commit on 7.x and 7.11 branches: https://github.com/elastic/go-elasticsearch/commit/3011c0845d704c1680f3114cf8940cb0e408d7f1
Not sure if you want to use the same commit or replace the old code with this new method, either is fine!

If that's all the same to you I'd prefer to use the method, first step to remove code duplication.


Train using ROCm?

Since there is no discussion section I post my question here.
Is it possible to use Pytorch and Tensorflow for ROCm to train NAM models? If so, does anyone have knowledge they can share on how to do this?
Regards.

This should be possible on computers on which PyTorch's ROCm backend is already supported.  Have a look at Start Locally | PyTorch to see more details.

I am trying it out now and will report back if I am successful.


In Video Player, do someone know where the video will be played at fullscreen at the start and minimize when it finish playing? Really need this one, thank you

Please only submit reproducible issues.
If you're not sure if it's a real bug or if it's just you, please open a topic on the forum: https://forum.magicmirror.builders/category/15/bug-hunt
Problems installing or configuring your MagicMirror? Check out: https://forum.magicmirror.builders/category/10/troubleshooting
When submitting a new issue, please supply the following information:
Platform: Place your platform here... give us your web browser/Electron version and your hardware (Raspberry Pi 2/3, Windows, Mac, Linux, System V UNIX).
Node Version: Make sure it's version 0.12.13 or later.
MagicMirror Version: Now that the versions have split, tell us if you are using the PHP version (v1) or the newer JavaScript version (v2).
Description: Provide a detailed description about the issue and include specific details to help us understand the problem. Adding screenshots will help describing the problem.
Steps to Reproduce: List the step by step process to reproduce the issue.
Expected Results: Describe what you expected to see.
Actual Results: Describe what you actually saw.
Configuration: What does the used config.js file look like? Don't forget to remove any sensitive information!
Additional Notes: Provide any other relevant notes not previously mentioned. This is optional.

Please refer to http://forum.magicmirror.builders for support and troubleshooting.
I'm closing this issue for now. If you feel this issue should remain open, please let me know.


How to show "Someone left the conversation" text in JSQMessageviewcontroller

Hi i want to  show "Someone left the conversation" text and the date in JSQMessagesViewController Please help me. @jessesquires

I suggest notifying the other user in the viewWillDisappear method of JSQMessagesViewController.
override func viewDidDisAppear(animated: Bool) {
    // Input method that notifies user(s) in message thread that a user is leaving
}

Thanks @srinathp90 !
v8.0 of the library will support system messages 😄

👍


Auth Methods

If I select Security Questions and Mobile app code in SSPR and Mobile app code and notification in MFA, I'm still only prompted for email and phone number upon registration.
Also, if I use Authenticator only in MFA, I am only prompt once I go through the scenario covered by the CA policy. I don't have the option to configure the authenticator ahead of time like I could with https://aka.ms/mfasetup.

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

ID: 91ad4424-8d90-7947-2f95-aec67acfec45
Version Independent ID: 0ebfde0e-5597-fed9-f87c-761fd778fb25
Content: Converged registration for Azure AD SSPR and MFA (public preview)
Content Source: articles/active-directory/authentication/concept-registration-mfa-sspr-converged.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

Can you try logging in incognito mode in a different browser? It's possible that your credentials are being cached.
@MicrosoftGuyJFlo can you confirm whether these scenarios are supported fully in this public preview? I will try to reproduce the issues in my lab.

In initial registration we only ask for the minimum required methods to get you registered. More than that was causing confusion and adding to the time required. You can always go back and add more methods in the future.
You can always go in and register methods beforehand if your admin has enabled them using https://aka.ms/setupsecurityinfo

#please-close

@jegilber We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Mat/Pat - Content Changes/Form Design Change for Maternity Allowance

Context

The allowance calculator felt like it was part of the form when it was not actually. To address this the calculator is now in a accordion style box
The info/alert boxes were not drawing attention as intended. To address this "check you eligibility" is now not inside a info box
Some content has also been moved from the "mat leave -length of leave" page to "allowance" page
A note has been added that AskMyHR will not be able to answer your EI related questions

New Design & Content
Maternity leave - top up allowance v3
Maternity leave - length of leave v3
AC

[ ] The content on both the pages above should match the new designs. Note that the text has also been refined a little bit.

Chris to review version 3 and talk to Andrew before starting this work.
@chriscaldwell-psa - please update the ticket with any info

Made 98% of the changes to the tab on Friday. I need to get some feedback next week to make sure the 'accordion style' calculator reveal is correct. When I moved the components, the calculate button does not work. I will need to connect with Andrew. The test env was downloaded as a backup. It's easy for me to revert any changes if necessary. I will update the hyperlinks on Monday.

FYI @chriscaldwell-psa notices these Maternity leave allowance screen -

Change the wording top-up allowance to leave allowance throughout the page.
Add missing links Private Zenhub Image
Typos in this text
"To avoid unforeseen, it's important to check your **legibility for EI before commencing maternity leave, especially if you 've re**cently completed parental or maternity leave or claimed EI due to any reason like unemployment, family caregiver leave or compassionate care leave. If you have any queries about your eligibility, you can contact EI. AskMyHr will not be able to help you with your questions about EI.

Also, note that your eligibility for EI will be officially confirmed only after you apply for EI when your maternity leave begins and your application is approved. "

Changes have been made to review. Chris

Tested in TEST environment.
Started a new Mat/Pat form.
Checked the form against the ticket's acceptance criteria
Also, reviewed Fazil's comments for updated changes in the form.
Private Zenhub Image

Private Zenhub Image

Looks good, changes match designs. Links not opening in new tabs, but that is a form wide issue that will be tackled in #1274


Add document usage for dynamic label config file

Add theme config for github page
Change template file extension for index
Fix err log with condition in config.go

See you evening to shabu


Multiple embedded resources issue

Using a simple starter template:
<Project Sdk="FSharp.NET.Sdk;Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>netstandard1.6</TargetFramework>
    <Version>1.0.0-alpha</Version>
    <EnableDefaultCompileItems>false</EnableDefaultCompileItems>
  </PropertyGroup>
  <ItemGroup>
    <Compile Include="Library.fs" />
  </ItemGroup>
  <ItemGroup>
    <PackageReference Include="FSharp.NET.Sdk" Version="1.0.*" PrivateAssets="All" />
    <PackageReference Include="FSharp.Core" Version="4.1.*" />
  </ItemGroup>
</Project>

Adding more than one .resx file to the folder results in the following build error:
C:\Program Files\dotnet\sdk\1.0.0-rc4-004771\Microsoft.Common.CurrentVersion.targets(2857,5): error MSB3577: Two output file
 names resolved to the same output path: "obj\Debug\netstandard1.6\.resources" [C:\Projects\test_fs\test_fs.fsproj]

P.S. C# templates work fine with multiple .resx

@ncave the resx where not converted to resource files at all 😄 now is implemented

@enricosada More than one .resx file in project works now?

@ncave yes, because resx was not implemented at all, so all resx collided to obj\Debug\netstandard1.6\.resources instead of obj\Debug\netstandard1.6\Res1.resourcesforRes1.resx` etc
i just need to publish FSharp.NET.Sdk 1.0.1 on nuget.org (really soon, probably tonight), i'll ping you and in the https://github.com/fsharp/FSharp.Compiler.Service/issues/700 issue

Would it work with other target frameworks too (net462)?

good test case.
Anyway yes, checked. i'll add to test suite (i need to use a better framework matrix, or anyway cleanup a bit)

@ncave just to be sure i added as tests in https://github.com/dotnet/netcorecli-fsc/pull/82.
if you have more test case in mind np, but there is just invocation of task built by vf# repo, but no need to duplicate testing a lot (but check target framework is ok ihmo)

@ncave this issue is fixed with the new published FSharp.NET.Sdk v1.0.1 package

Thanks!


Download Charts after fold?

Hello! Is there a way to download the charts containing predicted LDDT and predicted aligned error and such after the protein has been folded? I would like to have them as .png or .pdf or something similar.

Hi, @alanclayrichard, thank you for the question. You can refer to the AlphaFold notebook to see example code for plotting the PAE and pLDDT. You can add your code to save the plot image in there, or use the notebook as a reference for your own code. The function for saving a plot as image is: plt.savefig('filename.png').
I'll close this issue, feel free to reopen this issue or open a new issue if this is still a problem.


Minor problem with packages

In one of the latest commits the unit Neon.Core.Serializers was divided in three separate units: Neon.Core.Serializers.DB, Neon.Core.Serializers.RTL, Neon.Core.Serializers.VCL, but the package was not updated to reflect this change, so now it doesn't compile.

Hi Franco,
I can't see the issue in the actual code, can you confirm it's still present?
Paolo.

Hi Paolo,
I've just cloned your repo and the prolem is still there:
Package NeonRunTime.dpk references "Neon.Core.Serializers in '....\Source\Neon.Core.Serializers.pas'," that doesn't exists anymore. This is in all three packages (delphi 10.1, 10.2 and 10.3).
Can you confirm?

I also have the same problem. After modifying Neon.Core.Serializers to Neon.Core.Serializers.RTL, it can be compiled and installed.


Fail verification if contract is vacuous

Description
Fail verification if a contract's preconditions are unsatisfiable or its postconditions are unreachable.
This solution introduces a new Kani intrinsic, kani::internal::cover_contract. We insert calls to this intrinsic during macro expansion after the the precondition and before the postcondition, e.g.:
kani::assume(precondition);
kani::internal::cover_contract(precondition, "The contract's precondition is satisfiable");
....
kani::internal::cover_contract(postcondition, "The contract's postcondition is reachable");
kani::assert(postcondition);

This intrinsic has the same codegen as kani::cover (i.e., assert(!cond)). Just as with kani::cover, the CBMC property renderer intercepts the results of these property checks and updates them before returning them to the user.
Call Outs

I'd be curious if anyone can think of a simple solution to the never type fixme test that I included--if so, I'll fix it in this PR. Otherwise, I'll create an issue about it and link it in the test before merging.
I'm changing the result of undetermined checks to failure if there's unwinding failures. My rationale is that in the case of code like this:

#[ensures(...)]
fn foo() { loop{} }

We know that the postcondition is unreachable, so we should say so. I think this approach makes sense, but also happy to keep it undetermined if we'd rather not treat one type of undetermined check differently from another.
Resolves #2793
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Need to investigate why this breaks the standard library... not ready for review until then.

May I suggest a different approach? I think we should add 2 special cover statements to harnesses. One before we invoke the function and one after we leave the function. This would help us understand whether the pre-conditions are ever satisfied, and if the post-condition is ever checked.

Note that this fails the standard library job because a number of the unchecked_mul harnesses are restricted to ranges such that the precondition empties the search space. For example, verify::unchecked_mul_usize_edge performs lhs.unchecked_mul(rhs) on usizes between [u32::MAX / 2, u32::MAX], with a precondition that the product must not overflow. However, this product will always overflow, so the contract is vacuous and verification fails. If we decide we're happy with this behavior, I can create a PR to update those harnesses before merging this PR (or ping the CMU team who authored them).

hummm... that's an interesting point. I don't think the harness is incorrect though. I am not sure what the best approach is here to combine problem partitioning and the vacuous contract check. Maybe what we want is to combine harnesses that verify the same contract.

* I'm changing the result of undetermined checks to failure if there's unwinding failures. My rationale is that in the case of code like this:

I would prefer if we don't do that. We can discuss this further but I think it's out of scope of this PR.

Moving to draft for now -- we need to decide whether we want verification failure for these properties to be the default behavior, or a characteristic of the harness. This implementation has failure as the default behavior, but one could imagine that in the unchecked_mul case I mentioned above, you may want something like this:
#[requires(!self.overflowing_mul(rhs).1)]
pub const unsafe fn unchecked_mul(self, rhs: Self) -> Self {
  ...
}

#[kani::proof_for_contract(unchecked_mul)]
// without this attribute, verification would fail because of a vacuous proof,
// but maybe you want to sanity check your precondition
// by having a separate harness with just invalid inputs
#[kani::allow_vacuity]
fn unchecked_mul_usize_edge() {
  let lhs: u32 = kani::any_where(|num| num >= u32::MAX / 2);
  let rhs: u32 = kani::any_where(|num| num >= u32::MAX / 2);
  lhs.unchecked_mul(rhs);
}

// This harness will fail for a vacuous proof by default
#[kani::proof_for_contract(unchecked_mul)]
fn unchecked_mul_middle() { ... }

Partitioning harnesses like this is a reasonable thing to do (and in fact, an approach we recommend and would like to implement as a feature) so failing by default could produce a bad experience for users. Introducing a harness-level attribute requires further discussion, however.


Multiple RoutePrefix ignored

In our API I've used a custom route provider that stitches RoutePrefix attributes together from base classes, however this approach renders me unable to use NSwag as the URLs produced are incorrect.
The structure I have is as follows
[RoutePrefix("api/v1")]
public class BaseController : ApiController
{ ... }

[RoutePrefix("locations")]
public class LocationBaseController : BaseController
{ ... }

[RoutePrefix("stores")]
public class StoreController : LocationBaseController
{ ... }

[RoutePrefix("warehouses")]
public class WarehouseController : LocationBaseController
{ ... }

This will produce a route URL like ~/api/v1/locations/stores/ for the API, but NSwag doesn't see any of the RoutePrefix attributes resulting in a URL such as  ~/stores/.

Thanks for the quick answer, I didn't get the chance to mess with it too much on friday anymore but this seems to be a way to solve the problem : )

This is probably a “bug” as route prefix inheritance is not implemented. This is why we shouldnt close the issue yet....

It is not part of the default web api framework though, the default framework only supports 1 RoutePrefix. Any new ones will override the base class

Ah ok, then you should implement this rule in an operation processor.


4.03: Downloading Super Smahs Bros 64 asset crash the app

When you download the base config for Super Smash Bros 64, it crash the app and you can't launch it again

I didn't try with other game, just this one so don't know if it's specific to this game or not !
You just have to delete the ssb64 folder in /user_data to fix this

Damn, that's weird. Since then no more similar errors occurred?
I don't know how exactly but I should have some way of making the error logs be preserved, since I would need you to recreate the bug and send me the logs...

I tried to download different other assets base config (base_files), looks like it's only for smash 64 ?
Well, tell me if you know how to write somewhere error logs !

There are 2 log files in /assets/, but they reset every time you open the program again. So you'd have to make open the program, make it crash again, and send me these files

Log error is empty a,d log.txt looks "empty" :
log.txt

Still every time you try to download it, the program crashes?

I think I know the issue... some stage must have a utf-8 character in its name (like Pokémon Stadium) and I'm not loading the file with this in consideration. Will try to fix this as soon as possible!

Yes, only base_files, other asset from ssb64 is fine !
Ah, enconding problem, so much fun ! 😬

Actually, I'm unable to reproduce this issue :(
And I checked, I'm loading the files as utf-8...
Are you able to run the program using the .bat so you can get the error message for me?

I've tried reproducing the issue, downloading the SSB assets worked with the BAT file but not the EXE file

I found a workaround for the issue: On Windows 10, go to the Control Panel, open the Region settings, open the Administrative tab and click "Change system locale". From there, check the option "Beta: Use Unicode UTF-8 for worldwide language support" and reboot the computer

I suspect the EXE is crashing in normal mode, at least when the Administrative language is set to "French (France)" due to the SSB asset pack using the character Ω, which is not valid in Latin-1 encoding. For similar reasons, it is likely that the EXE would also crash when downloading any asset pack with é in it if the Administrative language was something like "Greek (Greece)", as é would not be valid in ISO-8859-7 encoding.

Fixed on 4.05!!


[keycloak - OID4VCI] Migrate Verifiable Credential Definitions from Client Attributes to Realm Level Attributes

This is a proxy of:

[ ] https://github.com/keycloak/keycloak/issues/32773

Closes https://github.com/keycloak/keycloak/pull/34697/

MR-https://github.com/keycloak/keycloak/pull/34796


Three listbox bugs in essence::count_of_exposed() and listbox::last_visible()

I have found three more listbox bugs:

drawerbase::listbox::essence::count_of_exposed():
this function counts an extra item over the real count when "with rest" is requested and all the items fit in the viewport (when the vertical scroll bar is not being used)

listbox::last_visible():
This function returns the item following the last visible item, because the item count returned by count_of_exposed() is added to the position of the first visible item. For example, if the first visible item is at display position 100, and 5 items are visible, the function returns display position 105, which is wrong. Since item 100 is itself part of the visible items, the function should return 104.

listbox::last_visible():
This function can return an empty index_pair object when all items fit in the viewport, because count_of_exposed() doesn't account for any empty space in the viewport. For example if the viewport can fit 10 items, but the listbox only has 5 items, count_of_exposed() will return 10 instead of 5. That value is then passed to drawerbase::listbox::es_lister::advance(), which returns npos.

I will submit a PR with my proposed fixes (feel free to ignore them if there are better solutions).

Thank you a lot for fixing these bugs.


Strange inability to rename files with Finder set to float

macOS mojave - 10.14.1
chunkwm --version
chunkwm 0.4.7

Here's a screen capture to show what I'm talking about.

Generally speaking, I've seen very strange issues with some of the elements that exist in Finder. The rename case is the simplest one for me to capture to demonstrate, but I also see it in File save dialogs where some of the save options are also not visible. An example that I saw yesterday was when exporting an HEIC file to JPEG via the Preview app. The export modal opened, but the target file type to select JPEG wasn't visible, and the JPEG quality slider wasn't visible to accompany the selected JPEG selector.
Both of these issues seem to be related, and as far as I can tell, they happen when the space is set to either bsp or monocle and Finder is set to float, either via a rule or by manually toggling the tiled Finder to float. Here's what I've tried:

Dark mode, light mode -- no difference
Changing the highlight and accent colors in the same System Preferences pane where you choose light/dark mode -- no difference
Set desktop to float mode -- works, but switching back to bsp or monocle ends up reordering my previous arrangement
Use skhd to chunkc set window_float_next 1 -- no difference
Started chunkwm with no rules -- no difference

.chunkwmrc contents
#!/bin/bash

export CHUNKC_SOCKET=3920

chunkc core::log_file /tmp/chunkwm.log
chunkc core::log_level debug

chunkc core::plugin_dir /usr/local/opt/chunkwm/share/chunkwm/plugins
chunkc core::hotload 1

chunkc set global_desktop_mode           bsp

chunkc set global_desktop_offset_top       0
chunkc set global_desktop_offset_bottom    0
chunkc set global_desktop_offset_left      0
chunkc set global_desktop_offset_right     0
chunkc set global_desktop_offset_gap       5

chunkc set 1_desktop_tree ~/.chunkwm/layouts/3_pane_middle_focus.txt

chunkc set bsp_spawn_left                  1
chunkc set bsp_optimal_ratio           1.618
chunkc set bsp_split_mode           vertical
chunkc set bsp_split_ratio               0.5

chunkc set window_focus_cycle            monitor
chunkc set mouse_follows_focus             intrinsic
chunkc set focus_follows_mouse             1
chunkc set window_float_next               0

chunkc set focused_border_width 1
chunkc set focused_border_radius 0
chunkc set focused_border_skip_floating    1
chunkc set focused_border_skip_monocle     1

chunkc set window_float_topmost            1

chunkc set mouse_modifier              \"alt shift\"

chunkc core::load tiling.so
chunkc core::load border.so
chunkc core::load ffm.so

Also, with chunkwm stopped, I tried to use the Accessibility Inspector to find the element when the rename is active, but I haven't had any luck in getting it to show me the UI element during the rename.

That's a really weird issue, since the rename does not spawn another window. Looking at it from a history point of view, I cannot remember kwm or chunkwm having this issue ever — kwm had all the issues with window layers you can imagine and then some.

Does it work if you just press return ↵ for renaming instead of using the context menu entry?
Have you tried rebooting?
Have you tried this on another user account with only chunkwm installed to make sure it is not other software interfering?
If and only if you are using the scripting addition chwm-sa, have you updated it to the most recent version?

The only thing I can imagine chunkwm does, that could cause this behaviour is the following setting: chunkc set window_float_topmost            1. Can't really say much more at this time.

@dominiklohmann:

re: return - yeah, I forgot to mention that in the summary, but I tried to show that in the screen capture. The first item was via context menu rename, and all the others that you see blank out one after the other are from just pressing the return key.
rebooting doesn't fix anything
haven't tried another user account yet
I use chwm-sa, and it's up to date

@koekeishiya - that's it -- setting that cvar to 0 fixes it for me

I'm not really sure if there is a way to fix this

@dominiklohmann

That's a really weird issue, since the rename does not spawn another window.

So it actually turns out it spawns a new element, and while this element is not technically a window, it does have an associated CGWindowId. We are able to observe the creation of these elements, and adjust their window level accordingly. I might actually have a fully working solution now. Testing an ongoing implementation of this in yabai at the moment.


Make any() accept anything

Hi, this makes any() (without any arguments passed) to accept anything. This actually has the same effect by simply removing the key from the schema file. However, in my use case, I would like the users to see key: any() and knows that this key is optional rather than unavailable. Also, things get a bit different in strict mode, which would be helpful when one wants some key to present but does not care about its type. Thanks for reviewing!

That make sense. Can you please update the README with the new functionality?

And add some additional tests

Hi, I have updated the README and add some tests. Thanks for reviewing.

Looks good, thanks for the PR and thanks for using Yamale!


Allow save/restore of parser state

So I'm convinced that rewind() from #165 was the wrong solution.
We should add instead a save/restore API, which the following advantages:

Does not require keepHistory
Allows you to manage memory usage yourself
Integrates much better with the Lexer interface

The idea is that e.g. nearley-codemirror will be able to save() the parser state at the end of each line. CodeMirror gives you a line at a time for syntax highlighting, and will go back to the first changed line after editing. So we can resume parsing at that point. :-)

@hardmath123 I removed rewind() in 8c33475. It made it into 2.8.0 and 2.7.15. If we do another release since then it will violate semver; either we assume no-one's using rewind(), or we re-implement it in terms of save/restore before we do another release. :-)

Sounds good to me. I don't think anyone uses rewind() yet, so a minor release should be fine.


Unit tests for event mixins

What could be improved
Currently the event mixins are not tested at, it would be good to improve the test coverage here.
Why should this be improved
So that we don't accidentally break anything when doing changes.
More information
https://github.com/flame-engine/flame/tree/main/packages/flame/lib/src/events/component_mixins
https://github.com/flame-engine/flame/tree/main/packages/flame/lib/src/events/flame_game_mixins
https://github.com/flame-engine/flame/tree/main/packages/flame/lib/src/events/game_mixins
Good to know
Make sure that you comment on the issue that you will be working on this before starting to program.
To set up the Flame environment you need to follow the instructions in our contributing.md doc, if you have any problems setting up your environment (or other issues of course) feel free to ask for help in the Flame channel on our Discord server.

@ShwetaChauhan18 here is an issue you can tackle!

Ok, I am taking this

@spydon : If you have any unit testing task then let me know

@spydon : If you have any unit testing task then let me know

This one is open if you want to tackle that one:
https://github.com/flame-engine/flame/issues/2031


Add Microsoft Azure support

Problem
Resolves #231. We would like to use ruby-openai with Microsoft Azure OpenAI, however its endpoints and authentication headers are slightly different, making it impossible currently to use this gem.
Solution
This PR adds a configuration setting api_type, which can be set to :azure, similar to how https://github.com/openai/openai-python supports Microsoft Azure.
Alternatives
I'm aware of the changes being worked on in https://github.com/alexrudall/ruby-openai/pull/150, it might be possible to create a subclass of OpenAI::Client instead once this has been implemented. I'm happy to rebase and refactor this PR once this is merged.
Additional context

Microsoft Azure currently only supports completions and embeddings, so these are the only endpoints implemented
A Microsoft Azure account is needed to rerecord the tests with VCR, and the account needs to have OpenAI access. Currently a form needs to be submitted for this. Also, a deployment has to be created for every model used in the tests, and the deployment name has to match the model name. Documentation should probably be written for this.

Now sure why circle is failing, it seems all the tests are missing the OPENAI_ACCESS_TOKEN

In cooperation with my colleague @rmachielse I've followed up on this. Initially I was looking at rebasing his changes, but I realized it would likely be easier to add the minimal required changes on top of main and slightly change the associated configuration changes.
In practice, the only differences that are really needed are to the uri and the headers, so I only ported those changes and added some tests for them. This works with the following (or similar) configuration (the example specifies config/initializers/openai.rb):
require 'openai'

OpenAI.configure do |config|
  config.access_token = Rails.application.config.app[:azure_openai_api_key]
  config.uri_base = "#{Rails.application.config.app[:azure_openai_uri]}/openai/deployments/gpt-35-turbo"
  config.api_type = :azure
  config.api_version = '2023-03-15-preview'
end

where azure_openai_uri is something like https://custom-domain.openai.azure.com.
In the process I realized there was an inconsistency with /es in the constructed URI, which I tried to fix with URI.join, however this resulted in some code constructions I didn't like so I ended up removing that part of the logic again. That unfortunately leads to the inconsistency where when using Azure, no trailing / is expected in the uri_base but when using the regular API a trailing / is expected and necessary.
I think the ability to be able to use Azure outweighs this inconsistency so I went ahead with the changes like this. If you want I can see if I can come up with a clean way of addressing this. My suggestion would be to do it in a separate PR though, since the behavior was already problematic (no trailing / results in problems with the current logic) and being able to use the gem with Azure outweighs delaying fixing this in my opinion.
Curious to hear what you think @alexrudall !

Looks good to me. By comparison, this is what the api would look like in Python:
import openai
openai.api_type = "azure"
openai.api_base = "https://your-resource-here.openai.azure.com/"
openai.api_version = "2023-03-15-preview"
openai.api_key = "your-key-here"

@alexrudall what do you think? These minimal changes should make it quite straight forward to at least allow usage of this gem with Microsoft Azure.

I'd also like to see this merged. I think more and more people will be transitioning across to the Azure version, especially within corporate environments so this would be very helpful.

I'm also hoping for this functionality. Been using this Gem for developing a proof of concept for a new feature, but my organization is moving to Azure OpenAI for a variety of reasons, so the ability to continue to use this gem with Azure would be great.

We're also moving to an Azure based version of OpenAI so would love to see Azure support in this gem.


Connecting sample and project

From @afrendeiro  #144

Can we reverse the removal of Sample-Project linking? It is true these don't need to exist, and probably shouldn't be used in the looper workflow but are extremely useful outside of it.

Further discussion:

vreuter: I'll address the other points separately, but I want to first focus on the samples/project deal. Why is that good/desirable? I see value in a sample knowing about certain attributes of a project that are sample-independent, but it would be bizarre for a sample to know about other samples via a project, especially with inter-sample independence advertised as a bedrock looper principle. I think it'd be good to facilitate access by each sample to a subset of project metadata, but I strongly oppose the existence of a direct reference of a sample to its project. Imports, function calls, object references -- I think circular is bad.

nsheff: Sample/Project link: I see both sides. I don't have strong feelings at this point. I think if you have a sample, you probably got it through a project, so why do you need the pointer? Just remember where you pulled the sample. But @afrendeiro if you can give a specific use case for when you'd have a Sample, need its Project, and not have accessed the sample through that Project in the first place (necessitating a link to Project from Sample), that would be interesting.

afrendeiro: Well I generally don't ask for features if I'm not using them:
My case is in when running a pipeline. Imagine your pipeline could make use of a few project-centric variables like a specific directory where you want to group a certain type of output for all samples (e.g. bigwig files, some pdfs, etc - this is not the norm for all of the sample's output but just one or two). This variable is common to all samples of the project and could simply be defined by extending the project's config with it. Instead of always having this passed as a parameter, one could simply read the sample's config (which is the only thing I require as input) and get that variable though the project's.
I do agree though that there must be a way to contain circularity and recursiveness specially in the serialization of the objects to yaml (I believe something was/is in place to assure this), so yes it is not straightforward to do this out of the box - it would be good if possible though.

Verdict?

To me there is no verdict yet...

Well, I'd be willing to meet you half way and have only access to the projects first-level attributes, but I also see Nathan's point.
Maybe we can leave it as-is for this release and keep thinking more about this. I'm curious about what other people that are/will be using mostly the models would say.

I'm good with the current level of connection; Sample has access to metadata, derived_columns, implied_columns, and trackhubs, so I'm good with closing this if others are?

Hey @vreuter with all the new attmap stuff, do you think this has been generally addressed? it seems that the attmap attribute filters concept has sort of made it possible to handle this pretty well while keeping the connection, am I right? if so maybe this can be closed?

I see no value in it, but c'est la vie--it's there. https://github.com/pepkit/peppy/issues/315


Optimize checkout - introduce OrderInfo

Optimize checkout - introduce OrderInfo

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4759 Optimize handle_fully_paid_order for order with customer email


Move to rollup from webpack

When i first wrote this library webpack was the only good solution for bundling. As of today rollup is widely adopted for library bundling. I wish if someone can help us with this.

I'd like to help with this one


Error reporting problem of util_filter.py file

When I run the code python evaluate.py, it shows that there is a syntax error in line 658 of util_filter.py. What are these codes for? It’s okay to delete them.

Hi, you can just delete them.


Fix #2825: Vefiry all SwitchToMainThreadAsync calls for CancellationToken usage

Also:

Fixed hanging in Microsoft.R.Editor tests caused by deadlock in TestEditorShell
Fixed a bug that EvaluateAsync->Stop->Start->EvaluateAsync sequence handged until console input is executed

👍


Add recommended build directory to bootstrap_configure

Problem
building in the package srcdir is bad practice
Summary of changes
auto-pick a build directory when invoked from srcdir

While I generally agree that building a package in its source directory is not good practice, I'd like to be less prescriptive about creating a build directory by default and mandating that the directory be created under abs_srcdir as build.
Ideally, there's an iteration of this that has an option(s) for:

Specifying that automatic build directory creation is desired.
a. Specifying the root of the build directory, in which case the script would auto-populate the named build directory beneath that and then configure in that. Failing that, it would default as proposed here.
b. Specifying the build directory, in which case it would configure in that. Failing that, it would default as proposed here.

With that in mind, perhaps:

--with-build-directory - Auto-generate a build directory under abs_srcdir based on config.guess and configure options and configure into it.
--with-build-directory=<DIRECTORY> - Manually specify the build directory as DIRECTORY and configure into it.
--with-build-directory-root=<DIRECTORY> - Auto-generate a build directory under DIRECTORY, auto-generate a build directory under it based on config.guess and configure options and configure into it.

Thoughts?

While I generally agree that building a package in its source directory is not good practice, I'd like to be less prescriptive about creating a build directory by default and mandating that the directory be created under abs_srcdir as build.
Ideally, there's an iteration of this that has an option(s) for:

Specifying that automatic build directory creation is desired.
a. Specifying the root of the build directory, in which case the script would auto-populate the named build directory beneath that and then configure in that. Failing that, it would default as proposed here.
b. Specifying the build directory, in which case it would configure in that. Failing that, it would default as proposed here.

With that in mind, perhaps:

--with-build-directory - Auto-generate a build directory under abs_srcdir based on config.guess and configure options and configure into it.
--with-build-directory=<DIRECTORY> - Manually specify the build directory as DIRECTORY and configure into it.
--with-build-directory-root=<DIRECTORY> - Auto-generate a build directory under DIRECTORY, auto-generate a build directory under it based on config.guess and configure options and configure into it.

Thoughts?

good thoughts. my first pass at this was trying to keep the interface the same as ./configure.
maybe another script?  I'd like to have another script that outputs the name of the build directory given a set of configure args...

closing because I don't really need this from nlbuild-autotools


[Bug] OSError: [Errno 9] Bad file descriptor

Describe the bug
Details：https://github.com/InternLM/InternEvo/actions/runs/8301063319/job/22722755612

Environment
python 3.10
Other information
No response

https://github.com/InternLM/InternEvo/pull/89，已修复


[Dataverse] Ontology connexion - Details page - Audit metadata

Feature: [Dataverse] Ontology connexion - Details page - Audit metadata

As a end-user navigating the portal,
I want to see the details of a resource
So that I can have more information about a resource registered in the Dataverse

Acceptance criteria
Scenario :  Audit metadata retrieval from ontology for a resource
Given that I am an end user navigating the portal
When I click on see more of a resource
Then the system should retrieve the current audit metadata available for that resource in the ontology
And display that content according to the UI in https://github.com/okp4/dataverse-portal/issues/120#issue-1635417468
Environment

staging URL: https://app.staging.okp4.network/

Additional informations
None.

Tech Spec: see here.


allow anonymous uuid generation to be customizable

:bulb: Motivation and Context

this makes it aligned with the js/ts sdk

https://github.com/PostHog/posthog-js/blob/55694356e168568700f3370f26d636cbe09bf795/src/posthog-core.ts#L169
https://github.com/PostHog/posthog-ios/pull/133

:green_heart: How did you test it?
unit test
:pencil: Checklist

[ ] I reviewed the submitted code.
[ ] I added tests to verify the changes.
[ ] I updated the docs if needed.
[ ] No breaking change or entry added to the changelog.

@jimexist thanks for the PR.
I left 2 more comments, please add a https://github.com/PostHog/posthog-android/blob/main/CHANGELOG.md#next entry under the Next section.


When spread array into object literal, Returns a type containing unnecessary properties

Bug Report

🔎 Search Terms
Spread Syntax, Array into object literal
🕗 Version & Regression Information
This is the behavior in every version I tried, and I reviewed the FAQ for entries about rest syntax.
⏯ Playground Link

Playground link with relevant code
💻 Code

const x = ["a", "b", "c"]
console.log(x)
const y = {...x}
//    expected {[x: number]: string} etc, but got {[x: number]: string, length: number, toString(): string .....}

console.log(y.length) // undefined
//  expected undefined, but got number

🙁 Actual behavior
y(spread array into object literal) 's property type is same as Array
🙂 Expected behavior
y(spread array into object literal) 's property has only array value.

Essentially a duplicate of #9726.


remove newline for date of birth to make personal details consistent …

…with banking style

Can you provide a minimum working example? I don't see a difference:


MF-838 - Migration to Go Module

What does this do?
It adds go.mod and go.sum into the project. Also, we can remove the Go Dep files.
Which issue(s) does this PR fix/relate to?
Resolves #838
List any changes that modify/break current functionality
There are no breaking changes, but if we remove go dep file, we break the Go Dep compatibility.
Notes
We can create specific go.mod for each independent component.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #840   +/-   ##
=======================================
  Coverage   85.58%   85.58%
=======================================
  Files          72       72
  Lines        4702     4702
=======================================
  Hits         4024     4024
  Misses        442      442
  Partials      236      236

Continue to review full report at Codecov.

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

@1995parham Mainflux project includes vendoring (we keep vendor folder in our GitHub). Please make sure that Go mod will include vendoring as well.
Take care that you remove Gopkg.lock and Gopkg.toml files (previously generated by dep tool) when you do module generation, so you can start with fresh go mod files.

@drasko I will update the vendoring folder, and I will remove the dep files. Thanks for mentioning theses.

@1995parham Any progress here?  We are very excited about switching, so if we can help somehow, let us know. Thanks!

@nmarcetic I think it is ready and I have updated the vendor folder. If all contributors prepared for it, we can merge it. 😄

@1995parham Please update your branch, sign your commits and remove WIP label when this PR is ready.

@1995parham I will do it right now

Is this PR ready? Can WIP be removed?
Also, please update the branch.

@drasko It is ready, I will update it.

@1995parham looks like PR got enught approvals. Please resolve the conflicts and update branch so we can merge it.

@drasko, I have updated my branch and resolve the conflicts.

@1995parham Can you resolve conflicts please ?

@nmarcetic I have resolved the conflicts, but there is an issue with MQTT library on the tests.

@nmarcetic I have resolved the conflicts, but there is an issue with MQTT library on the tests.

@1995parham Can you, please, update Paho lib version to 1.2.0?

@1995parham getting there. Only one comment left from my side, and I will approve.

@drasko Yes, you are right. I will check it.

@1995parham plase take into account @dusanb94 's remark for README and also update the branch.

@1995parham please update the branch

@1995parham we had a lot of big changes recently and now they are merged. Can you please update your branch now so that the team can merge your PR tomorrow morning.
Also, please take into account all new vendoring that might have happened in the meantime.

@drasko Yes, I will do it.

@1995parham be careful with DCO, it's failing.

Closing this one due to its high impact. It must be handled carefully by Mainflux core engineer.


Small documentation fixes

Fix various smaller things in the documentation:

Switch to the RTD theme in local builds.
Use Python 3 syntax highlighting (e.g. to highlight the await keyword)
Fix a broken inline link

@cmarqu good point, fixed!


Locust web interface not starting on Windows

C:\Development\Test>locust -f main.py
[2013-02-27 19:30:11,346] INFO/locust.main: Starting web monitor on port 8089
[2013-02-27 19:30:11,348] INFO/locust.main: Starting Locust 0.6.2

...And then it never gets past this point. I am using the example locust file from the website for main.py
from locust import Locust, TaskSet, task

def index(l):
    l.client.get("/")

def stats(l):
    l.client.get("/stats/requests")

class UserTasks(TaskSet):
    # one can specify tasks like this
    tasks = [index, stats]

@task
def page404(self):
    self.client.get("/does_not_exist")

class WebsiteUser(Locust):
    host = "http://<IP_ADDRESS>:8089"
    min_wait = 2000
    max_wait = 5000
    task_set = UserTasks

I am running Windows 7. Any ideas on what I could be doing wrong? Would be happy to clarify more information.

Hi there,
I have a similar question like above , the locust web interface does not appear.
Here are details for my try:
Recently i installed locust in windows and i used the http://docs.locust.io/en/latest/quickstart.html quickstart.
Currently i have locally a web application running in http://<IP_ADDRESS>:8080.
So i used the command line and i navigated to the directory with the file locustfile.py.
Subsequently i executed the command:   locust --host=http://<IP_ADDRESS>:8080
Unfortunatelly the http://<IP_ADDRESS>:8089 link does not display the locust web interface.
Do you have any idea what is the root cause and how to resolve it.
Thank you

I had the same problem.
After hopeless searching, I tried navigating http://localhost:8089 and web interface appeared like miracle.
Give it a try guys.

i have the same issue on windows7.
i follow the example in the quickstart  http://docs.locust.io/en/latest/quickstart.html
and type locust in the windows cmd window. it shows starting web monitor at *:8089 but failed to get it the web page on localhost:8089

@konstantinosalvertis hi, did you close this issue?  i am facing the same issue.

but failed to get it the web page on localhost:8089

@frostHit I don't know what that means.  did you open a browser and try loading http://localhost:8089?
btw, this issue was closed over 5 years ago.  Please open a new one if you actually have an issue.

@frostHit Hi, I couldn't find any workaround for the issue as I remember related to windows 7. Subsequently I set up a Linux server and locust worked fine.

Strange http://<IP_ADDRESS>:8089/ doesn't work but http://localhost:8089/ does.


Block metadata analyzer

A dedicated process/storage for containing calculated information on block level.

Changes are now in 0.12.0.


<EMAIL_ADDRESS>hash check failed

I tried to install emacs, but got the hash check error.
❯ scoop install emacs
Installing 'emacs' (29.4) [64bit] from 'extras' bucket
emacs-29.4.zip (150.5 MB) [=============================================================================================================================================================================] 100%
Checking hash of emacs-29.4.zip ... ERROR Hash check failed!
App:         extras/emacs
URL:         https://ftpmirror.gnu.org/gnu/emacs/windows/emacs-29/emacs-29.4.zip
First bytes: 50 4B 03 04 0A 00 00 00
Expected:    ad4cd18e671e0957762b4e5b9ac7520b7ab0632ac41071ffa98afb23b17bacbf
Actual:      a4573b62c2b7089d319c0c461d0ed61a597b81ead972c97033ffc0936137e0e0
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/ScoopInstaller/Extras/issues/new?title=emacs%4029.4%3A+hash+check+failed

I tried the command scoop update; scoop cache rm emacs;, it is still the same.
and I find the origin hash code is here.
it is a4573b62c2b7089d319c0c461d0ed61a597b81ead972c97033ffc0936137e0e0,
rather then ad4cd18e671e0957762b4e5b9ac7520b7ab0632ac41071ffa98afb23b17bacbf, which is in the https://github.com/ScoopInstaller/Extras/blob/master/bucket/emacs.json file, the hash code of this file might not been updated.

@kinthero Fixed in 7419159, thank you.


where is microros in CMakeLists.txt

I would like to know which is the corresponding file for this.  I could not find it.
libraries microros  in below code
target_link_libraries(pico_micro_ros_example
    pico_stdlib
    microros
)

pico_stdlib is in pico_sdk/src/common/ but where is microros

@pablogs9 thanks for your reply. but I dont understand why cannot write like this
target_link_libraries(pico_micro_ros_example
    pico_stdlib
    libmicroros
)

why microros instead of libmicroros .
and in libmicroros DIR i cant find something like microros.xxx

https://stackoverflow.com/questions/6561273/is-liblibrary-name-a-so-a-naming-convention-for-static-libraries-in-linux


added enable Cloud Build API

Closes #1814
Added enable cloud build API in step 4 here

This change is

/assign @neuromage

@sarahmaddox Done.

/assign
/ok-to-test
/unassign @neuromage
Preview: https://deploy-preview-1817--competent-brattain-de2d6d.netlify.com/docs/gke/deploy/project-setup/

Thanks @imskr!
/lgtm
/approve


Toggle display between file position and alphabetical

I'd like to see the symbols listed in the order they appear in the file so I get an outline of my massive one page mess. Please.

This would be useful for F# as well.

This is possible to configure using settings.


num_stages=0 becomes num_stages=2

Differential Revision: D65955484

@jiaqizhai Can you take a look, another pin update related change


Add standard metrics custom processor and request duration metric

This PR adds a custom span processor for extracting standard metrics. It also adds request/duration metric.
TODOs:

Add remaining metrics.
Additional tests which includes updated sampler.
Update Changelog.

From end user perspective this will be enabled by default on configuring TraceExporter using extension method.
Adding the complete design plan for extracting standard metrics below
Proposal: Standard metric collection in OpenTelemetry can be enabled by customizing following two parts in SDK:
Sampler: When the sampler decides to drop the data return SamplingResult.RecordOnly  instead of SamplingResult.None. This will achieve following two things:
Span will be created and will be populated with all the information (tags, events etc.)SampledFlag will still be set to false. So, this will not impact propagated sampling decision i.e. traceflags information.
Custom Processor:Add a custom processor which will extract standard metrics information as required. These metrics can then be reported as standard metrics using a metric exporter. Note: no extra processing will be needed on metric exporter side, as the metric data will already be in the format that is expected by backend.

API change check
API changes are not detected in this pull request.


Laravel 5.2 Support

Could you please update to support Laravel 5.2?

Hi @garlian
Run composer require appzcoder/crud-generator:1.0.x-dev
And follow this https://github.com/appzcoder/crud-generator/ for using laravel 5.2

Ok, runs now :-) Thank you!


Clean up for first release

Once the 0.2 published, and the ha-tahoma integration updated to target it, I suggest to rename the package name from tahoma_api to just tahoma. See https://www.python.org/dev/peps/pep-0008/#package-and-module-names
So
from tahoma_api.client import TahomaClient
from tahoma_api.exceptions import BadCredentialsException, TooManyRequestsException

would become
from tahoma.client import TahomaClient
from tahoma.exceptions import BadCredentialsException, TooManyRequestsException

For the name of the project on pypi, tahoma-api must be enough, we are on pypi, for sure it's a python project.


Bump post v0.5.0

Changes
Bumps post repo to v0.5.0 which includes implementation of post generation in Rust.
Test Plan
All existing tests should pass

bors merge


Duplicate badges

I do not know whether this is your job to fix or not, but the US-15 in Virginia has north and southbound routes mapped separately. They're both part of a super-route. Anyway, the badge shows up twice.

For reference, this way is a member of the northbound and southbound U.S. 15 route relations.
I can also reproduce this issue along this way that’s a member of the eastbound and westbound SR 32 route relations in Ohio:

The problem lies in the vector tiles:

This would be fixed in openmaptiles/openmaptiles#1361.

https://github.com/openmaptiles/openmaptiles/pull/1361 is merged now, but I don't know when the tiles become available.

I'll build a full-zoom Ohio so we can test the behavior.

We should probably decide how we want to administratively mark issues which are not resolved in the public demo but are fixed in the underlying software.  It'll probably be awhile before a new release of OMT and a subsequent update of planetiler.

From omt-tools render of Ohio:
(localhost link)

(localhost link)

Maybe fixed? Can anyone confirm?

Yes, the same spot in https://github.com/ZeLonewolf/openstreetmap-americana/issues/423#issuecomment-1157036014 now shows only a single state route shield:


threefold demo: should redirect to login after showing Terms & Conditions

Description
If you're not logged in, It'll show the T&C page, but after pressing "Agree", nothing changes.
Version information

OS: ubuntu 18.04
project commit/version: development
Browser: chrome

Installation method
Locally using poetry.
Steps to reproduce
1- Make sure you're not logged in
2- Go to /marketplace
3- Press "Agree"
Traceback/Logs/Alerts

is this still the case?

I think it's just caching, and it's only reproducible in case of logout, then going to /marketplace manually, and once you do a refresh, it works (and the user is already logged out).
Will close this issue, it's not a problem IMHO, as the user already got redirected to login page again normally.


[Snyk] Upgrade @tanstack/react-query from 4.35.3 to 4.35.7

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

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2023-10-01.

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

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:loudspeaker: Thoughts on this report? Let us know!.


SPIFFE URI Considerations and options

The Secure Production Identity Framework For Everyone (SPIFFE) Project defines a framework and set of standards for identifying and securing communications between application services.
SPIFFE spec recommends adding a unique identifier in the identity document.
In the case of Athenz, the identity document is a X.509 certificate and SPIFFE id is added in SanURI. Today ZTS adds a SPIFFE URI for service identity certificates and role certificates.
The current document is primarily concentrated on service identity certificates.
According to SPIFFE specification, SPIFFE URI takes a following format - spiffe://{trust_domain}/{path}
But SPIFFE specification does not mandate using a specific convention to represent identities.
Current convention adopted by Athenz:

Service identity certificate: spiffe://{athenz-domain}/sa/{athenz-service}
Role certificate: spiffe://{athenz-domain}/ra/{athenz-role}

The current SPIFFE naming convention satisfies the secure naming requirements, but the Istio community made a recommendation about the SPIFFE format involving using the trust domain of the SPIFFE URI.
Istio relies on the following format:

spiffe://{trust_domain}/ns/{namespace}/sa/{service-account}

Trust Domain Validation implementation in Istio rejects the requests from outside trust domains or if it's not in the TrustDomainAliases specified in the MeshConfig. So with the current convention, Istio service mesh would reject all requests coming from an Athenz service of a different domain than the target service’s Athenz domain, without further configuration.
To make using Istio service mesh simpler and easier, we can consider revising our SPIFFE trust domain convention.
According to the SPIFFE specification about trust domains, a trust domain represents a trust root of the system backed by cryptographic keys.
Recommended full SPIFFE URI format
Treat the company as a single trust domain - with a valid dns entry e.g. athenz.io
Service Certificates:
spiffe://{athenz-trust-domain}/ns/{namespace}/sa/{athenz-domain}.{athenz-service}
For non-k8s workloads (e.g. EC2 instances)  use "default" as the namespace name
Some examples:

Domain
Service
Namespace
SPIFFE URI

sports
api
N/A
spiffe://athenz.io/ns/default/sa/sports.api

sports
api
sports
spiffe://athenz.io/ns/sports/sa/sports.api

sports
api
prod
spiffe://athenz.io/ns/prod/sa/sports.api

sports.k8s
api
N/A
spiffe://athenz.io/ns/default/sa/sports.k8s.api

sports.k8s
api
sports-k8s
spiffe://athenz.io/ns/sports-k8s/sa/sports.k8s.api

sports.k8s
api
prod
spiffe://athenz.io/ns/prod/sa/sports.k8s.api

sports.k8s
my-api
N/A
spiffe://athenz.io/ns/default/sa/sports.k8s.my-api

sports.k8s
my-api
sports-k8s
spiffe://athenz.io/ns/sports-k8s/sa/sports.k8s.my-api

sports.k8s
my-api
prod
spiffe://athenz.io/ns/prod/sa/sports.k8s.my-api

fan-sports.k8s
my-api
N/A
spiffe://athenz.io/ns/default/sa/fan-sports.k8s.my-api

fan-sports.k8s
my-api
fan--sports-k8s
spiffe://athenz.io/ns/fan--sports-k8s/fan-sports.k8s.my-api

fan-sports.k8s
my-api
prod
spiffe://athenz.io/ns/prod/sa/fan-sports.k8s.my-api

https://github.com/AthenZ/athenz/pull/2228


Phone Number

Please change the phone number in the footer to have dashes, and call when it is clicked

Fixed by PR #69