Skip_rows, flags loading and estimation reshape fixing

Closes issue #15

You have to write it like this: closes #15 😉

The problem originated from the addendum of the normalization to simplest_example_1.ini, thus the name

The problem originated from the addendum of the normalization to simplest_example_1.ini, thus the name

I did not remember that that configuration file used the RowSelection method, and was wondering how it was possible that you fixed that bug. Thank you


Test Accelerate workflow with branch with fixes for changes due to updated transformers.

nv-accelerate test passed and required changes were merged upstream in https://github.com/huggingface/accelerate/pull/3196


VOC Security Vulnerability

Hi when I attempt to install this library one of its dependencies ( VOC ) has a security flaw and my corporate Nexus repo complains that it is quarantined.  Does anyone know which dependency is using it and if it is necessary as it may make applications vulnerable?

I raised this issue in the voc repository and it has been resolved.


Build Update Retention Task does not appear in On-Premise TFS

Azure DevOps Extension you are using
Build Update Tasks - specifically the Build Update Retention Task
Where are you running it?
On-premise TFS 2018 Update 4
Version of Extension/Task
1.16.1
Expected behaviour and actual behavior
That I would see this task in the list of available tasks to perform.
Steps to reproduce the problem
There are two other tasks that are part of this extension for variable tasks and I see both of those but the Retention Task does not appear.  Is there some reason that this would be the case that two of the tasks would appear in the list and one would not?

This task is set, via the task manifest, to only be available in releases.
The other two tasks in the extension are set to run in builds and releases

@SchulzDL I have been thinking, I am assuming you have a use case for setting build retention within a build. The reason the task does not allow this is because I did not have one. Do you think it is worth changing the flag?
It is worth noting that these build/release usage flags in the manifest are ignored if you are using a YAML build, so that is a wotkaround

Hi Richard,
Thanks for thinking about this.  I have a client where they can’t really use the release pipelines just yet.  They have different branches and builds for their dev/test and release paths.  When they build this final build they wanted that build to be locked and are doing this as the last step of this release build.  They are on TFS 2018 and YAML builds are not there yet.  When they upgrade to Azure DevOps Server 2019, I guess the YAML could be an alternative.  However, they would really like to start using this task now.
Thanks
Donald<EMAIL_ADDRESS>Senior DevOps Consultant  |  Imaginethttp://www.imaginet.com/  | MCTS, MCP
dschulz<EMAIL_ADDRESS>|  P<PHONE_NUMBER>  |  C<PHONE_NUMBER>  |  tw: @justimaginethttp://www.twitter.com/justimaginet  |  fb: justimaginethttp://www.facebook.com/justimaginet
From: Richard Fennell<EMAIL_ADDRESS>Sent: Friday, August 23, 2019 2:58 AM
To: rfennell/AzurePipelines<EMAIL_ADDRESS>Cc: Donald Schulz<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [rfennell/AzurePipelines] Build Update Retention Task does not appear in On-Premise TFS (#537)
@SchulzDLhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSchulzDL&data=02|01|dschulz%40imaginet.com|e6269a1095b346335f3008d727b06408|f5e8862b5513407a9f7db67b33a9001a|1|0|637021510860664904&sdata=dJoZQ36TPOq7%2BJsbEwKz5yUTYUo04gkUKCSYdS6ATc0%3D&reserved=0 I have been thinking, I am assuming you have a use case for setting build retention within a build. The reason the task does not allow this is because I did not have one. Do you think it is worth changing the flag?
It is worth noting that these build/release usage flags in the manifest are ignored if you are using a YAML build, so that is a wotkaround
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frfennell%2FAzurePipelines%2Fissues%2F537%3Femail_source%3Dnotifications%26email_token%3DADCOAHDKK4ETKIUCIKZYMPDQF6YCXA5CNFSM4IOWTV32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD47XZSI%23issuecomment-524254409&data=02|01|dschulz%40imaginet.com|e6269a1095b346335f3008d727b06408|f5e8862b5513407a9f7db67b33a9001a|1|0|637021510860674896&sdata=EFicr0WLD6pgVl5KnggJ9PqXHjBrRV8tROz8FFpMSu0%3D&reserved=0, or mute the threadhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADCOAHC6AX2YE7RXZF5ODVDQF6YCXANCNFSM4IOWTV3Q&data=02|01|dschulz%40imaginet.com|e6269a1095b346335f3008d727b06408|f5e8862b5513407a9f7db67b33a9001a|1|0|637021510860674896&sdata=DyJ9WIsNPF8DKas0bbPyX2D4cRSIHytMtTqPqAKdpiI%3D&reserved=0.

I will make the change to the manifest to enable the task in builds. I just need to check it will work, as it was not designed to

Thanks Richard
Donald<EMAIL_ADDRESS>Senior DevOps Consultant  |  Imaginethttp://www.imaginet.com/  | MCTS, MCP
dschulz<EMAIL_ADDRESS>|  P<PHONE_NUMBER>  |  C<PHONE_NUMBER>  |  tw: @justimaginethttp://www.twitter.com/justimaginet  |  fb: justimaginethttp://www.facebook.com/justimaginet
From: Richard Fennell<EMAIL_ADDRESS>Sent: Friday, August 23, 2019 10:05 AM
To: rfennell/AzurePipelines<EMAIL_ADDRESS>Cc: Donald Schulz<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [rfennell/AzurePipelines] Build Update Retention Task does not appear in On-Premise TFS (#537)
I will make the change to the manifest to enable the task in builds. I just need to check it will work, as it was not designed to
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frfennell%2FAzurePipelines%2Fissues%2F537%3Femail_source%3Dnotifications%26email_token%3DADCOAHCM3IX4OS3RZ7732B3QGAKDFA5CNFSM4IOWTV32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5AZBFQ%23issuecomment-524390550&data=02|01|dschulz%40imaginet.com|102cddfd0f424f5ebb3808d727ec02b2|f5e8862b5513407a9f7db67b33a9001a|1|0|637021766937381671&sdata=cC%2Ff8AjIiV%2BEkgsCxOLwrMGZIP7VT2lw4%2FYik7kVGKo%3D&reserved=0, or mute the threadhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADCOAHCHBWTL4CD6MG4U5JDQGAKDFANCNFSM4IOWTV3Q&data=02|01|dschulz%40imaginet.com|102cddfd0f424f5ebb3808d727ec02b2|f5e8862b5513407a9f7db67b33a9001a|1|0|637021766937381671&sdata=rCLEze1nxZrzPJjTfSXWGlOyS544uvTM%2BqSqGtampG8%3D&reserved=0.

1.17.5 has the change, the build retension task can now be used in a build to set the build's own retension.
All the other options for artifact selection are invalid in a build as they require access to details only stored in a release
Let me know how it goes

As per my standard practice I am closing this issue as I have heard nothing for a while. If this issue remains a problem please do not hesitate to re-open it


win10 pycharm can't run flask test on Chapter 7

File "C:\Users\F1334694\Desktop\python\flasky\app_init_.py", line 18, in create_app
config[config_name].init_app(app)
KeyError: <flask.cli.ScriptInfo object at 0x05548D50>
i can't figure out why this happened. i have set the FLASK_APP=flasky.py.
but when i set FLASK_APP=flasky.py, it worked.
pls help me solve this problem,thx

Does it work from the command line?

when work from the command line, it need set FLASK_APP=flasky.py first

That is also required for PyCharm.

thanks

Thank you

KeyError: <flask.cli.ScriptInfo object at 0x7f35046f8d30>
only problem solved for FLASK_APP=flasky.py or FLASK_APP=wsgi.py  in config file not working if i use app.py or anything else

Hello guys,
I had an error similar to the one reported.
When i tried to access flask shell typing: 'flask shell' after i declared 'FLASK_APP=main.py' in my terminal i had the same error as follow:
File "/Users/eduardorangel/Desktop/PythonProjects/SMSCode-generate_project/app/init.py", line 22, in create_app
app.config.from_object(config[config_name])
KeyError: <flask.cli.ScriptInfo object at 0x7f91d72c0310>
Follow is my code:
https://github.com/eduardohangel2018/SMSCode.git
Please, can anyone help me?

Hello guys,
I had an error similar to the one reported.
When i tried to access flask shell typing: 'flask shell' after i declared 'FLASK_APP=main.py' in my terminal i had the same error as follow:
File "/Users/eduardorangel/Desktop/PythonProjects/SMSCode-generate_project/app/init.py", line 22, in create_app
app.config.from_object(config[config_name])
KeyError: <flask.cli.ScriptInfo object at 0x7f91d72c0310>
Follow is my code:
https://github.com/eduardohangel2018/SMSCode.git
Please, can anyone help me?

KeyError: <flask.cli.ScriptInfo object at 0x7f91d72c0310>

Maybe you are doing the exercise locally and no in an env.
I exported the FLASK_APP environment variable. I used this:
$ export FLASK_APP=flasky.py


RPM repository update excessive download size (Linux)

This issue is with the Visual Studio Code binary redistribution infrastructure for RPMs, at https://packages.microsoft.com/yumrepos/vscode
The RPM repository https://packages.microsoft.com/yumrepos/vscode seems to have accumulated a very large amount of excess data, which makes running a local RPM update operation take a rather long time.
Note that this is not about the size of the Visual Studio Code product RPM itself, but about the amount of data that, for instance, a sudo dnf upgrade --refresh transfers to update the local package database before deciding whether upgrading is required.
Steps to Reproduce:

have Fedora Linux 36 as a baseline system (may apply to other distributions - such as RedHat, CentOS, OpenSUSE - as well)
have Visual Studio Code installed as a product

Note that this will add an RPM repository to the operating system; the content of mine is
❯ cat vscode.repo
[code]
name=Visual Studio Code
baseurl=https://packages.microsoft.com/yumrepos/vscode
enabled=1
gpgcheck=1
gpgkey=https://packages.microsoft.com/keys/microsoft.asc

Now run

sudo dnf upgrade --refresh
and observe the output; for me this is (as I am writing this)

Fedora 36 - x86_64                                     109 kB/s |  21 kB     00:00
Fedora 36 openh264 (From Cisco) - x86_64                10 kB/s | 1.5 kB     00:00
Fedora Modular 36 - x86_64                             118 kB/s |  21 kB     00:00
Fedora 36 - x86_64 - Updates                            20 kB/s |  16 kB     00:00
Fedora Modular 36 - x86_64 - Updates                    18 kB/s |  15 kB     00:00
teams                                                   14 kB/s | 3.0 kB     00:00
Visual Studio Code                                      15 kB/s | 3.0 kB     00:00
Visual Studio Code                                     709 kB/s |  28 MB     00:40

Note the very last line here - this is something server-side on https://packages.microsoft.com/yumrepos/vscode for Visual Studio Code which causes a total of 28 MB to be downloaded, for a total time spent of 40 seconds.
28 MB seems to be excessive and makes determining update packages take a long time.
Do look at the size of the update information for the complete Fedora product (comprising of, I guess, a great many more packages): This is on the order of 70 KB.
Expectation: The vscode rpm update information downloaded should be at most on the order of the size of the full Fedora distribution. I expect a reasonable size to 20 kilobytes - and not 28 megabytes.
This will drastically reduce the amount of data transferred, will accelerate distribution updates, for every single user of the Microsoft RPM repository for Visual Studio Code.

FWIW, splitting the repos for "insider", "exploration", and normal builds might help (and would be natural to do) - but I do not know anything about the impact that this has on the distribution pipeline.

This would be helpful

The vscode metadata file even bigger the whole Fedora, this doesn't make sense.

I think it is because the old versions are never deleted and the file lists keep adding up. https://packages.microsoft.com/yumrepos/vscode/
I downloaded the repodata from https://packages.microsoft.com/yumrepos/vscode/repodata/ and opened it in dbeaver.
select count(*) from filelist where filenames = 'node';

This shows 31971 entries! It's like NOTHING is ever deleted.

it's 2023 now. no change?

it's 2023 now, any progress on this ?

Any updates ? That would help most of the world that do not have multi Gb/s connexion...(me included)

easy solution is to move to VS Codium the open source version, then you don't have Microsoft tracking and spyware, and also this repo problem is fixed!

Now downloading 36mb each time.
The milestone is set back a lot of times, but is there maybe some progress?

Any updates on this?
dnf update is significantly slowed down with a 36MB daily vscode repo download vs the other system repos which are about 2MB combined.

just use vscodium, it's better anyway, and the repo works properly.

just use vscodium, it's better anyway, and the repo works properly.

Allthoug this does not fix the issue, it was a solution for me.
Thanks for the info. Did not know about vscodium but switched now.

Fixed with manual cleanup commands.
I'll create an internal issue to automate the process.
Verification steps for endgame:

Navigate to https://packages.microsoft.com/yumrepos/vscode/Packages/c/
Confirm that there are no code packages older than 1.78.0.

@rzhao271 It looks like there are a few random older versions; are these intentional?

No, but I'll create a new issue for that in addition to the automated cleanup feature request.


Configurable garbage collector

add cleaning to all cache for every period (configurable)

just rerun the image into new container and problem solved (so far)


Create SECURITY.md

@KOSASIH do you want me to merge that example file or do you want me to fill it ? 😊

@KOSASIH do you want me to merge that example file or do you want me to fill it ? 😊

Doing the best you can do Sir...  ☺


raftstore: queue vote

If leader starts campaign after split immediately, follower may not apply the split yet. But if the follower is ready to apply the split, leader should probably apply the split already, so it should be more efficient to let follower start campaign than leader.
ref #1639
@siddontang @zhangjinpeng1987 @hhkbp2 PTAL

PTAL

I prefer using prevote here.

Any data on performance improvement of this PR?

This pr tries to resolve #1675. The policy has changed several times, I will clean the pr after it's verified to work.

I prefer RingBuffer too.
Rest LGTM

PTAL @hhkbp2
LGTM

Ping @hhkbp2

LGTM


[Snyk] Security upgrade php from 7.1-apache to 7.4.26-apache

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 2 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

httpd/ssi-rce/Dockerfile

We recommend upgrading to php:7.4.26-apache, as this image has only 311 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Server-Side Request Forgery (SSRF) SNYK-DEBIAN10-APACHE2-1585740
  929

Server-Side Request Forgery (SSRF) SNYK-DEBIAN10-APACHE2-1585740
  929

Server-Side Request Forgery (SSRF) SNYK-DEBIAN10-APACHE2-1585740
  929

Server-Side Request Forgery (SSRF) SNYK-DEBIAN10-APACHE2-1585740
  929

Out-of-bounds Write SNYK-DEBIAN10-APACHE2-2322058
  886

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Server-Side Request Forgery (SSRF)

Checkmarx One – Scan Summary & Details – 8548136e-223f-42fa-b9e6-5d1e1e456577
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

Path_Traversal
/base/oracle-java/utils/list-versions.py: 57
Attack Vector

Fixed Issues

Severity
Issue
Source File / Package

Client_DOM_XSS
/electron/CVE-2018-15685/src/index.html: 17

Client_Potential_XSS
/electron/CVE-2018-15685/src/index.html: 17

Path_Traversal
/python/PIL-CVE-2017-8291/app.py: 37

Path_Traversal
/base/oracle-java/utils/list-versions.py: 57


Ats1.8 add uploading a file howto

I have added a folder that contains a document that is shown when you click a link in the uploading-a-file-howto. It should not be visible.

What should not be visible, exactly?

The Build said that the selenium-files folder is rendered but is not visible. But that is exactly what I wanted, only the build failed. I tried to create the same mechanic as Mendix uses for downloading the how-to template.


Getting array element does not work

It doesn't seem to be possible to get something when the top-level element is an array.
Rodash.get([{'a' => 1},{'a' => 2}], '[0].a')

... results in:
Traceback (most recent call last):
        5: from /usr/local/bin/irb:11:in `<main>'
        4: from (irb):2
        3: from /usr/local/lib/ruby/gems/2.5.0/gems/rodash-3.0.0/lib/rodash.rb:54:in `get'
        2: from /usr/local/lib/ruby/gems/2.5.0/gems/rodash-3.0.0/lib/rodash.rb:123:in `baseGet'
        1: from /usr/local/lib/ruby/gems/2.5.0/gems/rodash-3.0.0/lib/rodash.rb:167:in `isKey'
NoMethodError (undefined method `has_key?' for [{"a"=>1}, {"a"=>2}]:Array)

reference PR #4

@this-chris awesome, thanks man! Maybe @jakhog, @sdalgard or @brice-morin could help out with merging this? I see they have been active pretty recently in this company repo.
Thanks in advance!

@brice-morin thank you for merging.
For everyone else still using this Gem: In order to have this issue solved, you'll need to gem 'rodash', :git => 'https://github.com/SINTEF-9012/Rodash.git' in your Gemfile, as the fix provided by @this-chris was not released as a new version yet.
@SINTEF-9012 feel free to close this issue as soon as someone from your side has released a new version of this Gem that contains the bugfix. Thanks!

Hi @mrusme
I created a repo that contains almost all Lodash functions.
https://github.com/Attrash-Islam/rudash
I hope you can get what you're looking for.

The gem has finally been updated. Sorry for the wait.


fix(requirements): remove extra empty permission requirement statuses

Description:
Fixes ALM-751

/lgtm
look forward to rebasing on this


Installed putio-automator on linux subsystem error on putio torrents add

I just installed this on Windows 10 subsystem last week and it was working just fine. Today I was doing a clean install and now I get this error when I try to add or watch torrents.
krash@Karens-HP-Laptop:~$ putio torrents add
Traceback (most recent call last):
File "/home/krash/.local/bin/putio", line 11, in
sys.exit(main())
File "/home/krash/.local/lib/python2.7/site-packages/putio_automator/manage.py", line 47, in main
manager.run()
File "/home/krash/.local/lib/python2.7/site-packages/flask_script/init.py", line 412, in run
result = self.handle(sys.argv[0], sys.argv[1:])
File "/home/krash/.local/lib/python2.7/site-packages/flask_script/init.py", line 383, in handle
res = handle(*args, **config)
File "/home/krash/.local/lib/python2.7/site-packages/flask_script/commands.py", line 216, in call
return self.run(*args, **kwargs)
File "/home/krash/.local/lib/python2.7/site-packages/putio_automator/commands/torrents.py", line 52, in add
with_db(app, func)
File "/home/krash/.local/lib/python2.7/site-packages/putio_automator/db.py", line 15, in with_db
func(connection)
File "/home/krash/.local/lib/python2.7/site-packages/putio_automator/commands/torrents.py", line 29, in func
conn.execute("select datetime(created_at, 'localtime') from torrents where name = ? and size = ?", (name, size))
sqlite3.ProgrammingError: You must not use 8-bit bytestrings unless you use a text_factory that can interpret 8-bit bytestrings (like text_factory = str). It is highly recommended that you instead just switch your application to Unicode strings.

Argh, this will probably go away if I upgrade to python3. I'll have a look, thanks for the report. Perhaps you can get around this by renaming one of the torrent files.

#34 uses python 3. please see if this is still an issue.


PYTHON_ARCH does nothing in appveyor

pretty sure this does nothing? (especially because we have the wrong value for py34-x64)

Coverage remained the same at 94.464% when pulling 35971cd6427db7a894db1c043429c9a5fc98a51b on asottile:appveyor_cleanup into 63933eb53bd0d867940e35f7df216c9e0806b49d on dahlia:master.


Upgrade to Orleans 3.5.0 and net5.0 and bump package version 0.0.42-pre

This brings the package up to date with latest libs.

Thanks for the PR... is it possible to update the GitHub Actions as this breaks the build currently by trying to build with dotnet 3.1.
https://github.com/OrleansContrib/Orleans.Sagas/blob/master/.github/workflows/dotnet-core.yml

Yeah, I'll do so.

Thanks!


Error when adding Product Block to Page

When I try to add the product block to the product page, I get the following error.
Doctrine\Common\Proxy\AbstractProxyFactory::getProxyDefinition(): Failed opening required '/home2/wichitc9/public_html/ictmag_v2/application/config/doctrine/proxies/__CG__ConcretePackageCommunityStoreSrcCommunityStoreProductProductOptionProductOptionItem.php' (include_path='/home2/wichitc9/public_html/ictmag_v2/updates/concrete5-8.2.1_remote_updater/concrete/vendor:.:/usr/php/56/usr/lib64:/usr/php/56/usr/share/pear')
I only seem to get the error when the product has options.

I've not seen this before, it looks like something to do with Doctrine's cache files.
Have you tried clearing your concrete5 cache?

Yes I have. Just did it again to be sure.

Can you provide a bit more detail as to what you are doing to trigger this - when you say you are adding a product block, is this on any page, or is on a page type default template?
I've tried adding a product block to a page, picking a product that has options and I didn't have any trouble.
Have you recently moved this site by any chance?  I'm wondering if you don't have write permissions on that config directory.  What about your caching settings, does it make a different with caching on or off?

I just ran through the options you gave me so I could give you exact answers.

It happens whenever i add the product block to any page, and I select a product that has options.

The site is in the same location on the server, but I did recently change the base URL. It was setup on a subdomain while in development and the site has since been made live, so it is no longer on that subdomain. I don't know if that would affect it. I didn't change anything with the site when we went live, I thought concrete5 got rid of the base URL option.

Changing the cache settings does not have any effect on the error.

Thank you for your help and speedy responses on this, I really appreciate it!

I think you should try to find the file mentioned in the error message. In my local install I can find the file __CG__ConcretePackageCommunityStoreSrcCommunityStoreProductProductOptionProductOptionItem.php
I can trigger this same error if rename this file to something else. Check whether the file exists and it has read permissions. If you can't find it, the easiest thing to do is to find a copy from some other install of the same community store version.


Remove Roslyn feed once RTM packages are available.

Revert this commit once the Roslyn RTM packages are available.

https://github.com/aspnet/Razor/commit/271de04d15841d1b379dfab5c71d9a2998a68ad6


Use lambda-packages during zip creation

Hey garnaat!
It might be cool if Kappa could automatically use the lambda-packages during zip creation, to automatically use lambda-compatible versions of popular dependencies.
Just a thought,
R

That would require rewriting application code to inject them, which I don't think is the right way to go here. It'll almost certainly break user's applications in weird ways, as people often build targeting specific versions of a package.


MapBuilder

Create a first attempt for a MapBuilder.
@pauldendulk Is this the way you thought about it?

Oops, merged the wrong merge request. First time ever for me. Reverted it just now.


Why do I need to install Visual Studio for using this?

I am trying to use this but while trying  to compile, it is asking that I need to install VS. Just wondering why is that?

Gotcha. Thanks for the response. I appreciate it. I ended up installing VS
pending reboot which I will do later :) . Out our curiosity, is this
evaluating the Static code analysis or just the simple libraries. I am
trying to do Static Code analysis and following this guide for evaluation
https://analysis-tools.dev/tag/rust
most of the tools are not good. Any recommendations for running SAST.
On Wed, Sep 23, 2020 at 8:21 AM Shnatsel<EMAIL_ADDRESS>wrote:

The x86_64-pc-windows-msvc target always requires Microsoft components.
That's just how that target works. This is not specific to this crate, this
is just how Rust works in general.
I suggest either installing x86_64-pc-windows-gnu target, which is
self-contained, or following an online guide to install the necessary
Visual Studio components.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Shnatsel/rust-audit/issues/25#issuecomment-697532733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANPYDW4QAOA7KESFQCYER3SHIHAXANCNFSM4RXDZLDQ
.

Well, the Rust compiler is already an in-depth static analyzer, so you typically don't need much else. clippy can catch some more issues.
See also https://github.com/rust-secure-code/projects


Group Accounts API structs

Added Group Accounts API response structs, made chrono dependency no longer optional, and also added the optional group_name for create project request

CC @janasheehan


Multiple instances on a dashboard?

Is this possible? At the moment I can only get it to display in one dashboard group, even if selected to others.

Same here. I've tried a few things, but not managed to overcome this.
Contributions welcome.


I found a broken link

Hello

Go on : https://docs.docker.com/v1.7/reference/run/
Click on "Option types"
Broken link on "https://docs.docker.com/v1.7/reference/reference/reference/commandline/cli/#option-types"

That page is archived, I believe what you're looking for is here.

Again, thank you jaredstroud! Closing ticket.


How to only display specific instance types?

I tried keeping 3 instance types (c3.large, c4.large, m3.large) but I cannot currently find how to do that. I seem to recall this was doable in the past.
As well, browsing to this page (link generated by adding a search term) raises a javascript error in the console sometimes, or the filter isn't respected (maybe stored in cookie?).
All in all - is there a way to quickly compare 3 instance types on the current filter system? Thanks!

Searching as a regex like (c3.large|c4.large|m3.large) will do this, though the storing of the filter term in the URL is broken as you've pointed out

Oh thanks, I had no idea! I'll try to submit a quick PR to document this directly on the site, if you'd like.

I had no idea either. Had to go poke around those docs. :)
If I remember correctly you won't be able to edit that search box UI directly since the library takes care of drawing it.

Closing this since it's not really a bug/issue


Entrypoint error when running app-start.sh

Describe the bug
Getting the following error when trying to run app-start.sh:
env: can't execute 'bash
': No such file or directory

Looks like it is coming from here:
https://github.com/Unstructured-IO/unstructured-api/blob/2a0f3b7fa0966747b965afea15ec7f71b88b118a/scripts/app-start.sh#L1
To Reproduce
Tag: 0.0.80
Build Dockerfile:  make docker-build.sh
note: had to modify because there is a \r line separator that needs to be removed. Check for this in app-start.sh, but doesn't seem to be the issue.
Start app: make docker-start-api
Get error:
': No such file or directory
Note: Also entered container with make docker-start-bash and ran scripts/app-start.sh with same error.
Environment:

Build Env: Windows 11, MINGW64 terminal; Also tested on WSL2 Ubuntu with same results

Additional context
I see a similar issue #261 referencing a whitespace, but that is not present here.

There's some hidden value on the new line below.  Removing the empty line and recreating it fixes the problem.  There seems to be a line separator or other hidden character in a lot of these .sh files that causes issues in certain environments. Is there a way to remove them all and make sure future commits are clean?


Drawing icons on the WebGLPoints layer has jitter and position deviation problems on some mobile devices

https://user-images.githubusercontent.com/60699706/220842783-83850e1b-717c-45a8-b5f4-c1566d0cde47.mp4
Not all devices have this problem. Is it a device problem? What should I do with my device or whether I need to pay attention to some problems when rendering,

Hi and thanks for the bug report. It looks like a floating point precision issue.
Can you please tell us:

which version of OL you are using?
which device was used for this recording?

and either provide us with a reproduction case using e.g. Codesandbox, or at least an extract of the dataset containing the points in the video? Thanks

My ol version is ^ 7.2.2 "My device is directly customized with the manufacturer. What parameters do you need for the device。 @jahow

Then could you maybe share a capture of this page? https://webglreport.com/
More specifically the orange boxes on the right showing vertex and fragment shader float precision.

Is this the screenshot? @jahow

Thanks! is there any way for you to provide a simple reproduction case?

Is there a problem with my device parameters？

Looking at the screenshots you attached, your device looks quite capable of providing a normal rendering of these points. I'm leaning towards a problem with how float precision is specified in shaders. I think doing something like this might help, if anyone feels like contributing a fix:
https://github.com/openlayers/openlayers/commit/e8261410c36fd49ac1f6a80c9947326b4d2fac68#diff-d7b28b24eafd2f092d93aa0154e9637ab92aee319f6a8d89cefb8f105bb3e5dbR34-R38

This problem bothers me very much. Can you help me to ask for advice from people who may be familiar with this problem? I need to use these methods to carry out new projects next, so I'm a little urgent @jahow


adding review

Some things I would like @GoRedbirds to review.
Please look over because I do not think some of it is working.

Was the codebit ready?


[BUG] Network Config Creation Issues JSON Unmarshalling

Describe the bug
When you're building a Network Config for your Cluster Network, there is a UI issue with JSON unmarshalling.
To Reproduce
Steps to reproduce the behavior:

Go to Networks
Go to Cluster Networks/Config
Create Cluster Network
On created cluster network, then click Create Network Config
Build Network Config, with a network and nic
Go back, build another network config with the same nic
Input Miimon: 100
Input MTU: 0
Click "Create" ,you should see some error like: admission webhook "validator.harvester-system.harvester-network-webhook" denied the request: Internal error occurred: could not create vlanConfig test because it overlaps with other vlanconfigs matching node(s)*
Remove then the MTU & Miimon
The error should be displayed of something like: admission webhook "mutator.harvester-system.harvester-network-webhook" denied the request: Internal error occurred: json: cannot unmarshal string into Go struct field BondOptions.spec.uplink.bondOptions.miimon of type int

Screenshots

Expected behavior
Not seeing JSON unmarshalling errors on BondOptions.spec.uplink.bondOptions.miimon
Support bundle
supportbundle_a55276f1-d2d3-4885-a91f-905a3e6d9aa0_2022-10-06T20-53-24Z.zip
Environment

Harvester ISO version: v1.1.0-rc2
Underlying Infrastructure: Dell PowerEdge R720

Additional context
( POST
https://harvester_VIP_IP/v1/harvester/network.harvesterhci.io.vlanconfigs )
Request:
{"type":"network.harvesterhci.io.vlanconfig","metadata":{"name":"eno4-net"},"spec":{"uplink":{"nics":["eno4"],"linkAttributes":{"mtu":""},"bondOptions":{"mode":"active-backup","miimon":""}},"clusterNetwork":"extra-enonet"}}

Response:
{"type":"error","links":{},"code":"InternalError","message":"admission webhook \"mutator.harvester-system.harvester-network-webhook\" denied the request: Internal error occurred: json: cannot unmarshal string into Go struct field BondOptions.spec.uplink.bondOptions.miimon of type int","status":500,"type":"error"}

Thanks for the fix @n313893254 !
I've validated on
Harvester, ISO: master-39c2bb2a-head
This looks great! :+1: :smile:
I'll go ahead and close this out.


Support using the oDrive encoder as positional source

This is actually two stages.
First read the value from the oDrive.
Second, when the CC3D is set to use the oDrive as source, switch to position control altogether.

Hi Werner, I read your progress about oDrive. Are you working on this project? Or not? How about position control and slipping?

Hi Werner, some news? I read about positional control, can you give me details?

The current code is working. I would like to test it some more, though.


Automatically decode x-ms-client-principal header

Add a user property to the HttpRequest that contains an object representing the logged in user.
If a user is logged in, the platform sets the x-ms-client-principal header, which is in base64 encoded JSON. Both AppService/Functions authentication and Static Web Apps authentication support this header. However, they each have a slightly different schema.
Option 1: automatically decode the header into req.user and leave it to the developer to figure out which type they are working with.
Option 2: automatically decode the header into req.user in a common type. The worker takes care of mapping fields from each schema into the corresponding fields in the common type. The developer can expect the same type no matter which auth the app is using.

Option 3, which I think is the one I'd suggest:

Add a user property to the http request object if x-ms-client-principal exists:{
  "type": "AppService", // or "StaticWebApps"
  "id": "b18031c7-46de-49c5-b68a-b562af6876a7",
  "username"<EMAIL_ADDRESS>  "identityProvider": "aad",
  "claimsPrincipalData": { ... }
}

claimsPrincipalData - base64 decoded and JSON parsed x-ms-client-principal
type - determine based on the properties in claimsPrincipalData

Property
AppService
StaticWebApps

id
x-ms-client-principal-id header
userId in client principal

username
x-ms-client-principal-name header
userDetails in client principal

identityProvider
x-ms-client-principal-idp header
identityProvider in client principal

Closing this issue, implemented in this PR #483.
Basically following @anthonychu's option 3. In HttpRequest, the user property is added:
/**
  *  Object representing logged-in user, either through
  *  AppService/Functions authentication, or SWA Authentication
  *  null when no such user is logged in.
  */
user: HttpRequestUser | null;

which has the following interface:
/**
 *  Object representing logged-in user, either through
 *  AppService/Functions authentication, or SWA Authentication
 */
export interface HttpRequestUser {
    /**
     * Type of authentication, either AppService or StaticWebApps
     */
    type: HttpRequestUserType;
    /**
     * unique user GUID
     */
    id: string;
    /**
     * unique username
     */
    username: string;
    /**
     * provider of authentication service
     */
    identityProvider: string;
    /**
     * Extra authentication information, dependent on auth type
     * and auth provider
     */
    claimsPrincipalData: {
        [key: string]: any;
    };
}


422 error on login with github

@maximepvrt, could you provide more information on this issue?
Ideally, screenshots, console logs and a way to reproduce the error.

https://github.com/nuxt/nuxters/assets/1840026/3c96c620-0b74-45a2-a93d-e40f29dd0f82
hello,
I encountered an issue while trying to connect to github using oauth through the brave mobile browser. whenever I attempt to log in, the browser redirects me to chrome for the authentication process. however, even after successful authentication, i receive a 422 error.
Steps to reproduce:

Open Brave mobile browser on my Google Pixel 5 running Android 13.
Navigate to the GitHub login page and click on the "Sign in with GitHub" button.
Brave redirects to Chrome for authentication.
Complete the authentication process on Chrome.
After successful authentication, I encounter a 422 error.

Expected Behavior:
I expect to be able to log in to GitHub using OAuth directly through the Brave mobile browser without any errors or redirections to other browsers.
Additional Information:
Interestingly, even when I use the Chrome browser on my device, I still face an inconvenience. When I click on the "Sign in with GitHub" button, I am prompted to select the Chrome browser again for the authentication process, which seems redundant.
Best,

The errors comes from https://github.com/nuxt/nuxters/blob/373768a7995dfcc3cc4b970ad539e4c75d8e9ec1/server/routes/oauth/github.ts#L10-L16 it seems that it cannot keep the cookie between the two apps 🤔

Thank you for your response @Atinux and for identifying the source of the issue.
Regarding the behavior of the mobile browser, I have noticed that when I click on the "Sign in with GitHub" button from both the Chrome and Brave browsers on my mobile device, I am prompted to choose an application for the authentication process.
Could you kindly shed some light on why this app selection prompt is appearing? Is there a specific reason for this behavior, and is there a way to configure it so that the authentication process can be streamlined without the need for manual app selection?

https://github.com/nuxt/nuxters/blob/main/components/home/HomeCard.vue#L92
Why not use the "to" attribute of the UButton ? currently it generate a <a> inside a <button>

Thank you @Atinux. I tested, but …
In regard to the mobile browser behavior, I've come to realize that the prompt to select an application for the authentication process occurs due to the Android GitHub application intercepting all GitHub URLs. Subsequently, it suggests redirecting the externally generated GitHub link to a browser (without brave, my browser 😅)

https://github.com/orgs/community/discussions/51846


Added dependency on rspec-its

This was added because we hit a need for its() when trying to publish an update to rs_jmx_bridge.  It required us to update the version of cookbook development, which didn't currently include rspec-its.


How to delete a topLevelCell?

I want to delete the top level cell named "SMOKE" in the example, how can I get that?

Do as follow:
1.delete data from datasource
2.update topCellLevel
[topCellLevel reset];
[topCellLevel setCellCount:[handler noOfRowsInRootLevel]];
3.delete cell


It doesn't appear to be working...

the server has been on for multiple days and doesn't appear to actually be renewing the vaults

Getting the data for the block, it would seem that the time doesn't actually seem to be going down.

It's working as intended. It's not an actual cooldown that is ticking, rather, it shows when the cooldown is going to end. I made it this way because if I used a cooldown, the Vault block would have to be loaded all the time for the cooldown to tick down. When the world time reaches the value of the 'cooldown_ends_at' variable, the Vault block removes the player for whom the cooldown is over.

Also the cooldown for the non-ominous variant of the Vault block is 10 hours in the real world and of couse you would have to be in the world itself for the world time to increase and therefore the "cooldown" of the vault block to "tick down".

Yeah no the problem is I last time opened the vault more than 40 hours ago.

But when you opened it 40 hours ago, have you played in your world for a total of 10 hours or more since then?

I can confirm that this is a bug, when no one is on the server the Vault block increases the value of the "cooldown_ends_at" variable.


Bump up version number to 3.3.0dev

Motivation

For the next release.
Description of the changes

@gen740 Could you review this PR?

Codecov Report

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@          Coverage Diff           @@
##             main     #27   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?      13
  Lines           ?     607
  Branches        ?       0
======================================
  Hits            ?       0
  Misses          ?     607
  Partials        ?       0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Login error (2401): username or password wrong

Hello,
I can't log in and all my details are correct.
Any idea ?
Is the email address required for the tuya app or for the Tuya Developer website ?

same problem

same problem here. Is it possible that it only works with Smart Life login and not with Tuya App login. Which would be a pain, because there is no easy way to migrate from Tuya App to Smart Life App and vice versa.

Change your password in Tuya APP, solved for me.


DO NOT MERGE! Fast Forward stable to 062e9fd8b484591e51ff17a682f484383f5149e8.

This PR will be merged automatically once checks are successful.

Jenkins job istio-testing/pr-stable passed


OAuth1a authorization feature implementation

Current PR contains the implementation of the OAuth1a feature. With this integration, a shift of paradigm is introduced from the initial authorization model established with OAuth2.
An overview of the two authorization models.
OAuth2
For OAuth2, getting the user authenticated is a two-step process:

Step 1:

Clients sends request to get authenticated and is redirected to the service's server for providing authorization credentials.
On successful operation, the server returns an authorization code.

Step 2:

Using the authorization code from step 1, Client sends it to the service's server in exchange for an access token and a refresh token.

Handling requests
Requests made to the API will contain the access token from the database in the authorization header.
OAuth1a
OAuth1a introduces an additional step, as the authorization flow will look like this:

Step 1:
a. Client sends a POST request to get the OAuth request token with the following parameters:
- oauth_callback
- oauth_consumer_key (represented by ClientId)
- oauth_nonce
- oauth_signature - HMAC-SHA1 encrypted string of this format: [HTTP Method(Uppercase)]&[Request URL]&[Authorization Parameters]. The encryption key is a string with this format: [Consumer Key (represented by ClientSecret)]&[OAuth Token] (in this step, we do not have a token, so I pass an empty string).
- oauth_signature_method=HMAC-SHA1
- oauth_timestamp
- oauth_version=1.0
Parameters need to be passed in an alphabetical order.
b. Server's response will contain several parameters passed as query string values: ?oauth_token=...&oauth_verifier. The OAuth token is now cached - the reason behind this is that in the OAuth1a flow I cannot use a state parameter to identify in the next step which service was authorized (I can pass it, but the service does not send it back), so I use the token to identify the service.
c. Using the response previously received, the authorization URL is built.
Step 2:

Clients makes an authorization request using the URL created in the previous step and is redirected to the service's server where he will provide the authorization credentials.
Server responds with an OAuth Verifier code.

Step 3:

Using the OAuth Verifier code from Step 2, and the OAuth request token from Step 1, Client sends them in exchange for an access token.
Received access token is saved into the database.

Handling requests
Requests made to the API will contain the parameters passed in Step 1.a, but additionally will include the oauth_token. This means that the signature will be hashed using a different key format.
In the implementation process the following services have been tested and used as reference:

Twitter
Flickr

With the latest commit I have also included the feature to add manually an access_token and a token_secret for OAuth1 based services.
For consistency, I have aligned the UI for the 3 components used to manually add details.
OAuth1

OAuth2

Api Key

Additional updates included:

when issuing sample requests, I changed to pick the path from the service detail instead of the query string, as in the case of Flickr ("/rest?nojsoncallback=1&format=json&method=flickr.test.login) an URL contains already query string parameters that got removed.
added a flag called UseRequestTokenWithExtendedParametersList to toggle the append operation of additional parameters when requesting the access token. And it will take into consideration the RequestTokenMethod - usually the services use a POST request, Flicker uses a GET.
on receiving the oauth_token_secret, Flickr uses it for retrieving the access token, therefor I cache it with the oauth_token.


V0.12.1.x sentinel pr 2

Some JSON parsing fixes in the governance code, now in sync with the current version of sentinel

utACK 24b3364


Cannot assign to property: 'selectedTitleColor' setter is inaccessible

When I first create the Segmented Control programmatically, I can set this property via the Options:
self.modeSelector = BetterSegmentedControl(             frame: CGRect(x: self.view.bounds.midX - view.bounds.width/8, y: 25, width: view.bounds.width / 4, height: 30.0),             titles: ["?","18+"],             index: 0,             options: [.backgroundColor(Constants.ColorScheme.Seafoam),                       .titleColor(.white),                       .indicatorViewBackgroundColor(.white),                       .selectedTitleColor(Constants.ColorScheme.Seafoam),                       .cornerRadius(self.modeSelector.frame.height/2),                       .titleFont(UIFont(name: "HelveticaNeue", size: 16.0)!),                       .selectedTitleFont(UIFont(name: "HelveticaNeue", size: 16.0)!),                       .bouncesOnChange(false),                       .panningDisabled(true)])
I'm trying to programmatically set the selectedTitleColor but Xcode is telling me this property is inaccessible:
self.modeSelector.selectedTitleColor = Constants.ColorScheme.Coral
This part of my code was working fine before I updated to Xcode 9 and Swift 4 but now it's failing the build. Has anyone else had this issue? I can't figure out how to get around it other than commenting it out.

Yes, indeed.
Configuration is now possible via the options property. I maybe need to add the change the to the README.
You can replace your old line with the following:
self.modeSelector.options = [.selectedTitleColor(Constants.ColorScheme.Coral)]
Let me know if that worked for you.

That did it, thanks!!

@gmarm Unfortunately not working for me because
Type 'BetterSegmentedControlOption' has no member 'selectedTitleColor'


Add ostree.passwd stage to edge types

Requires https://github.com/osbuild/osbuild/pull/620
When an edge/ostree image type is requested and a parent commit is specified, we want to pull the commit and make sure that existing user and group IDs don't change during an upgrade.

Done in #1659


Modal

Modal
A modal is a dialog box that is displayed on top of the current page or another dialog box.
Functionality
→ Appear as a result of user interaction.
→ Used to provide additional information in context.
→ Used when you want to interrupt the user flow to something more important (e.g alertdialog)
→ Used for requesting input from the user.
Best practices

always give clear exit routes to the user.
give a descriptive title
give the action buttons inside the modal clear names
give context to the modal (it should never occupy the whole screen)
add a darkened/dimmed layer underneath the modal
never disrupt the user journey, the modal should be triggered as a result of user interaction with the page, such as a button click.

Implementation
TODO
Examples
https://react-bootstrap.github.io/components/modal/
https://reach.tech/dialog/
https://www.w3.org/TR/wai-aria-practices/examples/dialog-modal/dialog.html
https://design.nib.com.au/language/master/#/organism/modal?_k=bezgi3
Accessibility
https://www.w3.org/TR/wai-aria-practices-1.1/#dialog_modal
https://www.w3.org/TR/wai-aria-1.1/#dialog
Resources
https://uxplanet.org/best-practices-for-modals-overlays-dialog-windows-c00c66cddd8c
https://xd.adobe.com/ideas/process/ui-design/best-practices-for-designing-overlays/
https://www.w3.org/TR/wai-aria-practices-1.1/#dialog_modal
https://www.w3.org/TR/wai-aria-1.1/#dialog
Visuals
 Any relevant visuals that help describe the component.
Additional context
 Anything that does not fit into the other categories.

Hey, thanks so much for the submission! This is a great start.
A few notes:

A modal is a dialog box that is displayed on top of the current page or another dialog box.

I've used the word "overlay" previously. What do you think about something like:

Modal is an overlay displayed on top of the current page.

I think that nested modals aren't very common so it might be unnecessary to specify "or another dialog box"

I like the points under best practices! Do you think we could elaborate a bit on some of those points?
For example:
When mentioning clear exit routes, specify what kind of exit routes do we specifically mean? Clicking on the backdrop, a visual close button, and Escape key come to mind at first.
Speaking of descriptive titles, let's add some example use cases and copywriting tips. Something like we have for Notification.
I think we can also take some content from the following resources I've used on Modals:

https://www.nngroup.com/articles/modal-nonmodal-dialog/
https://polaris.shopify.com/components/overlays/modal

If you'd like, feel free to start working on the article locally and throw up a pull request whenever you feel ready. We can iterate from there onwards!

Hey! thanks for the tips, I`ll work on the article this weekend :D


Adding the trajectories endpoint to OPTIMADE

During the OPTIMADE meeting in June 2021, we discussed in room 2 how the OPTIMADE standard could be expanded to allow the sharing of trajectory data. Based on the discussions with amongst others @giovannipizzi, @gmrigna, Adam Hospital, Tristan Bereau, Amit Gupta, Corey Oses, Ellad Tadmor, Maithilee Motlag, and Daniel S Karls, I have written a first draft for adding a trajectory endpoint. This will hopefully be a good base for further discussion.
The PR adds a trajectories section (in this PR section 7.3), that defines how a client can query and retrieve data from trajectories and how the server can share this data. We have tried to do this in a general way so that many different kinds of trajectories can be shared.

Can we add and endpoint for bulk downloads?

Can we add an endpoint for bulk downloads?
What would you want to do with that endpoint that could not be done with the endpoints that are already under discussion?
The server is already allowed to send back as many trajectories as it wants in one go unless the client specifies a page limit.

Can we add an endpoint for bulk downloads?

What would you want to do with that endpoint that could not be done with the endpoints that are already under discussion?
The server is already allowed to send back as many trajectories as it wants in one go unless the client specifies a page limit.

Both the client and the server can impose limits. What I'm thinking at is that the server might have to impose limits (we're returning data in JSON, that in most implementations has to be fully loaded in memory to be parsed, so the server will have to paginate). However there are other formats that would allow a simple file download (possibly also with streaming, reading one line at a time). Or, simply serve the file in the format stored in the underlying database.
Therefore, having a way to allow bulk downloads I think is very important - otherwise for big trajectories, the server will have to convert to JSON, paginate, the client will have to perform many paginated request, put all together, and reconstruct the trajectory in some other format. A bulk download would be much faster.

We could allow references/links that point directly to the (compressed) output files that contain the trajectory data.
Such as xyz, pdb etc.
This would however also apply to the structures end point, where databases may also want to supply the original pdb and CIF files. So I do not see this as an issue specifically for the Trajectories.
There is also PR#360 that suggests a files end point.
If that PR gets merged, we can also update our description for the trajectory endpoint and add a piece about how to share the trajectory data in these file types.

This is a large addition to the specification, albeit well-written. I personally do not deal with trajectories thus my comments may look naive, but wouldn't it be simpler and sufficient to define a trajectory by a sequence (collection maybe, #386) of OPTIMADE structures? I am aware that the current proposal introduces means to "pack" values using frame_serialization_format mechanism, but I wonder whether added complexity vs. benefit ratio is optimal.

Thank you for your comments.
There are several reasons why a trajectory can not be stored as a series of structures in a good manner.
An MD trajectory can easily contain 100.000 time steps.
To save storage space, not all the fields will be stored for every frame. In a project I did as a student I would store the constraint force every 5 time steps. Yet, the positions I would only store every 100 steps.
The cartesian_site_positions are a mandatory field, so I would not be able to create a proper structure for all the time steps where I have data on the constraint force. Even if I had that data, storing that many structures would take up a lot of hard drive space.
There would also be so many structures that queries with multiple properties would be very slow.
These structures would also drown out structures which occur only once. So I think we would have to, somehow, distinguish them from the normal structures any way.
To keep things as close as possible to the current OPTIMADE we have therefore implemented the reference frame. So you can still query trajectories in the same way as you would do with a normal structure. There are still some rare cases where this could cause problems, but I think it should be sufficient for the vast majority of the trajectories.

Merkys: Both requesting frame ranges via URL query parameters and returning their values inside trajectory entries also seem to deviate from OPTIMADE-ish and JSON:API-ish way of doing things. Maybe frame range could be made a subresource of a trajectory? For example, instead of requesting /trajectories/1?first_frame=10&last_frame=100&frame_step=20 we could request /trajectories/1/10-20-100? This would then return fields describing the frame range, but not the trajectory.

@giovannipizzi requested that I would include the frame ranges within the response. If I remember correctly, he found it clearer to return it with the trajectory property as it may not be immediately clear what the first frame and last frame are in case the frame_serialization_format is explicit_regular_sparse.
The method you suggest for selecting frames is quite universal. It should also apply to other future endpoints that could have some form of indexing. So we will need to consider such future use cases as well.
I think that 10-20-100 would at present be a valid id. So we would need to add a rule like:   "id's cannot contain a dash '-' ".
Do I understand correctly that with " This would then return fields describing the frame range, but not the trajectory."  you mean you would not return the fields: available_properties, reference_structures, nframes and reference_frame when only a subrange of the trajectory is requested?


fix!: Failing validations of strings with extended ascii characters

Hi,
DVT while performing row validations converts all columns to upper case before computing the hash function. This is generally OK, except when the upper case function works differently in different databases. For example in Teradata applying upper case to an extended ascii character (e.g. á, â or é) has no effect. Whereas in BQ applying upper case transforms them to Á, Â and É. The hash comparisons then fail even when the columns hold the same values.
This pull request removes the upper casing so these hash comparisons will succeed.
BREAKING CHANGE: Validations that succeeded in the past may now fail. For example a column holding "November" would successfully validate with a column holding "NOVEMBER". Without the implicit upper case, such a validation will fail. One option is for the user to use Custom Calc Fields
Sundar Mudupalli
Closes #1170

/gcbrun

Team,
Here is how Teradata compares character columns from their documentation. This maybe why we initially did rtrim and upper.  Source is here
For case blind comparisons, any lowercase single byte Latin letters are converted to uppercase before comparison begins. The prepared strings are compared and any trailing pad characters are ignored.
A case blind comparison always considers lowercase and uppercase Cyrillic, Greek and full-width ASCII letters to be equivalent. To distinguish lowercase and uppercase Cyrillic, Greek, and fullwidth ASCII letters you must explicitly declare CASESPECIFIC comparison.

/gcbrun


How to close a websocket connection using the lib from a Node.js app?

Hi,
I'm using the lib in a Node.js app.
Is it possible to programmatically, explicitly, close a connection (close the related websocket) to a bot from the app?
Many thanks.

No, unfortunately, this is not an option. The web socket is part of the server running the bot. In order to close the web socket, the server would have to be restarted with the web socket disabled.

Hi @stevkan, thank you for your reply. My question was about closing the websocket client-side, not on the bot server. In order to clean my app sockets connections.


contenteditable within Shadow DOM does not fire 'onSelect'

It seems that a contenteditable <div> within ReactShadow does not fire an onSelect event. This event is usually fired when you click anywhere within a contenteditable.
I'm actually not certain whether this is a problem with the Shadow DOM spec or a problem with react-shadow, but I thought there might be some insight here into the problem. I've made a minimal codepen that illustrates the issue: https://codepen.io/anon/pen/JJPBaL?editors=1011
If you click within the red area (outside the Shadow DOM), you'll see a log in the console, but if you click within the blue area (inside the Shadow DOM), you don't see anything written to the console.

As onSelect for contentEditable fields is non-native, I'd assume it's an issue buried somewhere in react-dom's SelectEventPlugin. Without stepping through that code I'm not sure. However I'd presume it's attaching the event to a node above the shadow boundary, which is blocked during its bubble phase by the boundary.
What is the purpose of having an onSelect event on the  contentEditable field? As onClick seems to achieve the same thing. I must be missing something.


Automatically update available Exoscale DBaaS Versions

Context
In #50 and #49  we added validation of the provided Exoscale Database version through an enum. This however now requires us to manually update the valid version when there is an upstream change.
We should try to find a way to automatically update these versions. Maybe we could somehow make renovate aware of these changes.
Alternatives
If this isn't doable with reasonable effort we could keep the manual process, however we might be able to create some testcases to see if the current versions are still valid

Looking into this, adding a custom datasource to renovate for a single repo isn't reasonable. We could however add a scheduled github action that calls a Go script that updates the DBaaS versions and creates a PR.
That would mean we need to:

Write Go script that fetches the available versions using https://openapi-v2.exoscale.com/operation/operation-list-dbaas-service-types and patches them.
Add scheduled Github action that runs script daily and creates a PR if there is any change using https://github.com/marketplace/actions/create-pull-request or similar.
Make sure the PR is noticed by requesting review from schedar or making sure that there is a Jira ticket for it.


AdaptiveTheme.of(context).setTheme is not working

I am setting both Light and Dark Theme with initial Primary Color. Later in Settings screen, i am using setTheme() to change entire color theme with other primary color. But it is not changing to new color. I have checked in both iOS and Android.
void setApplicationColorTheme(
    {required BuildContext context, required Color primaryAppColor}) {
  print("primaryAppColor is $primaryAppColor");
  AdaptiveTheme.of(context).setTheme(
      light: getLightThemeData(primaryAppColor: primaryAppColor),
      dark: getDarkThemeData(primaryAppColor: primaryAppColor),
      notify: true);
}

Expected behavior
It should change entire app theme with new primary color.
Please suggest me to fix this issue.

My Bad. I forgot to change initial color to Theme.of(context).primaryColor.
It is working fine.
Thank you.


Columbia epidemiology data has bad date values.

Dates are not parsed correctly, and the country subregions don't appear to follow ISO 3166 standard.
Example output
1899-12-31,CO,2760,0,0,,2760,0,0,
1899-12-31,CO_05,145,0,0,,145,0,0,
1899-12-31,CO_05_05001,53,0,0,,53,0,0,
1899-12-31,CO_05_05045,4,0,0,,4,0,0,
1899-12-31,CO_05_05051,1,0,0,,1,0,0,
1899-12-31,CO_05_05088,4,0,0,,4,0,0,
1899-12-31,CO_05_05107,1,0,0,,1,0,0,
1899-12-31,CO_05_05120,1,0,0,,1,0,0,
1899-12-31,CO_05_05147,4,0,0,,4,0,0,
1899-12-31,CO_05_05234,1,0,0,,1,0,0,
1899-12-31,CO_05_05266,2,0,0,,2,0,0,
1899-12-31,CO_05_05308,1,0,0,,1,0,0,
1899-12-31,CO_05_05318,1,0,0,,1,0,0,
1899-12-31,CO_05_05360,12,0,0,,12,0,0,
1899-12-31,CO_05_05361,32,0,0,,32,0,0,

Date parsing needs fixing. Not all regions follow ISO 3166 standard, we try to stick to that as much as we can but the choice of region codes is mostly informed by the availability of epidemiological data. In the case of Colombia, the data is reported using their own standard called DIVIPOLA codes...

Ah thanks, understood.

Update: Some of the date values in the raw data are causing the issue. The following columns provide dates,
Date of notification,
Date of death,
Date of diagnosis,
Date recovered,
date of web report
In the offending entries (2760 of them), one of these dates is set to 1899-12-31 which looks like a user input error. We only make use of date of death/diagnosis and recovered, so it should be enough to filter entries where the date is 1899-12-31.


sync: TestMutexFairness flaky on openbsd-*-62

Looks like TestMutexFairness has somehow recently regressed on OpenBSD 6.2:
https://build.golang.org/log/1072b7783fa59d0db61b3dc39f893330491a7722
https://build.golang.org/log/4978ed9696cf9aed6c6e26c4fa7cb97ad55c99a7
https://build.golang.org/log/8bfbe3f24eb9dc9cf8b5f223130c208e2c786892
https://build.golang.org/log/5dd265480a37db6270267bc25822de1a62e04163
CC @bradfitz @aclements

I was able to reproduce this reliably at the change 02a5502ab8d862309aaec3c5ec293b57b913d01d listed in one of the failure instances, by repeating this 100 times:
gomote run  user-danscales-openbsd-amd64-62-0 go/bin/go test sync -test.run TestMutexFairness -test.count=1 -test.timeout=15s
Using this same way to reproduce, I found that it started happening when Ian turned on the new timer code.  It is fixed (not reproducible) in the current master, and by bisecting, I found it was fixed by Ian's commit
d80ab3e85a :  runtime: wake netpoller when dropping P, don't sleep too long in sysmon
@ianlancetaylor  does that make sense to you (seems reasonable to me based on the description of the change and the bugs that it fixed).  If so, I think we can close this bug.
(Also, doesn't seem like it has been happening on the builder for the last week, as expected.)

Yes, I expect that was fixed by the timer change.


[NNotepad] Fix serialization of non-tensor arrays

Most MLGraphBuilder method arguments are operands, but sometimes they're arrays or numbers. There was a mechanism in place to determine (via a table) whether a given method's argument was an operator or not, but that wasn't enough to distinguish an array of numbers (like reshape() needs) from an array of operands (which concat() needs).
Rework the lookup from returning a boolean to returning an argument type, and adjust serialization appropriately.
Fixes #263

@Honry and or @huningxin - could you please review / merge?


Cannot remove integration and it's dependencies from HA

Version of the custom_component
v2.4.3
-->
Configuration
I'm not using any yaml configuration, I just installed the integration via hacs and added it to my system through the web interface and the "add integration" button

Describe the bug
There is no clear way on how to remove this integration from HA as it misses the ... menu.
Removing the package from the homeassistant/custom_components directory (or removing it through hacs) leaves stuff around in a way it makes it apparently impossible to "reset" it.

For some reason, all my meros plugs went unresponsive kind of out of the blue.

In a couple of days I both updated HA to 2021.12.3 as well as revamped my lan+dhcp config, which very likely changed IPS for all my meros plugs. None of them has a reservation in my dhcp so it's hard to tell.
Never the less it took me a couple of days to realize all of them went unresponsive (one of them takes care of turning on an extra heater in my bathroom to warm it up for my morning shower. I had to shower in the cold to realize something was wrong), so not really sure which change did it.
All my plugs show up and work perfectly fine in the meros app on my iPhone
It didn't seem like a big issue: I'll just remove the integration, HA will forget about them, add the integration back and it'll detect the plugs again. King of the traditional "fix the unresponsive device" dace I'm sure many went through with many integrations over the years. Or so I thought.
Going into the Integration page I found myself staring at the meros integration lacking any apparent way to remove it. Fine, I'll delete the module, reboot HA, and install it again. Seemed yet again enough to have those plugs been rediscovered.
I turned about to be wrong again: as soon as I add the module back all the stale devices show up again as nothing happened, and I'm back to square one.
Now, besides my showering in the cold problems, there should really be a clear and clean way to just remove this integration and all its dependencies from the system. Wouldn't you agree?
Debug log
there are no real logs possible for this kind of issue

Sorry turns out there is, it's just confusing and it's due to HA. Nothing this module can do. My bad.

Ah ok, just read this after commenting the same issue in another thread ;)

@unlucio and how did you finally remove these devices?


Error accessing uninitialized property

I finally could update a big Symfony app to 7.0. When trying to browse the app, I see this error:
Typed property Symfony\Component\HttpKernel\Bundle\Bundle::$extension
must not be accessed before initialization

The error happens here:
https://github.com/symfonycorp/connect/blob/ef075c748c87d1198b77f354395f3442df70624d/src/SymfonyConnectBundle.php#L30
@nicolas-grekas did you expect this issue? What would be the best way to solve this? Thanks!

I guess this can be closed as fixed by https://github.com/symfony/symfony/pull/52438 :slightly_smiling_face:


Access to the path is denied

PowerShell 5.1
When I attempt to use the -OutFilePath parameter, whether I'm using Protect-PGP or Unprotect-PGP, I'm getting an Access to the path  is denied error.
If I call Protect-PGP without the -OutFilePath parameter, it'll place the encrypted files in the same directory as the source files.
I do not have that option when attempting to run Unprotect-PGP.  It seems to require the -OutFilePath parameter to be defined.
This is not a permission issue as I can write to the directory without problems.

...
Decrypting testFile8.csv.pgp...
WARNING: Unprotect-PGP - Remove PGP encryption from D:\Sage\Concur\Import\Processing\testFile8.csv.pgp failed: Exception calling "DecryptFile" with "4" argument(s): "Acce
ss to the path 'D:\Sage\Concur\Import\Processing' is denied."
Decrypting testFile9.csv.pgp...
WARNING: Unprotect-PGP - Remove PGP encryption from D:\Sage\Concur\Import\Processing\testFile9.csv.pgp failed: Exception calling "DecryptFile" with "4" argument(s): "Acce
ss to the path 'D:\Sage\Concur\Import\Processing' is denied."

PS C:\Users\sage100> Unprotect-PGP -FilePathPrivate $pgpPrivateKeyFile -Password $pgpPassword -FilePath "D:\Sage\Concur\Import\testFile1.csv.pgp"
cmdlet Unprotect-PGP at command pipeline position 1
Supply values for the following parameters:
OutFilePath: .
WARNING: Unprotect-PGP - Remove PGP encryption from D:\Sage\Concur\Import\testFile1.csv.pgp failed: Exception calling "DecryptFile" with "4" argument(s): "Access to the p
ath 'C:\Users\sage100' is denied."

PS C:\Users\sage100> Unprotect-PGP -FilePathPrivate $pgpPrivateKeyFile -Password $pgpPassword -FilePath "D:\Sage\Concur\Import\testFile1.csv.pgp" -OutFilePath C:\Temp
WARNING: Unprotect-PGP - Remove PGP encryption from D:\Sage\Concur\Import\testFile1.csv.pgp failed: Exception calling "DecryptFile" with "4" argument(s): "Access to the p
ath 'C:\Temp' is denied."

PS C:\Users\sage100> New-Item "D:\Sage\Concur\Import\Processing\testfoo.txt"

    Directory: D:\Sage\Concur\Import\Processing

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----        8/27/2024   4:54 PM              0 testfoo.txt

The outputfilepath requires full path to file, and not just a folder... if you want to use folder you should use OutputFolderPath

This works.  Thank you!


Facebook Messenger Bot: No replies to others

I've successfully created a bot and integrated with Skype and Facebook Messenger.
The Skype interface works for everyone but the Facebook one replies only to my Facebook account.
The bot is not published and in the channels list, Facebook and Skype is enabled but not published.
I've searched online but I can't find much information, so I'm creating this issue as means to get some help and produce help for similar issues.

Hi!
Do you see some errors in dev.botframework.com ?
I had similar issue. I tried to test unpublished facebook bot from account, that not linked to project as developer or QA.
Here is error from facebook in dev.botframework.com

Cannot message users who are not admins, developers or testers of the app until pages_messaging permission is reviewed and the app is live.

From your description, I guess you have same issue.
Seems like it's just related to facebook development specific.

@hardsky I just noticed the errors. If you hadn't mentioned it I would have probably never seen it.
I do have the same error. Any suggestions for the fix?
Should this be added to the guide? I suggest this because the guide for skype works well for all but the guide for facebook doesn't and it's a pity.

For future reference to anyone having this issue. There are two possible actions:

Easy. From developers.facebook.com add the specific people you want them to engage your bot as testers.
Complex. From developers.facebook.com add the pages_messaging to submission. This is more complex because

it requires a 1024x1024 icon (I resized the default from the bot-framework.
you must add a Privacy Policy URL. I used the iubenda.com.
you must add to submission the pages_messaging. You need the above before you can proceed with this step.

If you are looking for a couple of people to play with your bot then option 1 is enough. If you are looking to a more dynamic experience where you share your bot link and people have then option 1 is not enough and you need to proceed with option 2.
Is it possible to add a more refined information in the dev.botframework.com?

Hi,
I need the Option 2 but whenever I am submitting the app for getting the permission for pages_messaging they are rejecting it as the Bots are not replying to them. The bots will not reply to any user outside of Developers and testers but I am unable to explain the same to the Facebook review team. Can you please guide me on how can I take it forward?


Update to omniauth-bigcommerce 0.3.0

@philipmuir @pedelman @saurabh-g

Looks good to me! :+1:

👍


boards: arm: 96b_wistrio: Use stm32flash runner

This feature is mentioned here.
root:zephyr# west build -b 96b_wistrio samples/hello_world/
root:zephyr# west flash -r stm32flash --device /dev/ttyUSB0
-- west flash: rebuilding
ninja: no work to do.
-- west flash: using runner stm32flash
-- runners.stm32flash: Board: write 10172 bytes starting at 0x08000000
stm32flash 0.5

http://stm32flash.sourceforge.net/

Using Parser : Raw BINARY
Interface serial_posix: 115200 8E1
Version      : 0x31
Option 1     : 0x00
Option 2     : 0x00
Device ID    : 0x0429 (STM32L1xxx6(8/B)A)
- RAM        : 32KiB  (4096b reserved by bootloader)
- Flash      : 128KiB (size first sector: 16x256)
- Option RAM : 32b
- System RAM : 4KiB
Write to memory
Erasing memory
Wrote address 0x080027bc (100.00%) Done.

-- runners.stm32flash: Board: finished 'write' .
root:zephyr#

*** Booting Zephyr OS build zephyr-v1.13.0-13077-gdbd1f9f13785  ***
Hello World! 96b_wistrio

@KwonTae-young Can you please update the board doc also? Just remove the separate build section and merge it with Using ROM bootloader section as below:
Here is an example for building and flashing the :ref:`hello_world` application.

.. zephyr-app-commands::
   :zephyr-app: samples/hello_world
   :board: 96b_wistrio
   :goals: build flash


Allow configurable assignPublicIp and platformVersion parameters

• Allow to define platformVersion (e.g. 1.4.0 which supports NFS mounts)
• Allow public IP to be assigned to the task in order to get access to external repositories when running in public subnets

Closing since I believe this is now possible using the latest API set at https://github.com/uktrade/fargatespawner/commit/72b20163825af557f401268a9f7108174e2dd83b


Follow-up work for large data, increased protection and testability

Creating on behalf of @ukengineer925.  Same source branch as #361, but this PR has master branch specified as its merge target.
This PR contains follow-on work identified as non-critical fixes or wider changes from #204.

Closing as the original PR just needs the target branch updating after the original one got merged


ImGui example crash

ImGui example crashes on Windows and MacOS. Dependencies are installed via Nuget.

Same result with cloning and using the repos.

Hey @elvodqa,
Thanks for your interest.
I was able to reproduce this problem. I had to have ImGui go back a few commits and use those bindings before I was able to have it working correctly.


Allow keyword arguments to Base.close

WebSockets.close has a couple of keyword arguments which I'd like to use :slightly_smiling_face:

cool! thanks you  for you feedback. @christopher-dG


[AIRFLOW-2185] Use state instead of query param for redirect_uri

JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2185

Description
Both the Google OAuth2 and GHE authentication plugins include the next_url as a query parameter in redirect_uri. This breaks at least Google OAuth2, unless you include the query parameter in the authorized redirection URI. This isn't the most flexible solution, as you would have to do the same for every potential next URL, and seems to go against the OAuth2 spec.
Instead the next_url should be sent via the state parameter which MUST be maintained by all spec compliant OAuth2 implementations, and is not used when comparing redirection URIs.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
This is not easily unit testable, but I have tested it with Google OAuth2. Assuming GHE follows the OAuth2 spec, it should work there as well.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

@samschlegel Would there be any way to test this?

Please rebase onto master to fix the CI

Could use Flask-OAuthlib to create a mock OAuth2 server for all of the OAuth2 related auth backends, special casing for any quirks they have.
But if you wanted to actually test with Google OAuth2, that's a much more involved process that would probably require a headless browser and a GCP account. I'm not sure what the process would be with GHE, but given you need a license to run it I'm not sure it would be much easier.
Could testing this be added as a separate Jira ticket since it's non-trivial?

It 100% works with GCP for us, but since I can't test with GHE I can remove that part of the change, unless someone else can test it.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3103   +/-   ##
=======================================
  Coverage   73.07%   73.07%
=======================================
  Files         180      180
  Lines       12578    12578
=======================================
  Hits         9191     9191
  Misses       3387     3387

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 0f9f460...a7de7de. Read the comment docs.

Would love for this to be merged. @Fokko are you blocking this on testing?


[feat] Zone details

#65 should address this same issue. I'm just finishing up the final bits for that (EditZone).

@jamesog Awesome, closing this PR. Feel free to reopen if necessary.


feat: Functions to write to DB

Initial work for caching:

Function to receive webhook + write to queue
Function to read from queue + write to DB
Classes/interfaces for webhook + DB entities
Initial mapping classes
SQL script to create DB tables
Majority of terraform for infrastructure

I've approved the PR, obviously with the caveats of the things we just discussed in our session - get the caching branch pulled into other local branches, fix the merge conflicts - then let's try get this caching branch into Development branch ASAP so we can get back to proper branching strategy/ways of working


Removed old leftovers from bower and ionicons. These caused problems.…

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
feature and bug fix

What is the current behavior? (You can also link to an open issue here)
No local docker support, missing ionicons

What is the new behavior (if this is a feature change)?
Introduces support for local Docker environment, execute by using:
./app npm

Also removed leftovers from old bower setup

Other information:

Hi @RonnyRoos, many thanks for this PR, and sorry for taking so long to get to it. We would love to get rid of the last bower dependency, however we cannot introduce such breaking change as some of the users may already use ionicons in their projects. Probably the other way is to just move ionicons to the assest folder and commit them to git, as the ionicons package is not updating much. If you would like to take a look at this we would be grateful.
As for the menu fix - could you post a separate PR?
Thanks again!

@RonnyRoos I merged the fix for menu manually, will be in master soon

Hi Dmitry,
Haven't replied to mails from github before, so no ideas if this ends up in
the right place.
Anyhow, as I mentioned in my other mail. Some kind of task system or
roadmap so that people know who is doing what would be great.
I am considering looking into upgrading to the alpha router. Would you be
interested in this?
I will look into moving bower to local assets as well and submit a PR.
BR.
Ronny
ons 29 juni 2016 kl 11:02 skrev Dmitry<EMAIL_ADDRESS>
@RonnyRoos https://github.com/RonnyRoos I merged the fix for menu
manually, will be in master soon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/akveo/ng2-admin/pull/40#issuecomment-229298993, or mute
the thread
https://github.com/notifications/unsubscribe/AGR173K1suOwxAUx6L5eudCB8QTFORQBks5qQjScgaJpZM4I8In5
.

Hi @RonnyRoos, I guess the github issues is a great place for issue communication, so basically you can just drop a line in the issue you are working so that everyone knows it's assigned.
Yes, working on the alpha router would be a great help!
Thanks again.

Cool,
Let's try that. I will drop a note when I start the work. Just have to
verify that I have some time off this weekend :-)
You guys don't use Slack or such? Could be good for general directions on
where you want to go on certain stuff.
/Ronny
tors 30 juni 2016 kl 17:55 skrev Dmitry<EMAIL_ADDRESS>
Hi @RonnyRoos https://github.com/RonnyRoos, I guess the github issues
is a great place for issue communication, so basically you can just drop a
line in the issue you are working so that everyone knows it's assigned.
Yes, working on the alpha router would be a great help!
Thanks again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/akveo/ng2-admin/pull/40#issuecomment-229703138, or mute
the thread
https://github.com/notifications/unsubscribe/AGR17zKf9QKVxHsFFOROjBo2S48zEa6-ks5qQ-bggaJpZM4I8In5
.


Deserialisation Error for  info_get_deploy on Key CL Types

info_get_deploy does not correctly deserialise ByteArray and Key CL Type
info_get_deploy does not correctly deserialise ByteArray and Key CL Type when deserialising a deploy via  info_get_deploy.
To Reproduce
Identify if any Prequisites:
Use the following JSON as a response to the info_get_deploy RPC call:
info_get_deploy.json.txt
Expected behavior
CL type key can be deserialised
Exception
Error: Tried to unwrap Error: Left over bytes
    at ErrImpl.unwrap (/Users/imills/casper/cspr-typescript-sdk-standard-tests/node_modules/src/result.ts:123:15)
    at Function.fromJSON (webpack://casper-js-sdk/./src/lib/CLValue/Abstract.ts:70:67)
    at from (webpack://casper-js-sdk/./src/lib/RuntimeArgs.ts:49:34)
    at Function.from (<anonymous>)
    at Object.l [as deserializer] (webpack://casper-js-sdk/./src/lib/RuntimeArgs.ts:48:11)
    at /Users/imills/casper/cspr-typescript-sdk-standard-tests/node_modules/typedjson/src/deserializer.ts:301:50
    at Map.forEach (<anonymous>)
    at convertAsObject (/Users/imills/casper/cspr-typescript-sdk-standard-tests/node_modules/typedjson/src/deserializer.ts:294:36)
    at Deserializer.convertSingleValue (/Users/imills/casper/cspr-typescript-sdk-standard-tests/node_modules/typedjson/src/deserializer.ts:132:20)
    at TypedJSON.parse (/Users/imills/casper/cspr-typescript-sdk-standard-tests/node_modules/typedjson/src/parser.ts:411:40)

TypeError: Cannot read properties of undefined (reading 'toBytes')
    at toBytes (webpack://casper-js-sdk/./src/lib/DeployUtil.ts:690:37)
    at e.toBytes (webpack://casper-js-sdk/./src/lib/DeployUtil.ts:757:29)
    at serializeBody (webpack://casper-js-sdk/./src/lib/DeployUtil.ts:1285:54)
    at validateDeploy (webpack://casper-js-sdk/./src/lib/DeployUtil.ts:1570:26)
    at Object.deployFromJson (webpack://casper-js-sdk/./src/lib/DeployUtil.ts:1502:17)
    at webpack://casper-js-sdk/./src/lib/CasperClient.ts:277:28
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Additional context
Add any other context about the problem here.
Defect Type

[ ] Regression
[ ] New
[ ] Escape

hi @meywood, thanks for raising issues, which version of sdk are you using?

Sorry for the very late reply I didn't see a notification on GitHub for this was tested using V2.15.2

@meywood , seems this issue is the duplicate of https://github.com/casper-ecosystem/casper-js-sdk/issues/380


Upgrading envoy image and org json library

Purpose
fixes: https://github.com/wso2/apk/issues/1830

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

Comparison is base (1bc6018) 26.00% compared to head (bec6903) 28.34%.
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1831      +/-   ##
==========================================
+ Coverage   26.00%   28.34%   +2.34%
==========================================
  Files         250       61     -189
  Lines       29536     4135   -25401
  Branches     4894     2201    -2693
==========================================
- Hits         7680     1172    -6508
+ Misses      21594     2963   -18631
+ Partials      262        0     -262

Flag
Coverage Δ

adapter
?

common-controller
?

config-deployer-service
28.34% <ø> (ø)

idp-domain-service
?

router
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 189 files with indirect coverage changes

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


Icons aren't displaying

https://i.imgur.com/1LWsLZp.png
https://i.imgur.com/eLC9YQN.png
Using Arch with i3wm, also tried using Cinnamon but still had the same issue.
This occurs with the AUR build as well as the most recent Github release build.
The text overlaps everywhere, making it hard to read anything.
Any ideas?

Weird... I don't have any problems with i3wm. Are those problems on full-screen too?

Yes, the problem occurs in fullscreen as well as in other WMs/DEs.  Is there any way I can get a detailed log or something that could potentially help figure out what's causing this?

You can use Ctrl+d to open the console.


Deleting older images breaks date sorting in walk mode

In case it matters the OS is Windows 11
When deleting older images the entire rest of that folder gets pushed up as most recently updated.
Shouldn't sorting be done based on file property and not folder? Because as you can see in screenshot, the files are still at their original date but the folder has been changed and thus I assume you are sorting based on folder property rather than file.
Thing is. Technically you only need to getch property of the most recent file in a folder rather than only the folder to solve this.

It may be because you modified the default sorting method.

You don't seem to understand what I was trying to put forth in this issue.
I am not trying to sort by created date. I am sorting by updated date. But the sorting for updated date is not based on file updated date but rather directory updated date.
When a file inside a directory is deleted, the updated date for the directory is changed but not the updated date for the other containing files.
When sorting is set to updated date it seems you are currently sorting files in walk mode in the order of directory updated date then file updated date with the directory taking priority? Is this correct?
If this is correct I would suggest you inverse that priority so that it first and foremost sort based on files updated date rather than the directory properties updated date since the directory updated date is not inherited by the files.
example: we have 5(five) files inside 3(three) directories. Files have the following ID: A, B, C, D and E. Directories have the following ID: X, Y and Z.
Last updated date is specified with semi-colon
;
after an ID followed by date in the
YYYY-MM-DD
format (ex. Y;2024-02-16).
Indented line signifies that file ID is a child of Directory ID.
Current date in example is set to 2024-08-01(remember this). Initial structure and sort order(we only see files) based on updated:
X;2024-01-15
  A;2024-01-15
  B;2024-01-11
Y;2023-10-30
  C;2023-10-30
Z;2023-07-18
  D;2023-07-18
  E;2023-07-11
Viewed order in UI: A, B, C, D, E

As we can see all is sorted in order here. Now lets delete file D and based on the current sorting schema in infinite image browsing the following is now the order in which we see the files:
Z;2024-08-01
  E;2023-07-11
X;2024-01-15
  A;2024-01-15
  B;2024-01-11
Y;2023-10-30
  C;2023-10-30
Viewed order in UI: E, A, B, C

You might notice that even though file E was updated way back far before file A, B, C, we still see file E first even though that file has not been updated since 2023-01-15.

You might want to try using the Fuzzy Search feature instead. It preprocesses the files into a database and then strictly sorts them by modification time, ignoring irrelevant folders. The current implementation actually has significant benefits in many cases.

Here. Also datetime is not optimal for this. You are better of using time rather than datetime since you are doing comparison and not indexing cause that means you only need to compare floating point values for modified time rather than compare a string. The time module returns a Unix timestamp in floating point format.


Incorrect instructions in getting started tutorial for .NET v4

You don't have to create a new bot configuration in the emulator. You can simply click "Open bot" and navigate to the .bot file that is created when you use the template
https://github.com/MicrosoftDocs/bot-framework-docs/blob/live/articles/dotnet/bot-builder-dotnet-sdk-quickstart.md

For this example, it works either way.


DSEGOG-316 Switch between parallel and sequential ingestion for Echo ingest script

This PR adds some config that switches between sequential and parallel ingestion. As such, I've added the parallel ingestion back in that existed in previous versions of this script. The intention is to use sequential for GitHub Actions and use parallel on large cloud VMs for ingestion of the simulated data on the dev server.
There's a couple of other changes I've made:

In the ingestion script, I've replaced mongo with mongosh - from 6.0, the mongo command disappeared so using mongosh is a solution to support more MongoDB versions (at least 5.0-7.0, if not more)
Added default sorting (_id ASC) on data export when no order is provided in the request. When I tested parallel ingestion on GitHub Action, one of the export tests failed because the nature of parallel ingestion means files aren't always ingested in chronological order so the order of export could change a bit.
Added a 5 second sleep on the ingestion script after the database collections are dropped. If the Echo bucket is empty or the config tells the script to not wipe the bucket, the script can encounter a race condition where the channel manifest file is ingested first, before all collections have been dropped. This would cause a situation where no manifest is present on the database and because the validity checks rely on using the manifest, the ingestion wouldn't work.

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

Project coverage is 94.99%. Comparing base (1733055) to head (1810cdb).

Additional details and impacted files
@@                    Coverage Diff                    @@
##           DSEGOG-305-reingest-data     #120   +/-   ##
=========================================================
  Coverage                     94.99%   94.99%
=========================================================
  Files                            50       50
  Lines                          2837     2838    +1
  Branches                        297      297
=========================================================
+ Hits                           2695     2696    +1
  Misses                          105      105
  Partials                         37       37

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

Making this PR ready to review as I've finished the ingestion and no further changes were made.

Just requested a re-review - hopefully addressed all the comments


Test project-meta.js:write() should merge changes with .blinkmrc.json file does not clean up after itself

Fixed in #149


[Bug]:  Other pipeline can't be used

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
i wan't use image-embedding-vgg16-bn pipeline, it can't work.
def init(self):
self.embedding_pipeline = pipeline('towhee/image-embedding-vgg16-bn')
i have got this error message:

why only default pipeline is ok , other pipeline not work

Which version of towhee are you using?
When I tried to reproduce it. I can get the pipeline with towhee0.4.0 but get the error to run with img:
File "~/.towhee/operators/image_decoder&towhee$main/image_decoder.py", line 5, in <module>
    import cv2
ModuleNotFoundError: No module named 'cv2'
from timm.data.transforms_factory import create_transform
ModuleNotFoundError: No module named 'timm'

I think that @filip-halt has lost the requirements.txt in operator.

0.2.0

i update the version to 0.4.0 , it works

Woops, recently transfered the timm operator to towhee account. Good catch. Let me change it real quick.

Yes, being that towhee is still in its early stages, we made quite a few changes between 0.2 and 0.4 in regards to the hub, so if you can update towhee the issue should be resolved. @annisun2020

I have update version to 0.4.0 , but get this error message
2022-02-11 11:02:38,919 -<PHONE_NUMBER>13504 - runner_base.py-runner_base:133 - ERROR: image_decoder:0 runs failed, error msg: startswith, Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/towhee/engine/operator_runner/runner_base.py", line 160, in _call_op
outputs = self._op(**inputs)
File "/Users/liyan/.towhee/operators/image_decoder&towhee$main/image_decoder.py", line 36, in call
if image_path.startswith('http'):
File "/usr/local/lib/python3.9/site-packages/PIL/Image.py", line 546, in getattr
raise AttributeError(name)
AttributeError: startswith

I have update version to 0.4.0 , but get this error message
2022-02-11 11:02:38,919 -<PHONE_NUMBER>13504 - runner_base.py-runner_base:133 - ERROR: image_decoder:0 runs failed, error msg: startswith, Traceback (most recent call last): File "/usr/local/lib/python3.9/site-packages/towhee/engine/operator_runner/runner_base.py", line 160, in _call_op outputs = self._op(**inputs) File "/Users/liyan/.towhee/operators/image_decoder&towhee$main/image_decoder.py", line 36, in call if image_path.startswith('http'): File "/usr/local/lib/python3.9/site-packages/PIL/Image.py", line 546, in getattr raise AttributeError(name) AttributeError: startswith

The input must be an image path, it could be a local path or  HTTP URL, not a PIL image.

I have update version to 0.4.0 , but get this error message
2022-02-11 11:02:38,919 -<PHONE_NUMBER>13504 - runner_base.py-runner_base:133 - ERROR: image_decoder:0 runs failed, error msg: startswith, Traceback (most recent call last): File "/usr/local/lib/python3.9/site-packages/towhee/engine/operator_runner/runner_base.py", line 160, in _call_op outputs = self._op(**inputs) File "/Users/liyan/.towhee/operators/image_decoder&towhee$main/image_decoder.py", line 36, in call if image_path.startswith('http'): File "/usr/local/lib/python3.9/site-packages/PIL/Image.py", line 546, in getattr raise AttributeError(name) AttributeError: startswith

The input must be an image path, it could be a local path or HTTP URL, not a PIL image.

I have change to image path, it works ,thanks

Hi @annisun2020, would you like to join towhee slack where people can discuss anything about towhee.


add rails_env back to the shard selection key

#51 removed rails_env from the shard selection key which breaks rake db:test:prepare without a RAILS_ENV specifically set.
(https://github.com/zendesk/zendesk_database_migrations/blob/master/lib/zendesk_database_migrations/tasks/shared.rake#L152)
@osheroff @bquorning @grosser

fun :+1:

Is this an "upgrade as soon as possible" change? I'm thinking not, right?

I don't think it matters for help_center since you use the maintain_test_schema! feature of Rails >= 4.


Invalid argument number in some notify calls?

Something fishy seems to be going on in that line:
https://github.com/elm-lang/elm-reactor/blob/master/frontend/debugger-implementation.js#L387
and in that line:
https://github.com/elm-lang/elm-reactor/blob/master/frontend/debugger-implementation.js#L443
In both lines, notify is called with three arguments. But all the notify functions that exist, be it from the "normal runtime" defined in core, or be it from notifyWrapper, have only two arguments.
Could it be that this is a remnant from a time when a third argument for the current timestamp was used, which got refactored away (but JavaScript not being strongly typed didn't notice that this should have led to a change in those two lines)?

I am guessing it was an optional argument to be used with the debugger, but I don't remember :/
In any case, looks like it's sorted out, thank you for opening this issue!