get real enum name for csharp emitter

Description
Fix https://github.com/Azure/autorest.csharp/issues/4042

When the enum has projected name or friendly name, need to set the usage to the enum with the projected name/friendly name.

Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

No one would see the generated test code later. We should have unit tests to check the model usage by checking for example the serialization method.

No one would see the generated test code later. We should have unit tests to check the model usage by checking for example the serialization method.

I added Unit test to test the usage of the renamed enum and model.


cmake template: remove panic_immediate_abort for new idf versions

build-std-features=panic_immediate_abort causes panics to not print
their error messages which is a bad developer experience. Similar to
the Cargo template, do not set this flag for newer espidf versions.

On the other hand it erases 100 to 200K from the final binary.

Perhaps we need a .cargo/config.toml for the cmake-first build as well? This way we can put the immediate abort setting as commented out by default, but propery explained with comments?

I introduced a minimal .cargo/config.toml to the cmake build. I agree it is better. I add a comment about binary size to that file to both the cargo and cmake build.

currently already removed in cmake builds with "std" + "std and hal" builds. So this PR is outdated. Thanks for your initial contribution!


add x86 build and run unittests to backend pipeline

Here is some more pipeline work for your consideration. This adds a x86_64 build for the backend, in addition to the existing aarch64 build. Also added running the backend unittests on both platforms.
Relates to #1032 and #981
Added backend x86_64 build
The changes made are:

Build the backend for x86_64
Use the target names x86_64 and aarch64 (instead of amd64 and arm64) consistently in the pipeline
Use the same rust version everywhere
Remove some unnecessary build steps, or add an if when platform specific

The current Makefile only targets aarch64, so for the build and test commands I've had to ignore the Makefile.
Added executing unittests in pipeline
To run the unittests I found that some tests expect eOS environment features, such as existing files or directories. Perhaps they should be classified as integration tests and run in the eOS environment. I skipped those tests for now.
Running the tests on aarch64 was a little tricky. I found a couple of ways that didn't work. The obvious one was running cargo test in a aarch64 container. However cargo detects a change in filetime or signature and rebuilds everything, taking multiple hours on QEMU. I don't know how to fix that. The solution I found was to use nextest's archive option. Unfortunatly nextest doesn't release aarch64 binaries, so I added a stage to create them.

There may be some considerations with the upcoming x86 builds, @dr-bonez will know more.  Thank you again for your contributions :)

FYI nextest now publishes aarch64 Linux binaries.

@dr-bonez , just a note after merge., I notice a few more "util::http_reader::*" unittests are now failing because the container image the unittests are running in does not have the ca-certificates apt package installed. This is intentional, because a unittest that goes onto the internet is a too large unit in my view. We could add them to the skip list for now, or fix them if possible.

I'd accept a pr that switches to using a local fileserver to test the code, but for now I'd rather fix the container to have ca certs, and keep the tests.


Add deny grant capability to storages

Isn't this pkg/storage/utils/grants/grants_test.go file the one written by Gianmaria to extend the tests?

Those are not needed anymore as they were there only for ensuring correct position but since we have the EOS implementation now these tests (that GIanmaria added) are enough:
pkg/storage/utils/eosfs/eosfs_test.go


Word wrap implementation for text editor (f.e like in SublimeText)

It would be usefull to implement Word Wrap, which supports now in all modern Text Editors (f.e Sublime on screenshot):

SublimeText:

vscode:

Also would be usefull to implement such features as Word Wrap Column and Ruler

VSCode supports word wrap. Use "editor.wrappingColumn": 0 in the settings. (File > Preferences > Workspace settings).


Create empty directory 'templates' on first start

Like languages, logs and static there should be an empty directory called 'templates' being created on first weppy launch.

The only reason I'd argue against that is if you were using a weppy instance strictly as an api for backend processes. Though IMO and in your defense it is always helpful for even a backend api behind a vpn/vpc to have an "about" or "doc" page for developers using it. Maybe weppy is overkill for a simple backend api, but it's just so darn convenient.

@GiantCrocodile the reason templates folder is not created on application launch are mainly:

because you can customize the folder name, so it would be pointless to create the directory if you're using another directory for the templates
because as @mijdavis2 said, if you're using weppy to make just APIs, you don't need templates at all

I think this behaviour wouldn't be changed. Probably, the changes on folder creation will be to avoid creating them at all, but this will be inspected carefully considering the next versions and the development process.

Okay but why do we create other folders then? @mijdavis2 argument is applying too if you have no static files or no need for different languages.
I've created this issue because when I started a new project after some time I forgot where to place the templates. So I looked it up in the docs and I found with bad luck a place where the folders default name for templates was wrong (by a small typo: template, not templates). Then I struggled because it won't render still. This could have be prevented with an already existing folder like the other folders.


Use update4j for auto-updates

It has always bothered me that users need to:

Download the newest version
Close the client
Execute the installer

Even if the version just contains a small bugfix.
2y ago, update4j came into existence and today, it seems quite mature. I'd like to try implementing it into the client.
Best case scenario, the client will auto-update itself in the background (after asking for permission to do so) and the updated version will be loaded upon the next start.

Like u personal 😱 ? Nice

Hi, update4j contributed here.
What's your experience so far? Feel free to ask, I'll try my best to assist.

@Sheikah45 just wanted to ask if you didn't want to continue on this, now I see you committed something :) Nice, I would love to see this implemented. IIRC the remaining issue were:

The client installs e.g. 1.0.0, then despite auto-updating, Windows will still see 1.0.0 installed
The generated JRE module file somehow changes every time (adding ~30mb) even though nothing changed. Still better than downloading the whole client, though.
Right now, files are uploaded to a private server of mine. We'd probably want a new container on the server that allows uploading files via SCP. Alternatively, we could upload it via API but I think this requires unnecessary work and ends up being less flexible.

Let me know if you have any questions or want to work on it together.

Yeah I was working on rebasing this and was going to start again on it once I finished the gson to Jackson and oauth PRS.
I will definitely reach out to you probably for assistance.
With regards to the update files and the server that is definitely something we will need to work with Brutus on to see if he thinks that is feasible.
Although why do you think they could not just be on the git release?

The idea of the auto updater (and the way update4j works) is to download only files that changed. Therefore, all client files need to be hosted relative to some URL. With GitHub releases, this isn't viable nor possible.
Keeping such releases around on our servers is quite space consuming. To reduce this, the deployment script makes use of hardlinks; if a file didn't change it's only stored once. This way, new releases usually only take up a few KiB of additional disk space.
Once this is in place, the question becomes whether we still want to publish offline installers or if we'd want to provide a web-installer that's small and will download the files as needed. Such an installer could even allow selecting the version to install, making it easy to install an older version, too.


Publish 1.6.0 to NPM?

Noticed that there is a 1.6.0 release here, but it is not on NPM?

#542

Perfect! Thanks!


BulkUpdate: Owned types are not updated anymore

Hi.
Owned types are not updated anymore when using BulkUpdate API. However, it works with BulkInsert.
I have upgraded to .NET 5. This has worked previously so I am not sure if this issue is related to .NET 5 or this project.
The following fix in TableInfo (Line 404) fixes the issue, but I am not sure if this is the correct way to fix it.
if (doAddProperty)
{
    string columnName = ownedEntityPropertyNameColumnNameDict[ownedProperty.Name];
    PropertyColumnNamesDict.Add(ownedPropertyFullName, columnName);
    OutputPropertyColumnNamesDict.Add(ownedPropertyFullName, columnName);
+   PropertyColumnNamesUpdateDict.Add(ownedPropertyFullName, columnName);
}

It was a problem in the lib, and you've already made a fix.
Published v3.5.8


Migrate to angular 9 (WIP)

Certain libraries are not yet ready for angular 9

[x] ngrid has still some issues with ivy
[ ] Angular TinyMCE still seems to require an

ngx-file-drop also requires angular 8 libraries. Tho there would be alternatives

@tsiegleauq try version 2.0.0-rc.0 of ngrid, should work with angular 9.0.6 +

90% of OpenSlides is already working fine.
The thing nor working right now it the projector - this still WIP but can be tested.
Angular 9, AOT, Ivy, Postinstall script.

@shlomiassaf in our case, nGrid works absolutely perfect in angular 9 :)

@tsiegleauq What is currently not working on the projector?

@FinnStutzenstein basically the slide manager service.
Apparently, JIT compiling the slide components won't work with AOT.

Is this PR rebased on master, so that this commit (https://github.com/OpenSlides/OpenSlides/commit/76bd184ff41483d0877b304486d686e5ff1cc43f) is present in this branch?

Yes it's ts. This commit, as well as the whole voting, is already there. You might want to take a look at the projector errors. First build might take a while due to ivy

Bazel for angular currently offers no clear migration path for most projects.
Our sass imports (mainly ~) are currently tief to WebPack. To adjust bazel building, we would have to keep the configuration files in out project which is discouraged.
I'd recommend to stick with WebPack for now.

@tsiegleauq Glad to hear you are now in angular 9.
We still didn't migrate, so your project is our front-line battle ground for nGrid.
Please feel free to contact me directly, if any issues arise.
Thanks!

Tested PWA on Local nginx


Incorrect debug line number

Crystal 0.24.1, mac osx
With this code:
def foo
  [1][2]
end

foo

I get this output:
Index out of bounds (IndexError)
  from src/indexable.cr:0:17 in 'at'
  from src/indexable.cr:73:5 in '[]'
  from foo.cr:2:3 in 'foo'
  from foo.cr:5:1 in '__crystal_main'
  from src/crystal/main.cr:11:3 in '_crystal_main'
  from src/crystal/main.cr:112:5 in 'main_user_code'
  from src/crystal/main.cr:101:7 in 'main'
  from src/crystal/main.cr:141:3 in 'main'

Note this line:
  from src/indexable.cr:0:17 in 'at'

It has line 0, column 17. It's not correct.
I checked the .ll file produced by this program and I couldn't find any line: 0 in it (plus I debugged the compiler and it didn't happen either), which might mean the dwarf parser has a bug or something is missing.
I didn't try this on linux to see if it happens there too.
/cc @ysbaddaden (in case you are interested in investigating this)

The backtrace I'm getting now is:
Unhandled exception: Index out of bounds (IndexError)
  from /usr/local/Cellar/crystal/0.25.1/src/indexable.cr:596:8 in 'at'
  from /usr/local/Cellar/crystal/0.25.1/src/indexable.cr:73:5 in '[]'
  from foo.cr:2:3 in 'foo'
  from foo.cr:5:1 in '__crystal_main'
  from /usr/local/Cellar/crystal/0.25.1/src/crystal/main.cr:104:5 in 'main_user_code'
  from /usr/local/Cellar/crystal/0.25.1/src/crystal/main.cr:93:7 in 'main'
  from /usr/local/Cellar/crystal/0.25.1/src/crystal/main.cr:133:3 in 'main'

which is almost correct: the error happens at line 591, not 596. But at least it's not 0 anymore :-P

Oh, cool ?

In Crystal 0.31.1 it's no longer points to line 0
def foo
  [1][2]
end

foo

It points to the correct location https://github.com/crystal-lang/crystal/blob/0.31.1/src/indexable.cr#L73
$ crystal e.cr
Unhandled exception: Index out of bounds (IndexError)
  from /usr/local/Cellar/crystal/0.31.1/src/indexable.cr:73:20 in '[]'
  from e.cr:2:3 in 'foo'
  from e.cr:5:1 in '__crystal_main'
  from /usr/local/Cellar/crystal/0.31.1/src/crystal/main.cr:97:5 in 'main_user_code'
  from /usr/local/Cellar/crystal/0.31.1/src/crystal/main.cr:86:7 in 'main'
  from /usr/local/Cellar/crystal/0.31.1/src/crystal/main.cr:106:3 in 'main'

$ crystal --version
Crystal 0.31.1 (2019-10-02)

LLVM: 8.0.1
Default target: x86_64-apple-macosx

Probably fixed by #8234


Listen to close instead of finish when making static zips

We have been investigating an error where zip files that are produced from this code end up being corrupt when we try to unzip them. It seems that since we are using a file here that we might want to listen to the close event instead of the finish event, since close will wait until the file handle is closed. I think that if we resolve a little too early that we might end up sometimes uploading zip files that are not fully written to disk.
I also noticed that in other places where we are creating zip files that we do not use a file on disk to write them to, and instead we write to a buffer that we eventually upload. In these cases it probably works fine and actually makes sense to use the finish event. It might be worth revisiting how this works and potentially refactor it to use that pattern instead.

Regarding the temp file, I found this commit: fd6d95a92e657f42fbfa47e56fe6fb803ad040a8


for ubuntu provision append hostname to <IP_ADDRESS> line in /etc/hosts

we should do it this way instead of appending a record for <IP_ADDRESS> to the file itself. Two reasons for this:

you should not have multiple lines in /etc/hosts with the same ip address
ubuntu/debian use <IP_ADDRESS> for this purpose as described here:

https://www.debian.org/doc/manuals/debian-reference/ch05.en.html#_the_hostname_resolution
and further discussed here:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=316099
Signed-off-by: Aaron Welch<EMAIL_ADDRESS>

I'm for it.
Could I get you to add an integration test which checks for this after create please?

@nathanleclaire I'll give it a shot :)

@nathanleclaire ok, how does that look? for some reason the busybox test was failing for me, but maybe that's because I'm on a plane and the wifi is spotty, the rest worked just fine...

@nathanleclaire does the test look ok?

@crunchywelch Yep, looks good - thanks for following up.  Just one last request: Could you please split the bit after the && into a separate SSHCommand command?  I think it will make the code more readable.  This will be LGTM for me after that.

Oh, and also a comment inline explaining the motivation (similar to what you have here on the inline code commentary) would be fantastic!

@nathanleclaire okies, here you go!

@nathanleclaire actually, this won't work, for the same reason I wanted to reconfigure the provision logic from issue #961 because you return an ssh cmd object, you don't actually execute the command within that method. I could run the first set directly, then return the second cmd, but that seems wrong. Would you like me to make a bigger PR moving the logic around to make this work if you want multiple commands?  For now, I'm reverting to a single command...

(fwiw, I think the single line way is more readable just due to error checks, etc. happy to change it back if you agree)

I'd rather keep it "one concern per command" - one sets the hostname, the other replaces <IP_ADDRESS> with <IP_ADDRESS>.
Just FYI, it's slightly more idiomatic to do this if a function call only returns one value, like so:
if err := cmd.Run(); err != nil {
    return err
}

You can change to that form if you want, but I don't really think of it as a hard requirement.
LGTM

too right, updated!

OK one last comment - LGTM - cc @ehazlett for review

cool, updated...

Great, thanks for being responsive.

@crunchywelch would you mind rebasing?  thanks!

ooook

cc @ehazlett

I'm getting a duplicate entry for the hostname -- is that expected?
<IP_ADDRESS> test-do test-do

Only if you somehow ran this twice on the same host, is that possible?

It was a fresh host in DigitalOcean.  I'll try again.

This also doesn't appear to be working on EC2:
<IP_ADDRESS> localhost

That's the only thing in /etc/hosts (besides the ipv6 entries).

Same with Rackspace

hrm, ok, let me play with it and figure out what's up.
On Apr 22, 2015 8:15 AM, "Evan Hazlett"<EMAIL_ADDRESS>wrote:

Same with Rackspace
—
Reply to this email directly or view it on GitHub
https://github.com/docker/machine/pull/960#issuecomment-95153327.

@crunchywelch thanks!

@ehazlett  this works as expected for me on DO:
root@ubuntutest:~# cat /etc/hosts
# Your system has configured 'manage_etc_hosts' as True.
# As a result, if you wish for changes to this file to persist
# then you will need to either
# a.) make changes to the master file in /etc/cloud/templates/hosts.tmpl
# b.) change or remove the value of 'manage_etc_hosts' in
#     /etc/cloud/cloud.cfg or cloud-config from user-data
<IP_ADDRESS> ubuntutest
<IP_ADDRESS> localhost

# The following lines are desirable for IPv6 capable hosts
::1 ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
ff02::3 ip6-allhosts

Can you tell me the parameters you are using for the create command to make sure I'm not missing something?
Looking into rackspace and aws now...

@ehazlett ok, so that double hostname entry is the default behaviour on DO, this was the result of me creating a droplet from the control panel, not using machine:
root@hostnametester:~# cat /etc/hosts
# Your system has configured 'manage_etc_hosts' as True.
# As a result, if you wish for changes to this file to persist
# then you will need to either
# a.) make changes to the master file in /etc/cloud/templates/hosts.tmpl
# b.) change or remove the value of 'manage_etc_hosts' in
#     /etc/cloud/cloud.cfg or cloud-config from user-data
<IP_ADDRESS> hostnametester hostnametester
<IP_ADDRESS> localhost

# The following lines are desirable for IPv6 capable hosts
::1 ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
ff02::3 ip6-allhosts
root@hostnametester:~#

So I suspect the provision failed, or did not run this code at very least. I've been getting this on digital ocean, which I'm troubleshooting now, but results in a working droplet, but that isn't fully configured:
INFO[0000] Creating SSH key...
INFO[0001] Creating Digital Ocean droplet...
DEBU[0003] Created droplet ID 5022362, IP address <IP_ADDRESS>
DEBU[0003] Getting to WaitForSSH function...
ERRO[0137] Error creating machine: Process exited with: 1. Reason was:  ()
WARN[0137] You will want to check the provider to make sure the machine and associated resources were properly removed.
FATA[0137] Error creating machine

Will continue to look into rackspace and ec2, I think that just requires a check for the line and if it not there appending it, or replacing if it exists...

@ehazlett ok, got this working on rax, DO and aws, let me know if you notice any funny business.
As an aside, might be nice to add some debug statements in the provision method so you can see where things fail if they do.

@ehazlett polite bump, would love to get this one in so I can merge on my other branches and get these other PRs issued if possible, is looking ok?

@crunchywelch looking now -- thanks for the bump!

@crunchywelch hey if i run this on DO i still get the duplicate on the first entry -- not a huge deal.  On ec2 I get the following:
ubuntu@mcn-test-ec2:~$ cat /etc/hosts
<IP_ADDRESS> localhost

# The following lines are desirable for IPv6 capable hosts
::1 ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
ff02::3 ip6-allhosts
<IP_ADDRESS> mcn-test-ec2

The hostname is appended at the end.  I'm not sure what is going on.

@ehazlett that's really odd, there was a rev with a typo at one point that appended the localhost IP that you see there, but it should be gone now. Are you certain you compiled against this newest rev?
Here are my results just now:
AWS - https://gist.github.com/crunchywelch/1cb0616d33be039d9bd1
DO - https://gist.github.com/crunchywelch/f34f4170151a4f13a568
Rax - https://gist.github.com/crunchywelch/7e06a949b4aac52a93e5

@crunchywelch thanks -- it was a fresh fetch -- i will try again -- thx!

Fresh fetch and this looks good.  Thanks!

LGTM


Error when activating Module

Hi. I get an error message when I want to run the module. The error message is as follows:
Cache cleared successfully.
Nothing to migrate.
Error occurred creating [/usr/home/xzihen/domains/DOMAIN.pl/public_html/public/modules/sidebarwebhook] symlink: symlink(): No such file or directory
The [/usr/home/xzihen/domains/DOMAIN.pl/public_html/public/modules/sidebarwebhook] symlink has been created
Clearing cache...
The compiled services & packages files have been removed.
Cache cleared successfully.
Compiled views cleared!
Configuration cache cleared!
Configuration cached successfully!
Created: storage/app/public/js/vars.js

I installed the module according to your instructions. Please help me to run the plugin ;)

Please check this
https://github.com/fulldecent/freescout-sidebar-webhook?tab=readme-ov-file#troubleshooting
I think this is a filesystem permissions issue on the server. But please add more notes and reopen if still an issue.


wd and ft for goem1240 contain replacement characters

The word form wd and text tx fields in wd.csv and ph.csv for goem1240 contain lots of replacement characters (�) hinting at a wrong encoding used to include the data in the CSV files.

This was a known issue and should have been fixed with version 1.1. We are investigating what went wrong, and will provide corrected files with the next update.


[ci skip] adding user @colinbrislawn

Hi! This is the friendly automated conda-forge-webservice.
I've added user @colinbrislawn as instructed in #29.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #29

Welcome Colin!


refactor merge_documents to be dataframes

@lily-mayfield @jodizzle you may find this interesting, converts a simple Python job to one in pure spark dataframes (a lot like SQL really)

Don't need try or log here, that's for Python functions only.

Does there need to be some other type of exception handling, then?

No
On Dec 21, 2017 6:08 AM, "jodizzle"<EMAIL_ADDRESS>wrote:

Does there need to be some other type of exception handling, then?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opensyllabus/osp-pipeline/pull/99#issuecomment-353324430,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAe2AHae968Nur2c9cQ7pfLxQ4GtEqTPks5tCjxAgaJpZM4RHUnC
.


Fix title patterns to no longer use unsupported proc

Currently the title patterns with proc prevent puppet generate types from working:
Error: /etc/puppetlabs/code/environments/production/modules/openldap/lib/puppet/type/openldap_access.rb: title patterns that use procs are not supported.
Error: /etc/puppetlabs/code/environments/production/modules/openldap/lib/puppet/type/openldap_dbindex.rb: title patterns that use procs are not supported.
Error: /etc/puppetlabs/code/environments/production/modules/openldap/lib/puppet/type/openldap_overlay.rb: title patterns that use procs are not supported.

https://tickets.puppetlabs.com/browse/MODULES-4505?focusedCommentId=418416&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-418416
https://github.com/treydock/puppet-module-keycloak/pull/21

I have not yet come up with a way backwards compatible way to remote the pattern from openldap_access type access property.  Using munge does not work as munge seems to iterate over the values in an array so generates a nested array.  Maybe this is okay and can be worked around by flattening the array in provider.

This PR is not complete until access property is handled

I can squash commits if desired.  Having to use flatten in several places seems ugly but seems to retain old behavior without using title pattern proc.

@raphink could you please review this? I'm not sure to understand exactly the impacts.

@treydock I wasn't sure what to do with openldap_access either.  See https://github.com/camptocamp/puppet-openldap/issues/214


Extension issue

Issue Type: Bug
Extension Name: azurite
Extension Version: 3.10.0
OS Version: Windows_NT ia32 10.0.19042
VSCode version: 1.52.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:
{
	"messages": [],
	"activationTimes": {
		"codeLoadingTime": 34735,
		"activateCallTime": 4,
		"activateResolvedTime": 90,
		"activationReason": {
			"startup": true,
			"extensionId": {
				"value": "Azurite.azurite",
				"_lower": "azurite.azurite"
			},
			"activationEvent": "*"
		}
	},
	"runtimeErrors": []
}

@Nava313
Thanks to report this issue!
This  Azurite start efficiency issue is already checked in : https://github.com/Azure/Azurite/issues/653


Support evolution on MySQL 5.6

Mysql 5.6 didn't like a index being made with keys longer setting than 767.    So this supports that change.

Hi all, I'd love to get some feedback on this change, as we'd like to use a released version of SMUI with this change made.

Thanks for the patch. It seems that we support only MySQL 5.7+ for now as we have indexes on VARCHAR columns wider than 767 chars (a limitation lifted in 5.7). We can't really change old Play evolutions because this would potentially lead to data loss for existing users. To add support for older versions of MySQL or new systems such as SQL Server (as in #66) with incompatible DDL scripts I see these options:

Add a distinct set of migrations in a folder in /conf/evolutions/mysql-pre-57 and /conf/evolutions/sqlserver which reflect the mainline migrations but adjust the tables where needed. Explicitly dispatch to these migrations based on some application setting.
New major. We provide a new 1.sql initial DB evolution that describes the DB in a way that is compatible with all DBs we want to support. For existing customers we also provide a script to run on the existing DB which alters/removes the evolution history and data tables to reflect the expected state.
New major and automated transition. Same as 2 but with automated script execution by SMUI.

@pbartusch, @renekrie  Opinions?

I may have goofed with git..   I wanted to just cut a quick branch that supports mysql and mssqlserver on Java 1.8, to get through a deploy with two clients..   I thought the commits would all be on https://github.com/o19s/smui/tree/support_multiple_db_types_on_java18.    Not sure how to unwind things!

Hi @epugh ,
I very much appreciate the contribution for a broader database support for SMUI.
As @mkr pointed out (https://github.com/querqy/smui/pull/67#issuecomment-801289687) , the suggested implementation would cause this SMUI version to be incompatible with previous SMUI installations.
But I understand SMUI's data model could be improved at this point to offer broader support for DBMSs.
My 2 cents on the topic:
Regarding MySQL < 5.7: SMUI should not encourage supporting old DBMSs , that might also pose a security risk to their users (as I found some CVEs , that seem related to that MySQL version).
Regarding MS SQL Server: This seems to be a legit new requirement for SMUI. This comes with the need to change the database of SMUI.
As for the options @mkr highlighted (https://github.com/querqy/smui/pull/67#issuecomment-801289687) , I think:

Will be hard to maintain , as every future add-on to the database structure needs to be synch'ed with all configurations (Play evolutions).
Might be possible , but seems hard to be reflected in an automated test case.
Seems the best option to me here , but comes with a bit of effort especially for the test automation.

As before this PR , my strategy regarding persistence in SMUI was mainly driven by the idea, to further constraint the support of DBMSs (e.g. Postgres-only) to avoid compatibility issues often encountered while extending SMUI in the past. This PR heads into the opposite direction , where you seem to see a market need for that.
I'd like to learn more about that , to reinforce a strategic path for this question (as for extending the PR according to one of the options 2. or 3.).
What do you think?

I am very much conflicted....!      Indeed I argued the same as you @pbartusch on we should treat the database as a implementation detail.   Especially in Docker friendly environments.
I'm a bit stuck though, and only because I have two clients at the same time who don't use Docker in production (one doesn't even know what Docker is), and neither uses mysql5.7.    The first is on mysql 5.6, the other is a .net shop so MS SQL Server.
I also am starting to better understand that the database is important from the sense that it represents the accumulated knowledge of all the people and they don't want to risk losing it, and many ecomm tech teams are stretched for time and resources, so they want to use the database they already know and understand.   I dread the day that a team I worked with last year brings me back in and they want DB2 support ;-).   Big WCS user ;-)
I do think that regardless of what we may do, we need to solve the integration testing issue.   I'd love to see the testcontainer work that @tboeghk made part of this project.
I could keep trying approach 1 in my o19s/smui branch, and seeing how viable the multiple database thing really is.   For example, I found I need to change the code parsing the database response for MS SQL Server.   If those changes to the app layer aren't pretty simple (and work across all our databases) then the whole effort on MS SQL Server might be for nought!

alright @epugh .

I could keep trying approach 1 in my o19s/smui branch, and seeing how viable the multiple database thing really is.
sounds like a fair temporary solution. just keep in mind, that there is a risk the branch will get outdated soon as SMUI development proceeds. so , for installations based on the branch it might become necessary to migrate the data anyway (which might lead to solution 2. or even 3. naturally - let's see ;-) )

I'd love to see the testcontainer work
me too , actually ... thanks for pointing that out. @tboeghk , if you could share that container or drop a link , that would be awesome!

for this PR , @epugh :

I will close it as for the compatibility issue.
I will open an issue for a broader database support - including DB2 of course ;-)

[...] seeing how viable the multiple database thing really is
Let's channel further communication on this topic on the issue page (maybe we can introduce some kind of voting in the future).

PR closed (without merging for compatability issues, see above). Issue tracked as enhancement #69 .


how to get script real line number?

hi,
i want to add debug function into goja ,and i can't find real line number in compile.* and vm.* file,
in the following code, i get the source code line number if current program srcPos=codeLine, add wait the next  command to debug code, is that possible?

the following picture show the line and srcPos，but  my script code just have 3 lines

You should probably have a look at StackFrame.Position() method


[DO-NOT-MERGE] Testing Dev

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


change debian mirror to avoid http.debian.net bug

resolves https://github.com/zhihu/kids/issues/31

I can't find ./docker/sources.list. Not push yet?


codo guru reviewer

@cstudio-li Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Spam?


[WIP] Ranges, getindex, and compute_inplace

This PR introduces several new features!

Ranges for collecting Julia ranges into Banyan Julia arrays (e.g., BanyanArrays.collect(1:10:10_000_000))
getindex for indexing Banyan arrays (e.g., array[:, :, [2, 6], 1:10])
compute_inplace for computing some value without returning it to the client side (e.g., compute_inplace(results)

We also fixed some bugs in the annotation of map and the implementation of Copy* PFs.

WIP because I'm trying to fix a weird internal Julia stack error when compiling DelayedTask :weary:


Update pull_request_template.md

Labs 29 Story Squad Team C Template Update
Description
Updates PR template for Labs 29

Health check is failing due to .env credentials, but this PR is only to update the template. Should be good to merge

Failing check but it doesn't have to do with the template. Need to check more into this


Virtual Container representation

In current PISA Pi, when containers are linked together, the first container in the list is chosen as a representative for all containers, without perfoming checks.....this is a bit sub-optimal

linking looks fine to me now

As far as I can tell from https://github.com/icecube/pisa/blob/master/pisa/core/container.py#L153, the issue still persists.


[BUG]: ManualValidation result is empty on multiple attempts

Task name
ManualValidation
Task version
0.198.0
Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
ubuntu-latest
Task log
No logs

Relevant log output
No logs

Aditional info
If a stage's jobs were run more than one time with Rerun failed jobs options, the result of the ManualValidation task for this stage is empty presented (non information about validation is shown).

Here is no any reaction for more than a month. Could anybody please react somehow?


installtion of deproject

Dear sir,
I am using CIAO4.14 IN Ubuntu20.04LTS system. I am trying to test using  "sudo python3 setup.py test". But I get following errors:
skk@skk-HP-15-Notebook-PC:~/deproject-main$ sudo python3 setup.py test
running pytest
running egg_info
writing src/deproject.egg-info/PKG-INFO
writing dependency_links to src/deproject.egg-info/dependency_links.txt
writing requirements to src/deproject.egg-info/requires.txt
writing top-level names to src/deproject.egg-info/top_level.txt
reading manifest file 'src/deproject.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'src/deproject.egg-info/SOURCES.txt'
running build_ext
======================================================= test session starts ========================================================
platform linux -- Python 3.8.10, pytest-4.6.9, py-1.8.1, pluggy-0.13.0
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/skk/deproject-main/.hypothesis/examples')
rootdir: /home/skk/deproject-main
plugins: openfiles-0.4.0, doctestplus-0.5.0, remotedata-0.3.2, arraydiff-0.3, hypothesis-4.36.2, astropy-header-0.1.2
collected 0 items / 2 errors
============================================================== ERRORS ==============================================================
_____________________________________________ ERROR collecting tests/test_deproject.py _____________________________________________
ImportError while importing test module '/home/skk/deproject-main/tests/test_deproject.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tests/test_deproject.py:13: in
from deproject.deproject import Deproject
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/init.py:12: in
???
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/deproject.py:25: in
???
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/plot/init.py:59: in
from sherpa.utils import NoNewAttributesAfterInit, erf, SherpaFloat,
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py:43: in
from sherpa.utils import _utils, _psf
E   ImportError: cannot import name '_psf' from partially initialized module 'sherpa.utils' (most likely due to a circular import) (/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py)
____________________________________________ ERROR collecting tests/test_simplegraph.py ____________________________________________
ImportError while importing test module '/home/skk/deproject-main/tests/test_simplegraph.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tests/test_simplegraph.py:10: in
from deproject.simplegraph import SimpleGraph
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/init.py:12: in
???
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/deproject.py:25: in
???
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/plot/init.py:59: in
from sherpa.utils import NoNewAttributesAfterInit, erf, SherpaFloat,
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py:43: in
from sherpa.utils import _utils, _psf
E   ImportError: cannot import name '_psf' from partially initialized module 'sherpa.utils' (most likely due to a circular import) (/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py)
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 2 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
===================================================== 2 error in 1.16 seconds ==============================

Questions:

How did you install CIAO 4.14?
Is the CIAO environment set up when you tried the setup.py call?
What does ciaover return?
Why are you using sudo here?

Dear sir,
Thank you for your reply.
1. I have installed CIAO-4.14 using ciao-install script.
*2. yes. I tried setting Ciao environment. but it shows error message which
I mentioned. *
3. ciaoverCIAO 4.14 Tuesday, November 23, 2021  bindir      :
/home/skk/CIAO-4.14/ciao-4.14/bin  CALDB       : 4.9.8
*4. When I used without sudo, permission denied. So I used sudo command. *
The output without using sudo command:
@.**:~/deproject-main$ python3 setup.py testWARNING:
The wheel package is not available.running pytesterror: can't create or
remove files in install directoryThe following error occurred while trying
to add or remove files in theinstallation directory:    [Errno 13]
Permission denied: './.eggs/test-easy-install-5397.write-test'The
installation directory you specified (via --install-dir, --prefix, orthe
distutils default setting) was:    ./.eggsPerhaps your account does not
have write access to this directory?  If theinstallation directory is a
system-owned directory, you may need to sign inas the administrator or
"root" account.  If you do not have administrativeaccess to this machine,
you may wish to choose a different installationdirectory, preferably one
that is listed in your PYTHONPATH environmentvariable.For information on
other options, you may wish to consult thedocumentation at:
https://setuptools.readthedocs.io/en/latest/easy_install.html
https://setuptools.readthedocs.io/en/latest/easy_install.htmlPlease make
the appropriate changes for your system and try again.
On Mon, 11 Jul 2022, 11:44 pm Doug Burke, @.***> wrote:

Questions:

How did you install CIAO 4.14?
Is the CIAO environment set up when you tried the setup.py call?
What does ciaover return?
Why are you using sudo here?

—
Reply to this email directly, view it on GitHub
https://github.com/sherpa-deproject/deproject/issues/19#issuecomment-1180717133,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ7ZEDTLZWGKDRJ5SBVWN3LVTRQBBANCNFSM53E2MWDQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Unfortunately I can not replicate this.
From you r original error report I see paths like
ImportError while importing test module '/home/skk/deproject-main/tests/test_deproject.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tests/test_deproject.py:13: in
from deproject.deproject import Deproject
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/init.py:12: in
???
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/deproject.py:25: in
???
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/plot/init.py:59: in
from sherpa.utils import NoNewAttributesAfterInit, erf, SherpaFloat,
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py:43: in
from sherpa.utils import _utils, _psf
E ImportError: cannot import name '_psf' from partially initialized module 'sherpa.utils' (most likely due to a circular import) (/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py)

These are not the paths I would expect if you had CIAO installed. If you had used ciao-install then it should be talking about some path relative to the CIAO installation - in your case /home/skk/CIAO-4.14/ciao-4.14/(it you had installed CIAO 4.14. with conda it would be a different path, but again not in /usr/local/).
What is the output of ciaover -v?

I have installed CIAO using ciao-install.
The output of ciaover -v :
@.***:~$ ciaover -v
The current environment is configured for:
CIAO        : CIAO 4.14 Tuesday, November 23, 2021
Tools       : Package release 1  Tuesday, November 23, 2021
Sherpa      : Package release 1  Tuesday, November 23, 2021
Obsvis      : Package release 1  Tuesday, November 23, 2021
Core        : Package release 1  Tuesday, November 23, 2021
Contrib     : Package release 2  Tuesday, April 19, 2022
bindir      : /home/skk/CIAO-4.14/ciao-4.14/bin
Python path : /home/skk/CIAO-4.14/ciao-4.14/ots/bin
CALDB       : 4.9.8
CIAO Installation: Linux
System information:
Linux skk-HP-15-Notebook-PC 5.15.0-46-generic #49~20.04.1-Ubuntu SMP Thu
Aug 4 19:15:44 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
On Wed, 14 Sept 2022 at 01:36, Doug Burke @.***> wrote:

Unfortunately I can not replicate this.
From you r original error report I see paths like
ImportError while importing test module '/home/skk/deproject-main/tests/test_deproject.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tests/test_deproject.py:13: in
from deproject.deproject import Deproject
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/init.py:12: in
???
/usr/local/lib/python3.8/dist-packages/deproject-0.2.1-py3.8.egg/deproject/deproject.py:25: in
???
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/plot/init.py:59: in
from sherpa.utils import NoNewAttributesAfterInit, erf, SherpaFloat,
/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py:43: in
from sherpa.utils import _utils, _psf
E ImportError: cannot import name '_psf' from partially initialized module 'sherpa.utils' (most likely due to a circular import) (/usr/local/lib/python3.8/dist-packages/sherpa-4.14.1-py3.8-linux-x86_64.egg/sherpa/utils/init.py)
These are not the paths I would expect if you had CIAO installed. If you
had used ciao-install then it should be talking about some path relative
to the CIAO installation - in your case /home/skk/CIAO-4.14/ciao-4.14/(it
you had installed CIAO 4.14. with conda it would be a different path, but
again not in /usr/local/).
What is the output of ciaover -v?
—
Reply to this email directly, view it on GitHub
https://github.com/sherpa-deproject/deproject/issues/19#issuecomment-1245899370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ7ZEDVM4XPG66JJVZXBOOLV6DNC5ANCNFSM53E2MWDQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I have just uploaded version 0.2.2 of deproject to pypi - see https://pypi.org/project/deproject/ - so hopefully you don't need to try installing from github and should just be able to say
pip3 install deproject
from within a CIAO environment.

Thank you sir. I will try to install it.
On Thu, 15 Sept 2022 at 01:43, Doug Burke @.***> wrote:

I have just uploaded version 0.2.2 of deproject to pypi - see
https://pypi.org/project/deproject/ - so hopefully you don't need to try
installing from github and should just be able to say
pip3 install deproject
from within a CIAO environment.
—
Reply to this email directly, view it on GitHub
https://github.com/sherpa-deproject/deproject/issues/19#issuecomment-1247253920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ7ZEDWX74KMEH7UWDUCHSLV6IWYDANCNFSM53E2MWDQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Dear Sir,
I successfully installed deproject v 0.2.2 using the link you provided.
I ran "ciao4.14" and after that "pip3 install deproject" in the terminal.
Now, I am able to use deproject.
Thank you very much sir for giving your valuable time to solve my problem.
Sincerely yours,
Sonali
On Thu, 15 Sept 2022 at 01:43, Doug Burke @.***> wrote:

I have just uploaded version 0.2.2 of deproject to pypi - see
https://pypi.org/project/deproject/ - so hopefully you don't need to try
installing from github and should just be able to say
pip3 install deproject
from within a CIAO environment.
—
Reply to this email directly, view it on GitHub
https://github.com/sherpa-deproject/deproject/issues/19#issuecomment-1247253920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ7ZEDWX74KMEH7UWDUCHSLV6IWYDANCNFSM53E2MWDQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sorry it took so long to resolve.

The output is :
@.**:~$ which python
@.:~$ printenv
SHELL=/bin/bash
SESSION_MANAGER=local/skk-HP-15-Notebook-PC:@
/tmp/.ICE-unix/1699,unix/skk-HP-15-Notebook-PC:/tmp/.ICE-unix/1699
QT_ACCESSIBILITY=1
COLORTERM=truecolor
XDG_CONFIG_DIRS=/etc/xdg/xdg-ubuntu:/etc/xdg
XDG_MENU_PREFIX=gnome-
GNOME_DESKTOP_SESSION_ID=this-is-deprecated
LANGUAGE=en_IN:en
GNOME_SHELL_SESSION_MODE=ubuntu
SSH_AUTH_SOCK=/run/user/1000/keyring/ssh
@.=ibus
DESKTOP_SESSION=ubuntu
SSH_AGENT_PID=1656
GTK_MODULES=gail:atk-bridge
PWD=/home/skk
LOGNAME=skk
XDG_SESSION_DESKTOP=ubuntu
XDG_SESSION_TYPE=x11
GPG_AGENT_INFO=/run/user/1000/gnupg/S.gpg-agent:0:1
XAUTHORITY=/run/user/1000/gdm/Xauthority
GJS_DEBUG_TOPICS=JS ERROR;JS LOG
WINDOWPATH=2
HOME=/home/skk
USERNAME=skk
IM_CONFIG_PHASE=1
LANG=en_IN
LS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:mi=00:su=37;41:sg=30;43:ca=30;41:tw=30;42:ow=34;42:st=37;44:ex=01;32:.tar=01;31:.tgz=01;31:.arc=01;31:.arj=01;31:.taz=01;31:.lha=01;31:.lz4=01;31:.lzh=01;31:.lzma=01;31:.tlz=01;31:.txz=01;31:.tzo=01;31:.t7z=01;31:.zip=01;31:.z=01;31:.dz=01;31:.gz=01;31:.lrz=01;31:.lz=01;31:.lzo=01;31:.xz=01;31:.zst=01;31:.tzst=01;31:.bz2=01;31:.bz=01;31:.tbz=01;31:.tbz2=01;31:.tz=01;31:.deb=01;31:.rpm=01;31:.jar=01;31:.war=01;31:.ear=01;31:.sar=01;31:.rar=01;31:.alz=01;31:.ace=01;31:.zoo=01;31:.cpio=01;31:.7z=01;31:.rz=01;31:.cab=01;31:.wim=01;31:.swm=01;31:.dwm=01;31:.esd=01;31:.jpg=01;35:.jpeg=01;35:.mjpg=01;35:.mjpeg=01;35:.gif=01;35:.bmp=01;35:.pbm=01;35:.pgm=01;35:.ppm=01;35:.tga=01;35:.xbm=01;35:.xpm=01;35:.tif=01;35:.tiff=01;35:.png=01;35:.svg=01;35:.svgz=01;35:.mng=01;35:.pcx=01;35:.mov=01;35:.mpg=01;35:.mpeg=01;35:.m2v=01;35:.mkv=01;35:.webm=01;35:.ogm=01;35:.mp4=01;35:.m4v=01;35:.mp4v=01;35:.vob=01;35:.qt=01;35:.nuv=01;35:.wmv=01;35:.asf=01;35:.rm=01;35:.rmvb=01;35:.flc=01;35:.avi=01;35:.fli=01;35:.flv=01;35:.gl=01;35:.dl=01;35:.xcf=01;35:.xwd=01;35:.yuv=01;35:.cgm=01;35:.emf=01;35:.ogv=01;35:.ogx=01;35:.aac=00;36:.au=00;36:.flac=00;36:.m4a=00;36:.mid=00;36:.midi=00;36:.mka=00;36:.mp3=00;36:.mpc=00;36:.ogg=00;36:.ra=00;36:.wav=00;36:.oga=00;36:.opus=00;36:.spx=00;36:.xspf=00;36:
XDG_CURRENT_DESKTOP=ubuntu:GNOME
VTE_VERSION=6003
GNOME_TERMINAL_SCREEN=/org/gnome/Terminal/screen/b28aa8ca_56e6_441b_bc8b_958598602095
INVOCATION_ID=0f59f395d8834d239618429648087113
MANAGERPID=1478
HEADAS=/home/skk/heasoft-6.29/x86_64-pc-linux-gnu-libc2.31
GJS_DEBUG_OUTPUT=stderr
LESSCLOSE=/usr/bin/lesspipe %s %s
XDG_SESSION_CLASS=user
TERM=xterm-256color
LESSOPEN=| /usr/bin/lesspipe %s
USER=skk
GNOME_TERMINAL_SERVICE=:1.92
DISPLAY=:0
SHLVL=1
QT_IM_MODULE=ibus
XDG_RUNTIME_DIR=/run/user/1000
JOURNAL_STREAM=8:41700
XDG_DATA_DIRS=/usr/share/ubuntu:/usr/local/share/:/usr/share/:/var/lib/snapd/desktop
PATH=/home/skk/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
GDMSESSION=ubuntu
DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/1000/bus
OLDPWD=/home/skk/sam1/RXCJ1023.8-2715/9400
_=/usr/bin/printenv
On Wed, 14 Sept 2022 at 19:10, Doug Burke @.***> wrote:

For some reason it's trying to use a different python, not the CIAO one.
If you do which python do you get back /home/skk/CIAO-4.14/ciao-4.14/bin/python`
or something else?
Can you also send the output of printenv (you might want to remove any
private environment variables that are set first)?
—
Reply to this email directly, view it on GitHub
https://github.com/sherpa-deproject/deproject/issues/19#issuecomment-1246781600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ7ZEDRDPKZ4BG5T2QJC4JTV6HITJANCNFSM53E2MWDQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


new: Migrations

Adding Migrations service for moving liquidity between pools. It supports unstaking / staking in gauges.

Other than the test scenarios that are testing migrations from/to the same pool, I think everything is good to go 👍

Out of curiosity, why can't I see the diff for each commit? It feels like each commit is pushing the whole PR again 🤔 It's a bit tricky to follow only the latest changes 😬

True, I was squashing all the commits too early thinking it will be merged. I should have waited bit longer till everything is reviewed. Sorry about that.


Would it be possible to include the frame timestamp in OutputVideoFrame?

First, I want to thank you for making this crate. ffmpeg is a powerful tool but it can be a major pain to work with, especially via the rust bindings available in many crates. Dealing with the CLI is slightly better but still far from ergonomic. This crate makes it so much better!
I was wondering if ffmpeg emits the frame timestamp when outputting rawvideo, and if so if this data can be captured and included in OutputVideoFrame. I'm using your library to overlay images on top off the video frames and for aligning the images and the video frame it be a lot easier if I had access to each frame's timestamp. The Progress(FfmpegProgress) event does not get emitted for every decoded frame so it's not usable for this purpose.

Hey @avsaase. Great suggestion. I'll make a PR for this feature later this week. After that you can let me know if it works for your use case.


vdebug panels don't resize when terminal geometry is changed

When debugging is active, and let's say terminal window size is changed, vdebug doesn't resize its panels, and there is no obvious way to resize them manually, since they don't act like regular splits that can be moved with a mouse. Is there some way to resize them?

Oh, I managed to move them as a regular split. Not sure why it didn't work before. Sorry for false alarm.


✨ bsx landing top collection

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Feature

PR Context?

[x] Closes #4059
[x] related with deployment snek

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[ ] Fill up your KSM address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot 📸

after clicking See More the result is empty, is that ok?

related with #3654

nice ,let's go

yess


Add updates made to TOB Timeline are made to ICOB

Ensure any commits to TOB are also applied to ICOB (https://github.com/bcgov/indy-catalyst).
Updates were last sync'ed on July 26 so any TOB updates since then must be manually applied to ICOB.
Since the ICOB structure is somewhat different than TOB, this might take a bit of work, but you'll have to get into it to figure out how much.

@seanadipose Please plan on doing this after the Identity Kit work that was discussed yesterday.


[WIP] Markdown linter

Add a markdown linter for a better markdown :)
Ok, so what do you think? Is having a markdown linter too much? Linting inside a linter is too meta? 😄

[x] Add to CI
[x] No offenses left
[ ] Fix rake task doc generation

Most common offenses are line length and headers with same content but I disabled them for now:
>> puts `mdl -r #{`mdl -l`.lines[1..-1].map { |l| l.split.first }.join(',')} **/*.md`.lines.map { |l| l.split[1] }.group_by(&:itself).map { |a, b| [a, b.size] }.sort_by(&:last).reverse.map { |l| "- " + l.join(": ") }.join("\n")

# master
- MD013: 1188
- MD024: 530
- MD031: 214
- MD012: 155
- MD002: 54
- MD009: 18
- MD004: 14
- MD005: 12
- MD014: 11
- MD034: 5
- MD032: 5
- MD036: 3
- MD006: 3
- MD033: 3
- MD029: 2
- MD038: 1
- MD023: 1
- MD027: 1
- MD007: 1

# current markdown-linter branch
- MD013: 1187
- MD024: 530
- MD002: 54
- MD014: 11
- MD034: 5
- MD036: 3
- MD033: 3
- MD029: 2
- MD038: 1
- MD023: 1

RULES.md
scripts I wrote/used

Before submitting the PR make sure the following are checked:

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

That's an interesting idea.

Any progress here?

@bbatsov Actually not sure where I'm going with that, I'm gonna give myself one more week otherwise move on, at first I just wanted to remove the double newlines and the newlines in the end of the files but this was done in #4085

(A lot of the markdown files are generated so the "fixes" need to happen at the generation level / doc level)


added in getPersonsProjects

Added in new method for getting a person's projects list from the api.

Cool, thanks for the effort! Could you also add a test for this new method? Checkout the other tests for inspiration on how to do this. If you need help, let us know.

Hey I need help regarding the test setup, I started writing one using another as a reference. What is the JSON at the bottom, a sample expected response?

In the mock directory you should add a mocked response for the call you're testing. Basically it is just a response from the live basecamp api but stored as JSON. This way the tests don't depend on the Basecamp API. So, as you noticed it is indeed an expected response.

I added a test, the top was just copied from another test. Do I need to get that headers and stuff from Postman or Chrome Dev Tools? How are you submitting the API call? Thanks.

Sorry for the delay. This ticket went off my radar.
I copy the headers from the request which I do with the Guzzle client. It is possible to echo the response you receive from a call.


Linked to FileResponse in Content-Disposition docs

FileResponse handles non-ASCII characters in filenames in Content-Disposition.

I agree with Carlton.


get-started: Fix and clarify usage of two dashes in npm scripts

Ref: https://github.com/webpack/webpack.js.org/pull/293#discussion_r87222508

Thanks, I was wondering too...


Set window.cookie_consent via lambda function, allowing conditional execution of plugins

you should set a global variable window.cookie_consent to TRUE or FALSE within the lambda function, so it is possible to check the value during the page load, and disable trackers if it is NOT set to TRUE!
it is a violation of EU rights to monitor the first page load if the "victim" has not agreed first :santa:
as for the silktide.com page, this is cookie set BEFORE I agree:
_ga=GA1.2.660093563.1440424277
_gat=1
and the consent code is BELOW the Google Analytics code, that's also BAD = I assume the best position is within the HEAD section

We're considering this, but just to clarify this:

"it is a violation of EU rights to monitor the first page load if the "victim" has not agreed first"

The law various from country to country in the EU. For the UK at least, this isn't the case, but other countries (notably France and Greece) are more restrictive.

A conditional execution  would be great, since I have a multi-country site and the plugin will be active on some sites, but not on all.. For now, I am setting the JS Options inline, but it would be great, if I could move the options and initialization of the cookie plugin to my js code..

This is now possible in CookieConsent V3, as it has location and cookie law features built into it.
There are a few callback functions that can be used to enable/disable cookies.


Add links to popup in license submitting form issue#117

Add links for GitHub login in the popup in the license submission form. Edit 'Licenses list to License list'.
Fixes for issue #117.

@anshuldutt21  I am not sure if we need to give github login link in the dialog box, since we don't want user to navigate away from our site to github, rather we want them to navigate to the github authentication page where they allow our app to use their github account (this includes github login). This is what submit dialog box does.
So, providing a link for login is useless.

@anshuldutt21 I think you misunderstood the issue. We would like to show the OSI approved link in the form ,as done in this PR https://github.com/spdx/license-list-XML/pull/862/files
Now you have to decide when to show this link, where to show and how to show.

I guess we can add the link to the header only as its not possible in the description part since that is based on hover and so the link won't be clickable.

@anshuldutt21 It's your choice where you want to show the link, paste a Screenshot of the change and raise the PR and we will comment if we agree with the design.

I added the link after the OSI status label.

@rtgdk please review the new commit.

@rtgdk Please review the changes

@anshuldutt21 LGTM.
@goneall  @jlovejoy Can you please take a look at the above screenshot? If you approve, I'll merge.
Also should we show FSF Free/Libre? field in the form as done in https://github.com/spdx/license-list-XML/pull/862/files ?


12 Procedures

With branches and loops in place, let's move on to procedures, then functions.
We will have to add some features to allow procedures and functions to have their own frames for locally declared variables (if any). But not here, not yet. For now, our procedures will use the global frame.
A procedure with no parameters is defined as follows:
PROCEDURE <identifier>
 <statements>
ENDPROCEDURE

A procedure with parameters is defined as follows:
PROCEDURE <identifier>(<param1>:<datatype>,<param2>:<datatype>...)
 <statements>
ENDPROCEDURE

Parsing PROCEDURE statements
We match for the PROCEDURE keyword:
https://github.com/nyjc-computing/pseudo/blob/256476cdff7a680a934b2f7e77245c517ce16f06/parser.py#L312-L313
And implement it as follows:
https://github.com/nyjc-computing/pseudo/blob/ea894883e1d0dad20319865a7384c113fa68200d/parser.py#L292-L318
if-while is an ugly way to parse optional arguments, but we can figure out more elegant ways later. Let's just make it work first.
At first glance, procedures look very similar to functions. Couldn't we just use the same stmt structure for both procedures and functions?
Yes, kinda ... but we can only call procedures with the CALL keyword (i.e. calling a procedure is a statement in itself), while functions can be called anywhere in place of an expression.
This means we need a way to differentiate functions and procedures, such that function calls are evaluate()d while procedure CALLs are execute()d. For now we write them as two separate statements, and later on we can refactor.

Resolving PROCEDURE statements
This one is easy:
https://github.com/nyjc-computing/pseudo/blob/6621c777162d6e339336a47b20748c25121e188b/resolver.py#L104-L105
https://github.com/nyjc-computing/pseudo/blob/6621c777162d6e339336a47b20748c25121e188b/resolver.py#L86-L88
Procedures have no return type that need matching or storing, and the name and argument declarations have no get expressions. That leaves only the stmts that need verifying.

Executing PROCEDURE declarations
Declaring a procedure is the easy part:
https://github.com/nyjc-computing/pseudo/blob/d5a8920b8afd667203d9d21f11b3383a9116391c/interpreter.py#L93-L94
https://github.com/nyjc-computing/pseudo/blob/d5a8920b8afd667203d9d21f11b3383a9116391c/interpreter.py#L68-L74
We just store everything the procedure needs into the frame for now. Executing it is going to be another story ...
And meanwhile we catch a typo bug:
[09d2ecb27e33f710f437950af6fa35ddd3487481]

Testing
PROCEDURE SayHi
    OUTPUT "Hi!"
ENDPROCEDURE

Result (frame):
{'SayHi': {'type': 'procedure', 'args': {}, 'stmts': [{'rule': 'output', 'exprs': [{'type': 'string', 'word': '"Hi!"', 'value': 'Hi!'}]}]}}

DECLARE Person : STRING
PROCEDURE SayHi(Person : STRING)
    OUTPUT "Hi, ", Person, "!"
ENDPROCEDURE

Result:
  File "parser.py", line 299, in procedureStmt
    args[var] = {'type': typetoken, 'value': None}
TypeError: unhashable type: 'dict'

Oops. At this point, var is a token and not yet a name. I'll have to extract 'word' from the token first. Fix: [15113393535903855e1fb2b9624e70b719b4445c]
Result (frame):
{'Person': {'type': 'STRING', 'value': None}, 'SayHi': {'type': 'procedure', 'args': {'Person': {'type': {'type': 'name', 'word': 'STRING', 'value': None}, 'value': None}}, 'stmts': [{'rule': 'output', 'exprs': [{'type': 'string', 'word': '"Hi, "', 'value': 'Hi, '}, {'left': {...}, 'oper': {'type': 'symbol', 'word': '', 'value': <function get at 0x7f1d6607c940>}, 'right': {'type': 'name', 'word': 'Person', 'value': None}}, {'type': 'string', 'word': '"!"', 'value': '!'}]}]}}


bug: Erro na build de produção para hospedagem do PWA

Testing the app with the ionic cordova run browser command it works normally when I generate the production build with the ionic build --prod --release command and send it to my hosting as the documentation says or using firebase hosting generates the console errors.
main-es2015.bbc63cf846c6411c3ada.js:1 ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'subscribe' of undefined
TypeError: Cannot read property 'subscribe' of undefined
at main-es2015.bbc63cf846c6411c3ada.js:1
at a.invoke (polyfills-es2015.558cb2cc942b93a94480.js:1)
at Object.onInvoke (main-es2015.bbc63cf846c6411c3ada.js:1)
at a.invoke (polyfills-es2015.558cb2cc942b93a94480.js:1)
at t.run (polyfills-es2015.558cb2cc942b93a94480.js:1)
at polyfills-es2015.558cb2cc942b93a94480.js:1
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at Object.onInvokeTask (main-es2015.bbc63cf846c6411c3ada.js:1)
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at t.runTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at P (polyfills-es2015.558cb2cc942b93a94480.js:1)
at polyfills-es2015.558cb2cc942b93a94480.js:1
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at Object.onInvokeTask (main-es2015.bbc63cf846c6411c3ada.js:1)
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at t.runTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at _ (polyfills-es2015.558cb2cc942b93a94480.js:1)
at c.invokeTask [as invoke] (polyfills-es2015.558cb2cc942b93a94480.js:1)
at y (polyfills-es2015.558cb2cc942b93a94480.js:1)
at HTMLHeadingElement.k (polyfills-es2015.558cb2cc942b93a94480.js:1)
main-es2015.bbc63cf846c6411c3ada.js:1 ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'platform' of undefined
TypeError: Cannot read property 'platform' of undefined
at main-es2015.bbc63cf846c6411c3ada.js:1
at a.invoke (polyfills-es2015.558cb2cc942b93a94480.js:1)
at Object.onInvoke (main-es2015.bbc63cf846c6411c3ada.js:1)
at a.invoke (polyfills-es2015.558cb2cc942b93a94480.js:1)
at t.run (polyfills-es2015.558cb2cc942b93a94480.js:1)
at polyfills-es2015.558cb2cc942b93a94480.js:1
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at Object.onInvokeTask (main-es2015.bbc63cf846c6411c3ada.js:1)
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at t.runTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at P (polyfills-es2015.558cb2cc942b93a94480.js:1)
at polyfills-es2015.558cb2cc942b93a94480.js:1
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at Object.onInvokeTask (main-es2015.bbc63cf846c6411c3ada.js:1)
at a.invokeTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at t.runTask (polyfills-es2015.558cb2cc942b93a94480.js:1)
at _ (polyfills-es2015.558cb2cc942b93a94480.js:1)
main-es2015.bbc63cf846c6411c3ada.js:1 Native: InAppBrowser is not installed or you are running on a browser. Falling back to window.open.
Ionic info:
Ionic:
Ionic CLI                     : 5.4.4 (C:\Users\Pichau\AppData\Roaming\npm\node_modules\ionic)
Ionic Framework               : @ionic/angular 4.10.0
@angular-devkit/build-angular : 0.801.3
@angular-devkit/schematics    : 8.1.3
@angular/cli                  : 8.1.3
@ionic/angular-toolkit        : 2.0.0
Cordova:
Cordova CLI       : 9.0.0<EMAIL_ADDRESS>Cordova Platforms : android 8.1.0
Cordova Plugins   : cordova-plugin-ionic-webview 4.1.2, (and 10 other plugins)
Utility:
cordova-res : 0.8.0
native-run  : 0.2.8 (update available: 0.2.9)
System:
Android SDK Tools : 26.1.1 (C:\Users\Pichau\AppData\Local\Android\Sdk)
NodeJS            : v10.15.3 (C:\Program Files\nodejs\node.exe)
npm               : 6.4.1
OS                : Windows 10

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue? The minified errors don't really tell us too much information.

Obrigado pelo problema. Você pode fornecer um repositório com o código necessário para reproduzir esse problema? Os erros minimizados não nos dizem muita informação.

I have a console image after page load, can it be? About informing the repository I need to check because it is a private project of the company.

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue? The minified errors don't really tell us too much information.

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue? The minified errors don't really tell us too much information.

I have a console image after page load, can it be? About informing the repository I need to check because it is a private project of the company.

Obrigado pelo problema. Você pode fornecer um repositório com o código necessário para reproduzir esse problema? Os erros minimizados não nos dizem muita informação.

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue? The minified errors don't really tell us too much information.

Can it be a project link on google drive or does it need to be on github?

A project link on Google Drive is fine.

Um link de projeto no Google Drive está bom.

A project link on Google Drive is fine.

Attached is the link -> https://drive.google.com/open?id=1-5CSYIoZbc-jRqwcL5r4rg2ZQp26bez7

Thanks for the follow up. It looks like you have a bug in user.service.ts:38:
const devicePlatform = platform.win.device.platform.toLowerCase();

platform is undefined. This might be a variable that is only defined when running as a standalone app in Cordova. The other subscribe error also appears to be a bug in your code. I would recommend reviewing your app to ensure that these properties/variables are defined before accessing them.
I am going to close this issue as this is not a bug in Ionic Framework.


Proposal: "No active contributors" flag for abandoned katas

Katas with inactive authors and contributors are quite common. People come to CW, write some katas, but then addiction dries out and they are never here any longer.
I would call such katas abandoned, as they have poor support. In a kata where author and all contributors are inactive, resolving an issue is only possible for a moderator, which has little time to do that... and so these katas become a pile of issues, especially after 500 solutions lock.
I suggest to flag such abandoned katas and provide experienced warriors a list of these (for example, titled "Contributors wanted"), with an option to add self as a contributor if one wishes to. This will allow for much better support of old katas.
The criteria for warrior inactivity could be absence of any activity for a certain period of time. I suggest to make this period variable depending on warrior's level - from a week for 8 kyu to a month for 1 kyu, for example.

This would be really great. +1!
Currently there are quite a few Katas which have major issues or pending translation which will never get solved or approved.
An example:

http://www.codewars.com/kata/ackermann-function/java

The Java translation has a broken or better missing method signature, uses a method name which starts with an uppercase letter and only contains only 3 final tests.
Maybe I'm overreacting a bit and it only requires so little to verify the solutions, but I've seen 7/8 Kyu Katas with way more work put into them...

@jhoffner  please take a look to the ackermann it should be editable with your updates when language  has less than 500 solutions #123 but even if I have the contributor hability can't edit it. Is it normal?


Unify StateProviderFactory StateProvider types

Describe the feature
Currently, the factory has two different types for accessing State
https://github.com/paradigmxyz/reth/blob/1642d77dcd2cf2190f99189ba690ce3403989b55/crates/storage/provider/src/traits/state.rs#L20-L27
Which makes sense, since we need two different implementations I suppose? (@rakita)
eventually, we also need a Pending state.
in RPC it'd be more convenient to return a single type.
we could either replace LatestSP and HistorySP with a single associated type.
or add something like
enum StateProvider<H,L,P> {
 History(H),
 Latest(L),
 Pending(P)
}

But unsure how an implementation of History would differ from Latest
Additional context
No response

Latest touches PlainState tables while History touches HistoryIndex/Changesets/PlainState to get the needed past state. Latest is simpler and faster.
Pending is going to be added after work on BlockchainTree is finished.
Could you maybe cast them to StateProvider trait (with generic or box them), they are different structures but implement the same trait?


fix nethermind chainspec

Is this a required field in the Nethermind chainspec?

It is required. https://discord.com/channels/595666850260713488/892088344438255616/1055533562453049395

Please make a new release once you've merged it

Looks like the docker build has failed.
Error: buildx failed with: ERROR: failed to solve: process "/bin/sh -c git clone https://github.com/protolambda/eth2-testnet-genesis.git     && cd eth2-testnet-genesis     && go install .     && go install github.com/protolambda/eth2-val-tools@latest     && go install github.com/protolambda/zcli@latest" did not complete successfully: exit code: 2

hmm, let me try it as well. There were some changes recently to eth2-testnet-genesis and it worked locally atleast. But we might need to update go version or something.


Attempted import error: 'URLSearchParams' is not exported from 'url'

Hi, I'm using node-fetch-cache to deploy a site on vercel and I'm having this issue during deploy:

./node_modules/node-fetch-cache/src/index.js
Attempted import error: 'URLSearchParams' is not exported from 'url' (imported as 'URLSearchParams').

Here's my package.json:
{
  "name": "[masked]",
  "version": "1.0.0",
  "description": "[masked]",
  "main": "index.js",
  "scripts": {
    "dev": "next",
    "build": "next build",
    "start": "next start",
    "postbuild": "next-sitemap",
    "fonts": "fontello-cli install --config ./fontello-config.json --font ./public/font --css ./public/css"
  },
  "repository": {
    "type": "git",
    "url": "[masked]"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "bugs": {
    "url": "https://github.com/[masked]"
  },
  "homepage": "https://github.com/[masked]",
  "dependencies": {
    "@next/bundle-analyzer": "^10.1.2",
    "axios": "^0.21.1",
    "classnames": "^2.2.6",
    "cookie": "^0.4.1",
    "dayjs": "^1.10.4",
    "imagemin-mozjpeg": "^8.0.0",
    "imagemin-optipng": "^7.1.0",
    "imagemin-svgo": "^7.1.0",
    "isomorphic-unfetch": "^3.0.0",
    "js-cookie": "^2.2.1",
    "lodash": "^4.17.19",
    "mixpanel-browser": "^2.39.0",
    "next": "^11.1.2",
    "node-fetch-cache": "^3.0.4",
    "node-html-parser": "^1.4.9",
    "nookies": "^2.2.4",
    "nprogress": "^0.2.0",
    "numeral": "^2.0.6",
    "probe-image-size": "^6.0.0",
    "react": "^17.0.2",
    "react-copy-to-clipboard": "^5.0.2",
    "react-dom": "^17.0.2",
    "react-fade-in": "^1.1.1",
    "react-gtm": "github:Vadorequest/react-gtm#2.0.0",
    "react-hook-form": "^6.9.5",
    "react-html-parser": "^2.0.2",
    "react-lazy-load-image-component": "^1.4.1",
    "react-lazyload": "^3.0.0",
    "react-mailchimp-subscribe": "^2.1.0",
    "react-masonry-css": "^1.0.14",
    "react-modal-video": "^1.2.5",
    "react-multistep-wizard": "^3.0.1",
    "react-slick": "^0.27.3",
    "recharts": "^1.8.5",
    "sass": "^1.26.5",
    "simple-react-lightbox": "^3.6.9-0",
    "slick-carousel": "^1.8.1",
    "smoothscroll": "^0.4.0"
  },
  "devDependencies": {
    "fontello-cli": "^0.6.2",
    "next-sitemap": "^1.2.4"
  }
}

Found this thread: https://github.com/Azure/azure-sdk-for-js/issues/7381
But installing node/types doesn't solve my problem...
Any idea where this can come from ?

Hello, I'm not familiar with Vercel and that error is rather strange, almost like its injecting its own version of the url module. URLSearchParams is a global class now though, so importing it is not even necessary (as of Node v10). So I have removed the import and pushed new version v3.0.5 to npm. You can try installing it with npm install node-fetch-cache@latest and maybe it will work for you.

Omg thanks so much for that reactivity ⚡
That solved my issue :)

Argh, seems that I got happy too fast..
Now I have this weird issue:
Unhandled Runtime Error
TypeError: The "original" argument must be of type Function

Call Stack
promisify
node_modules/util/util.js (614:0)
eval
<EMAIL_ADDRESS>(25:0)
<EMAIL_ADDRESS>file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/pages/_app.js (716:1)
Object.options.factory
/_next/static/chunks/webpack.js (685:31)
__webpack_require__
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/webpack.js (37:33)
fn
/_next/static/chunks/webpack.js (354:21)
eval
node_modules/cacache/lib/entry-index.js (16:17)
Object../node_modules/cacache/lib/entry-index.js
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/pages/_app.js (1598:1)
Object.options.factory
/_next/static/chunks/webpack.js (685:31)
__webpack_require__
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/webpack.js (37:33)
fn
/_next/static/chunks/webpack.js (354:21)
eval
node_modules/cacache/ls.js (3:14)
Object../node_modules/cacache/ls.js
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/pages/_app.js (1686:1)
Object.options.factory
/_next/static/chunks/webpack.js (685:31)
__webpack_require__
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/webpack.js (37:33)
fn
/_next/static/chunks/webpack.js (354:21)
eval
node_modules/cacache/index.js (3:11)
Object../node_modules/cacache/index.js
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/pages/_app.js (1543:1)
Object.options.factory
/_next/static/chunks/webpack.js (685:31)
__webpack_require__
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/webpack.js (37:33)
fn
/_next/static/chunks/webpack.js (354:21)
eval
webpack-internal:///./node_modules/node-fetch-cache/src/classes/caching/file_system_cache.js (5:65)
Module../node_modules/node-fetch-cache/src/classes/caching/file_system_cache.js
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/pages/_app.js (4513:1)
Module.options.factory
/_next/static/chunks/webpack.js (685:31)
__webpack_require__
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/webpack.js (37:33)
fn
/_next/static/chunks/webpack.js (354:21)
eval
webpack-internal:///./node_modules/node-fetch-cache/src/index.js (13:95)
Module../node_modules/node-fetch-cache/src/index.js
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/pages/_app.js (4557:1)
Module.options.factory
/_next/static/chunks/webpack.js (685:31)
__webpack_require__
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/webpack.js (37:33)
fn
/_next/static/chunks/webpack.js (354:21)
eval
webpack-internal:///./pages/_app.js (10:74)
Module../pages/_app.js
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/pages/_app.js (40:1)
Module.options.factory
/_next/static/chunks/webpack.js (685:31)
__webpack_require__
file:///Users/guillaume/dev/mecanicus/app/.next/static/chunks/webpack.js (37:33)
fn
/_next/static/chunks/webpack.js (354:21)
eval
node_modules/next/dist/build/webpack/loaders/next-client-pages-loader.js?page=%2F_app&absolutePagePath=private-next-pages%2F_app! (5:15)
eval
node_modules/next/dist/client/route-loader.js (207:48)

That looks to me like it's trying to compile this module into your frontend code. Is that your intention? If so, I'm afraid this module will only work in backend Node.js code, it won't run in a browser.

yes this code is definitely supposed to be run on the server side... I don't get it, I guess that's a question for next.js community. Regardless, thanks a lot for your help :)


Backport 2.7: IOTSSL-1879 buffer overflow in server key exchange parsing

Backport of #1439 to 2.7

retest

Jenkins failed only due to timing self-test on FreeBSD (known issue), so ok to merge.


Google provider is not working

Description
NOTE: # Creating new statuspage with Google provider is throwing an error
[TIP]:  # ( Do NOT give us access or passwords to your New Relic account or API keys! )
Steps to Reproduce

Create new service.
Choose provider Google
Add any status page you want (I tried https://status.azure.com )

Browser name and version:
Version 85.0.4183.102 (Official Build) (64-bit)

Operating System and version: Windows 10 2004

Thanks for the issue Aren, I'm looking into this today. Will hopeful have something good to report a little later.

This error is caused by a CORS violation - a proxy configuration would be required.


Container steps also have stdout stored in result output parameter

Summary
The docs seem to suggest that only script steps will automatically capture stdout into the result output parameter. I did some testing and this doesn't seem to be the case and I couldn't see anywhere in the code that only makes this happen for script steps.
For this card I believe we should update the docs to reflect this and rename functions for capturing stdout to remove references to script (e.g. this function).
Use Cases
From the docs I was under the impression container steps wouldn't have their stdout automatically captured in the result output parameter.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Feel free to submit a PR to improve the docs.

I'd be happy to. Could I get some confirmation that I'm correct and not missing something before I do so?

Hey @lacarvalho91 I just wanted to let you know that I verified this by running a workflow that uses {{ tasks.previous-task.outputs.result }} as an input and it worked.


[Internet Explorer] Website Not Loading

Bug Description
Website not open In internet Explorer
Steps to Reproduce
Steps to reproduce the behavior:

Go to 'IE' (internet Explorer)
2.Enter Following Url "https://thenewboston.com/"

Expected behavior
Website supports Every Browser
Actual behavior
Website not supports internet explorer
Screenshots/Recordings
https://prnt.sc/vejidh
Source URL: https://thenewboston.com/
| Browser | Ie 11.0 |
| Operating System | Windows 10 |
| Resolution | 1920x880 |

I will fix this

@webdeveloperswj @angle943 I think this issue is a duplicate of #211

#542 Pull request submitted

Closing because this is a duplicate issue.


samples: bluetooth: periperal_hr: create minimal configuration

Create minimal configuration for the peripheral_hr. Which can
be used to build this sample for nRF52810 and nRF52811 targets.
Signed-off-by: Marcin Jeliński<EMAIL_ADDRESS>

@carlescufi Please merge.


Geode says it's in version 0.6.99999999 when attempting to install an incompatible mod

Platform
Windows
Geode Version
v0.6.1
Mods Installed
alk.editorcollab v1.3.0, mat.scroll-zoom 1.0.0, hjfod.devtools v0.0.5, mat.practice-music v1.0.1, com.mat.fps-bypass v1.0.0
(failed to load, can't check id or versions): acbypass, demonlist-button, noclip
Expected Behavior
Expected version to show up as 0.6.1 or 0.7.0.
Actual Behavior
Version showed up as 0.6.99999999

Steps to Reproduce

Manually install acbypass, demonlist-button or noclip by camila314
Open the game, a pop-up saying some mods failed to load should come up
Open the Geode menu
Press "Info" on any of the mods that failed to load

Additional Information
could be because of how different operating systems handle numbers? unsure

Fixed in 0.7.0 already
But thank you


Disable local storage

I have Frigate running in a docker container on a Synology NAS. So recordings, snapshots etc are stored on the nas.
I've installed this frigate integration in home assistant pointing to that container's url to have some switches and sensors and that works fine.
But the integration also saves the recordings etc locally on the HA drive, which is in my case unneccesary.
Further more, the HA drive is much smaller than the space on the nas.
I would like an option to disable the local storage for the integration.

The integration doesn't store anything locally. What are you seeing?

the integration is just showing you media provided by frigate, the data is only stored once


Updating test gribi_scale_test

Breaking more than 10k gribi entries from 1 modify operation into multiple modify operations with 10k entries each.

Pull Request Functional Test Report for #3437 / 5e6263daf2f559dfd1a72b2297e55566958d6c70
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

TE-14.1: gRIBI Scaling
cc0bf749
Log

Cisco 8000E

TE-14.1: gRIBI Scaling
e5460c91
Log

Cisco XRd

TE-14.1: gRIBI Scaling
677b2653
Log

Juniper ncPTX

TE-14.1: gRIBI Scaling
19ed7488
Log

Nokia SR Linux

TE-14.1: gRIBI Scaling
d5d59407
Log

Openconfig Lemming

TE-14.1: gRIBI Scaling
67a7f4a4
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

TE-14.1: gRIBI Scaling

Cisco 8808

TE-14.1: gRIBI Scaling

Juniper PTX10008

TE-14.1: gRIBI Scaling

Nokia 7250 IXR-10e

TE-14.1: gRIBI Scaling

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 55.314%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
1983

Relevant Lines:
3585

💛  - Coveralls


Scales Border color doesn't work as expected in chart.js v4.0.1

Expected behavior
Changing options.scales.y.grid.borderColor or options.scales.x.grid.borderColor  modifies scales border color, code example from docs:
const config = {
  type: 'line',
  data,
  options: {
    scales: {
      y: {
        grid: {
          borderColor: 'red'
        }
      }
    }
  }
};

Expected visual result:

Link to docs:
https://www.chartjs.org/docs/3.9.1/axes/cartesian/#border
Current behavior
Changing options.scales.y.grid.borderColor or options.scales.x.grid.borderColor doesn't change the border, border is grey:

Link:
https://www.chartjs.org/docs/4.0.1/axes/cartesian/#border
Reproducible sample
You can reproduce the issue in docs: https://www.chartjs.org/docs/4.0.1/axes/cartesian/#border
Optional extra steps/info to reproduce
Just compare generated chart (expected): https://www.chartjs.org/docs/3.9.1/axes/cartesian/#border and broken: https://www.chartjs.org/docs/4.0.1/axes/cartesian/#border
Possible solution
No response
Context
No response
chart.js version
v4.0.1
Browser name and version
Google Chrome 107.0.5304.107 (Official Build) (64-bit)
Link to your project
No response

https://www.chartjs.org/docs/master/migration/v4-migration.html#specific-changes


Disable unsecured PUT/POST/DELETE api methods

We gained full CRUD support, but I don't think we actually need it for anything yet, and when we add it, we probably should put some kind of authentication in front of it.

Would it be better to just change this issue to add authentication to the "modifiable" endpoints rather then disabling them? Also right now there is no code behind PUT/POST/DELETE only GET is implemented so others would fail anyway. We need other issues to do the work for these endpoints.

What updates are you thinking will come in via the API?   Is MGI thinking of pushing incremental changes to genes?


Improve remote processing performance

This PR main objective is enhance the remote file processing.
When a remote file with high number of pages is processed it takes huge time, because Imagick seems to process the whole file, and not just pages required.
We had facing a lot of troubles on own projects with saveImage first page of pdf, with > 300 pages, reaching more than 500 seconds for processing.
We have tested with this file:
https://www.rd.com.br/Download.aspx?Arquivo=g1va4C+5cfmJ7zPl9zlMTQ==
15MB 556pages
Time to save Image: 465 sec.
Then to fix it (or improve) we’ve updated the constructor to fetch remote file and process is as local.
Time do save Image after improvement: 59 sec.

Thank you for this, but I like to keep things package simple and currently don't have a need for handling large PDFs coming from the network.
Feel free to fork our code and maintain your own copy.


Add makefile

See https://github.com/Peripli/service-broker-proxy-cf/pull/21

Pull Request Test Coverage Report for Build 255

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.061%

Totals

Change from base Build 253:
0.0%

Covered Lines:
456

Relevant Lines:
490

💛  - Coveralls


No support for directories starting with dash

Same issue as #1, but different cause.
wipe-modules 1.1.1
bash 3.2.57(1)-release (x86_64-apple-darwin15)
OS X 10.11.6
(also tried with fishell 2.5.0)
(I'm working on a fix)

The issue were two folders named -test and -unlisted. Fixed in b70b109c2f66d903596bef1381a48db237b92eb7 using this suggestion https://serverfault.com/a/462740/207547

Awesome! @bfred-it 😄 thx


[Feature Request] Different MOTDs for hostnames (velocity)

Is your feature request related to a problem? Please describe.
I can't display different MOTDs for different hostnames -- in my scenario I have a build and a gameplay server that I want to differerntriate.
Describe the solution you'd like
Mabye some sort of subcategory using the hostname as a paramater?

Just bumping this suggestion.
In the meantime, a workaround is to put MiniMOTD on each server and use velocity's full motd passthrough option. I personally like having it controlled from the proxy though.

Just bumping this suggestion.
In the meantime, a workaround is to put MiniMOTD on each server and use velocity's full motd passthrough option. I personally like having it controlled from the proxy though.

implemented in - https://github.com/jpenilla/MiniMOTD/commit/50158a1a81aeb9a6fa722f02d9347ddb4e718a66
snapshot download: https://jenkins.jpenilla.xyz/job/MiniMOTD/lastSuccessfulBuild/

implemented in - https://github.com/jpenilla/MiniMOTD/commit/50158a1a81aeb9a6fa722f02d9347ddb4e718a66
snapshot download: https://jenkins.jpenilla.xyz/job/MiniMOTD/lastSuccessfulBuild/


Rule: no-watch

Watch have last  computation priority.
Frequent use of the watch can lead to races that are very difficult to debug. Often, the use of the watch can be avoided (by using target, for example).

But effects has the same computation priority 🤔
I agree, we can forbid `.w because it leads to imperative code, but I cannot find the connection with computation priority.

but I cannot find the connection with computation priority

forward({
  from: fooEvt,
  to: barEvt
});

fooEvt.watch(barEvt);

The second option can lead to an "unexpected" calculation order.

If u want I can try to implement this rule. Both my issues are not "do this for us" :)

So, I agree, it's a good rule. Let's do it. I think we ought to mark this rule as warn.

Maybe u can give me some recommendations how to write new rule? I wrote rules many-many years ago :c

Oh, I don't think so 🥲 my only advice: try to write as many tests as you can. ESLint plug-in documentation is a really poor.

Need a little help. What we need to do with the JS way branch? .watch can be called in a file that does not declare a watchable unit. We can't rely on unit naming, because there is no naming convention for events.

Such a good question. I don't have an answer for you.
Maybe, we ought to introduce https://github.com/effector/eslint-plugin/issues/32 before?
But, it isn't resolving a problem with something like sample(...).watch(...) and so on.
So, in my opinion, we have two ways:

Specify this rule only for TS-code
Don't make this rule

What do you think?

I like the first way. I'll explain why.
Don't take the rule away from developers with TypeScript and Effector projects just because not all projects have TypeScript.
Binding to name convention seems fragile because any naming rule can be disabled.

I agree. But we should notice it in documentation.


Removed debug print statement

Removed print call that looks like some kind of debug that should not be included into released code.

Ah, what a miss! 😄 Thanks for fixing this @stoicsm!


Operator of modification

I suggest to add the handy operator
x %~ f  === x = f x
like over in haskell lens library.
It somehow generalizes the .= to be more flexible

Try x |>= f; does that do what you want?

Exactly, thanks!


python312Packages.reuse: 3.0.1 -> 3.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.reuse is: A tool for compliance with the REUSE Initiative recommendations
meta.homepage for python312Packages.reuse is: https://github.com/fsfe/reuse-tool
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.2 with grep in /nix/store/bcn2y5c38kphwdn29y9fahhvpyz3a8gm-python3.12-reuse-3.0.2
found 3.0.2 in filename of file in /nix/store/bcn2y5c38kphwdn29y9fahhvpyz3a8gm-python3.12-reuse-3.0.2

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath
amberol
python311Packages.reuse
python312Packages.reuse
reuse
sbomnix

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bcn2y5c38kphwdn29y9fahhvpyz3a8gm-python3.12-reuse-3.0.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.reuse https://github.com/r-ryantm/nixpkgs/archive/a589193687af4287dc84c54e259cd6808e80c7da.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a589193687af4287dc84c54e259cd6808e80c7da#python312Packages.reuse

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bcn2y5c38kphwdn29y9fahhvpyz3a8gm-python3.12-reuse-3.0.2
ls -la /nix/store/bcn2y5c38kphwdn29y9fahhvpyz3a8gm-python3.12-reuse-3.0.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

7 packages built:

amberol
reuse (python311Packages.reuse)
reuse.dist (python311Packages.reuse.dist)
python312Packages.reuse
python312Packages.reuse.dist
sbomnix
sbomnix.dist

Maintainer pings
cc @FlorianFranzen @Luflosi for testing.

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

7 packages built:

amberol
reuse (python311Packages.reuse)
reuse.dist (python311Packages.reuse.dist)
python312Packages.reuse
python312Packages.reuse.dist
sbomnix
sbomnix.dist


copy apphook page

test checks if the error message appears when a page with the apphook gets copied

👍

Coverage remained the same at 87.94% when pulling 2d8e1427dfde65651864c0ac45f67e20c121c0ee on genti94:feature/test-copy-page-with-apphook into bbf3ec89e29c1e38231ed898989f6dba6d6cb6f4 on divio:develop.


New Feature Request - Album Art

Hi,
I am using a similar plugin in React Native. Among its features is the album art. Can you add the same feature?
RN example: https://github.com/leandrosimoes/react-native-android-notification-listener/issues/21

New version is published notification_listener_service: ^0.3.2


added self-employed conditional logic to jobs[i].complete block

fix #85
Changed the jobs[i].complete block to ask self employment questions if needed.

I'll work on the bug with the edit button, but do you want two separate tables and buttons on the review screen?  It seems like we should do the same with the Review your jobs screen.

I tried making two tables first on the Review your jobs signs, and I put it into the review screen, forgetting the review screen doesn't use tables.  But it could be an easier interface -

I could switch that to look like the rest of the review screen.  Or do we want to use tables more throughout the review screen.
Here is the Review your jobs screen

There's only one add job, which will then let somebody pick self-employed or not.  If that's okay, that will be easier to do (and it will be easier to do on the review screen).
If that looks okay, then I still have some cleaning up.

How about this for the review screen?

Ok, I am going to do a test with no jobs and I'll commit if it works.

Yes, I'm working on this.  I think the best way to do it may be to make "self employed" a "source" like "part time" or "full time".

This seems like an issue with .times_per_year being added to the self-employed income.  I can work on this after a meeting this morning.

I think the pdf is working better but the review tables aren't working right - edit isn't working in the self-employed jobs, and delete deletes the wrong thing (seems to delete jobs[0] when you delete the first item in the self-employment table, even  if that's a job with the employer)

This is also happening in the review screen immediately after the question.  I don't know if there is going to be a good way to fix this using two tables.  The easiest thing to do would be to have one table for all jobs, including self employed (we could have separate totals for employer jobs and self employed on the review screen with the table).