image import module fixed

Could you please reopen the existing pull request rather than resubmitting when you make a change? There's now three closed versions of this pull request.

For formatting headers see the coding conventions page. You should include a file header, as well as header guards.
Also, the indentation in the readcsv function now seems to be a combination of spaces and tabs.

Other than the mentioned issues, I think this is good to go.
It would be nice to have unit tests, though the unit test update is not actually merged yet. We may as well merge this once the remaining issues are fixed and then add unit tests later.

I made some changes, but I am still not sure if I am doing correct in the linked list part. If there are still things to be fixed, please let me know.

Did you consider using an existing csv parser? I am sure there are some good ones. https://github.com/ben-strasser/fast-cpp-csv-parser looks like it will do the trick.
I would also suggest a linked list wouldn't be the correct data structure in this scenario, since you aren't doing many inserts (just append and access). Using something like a vector (which is backed by an array iirc) would give benefits wrt cache locality and better performance for random access. Also worth noting that stdlib has a linked list implementation already (just called list), and I expect boost has something as well.

As was mentioned here in the pull request for grabbing frames from a video stream, we're going to have to abstract the image import interface so that we can support the various methods we're going to be reading images.
Essentially you should move the code you put in the ImageImport class to a new subclass of ImageImport (call it PictureImport or something similar).

Just so we keep things moving....Is this okay to merge with master?

No, we unfortunately can't. For one thing the ImageImport class isn't actually abstract at the moment (hence why PictureImport currently can't override next_frame). I didn't have the time to fix that before, but I'm making changes now so that both PictureImport and VideoImport can properly implement ImageImport.
The other thing is that now that we're splitting up the importing code, the telemetry reading code really should be delegated to an external class that can be passed to and used by all variants of ImageImport (as a parameter to the base class). Note that this also means it has to scan the telemetry log in real time and update the stored data as entries are added to the log on the filesystem..

Look at this: http://www.learncpp.com/cpp-tutorial/79-the-stack-and-the-heap/

There's also now a global testdata directory. We want to avoid having large numbers of image files in the repository since they will quickly bloat it, so any image files should be in the testdata directory (there's a cmake variable that stores the location so you can get the path from anywhere in the project).
Also, since the picture import tests don't really depend on image contents, it would be best if we could use aerial photos that are vaguely relevant (essentially, take them from the photos we took at the competitions), then the images can be reused for things like object detection tests.

Also, if you're going to be working on multiple things at once, please use feature branches (put each feature on its own branch). Having multiple features on a single branch, and thus a single pull request, makes reviewing and merging them more difficult.

@ZheyuanZhang I need to integrate your code with mine. I need you to finish this in the next two days.

@chrishajduk84 You say 'finish' do you mean the test part? i think I have finished the code part.

Yes. The test part.

@chrishajduk84 Should I write a main function for test in src? Or I only need to store some photos and csv data in testdata folder?

@ZheyuanZhang refer to https://github.com/UWARG/computer-vision/blob/master/modules/targetid/test/test.cpp
Note that it is in the test folder.

Better yet. Refer to the WIKI:
https://github.com/UWARG/computer-vision/wiki/Writing Tests

@benjaminwinger I talked to @zheyuanzheng to rewrite the CSV test to be more useful.  What do you think?

Is it good to go now?

While it seems fine to me, the test results on Jenkins are empty for the two tests. This is possibly because none of the BOOST_CHECK statements are being evaluated, which may be a product of all your checks being inside loops which are exiting immediately.
You should check the number of frames/log entries read as well as their contents. That is, read and store the frames/log entries first, check the number of frames/log entries, and then check their contents.
This should at least expose the issue if is with your code, though I can't tell what it is at a glance (if you do the check on the number of frames and it passes then I guess it's an issue with the Jenkins setup).

I have found the problem: the imread function in my code does not read in anything during the test. But I do not know why. Last week, when I use my own test, it worked out just fine. (I used g++ to compile instead of cmake). Perhaps I should add testdata directory into the CMakeLists.txt in imgimport folder?


[DOCFIX] Add distributedLoad command to the caching docs

What changes are proposed in this pull request?
Add distribtuedLoad command to the caching docs
Why are the changes needed?
Otherwise people don't know distributedLoad command can be used to accelerate the data loading speed
Does this PR introduce any user facing changes?
change docs

alluxio-bot, merge this please


dotnet tool sonarscanner no longer recognizes params /k:___, /d:___ and /o:___

.Net 6.0.301
Sonarscanner tool 5.5.3
VS Code 1.68.1
Sonar Dotnet Extension 1.1.0

Ok, problem was in Git Bash:
https://github.com/SonarSource/sonar-scanner-msbuild/issues/548
Switched to Command Prompt terminal, now it works


pull request for e2e test at 11/23/2016 07:56:03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Speed accuracy estimate - e.g. SDOP / SDOS / EHVE / sAcc equivalent

It is not clear whether the equivalent is available from the Sony chip but in speedsurfing we make use of SDOP / SDOS (Locosys) and sAcc (u-blox).
SDOP was introduced in the SiRFstar III and was first made available to GP3S in the Locosys GT-31. It was also available in data from the Locosys GW-52 and GW-60 which probably use the SiRFstar IV.
The usefulness of SDOP was well documented by Tom Chalko in his paper from 2009.
SDOP is calculated by the GPS chip and is based using the covariation matrix within the Kalman filter. It is essentially giving a +/- figure for the uncertainty in the speed, based on 3 sigma (99.7%).
sAcc is similar to SDOP / SDOS (in principle) and is available from u-blox GPS chips. It is therefore available from GPS devices such as the Motion, Gyro1 and ESP-GPS which all use u-blox 8 variants. It is more than likely calculated in the same way as SDOP (using the covariance matrix) but is based on 1 standard deviation.
A recent investigation that I completed suggest that SDOP / SDOS may be derived in a similar way to the SiRF EHVE parameter but the important point is that it can only be done within the GPS / GNSS chip itself as it makes use of data within the Kalman filter.
It is unclear whether an SDOP / SDOS / EHVE / sAcc equivalent exists for the Sony chip (APEX Pro / VERTIX) or Airoha AG3335M (VERTIX 2). I have yet to see any such item in a MediaTek or Airoha specification. I can find no reference for any NMEA commands relating to this kind of data but I have no access to the technical specification for the Sony or Airoha chips used by COROS.
If Sony or Airoha do provide speed accuracy estimates then please add them to the FIT data so that it can be used for subsequent analysis of results.
If this data is unavailable then it would be useful if this point can be confirmed.

I have stumbled across the user's manual for the Sony CXD5603GF GNSS chip which I believe is in the APEX Pro and VERTIX:
https://github.com/Xinyuan-LilyGO/LilyGO-T-Motion/blob/master/datasheet/CXD5603GF_UsersManual_E_013_TianQin.pdf
It is clear that the Sony chip cannot provide speed accuracy data as it only outputs the 8 standard NMEA sentences; GGA, GLL, GNS, GSA, GSV, RMV, VTG, ZDA. I am very familiar with these NMEA sentences and I know that speed accuracy is not included.
I have applied the label "wontfix" but I have left the issue open so that this recent finding is clearly visible.

Closing this issue because it cannot be implemented.


Flakey MD5 loading when models have floats with two-digit iparts

What you were trying to do
Load an md5 animation and skeleton file
What actually happened
Errors were thrown while parsing apparently valid md5
How to reproduce
Edit the deer.prefab in WildAnimals to use the attached skeletonMesh or animation. The loader will throw a parse error.
Log details and game version
Terasology develop: f5bdc4099b2d0407ebbe166d0bd8f73d6c3f81d2
java.io.IOException: Invalid bounds line: " ( 0.219587 0.219587 0.219587 ) ( 0.219587 16.896273 13.914334 )"

The files were taken unmodified from http://opengameart.org/content/boneman
Other errors are encountered if you manually edit the file to fix the one above.
I noticed that the MD5 loader float regex is dubious. It appears to only allow one digit before a decimal point? So 16.xxxx will always break it?
https://github.com/MovingBlocks/Terasology/blob/develop/engine/src/main/java/org/terasology/rendering/md5/MD5SkeletonLoader.java#L51
https://github.com/MovingBlocks/Terasology/blob/develop/engine/src/main/java/org/terasology/rendering/md5/MD5AnimationLoader.java#L50
boneman.txt
boneman_idle.txt

Just cross-posting here from the forum to make sure it is covered - @flo posted an update at http://forum.terasology.org/threads/help-importing-minetest-player-model-skins.1593/#post-13897 having tweaked the Blender addon some more. Would be good to know if it helps this! If the RegEx still needs a tweak maybe we can cover that with this issue :-)

yes, the regex needs to be adjusted. I haven't adjusted the bounding box calculation yet.
The model above has a corrupt bounding box as minX and maxX is the same.
@glasz  tried to fix it, but I think i still saw an issue where it was smaller than expected.

@jdrueckert can this be closed now that we sunsetted md5 and switched to glTF?

We don't use md5 assets anymore.


Limit the number of worker processes to some arbitrary maximum

The current approach is to spin up as many processes as there are targets, and let them slug it out. While viscerally satisfying, this doesn't seem entirely smart.
Find the best way of limiting the number of active processes to some operator-specified maximum, and automatically join() processes as they exit.

Won't fix, as I've abandoned this project.


Enable FEATURE_PERFMAP on OSX, and update perfjitdump.cpp to work on OSX

With this, running Unity with DOTNET_PerfMapEnabled=1 set in the environment writes /tmp/jit*.dump files that https://github.com/mstange/samply understands. This gives mixed C++/C# profiles on macOS, similar to what you'd get with Superluminal on Windows. (Note that samply needs to be built/installed from source; the cargo-installed binary version seems to have odd issues.)
e.g.:
DOTNET_PerfMapEnabled=1 ~/proj/samply/target/debug/samply record -- ./build/MacEditor/arm64/Debug/Unity.app/Contents/MacOS/Unity -projectpath FPSSample/FPSSample

I have no idea what's up with CI:
[00:23:27.493 Information] git checkout -f --detach FETCH_HEAD
[00:23:27.797 Information] error: unable to read sha1 file of docs/area-owners.md (dc90f47a674c8cf62d6d250db607f5ebe2b32b3f)
[00:23:27.802 Information] error: invalid object 100644 dc90f47a674c8cf62d6d250db607f5ebe2b32b3f for 'docs/area-owners.md'
[00:23:27.834 Information] HEAD is now at 0a2ff6d93 Enable FEATURE_PERFMAP on OSX, and update perfjitdump.cpp to work on OSX

I have no idea what's up with CI:

I reran and it looks better


[FEATURE] Consider integrating with STNO

Serverless transit-network-orchestrator (STNO) is higher level of automation supported in automatic subnet addition to transit gateway route table.
More details https://aws.amazon.com/solutions/implementations/serverless-transit-network-orchestrator/
Github: https://github.com/awslabs/serverless-transit-network-orchestrator
We can either migrate to that, or vice-versa, add the automations to this repo. Important one is the web interface which is created to help control, audit, and approve (transit) network changes.

One more pre-requisite for this is to add the capability of attaching tags to subnet, Current SubnetDefinitionConfig lacking this.

We already perform many of those capabilities and plan to enhance via: https://github.com/aws-samples/aws-secure-environment-accelerator/issues/494 design spec.  Tags: what is your ask?  A customer could provide a key:value pair in the subnet\definition and have it applied to a subnet?

Actually that is not the main requirement but having an approval workflow (not so technical) is much preferred.

It is understated but the approvals process is part of "step 8", which is not yet fully refined.
the vpc-template, which is marked as opt-in, would have an approval-required: true/false flag.
if true, when the Service Catalog item was executed, it would not result in a VPC creation until someone from the cloud team approved/took action...
would this meet your requirement?

yes, this would be very helpful


Search History

It would be cool if the visitor's search history was retained and displayed when the visitor comes back.

[ ] Store user search history in localStorage via Mantine's Local Storage hook
[ ] Replace default results in search dropdown with user history
[ ] The last searched item should be used if the user comes back

I will work on this!


Hex-Colors

The budgeteer uses HexColors. Chart.js samples did not cover this type. Since rgb and rgba colors exists, it should be easy to implement. Example for usage:
new HexColor("#00c0ef")

Added Hex Colors.


Remove fields that are not used or unreachable from the entry points.

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[X] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
<!--

Functional tests re-running here: https://github.com/liquibase/liquibase-pro-tests/actions/runs/4928280271


Updating MS impersonation rule to include password expiry emails

An example:

The new iteration is being weird for detection, so I'm working on that.
See https://sublimecommunity.slack.com/archives/C02N1F7FP7C/p1657161669007889 for more info.

This rule doesn't work on weird obfuscations of words, as we've discussed in the Slack community, but the rule still works. I'm good to push this iteration, and open a new PR once we have a solid idea on how to catch obfuscations easily.
cc @rw-access for this discovery in Slack

thanks for this @aidenmitchell!
we're doing some testing on our side and will share results soon!

This rule doesn't work on weird obfuscations of words, as we've discussed in the Slack community, but the rule still works. I'm good to push this iteration, and open a new PR once we have a solid idea on how to catch obfuscations easily.
cc @rw-access for this discovery in Slack

That's been added in 0.31.2, with this PR open, I added it here instead of a new one :)

just wanted to give a quick update on this. we've been doing some testing in larger environments and are detecting quite a few net-new attacks that weren't previously detected by this rule, which is awesome 🔥. we're also seeing some FPs, so we're working on tuning some of the logic to get a bit higher efficacy. hope to have another update to share soon!
thanks for your patience, @aidenmitchell !

hey @aidenmitchell . does this version of the rule still flag your attack?
type.inbound
and (
  (
    ilike(subject.subject, '*Microsoft 365*')
    and ilike(subject.subject, '*is expired*')
  )
or
  (
    // should catch any instance of the word "expired"
    ilike(body.html.display_text, "*expir*")
    and ilike(body.html.display_text, "*password*")
    and ilike(body.html.display_text, "*microsoft*")
  )
  or iregex_search(body.html.display_text, ".*reach you.*microsoft teams")
  or ilike(sender.display_name, '*new activity in Teams*')
  or subject.subject =~ 'Offline Message in Teams'
  or ilike(subject.subject, '*Teams Sent A Message')
  or sender.display_name in~ ('Microsoft Partner Network','Microsoft',
                             'Microsoft Feedback', 'Microsoft account team',
                             'Microsoft Support', 'Microsoft 365 Message center',
                             'Microsoft Azure')
)
and sender.email.domain.root_domain not in~ ('microsoft.com', 'microsoftsupport.com', 'office.com')

and (
    (
        sender.email.domain.root_domain in $free_email_providers
        and sender.email.email not in $sender_emails
    )
    or (
        sender.email.domain.root_domain not in $free_email_providers
        and sender.email.domain.domain not in $sender_domains
    )
)

It looks right to me, unfortunately the original .eml I used to build this rule has been erased from our system.
I'm good to push this :)

It looks right to me, unfortunately the original .eml I used to build this rule has been erased from our system. I'm good to push this :)

awesome. made that update, and also added you as an author on the rule 😄


Use always libraries from pypi

We should avoid installing code from git repos:
https://github.com/seecloud/ceagle/blob/master/requirements.txt#L5

@maretskiy please fix this

@maretskiy we should put "fix #24" in our commit message to automatically close it

Ok


Config flow could not be loaded

Hi,
I'm trying to add the integration, but, after a long wait  with a popup saying "PLease wait while the installation is being installed", I'm getting "Config flow could not be loaded".
I've restarted the server twice, but still doesn't work. Any clue on what to check?

Just got my cameras and am experiencing this too while trying to do initial setup.

Encountering this trying to set up.
core-2021.11.3
Home Assistant OS 6.6

Note - Rebooting host for home assistant resolved this for me.

there is a template to follow when you create a ticket.
Closed until better infos provided


error creating file (Error code: EISDIR)

I'm trying to make this plugin work and can't pass que creation of the file. I'm getting same error every time: Warning: Unable to write "src" file (Error code: EISDIR). Use --force to continue.. I've been reading that's an error reading and writing a directory but everything is configures as your example. I'm adding my configuration, -verbose and --stack output:
gruntfile
indexing: {
      options: {
        layout : 'src/file.html'
      },
      filter: 'isFile', // added this and created a file in the root but no output and same error
      src: ['dist/*.html'],
      dest: 'list.html'
    },

file.html
<body>
	{{replace}}
</body>

$ grunt indexing --stack
Running "indexing:src" (indexing) task
dist/index.html src
Warning: Unable to write "src" file (Error code: EISDIR). Use --force to continue.
Error: EISDIR, illegal operation on a directory 'src'
    at Object.fs.openSync (fs.js:438:18)
    at Object.fs.writeFileSync (fs.js:977:15)
    at Object.file.write (/Library/WebServer/Documents/test/node_modules/grunt/lib/grunt/file.js:296:10)
    at /Library/WebServer/Documents/test/node_modules/grunt-indexing/tasks/indexing.js:48:18
    at Array.forEach (native)
    at Object.<anonymous> (/Library/WebServer/Documents/test/node_modules/grunt-indexing/tasks/indexing.js:30:16)
    at Object.<anonymous> (/Library/WebServer/Documents/test/node_modules/grunt/lib/grunt/task.js:264:15)
    at Object.thisTask.fn (/Library/WebServer/Documents/test/node_modules/grunt/lib/grunt/task.js:82:16)
    at Object.<anonymous> (/Library/WebServer/Documents/test/node_modules/grunt/lib/util/task.js:301:30)
    at Task.runTaskFn (/Library/WebServer/Documents/test/node_modules/grunt/lib/util/task.js:251:24)

Aborted due to warnings.

$ grunt indexing -verbose
Running "indexing:src" (indexing) task
Verifying property indexing.src exists in config...OK
Files: dist/index.html -> src
Options: layout="src/file.html"
Reading src/file.html...OK
dist/index.html src
Writing src...ERROR
Warning: Unable to write "src" file (Error code: EISDIR). Use --force to continue.

Done, it's an error in gruntfile config:
should be:
indexing: {
  options: {
    layout: 'src/file.html'
  },
  files: {
    src: 'dist/*.html',
    dest: 'dist/list.html'
  }
}


Missing "in" on readonly struct method parameters?

Hi Andrew,
I noticed that most readonly struct method parameters are not passed using "in".
Is there a reason for this?

Not really. It requires more thought to add the in modifier and I almost never go to the trouble to consider it.
Some considerations:

Adding the in modifier is a binary API breaking change, so adding it to public API is not an option once the API has shipped.
Adding in passes the struct as a pointer instead of by value, so to see any perf benefit the struct must exceed the size of a pointer (>8 bytes on 64-bit processes).
Even for large structs, the perf benefit is very small, so only in the hottest perf paths is it even interesting to consider. Nerdbank.Streams doesn't have such paths as far as I know.

In contrast, the MessagePack-CSharp project that I contribute a lot to does use in on some of its methods for the ReadOnlySequence<T> struct. That struct is quite large and a great deal of MessagePack is perf critical.


[Removal]: Raspy Gingham Puma

Hotspot Name
raspy-gingham-puma
Hotspot b58 Address
1141WMeQk1jfCpP7rF4wg8CrdisohkMDP52a4v4Jd4msBXUNdDF
Discord Handle
Chris V.#3868
Hotspot Manufacturer
Cal-Chip Connected Devices
Removal Reason
Reason added unknown; this is one of our local hosts who also happens to be a family friend.
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information

Upon reviewing this again I'm recalling that this setup was originally on LTE/4G as we were doing R&D on using off grid deployments. I believe it has since been moved back to residential WiFi.

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


HPP Challenge?

Thought I'd suggest that you add an HTTP Parameter Pollution challenge.
Interestingly while I was poking around the heroku instance I found the following:

Search is somewhat impacted, if you specify two q params and values the search result page doesn't render properly. i.e.: http://juice-shop.herokuapp.com/#/search?q=hoodie&q=lemon
Contact form does seem vulnerable, though tampering POST/JSON may not be everyone's cup of tea... It's also a kind of weak example and doesn't really get the "attacker" anything useful in the end.

Example:
Request:
POST /api/Feedbacks/ HTTP/1.1
Host: juice-shop.herokuapp.com
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0
Accept: application/json, text/plain, */*
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate
Referer: http://juice-shop.herokuapp.com/
Content-Type: application/json;charset=utf-8
Content-Length: 63
Cookie: io=63cDkUKnjTNXRqYLAAAj
Connection: keep-alive
Pragma: no-cache
Cache-Control: no-cache

{"comment":"james was here","rating":2,"comment":"so was fred"}

Response:
HTTP/1.1 200 OK
Server: Cowboy
Connection: keep-alive
X-Powered-By: Express
Access-Control-Allow-Origin: *
X-Content-Type-Options: nosniff
X-Frame-Options: SAMEORIGIN
Content-Type: application/json; charset=utf-8
Content-Length: 150
Etag: W/"96-XqpXSFISEgqoVwW7hYfcjw"
Date: Wed, 22 Feb 2017 13:18:50 GMT
Via: 1.1 vegur

{"status":"success","data":{"comment":"so was fred","rating":2,"id":12,"updatedAt":"2017-02-22T13:18:50.000Z","createdAt":"2017-02-22T13:18:50.000Z"}}

I ran out of time to play and see what might be possible. I was hoping to come across a GET request where I could tamper something from view or add to delete or remove, but in the 20mins I had to poke around couldn't fine one.....
There might be other more simple or slick ways to implement. Anyawy, there's some food for thought.
Refs:
https://www.owasp.org/index.php/Testing_for_HTTP_Parameter_pollution_(OTG-INPVAL-004)
http://blog.mindedsecurity.com/2009/05/client-side-http-parameter-pollution.html
https://www.youtube.com/watch?v=Tosp-JyWVS4

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

So i found a Bug which allows adding products to somebody else's basket and is not listed in challenges. The following challenge can be designed around it.

Yes, sir! ☑


"This document may be out of date" advisory should be shown in the language of the page it's displayed on

This is a Bug Report

Problem:
The blue-box advisory that a translated version of an English documentation page may be out of date is presented in English, but should be presented in the language of the page it's displayed on.
Proposed Solution:
Display same-language translations of the warning box on translated pages.
Page to Update:
https://kubernetes.io/ko/docs/concepts/scheduling-eviction/topology-spread-constraints/

Additional Information:
The included page URL just happens to be one I stumbled across while doing research on a separate issue -- there are likely other pages in other languages that need this as well.

/retitle Outdated content warning note does not match page language
/area localization

/remove-kind bug
/kind feature
Localization teams can localize the warning message; if they've not done that, it's not a bug, it's just work that hasn't yet happened.

/priority backlog

/assign @Ayush9026

/triage accepted

/remove-lifecycle rotten

@Ayush9026 Are you still working on this issue?


Ability to validate multiple specified fields at the same time

Right now, you have three options:
post.validateAll() to validate all fields. Error messages are for all fields
post.validate() to validate all fields, stopping at the first error. Error Message is for first field only.
post.validate('name') to validate a single field. Error message is only for specified field.
Would it be possible to add something like
post.validate('name', 'date') to validate both and only those fields, and getting error messages for both? Doing something like if (post.validate('name') && post.validate('date')) won't work because if the first field fails, the second won't be validated.
An example use case for this would be for a User object where the application would have several forms for different parts of the object, such as settings, preferences, etc. It would be nice to have a single User class while being able to control what gets validated.
Please correct me if there is currently a way to do this that I am just ignorant of.
Thank you.
Mike

Hi, thanks for this issue. Actually it was already implemented in the newest version :) which is not published yet. You will be able to do:
user.validate(['name', 'date']);

It will take some time to publish it because it bigger update that changes a lot in Astronomy. So stay tuned :). For now I can recommend some workaround:
var validateMultiple = function(doc, fields) {
  return _.every(fields, function(field) {
    return doc.validate(field);
  });
};

// Usage
var user = new User();
validateMultiple(user, ['name', 'date']);

It should work. And when the new version is released, you will be able to use previously mentioned solution :).

For now I'm closing this issue because the feature is implemented and will appear in the next release.


Add missing Z_INTERNAL to some functions that should not be exported.

PR #1396 fails due to a few functions not having an explicit visibility set, this rectifies that.

This also needs updated pkgcheck lists in order to pass tests though

FYI the abi files get uploaded as artifacts when the build fails so you can use those if you want.

In most cases you will want to update the ABI files in separate commit, so abicheck script can reference the correct commit hash that changed the ABI.

New ABI files in b0168f736833a035309ec0fe23a91479c8558c5a.

This was quite the rabbit hole, hopefully all the tests will pass now (except the ones just requiring updated ABI files). 🤞

There we go, all tests pass, except the ones that need updated ABI files of course. Will have to add those in a separate PR after this goes in.

We need a CI test that always enable checking for missing exports in version scripts... After that is added, we can just bump the commit hash in test7abicheck.sh to make all tests pass again. I can upload the new ABI files to GitHub when those are needed.

@mtl1979 I agree, it would be a good thing to be ahead of. But it would probably need 4 CI runs; zlib-ng, zlib-ng-compat, and both again without gzfileops (those two would still fail currently though, we might need a new templating system for that probably).
What I had to add to the CMake command line locally to enable this testing:
-DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DCMAKE_LINKER=lld -DCMAKE_SHARED_LINKER_FLAGS=-Wl,--no-undefined-version

@mtl1979 I agree, it would be a good thing to be ahead of. But it would probably need 4 CI runs; zlib-ng, zlib-ng-compat, and both again without gzfileops (those two would still fail currently though, we might need a new templating system for that probably).
What I had to add to the CMake command line locally to enable this testing: -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DCMAKE_LINKER=lld -DCMAKE_SHARED_LINKER_FLAGS=-Wl,--no-undefined-version

I originally tagged @nmoinvaz, because he has worked a lot more with the (Linux) CI runs than I have... We might want a separate PR for the CI improvements, but that is open for discussion in my opinion.

I won't make any CI changes part of this PR.
I'd like to see this PR land tomorrow if the stars align and nothing else crops up.

I don't like delaying things too much either... I still have things I need to work related to my job... I just really hate making changes and then reverting them later because of regressions our CI didn't catch.

I have added the CI in PR #1430. I suggest you merge it first, then rebase this on top of it, to make sure errors are gone.

I have new ABI files waiting in commit aa2741e437dd03b9fb1ea00dd401e46dd4e460f4 as long as there is no unexpected errors in remaining CI runs...


Upgrade log4j2 to 2.17.0

2.17.0 upgrade
a) Fix string substitution recursion and hence preventing DOS attack
Refer: https://logging.apache.org/log4j/2.x/changes-report.html#a2.17.0

This solves CVE-2021-45105
Apache Log4j2 does not always protect from infinite recursion in lookup evaluation
Refer : https://logging.apache.org/log4j/2.x/security.html

Eventhough we don't use these (context lookups) in pattern layout in log4j2.xml.
Its better we upgrade as there by still undiscovered attack vectors

PS: Due to large attention the log4shell got, many security researchers
are trying out to find flaws in log4j2 and we might see some more
upgrades in coming weeks fixing the security flaws.

I  have tested log4j2 2.17.0 upgrade in aaa server, everything works. We can upgrade directly as it won't break any logging functionality.  refer https://logging.apache.org/log4j/2.x/changes-report.html#a2.17.0 for 2.17.0 changes

Please do the needful in other vertx /java related apps @kailash @swaminathanvasanth @rraks.


Crandall - practice a Galaxy Assembly workflow and give feedback to Stephen Shank

Recreate the assemblyQC and ngsQC workflows on Galaxy. Execute a test run using an assembly file and associated SRA data from an organism that has already been analyzed in HIVE. Prepare a brief report comparing the results across the two platforms.

Move to July task list

@penningtonea has a workflow been share with the Crandall team by Stephen?

This is being closed, because  we cannot finish.


Made my first solo plugin!

@BurningTreeC:
I just wanted to thank you for all the help you've been so far in how to make a TW-plugin, and wanted to announce that I've made my first solo plugin: text-to-speech. The plugin is really quite simple, and I got the idea from TiddlyTweeter's question about this functionality. I felt that it wouldn't fit this plugin well (a bit off-topic for this plugin), so I created a minimal implementation of something here.

Congratulations @flancast90 😊

@flancast90 I've seen that in your new plugin you export the name of the startup module as "speech-to-text" and I don't know if that doesn't clash with the name of the speech-to-text plugin, which is also "speech-to-text". I believe that you should change that name to "text-to-speak" or whatever you like

Whoops! I used the file structure of this plugin there, and must have forgotten to rename something! Thanks for the catch!


Special characters like "&" not working properly

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
Text is broken when using underline with "special" characters like "&" (ampersand)
See the video below.
To Reproduce
Steps to reproduce the behavior:

Drop a rectangle on the screen
Uncheck "Formatted text" (this is important, when "formatted text" is on it works properly)
Set rectangle text to "A&B"
Click "Underline" button
See error: "A&B" is changed to "A&B"

Expected behavior
The text stays "A&B"
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 21.0.2 (online)

Desktop (please complete the following information):

ALL

Smartphone (please complete the following information):

ALL

This doesn't happen for me using 21.0.2 on any browser. Is this private/incognito with all extensions off?

@davidjgraph
I just filmed it five a few minutes ago using the latest Chrome, and the current online version.

Actually, it is a very old bug. We are using the mxgraph library, just reporting some problems we encountered in a while :)

Thanks for the report. This will be fixed in the next release.

@alderg could you give some details on how it was fixed (so that the fix can be reused in a fork like maxgraph for example?)

Line 9426 in Graph.js was changed as follows:

Fixed in 21.0.4


Links are not drawn when name of source and/or target chromosome do not match name of visible chromosome

The following JSON contains a multiple alignment, but links are not drawn. In the function filterVisibleLinks all links are removed. Because the names of the targetKaryo and accordingly sourceKaryo do not match the keys of the visible Chromosomes.

@sonjahohlfeld
Thanks for that bug report. It the mapping of AliTV somewhere documented?
Best,
Frank

targetKaryo is the name which is setted in data.features.link['Sequence' ].karyo . This is the same for sourceKaryo. But in visibleChromosomes the names of karoy.chromosomes are stored.

Is this issue solved?


release 3.2.8 Ubuntu 14.04 packaged for 16.04?

Tried to install the latest 3.2.8-1 Debian AMD64 package for Ubuntu 14.04 from
https://download.arangodb.com/arangodb31/xUbuntu_14.04
in an Ubuntu 14.04 based docker container fails with the following error:
Preparing to unpack arangodb3-3.2.8-1_amd64.deb ...
Unpacking arangodb3 (3.2.8) ...
dpkg: dependency problems prevent configuration of arangodb3:
 arangodb3 depends on libssl1.0.0 (>= 1.0.2~beta3); however:
  Version of libssl1.0.0:amd64 on system is 1.0.1f-1ubuntu2.21.
 arangodb3 depends on libstdc++6 (>= 6); however:
  Version of libstdc++6:amd64 on system is 4.8.4-2ubuntu1~14.04.3.

dpkg: error processing package arangodb3 (--install):
 dependency problems - leaving unconfigured

The affected packages libssl and libstdc++6 are not available in the required versions on Ubuntu 14.04 with the standard repositories.
Same setting with ArangoDB 3.2.7 for Ubuntu 14.04 works fine, so maybe something went wrong during packaging?

hm, https://download.arangodb.com/arangodb31/xUbuntu_14.04 will only provide you with arangodb 3.1 releases in first place, so somethings wrong here from start.
The dependencies you're listing are only here with ubuntu 17 packages.
Can you please revalidate whether the problem persists when manually downloading
https://download.arangodb.com/arangodb32/xUbuntu_14.04/amd64/arangodb3-3.2.8-1_amd64.deb
and installing it with dpkg -i ?

sorry @dothebart the url was wrong but not used anyway, as I downloaded the package directly (so we don't require internet connection during docker build).
I am infact using the package from https://download.arangodb.com/arangodb32/xUbuntu_14.04/amd64/arangodb3-3.2.8-1_amd64.deb.

Hi,
Checking out the dependencies of this package:
dpkg -I arangodb3-3.2.8-1_amd64.deb
 new debian package, version 2.0.
 size 33667208 bytes: control archive=141252 bytes.
     177 bytes,    13 lines      arangobench.conf
    1670 bytes,    74 lines      arangod.conf
     520 bytes,    30 lines      arango-dfdb.conf
     158 bytes,    12 lines      arangodump.conf
     143 bytes,    10 lines      arangoexport.conf
     140 bytes,    10 lines      arangoimp.conf
     101 bytes,    10 lines      arango-init-database.conf
     161 bytes,    12 lines      arangorestore.conf
     101 bytes,    10 lines      arango-secure-installation.conf
     200 bytes,    14 lines      arangosh.conf
     359 bytes,    11 lines      conffiles
    2004 bytes,    83 lines   *  config               #!/bin/sh
     679 bytes,    21 lines      control
     267 bytes,    15 lines      foxx-manager.conf
  533264 bytes,  4805 lines      md5sums
    4201 bytes,   130 lines   *  postinst             #!/bin/sh
    1256 bytes,    46 lines   *  postrm               #!/bin/sh
     544 bytes,    17 lines   *  preinst              #!/bin/sh
     359 bytes,    15 lines   *  prerm                #!/bin/sh
    1715 bytes,    45 lines      templates
 Package: arangodb3
 Version: 3.2.8
 Section: database
 Priority: optional
 Architecture: amd64
 Depends: libc6 (>= 2.18), libssl1.0.0 (>= 1.0.1) <--------------------------------------------------
 Homepage: https://www.arangodb.com
 Conflicts: arangodb, arangodb3e, arangodb3e-client, arangodb3-client
 Installed-Size: 268879
 Maintainer<EMAIL_ADDRESS> Description: the multi-model NoSQL database
  A distributed free and open-source database with a flexible data model for documents,
  graphs, and key-values. Build high performance applications using a convenient
  SQL-like query language or JavaScript extensions.
  .
  Copyright: 2014-2017 by ArangoDB GmbH
  Copyright: 2012-2013 by triAGENS GmbH
  ArangoDB Software
  www.arangodb.com

I don't see anything about libstdc++. Are you sure you're not mixing downloads?

I'm puzzled but cannot reproduce this now, works with 3.2.8-1 - will try to shed some light on this tomorrow, feel free to close and sorry for the false alarm.

ok. Since we had other people mix versions up recently we may have to think about adding a way to prevent oneself to shoot in ones foot.  I will change the topic accordingly and mark this as a feature request.

Meanwhile we use static executeables that don't depend on a specific OS libc and other libraries anymore.
Marking as solved.
Please find ArangoDB 3.7 available for download:
https://www.arangodb.com/2020/08/arangodb-3-7-a-big-step-forward-for-multi-model/


Strongly-typed optional handles

I think I got the generator producing appropriate output. My next step is to update all the wrapper functions. So far the only surprise has been that apparently all the destroy functions accept null handles; I intend to hide that from the high level API rather than propagating it.

This change is

One solution is to take a page from std and default required handle fields to a nonzero but arbitrary value (say, !0 or 1). This will make the structs safe to construct/manipulate/print/etc and unsafe to use if not properly initialized, just like they're already unsafe to use if you violate any of the other numerous valid usage rules.
This has the downside of making it remain possible to inadvertently omit a handle from a struct, but that's the status quo, so it's still an improvement overall. I'll proceed with that later tonight unless someone has a better idea.

I think this is ready, if everyone's on board with the idea.

One interesting note is that this exposes us to breaking API changes from minor metadata updates like https://github.com/KhronosGroup/Vulkan-Docs/pull/759. Care will need to be taken when updating the supported vulkan version.

So I guess they would have to do something like SomeHandle::from_raw().unwrap_or(SomeHandle::invalid()), but then passing that to a destroy_ function would not be good.

Note that invalid is not exported. I was conflicted about this--invalid handles can be constructed by the user by allocating then immediately freeing an object--but in the end I failed to come up with a concrete use case and it's easier to make it public in the future than to start that way and make it private.
My feeling was that if from_raw gives you None you simply don't call the destroy function (or do anything else with it).

vk.xml also (in theory) has enough information to let us do this same trick for "flags" fields, which sometimes must be nonzero. I think I'll see how that works out too.

we should probably wait for the PR on the vk.xml

It's worth note that this change probably exposes us to minor errors like this in vk.xml far more than any existing project. I think the risk is worth it, though.

Optional bitflags seem to work okay, and are mostly ergonomic. The generated API for the flags types themselves is unfortunately higher-friction; many operations must now return Option, and some had to be removed entirely. Judging by the examples, the impact of this is limited, but I'd like to hear what others think.

I generally like it, although it leaves a sour taste in my mouth that it depends on the correctness of the vk.xml. We should be fine as long as khronos accepts our fixes.
I would also like to get some feedback from @kvark, as they are currently using ash and might want to use ash-sys directly.

Also while we at it https://github.com/KhronosGroup/Vulkan-Headers/blob/master/include/vulkan/vulkan_core.h#L57-L60
Should the handles be NonZeroU64? I don't fully understand the implications, I assume it might be less than 64bits on some architectures.

My understanding is the intention there is to replace uint64_t with opaque struct pointers on platforms where doing so does not affect the ABI, purely so that C++ users get better typechecking by way of handle types that don't implicitly convert to eachother. Our binding should be correct, and mirrors that used by vulkano.

To me it seems as if it were u64 on 32bit platforms and otherwise it would be a ptr / usize.
I just tested it,
#include<iostream>
#include <cstddef>

int main() {
    #if defined(__LP64__) || defined(_WIN64) || (defined(__x86_64__) && !defined(__ILP32__) ) || defined(_M_X64) || defined(__ia64) || defined (_M_IA64) || defined(__aarch64__) || defined(__powerpc64__)
        std::cout << "Ptr" << std::endl;
    #else
        std::cout << "U64" << std::endl;
    #endif
    std::size_t z;
    std::cout << sizeof(z) << std::endl;
}

➜  src clang++ test.cpp -o test; ./test
Ptr
8

and
➜  src clang++ test.cpp -o test -m32; ./test
U64
4

So it seems like it is an u64 on 32bit system but, your usize will be 32bits.

Forget what I said above. Everything is fine, I just thought you defined non dispatchable handles as usize.

It seems like the vulkan-docs repo isn't very fast-moving. Perhaps we could work off a fork for now, if we want this in for the next ash release?

Following discussion with @kvark in IRC, I've removed the bitflags component of this change; only handles are now affected. Our conclusion was that the practical benefit of encoding this information in the type system is dubious (it's e.g. difficult to forget that buffer usages are mandatory), and the added friction for gfx-hal, which relies on bitflags types having a zero value for its flag conversion logic, and the confusion for users expecting familiar bitflags semantics are therefore unjustified.

I'd rather wait until the Khronos PR is merged before I accept it, unless you are sure that it will be merged. I wouldn't want to maintain a separate vk.xml file in the future.
I am fine with the bitflag change.

There are months-old reasonable pull requests still open with little feedback, so it may be quite some time. It's a shame to cut a release with known breaking changes pending, but that's probably the way to go for now, then.

What PR is that? Lemme ping Khronos

On Aug 17, 2018, at 03:17, Benjamin Saunders<EMAIL_ADDRESS>wrote:
There are months-old reasonable pull requests still open with little feedback, so it may be quite some time. It's a shame to cut a release with known breaking changes pending, but that's probably the way to go for now, then.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

In general we've resisted additions that are only for external toolchains, mostly based on the fact that if we haven't got anything relying on it, it won't get tested and will almost definitely fall out of date.

I personally think this piece of information is probably somewhat useful to put into the xml though, and we might consider incorporating this information into the spec at some point too (giving it coverage).

Sadly this means that this PR will probably take a while until we can merge it.

https://github.com/KhronosGroup/Vulkan-Docs/pull/1597: Khronos seems to have started their first attempt at properly annotating optionals, with pNext. Anything we can do/salvage here?

It looks like that change (flagging pNext as optional in most cases) is still a long way from the pervasive cleanup envisioned here. IIRC the specific changes in this PR aren't especially clever, but might serve as a useful reference if the generator hasn't been completely rewritten if/when Khronos gets around to adding the metadata we want.

The pNext annotation is pretty much useless yes, I assume it's (already assumed to be) optional in most if not all places anyway?The generator has seen quite some (small and large) changes over time so this might be harder to rebase, but I still see potential in some of the changes here that we should not forget about. Will take another look when more annotations trickle in.


test(metrics): run e2e tests for both Node runtimes

Description of your changes
Changes in this PR.

refactor metrics e2e tests to make stack and resource name unique (so that they can be run in parallel)
refactor metrics e2e tests to be consistent with logger and move common code to commons
run e2e tests on both Node12 and Node14 runtimes

How to verify this change
Run npm run lerna-test:e2e at the root.
Every test should pass and metrics tests should be on both node14 and node12
Related issues, RFCs
#497
PR status
Is this ready for review?: NO
Is it a breaking change?: NO
Checklist

[x] My changes meet the tenets criteria
[ ] I have performed a self-review of my own code
[ ] I have commented my code where necessary, particularly in areas that should be flagged with a TODO, or hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have made corresponding changes to the examples
[ ] My changes generate no new warnings
[ ] The code coverage hasn't decreased
[ ] I have added tests that prove my change is effective and works
[ ] New and existing unit tests pass locally and in Github Actions
[ ] Any dependent changes have been merged and published in downstream module
[ ] The PR title follows the conventional commit semantics

Breaking change checklist

[ ] I have documented the migration process
[ ] I have added, implemented necessary warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks a lot for the PR, I like the direction you're giving to the integration tests. It'll make them much easier to maintain moving forward.
My only concern relates to exporting the e2e utilities in the commons package. This package is depended and bundled by all the other utilities and as such it will be included in the deployment package of each function that uses Powertools.
While I think there's definitely value in providing test utilities (see #445), I don't know if it should be in commons and in general, if it's code that should be deployed in Lambda.
What's your take?

I totally agree with you. We'll unnecessarily increase package size. I'll change this.
Found that I modify quite a few files with #676 . I'll wait until that one is merge first, resolve conflict, and change this.


svelte config uses JSDoc comment when typescript was selected npx sv create

The following svelte config was generated while I selected typescript to be used:
import adapter from '@sveltejs/adapter-auto';
import { vitePreprocess } from '@sveltejs/vite-plugin-svelte';

/** @type {import('@sveltejs/kit').Config} */
const config = {
	// Consult https://svelte.dev/docs/kit/integrations
	// for more information about preprocessors
	preprocess: vitePreprocess(),

	kit: {
		// adapter-auto only supports some environments, see https://svelte.dev/docs/kit/adapter-auto for a list.
		// If your environment is not supported, or you settled on a specific environment, switch out the adapter.
		// See https://svelte.dev/docs/kit/adapters for more information about adapters.
		adapter: adapter()
	}
};

export default config;

This is expected. The svelte config can only be configured in a .js file. See https://github.com/sveltejs/kit/issues/2576


Realm Custom-Token Login failed. [When switched to partition based sync to Flexible sync]

How frequently does the bug occur?
Always
Description
Hi
We have released the app with  change of Partition based sync to flexible sync configuration with new appId in the new release. User can able to login with JWT token credentials for Realm login. But when the user did not use the app for 3 or more then 3 days, after open the app all the user facing realm Custom token Auth Error, with error of
"'"appid"' not found in 'aud' claim" & and user can't able to have any access any features in through the app. When i checked with the jwt generated token in online decode platform the payload contains the old appId. How it was possible?.
Can anyone help me out this......?
The realm swift version was used is 10.28.2

Stacktrace & log output
No response
Can you reproduce the bug?
No
Reproduction Steps
No response
Version
10.28.2
What Atlas Services are you using?
Both Atlas Device Sync and Atlas App Services
Are you using encryption?
-- select --
Platform OS and version(s)
iOS
Build environment
Xcode version: ...14.2

Hi @Kathiravan001 Can you show us how you are generating the custom token on the client side?

@Kathiravan001 We can't help without your input. Have you resolved the issue?


some question about GAT

Hello, recently when I read paper of GAT again, I found a question and was confused. I hope to obtain your help.
The coefficient of $\alpha_{i,j}$ is decided by the features of node i and node j under the supervised learning of those training set nodes with corresponding labels. But if the situation in the traing set is: The two vertices of an edge belong to the training set and the test set respectively. Theoretically, node without label will not be able to use gradient descent for learning. In this way, how does GAT it works?
Thanks a lot !

Hello,
Thank you for your issue.
The answer to your question depends on whether you're doing transductive or inductive learning.
If transductive, the training algorithm sees all nodes (including test nodes), and then it is used for learning.
If inductive, the test nodes are masked out along with their edges during training and, for the purposes of the training algorithm, are treated as non-existing. At test time we add them back.
Hope that helps!
Thanks,
Petar

In this way, if transductive, the whole graph will be thrown into training, only the labels of test set are marked?

You are correct -- test and validation nodes won't be used for cross-entropy loss.

Thanks a lot！！


@inheritdoc not highlighted when used in NatSpec comments

Solidity offers a very useful tag called @inheritdoc, which enables us to specify that a function in a contract can inherit the NatSpec comments from another contract.
Here's a screenshot:

Bumping this up, as it'd be very useful for code readability


Distance for TR

Thanks @serefayar!
Numeral currently use a dot as a decimal mark, instead of a comma. I've updated the corpus file for Distance accordingly. Any chance you can update Numeral to use the comma as a decimal mark?

Hi @patapizza ;
Yes, I can do so.
thanks.


Cannot find 'Microsoft.Framework.DependencyInjection.Abstractions'

Hello.  I am trying to build just a sample .net 5 web API and I'm attempting to use the built-in DI container.  Whenever I try to run my API though, I keep getting the error:
"Could not load file or assembly 'Microsoft.Framework.DependencyInjection.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. The system cannot find the file specified."
I've looked through my project.json file and there's no reference to it, and I've done a search through the entire solution to see if there's a reference to it and I can't find it at all.  I'm using the new Microsoft.Extensions.DependencyInjection and it seems like something in my project is using the old one even though I can't find any references to it.  Could someone please help?  More than happy to provide more information if necessary.
Thanks.

Wrong place sorry.


Multiclass

Not to be merged yet.

@jameshensman I've tried to get the likelihood test suite working on the multiclass likelihoods but haven't succeeded so far. I have two problems at the moment:

Calls to

l.predict_mean_and_var(self.F, self.F * 0)
seem to be returning all nans.

Calls to

Likelihood.predict_density(l, self.Fmu, self.Fvar, y)
Seem to be getting broadcasting errors.
Any ideas? Thanks very much.

This was because of a sqrt function being called on self.F * 0. I added come clipping to try to help with this.

Short answer: test not applicable. Long answer: This test was designed to check analytical implementations of the density against numerical ones. For example, in the Gaussian likelihood, we know the formula for the predictive density. The base class has the quadrature code in, so I compared the performance of the base class (Likelihood.predict_density(l...) to the analytical code (l.predict_density(...). The broadcasting error happens because the base-class assumes that the likelihoods is univariate, i.e. multiple columns of Y and F (and F_var) are treated independently, which is not the case for the multi-class likelihood. My solution is to omit this likelihood from that test.

@jameshensman Did you push changes corresponding to your comment above? For instance in the case of 2) I can't see any changes to the likelihood test.
Also the tests seem to fail in a very similar way to how they failed before.

Sorry @alexggmatthews I didn't make any changes to the tests.
For 1., the call that you mentioned no longer gives nan, but sadly the tests still fail.

@jameshensman OK thanks for quick reply I've removed case 2) . I had a look into case 1)
There do currently seem to be some large differences. In some cases we have a relative difference of more than 10% .
I suggest this is a property maybe of the way we have had to implement it in TensorFlow?

Actually @jameshensman having done some more looking into it I don't think this is entirely a numerical problem.
I removed some of the clipping which improves the numerical stability of some of the tests. I hope this is OK?
The remaining tests look like they are failing from some sort of indexing error in predict_mean_and_var. If I call the test_mean function in the failing case I get:

In [1]: mu1
Out[1]:
array([[ 0.99900001,  0.001     ],
[ 0.001     ,  0.99900001],
[ 0.99900001,  0.001     ],
[ 0.99900001,  0.001     ],
[ 0.001     ,  0.99900001],
[ 0.001     ,  0.99900001],
[ 0.99900001,  0.001     ],
[ 0.99900001,  0.001     ],
[ 0.99900001,  0.001     ],
[ 0.99900001,  0.001     ]], dtype=float32)
In [2]: mu2
Out[2]:
array([[ 0.999,  0.001],
[ 0.001,  0.999],
[ 0.001,  0.001],
[ 0.001,  0.001],
[ 0.001,  0.001],
[ 0.001,  0.999],
[ 0.999,  0.001],
[ 0.999,  0.001],
[ 0.001,  0.001],
[ 0.001,  0.001]])

In some cases the output of predict_mean_and_var does not add up to 1! It is a similar case for the other failing tests. I don't think the result can be explained by numerical problems.

@jameshensman  OK so it turned out to be numerical after all :)

OK I'm making some progress now but still working on some more tests.

@jameshensman I think this one is good to go.
@alexisboukouvalas since James and I have both written code for this would you like to act as the reviewer? If you are happy with the content could you please merge it.

Nice work @alexggmatthews.
Could you tell us (me) a little about the changes you made in order to make the numerics work, please?

Hi @jameshensman I backtracked a bit so only ended up with one change . There was a division by the standard deviation that we needed to clip in the way you clipped another part. Ive tidied the test and rerun the notebook.

Hi @alexggmatthews, I will happily review and merge this. I will do so this afternoon.

@alexggmatthews, @jameshensman  just one thing to check:
In MultiClass(Likelihood) the class comments says:
'We use the softmax and RobustMax inverse-link functions; [..] '
Should this  be
'' A likelihood that can do multi-way classification.  Currently the only valid choice
of inverse-link function (invlink) is RobustMax.''
As there is no softmax option per se - just RobustMax.

Hi @alexisboukouvalas . Thanks for looking at this. Yes good spot I took out softmax because we only had a half implementation and I think RobustMax makes more sense algorithmically given that it requires only a one dimensional quadrature. I've made the correction you so suggest so I think we are good to go.

We should have the softmax too, it's nice if you're doing MCMC. RobustMax
won't MCMC because it's not differentiable. Softmax can wait for a future
PR though.
James.
On Tue, 28 Jun 2016 at 13:07 Alexander G. de G. Matthews <
<EMAIL_ADDRESS>wrote:

Hi @alexisboukouvalas https://github.com/alexisboukouvalas . Thanks for
looking at this. Yes good spot I took out softmax because we only had a
half implementation and I think RobustMax makes more sense algorithmically
given that it requires only a one dimensional quadrature. I've made the
correction you so suggest so I think we are good to go.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GPflow/GPflow/pull/109#issuecomment-229030024, or mute
the thread
https://github.com/notifications/unsubscribe/AAJbD8nnfwUSjVY7ri6-TLX6Mrr3LEQZks5qQQ50gaJpZM4I900G
.

@jameshensman To clarify I guess you mean full MCMC? We know RobustMax is OK with variational MCMC .

Yes.
On Tue, 28 Jun 2016 at 14:18 Alexander G. de G. Matthews <
<EMAIL_ADDRESS>wrote:

@jameshensman https://github.com/jameshensman To clarify I guess you
mean full MCMC? We know RobustMax is OK with variational MCMC .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GPflow/GPflow/pull/109#issuecomment-229045778, or mute
the thread
https://github.com/notifications/unsubscribe/AAJbD-eupoV8Qk5unP-r3MxIlbrfho5wks5qQR8IgaJpZM4I900G
.

OK thanks.


Show hook URL for the github app in cli info install

Closes: #874

Changes

Submitter Checklist

[X] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[X] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[X] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[X] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[X] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

it just adding the webhook url and its somewhat private 🙃


jx pipeline start ask for credentials even though they are supplied

jx pipeline start asks for credentials even though they are supplied with GIT_USERNAME / GIT_TOKEN or --git-username /  --git-token. It seems like it is git that asks for it during git clone.
If a git credential store with the crediental is set up this doesn't happen. So it's typically in a use case like mine it happens: Executing jx pipeline start in a cronjob.
This bug didn't exist in version 0.0.143. I have a hunch that the regression appeared with jenkins-x/lighthouse#1358.

Migrated to jenkins-x/jx#8421


fix(cli): specify type imports, update linter config

Component / Package Name:
Cli
This PR contains:

[x] bugfix
[ ] feature
[x] refactor
[ ] documentation
[ ] other

Are tests included?

[ ] yes (bugfixes and features will not be merged without tests)
[x] no

Breaking Changes?

[ ] yes (breaking changes will not be merged unless absolutely necessary)
[x] no

If yes, please include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.
List any relevant issue numbers:
Tentatively resolves #31

Description

Specify all type imports
Add ESLint rule to ensure consistent type imports

@lordelogos checked the build output from the changes to the other packages, and there's no difference in output, so no need to publish the others. Going to merge as-is.


Add jekyll-watch by default

$ docker run --rm --label=jekyll --volume=$(pwd):/srv/jekyll   -it -p $(boot2docker ip):4000:4000     jekyll/jekyll
Using jekyll-feed (0.4.0)
Fetching: jekyll-paginate-1.1.0.gem (100%)
Installing jekyll-paginate (1.1.0)
Using kramdown (1.9.0)
Configuration file: /srv/jekyll/_config.yml
Source: /srv/jekyll
Destination: /srv/jekyll/_site
Incremental build: disabled. Enable with --incremental
Generating...
done in 0.578 seconds.
Dependency Error: Yikes! It looks like you don't have jekyll-watch or one of its dependencies installed. In order to use Jekyll as currently configured, you'll need to install this gem. The full error message from Ruby is: 'cannot load such file -- jekyll-watch' If you run into trouble, you can find helpful resources at http://jekyllrb.com/help/!
jekyll 3.0.1 | Error:  jekyll-watch
ok: down: /etc/startup3.d/nginx: 0s, normally up
sh: can't kill pid 25: No such process

Actually, jekyll-watch appears to be included as of the new image mentioned in #76   Therefore closing.
@envygeeks should this inclusion get mentioned in the README?

The new image your are speaking of was for #78.  #76 never got completed as I was consuming locked resources on the build server so it didn't build the image and probably won't until I manually build it and ship it. Sorry.
On the question about adding it to the README.  I don't think it should be added but I'm not against adding it if somebody wants to send a pull request.  The reason I say I don't think it should be added is because that list is directly tied to: https://github.com/jekyll/docker/blob/master/opts.yml#L74 -- we pretty much only list gems that we explicitly install directly so that you know what you might need to bring in.

Thanks for the clarification.  I am ok with omitting it for that reason and for the fact that it isn't one you would load to add functionality to your site, afaik.


about spring cloud feign how to support springmvc @requestmapping annotation

hi,guys, spring cloud feign component support springmvc annotation @requestMapping with method ,and auto generate rest style http template,that is amazing!How it do,would you tell me the resolution how it work.
p.s: where is source code,pleasure!thank you

The source code is in this repo, I believe the class is SpringMvcContract.


No notification when client is not running due to modal dialog

Not sure if "modal dialog" is the correct wording - the issue is that suppose application is upgraded and there is a notification dialog that must be accepted by user before application connects.  If the application is upgraded by Android automatically, the user is not aware of this, and actually is not receiving any notifications about incoming messages (because client is not connected, so it's obvious).
Could there be a notification that user has to do something in order for application to connect?
I think similar issue is when server certificate changes, user must accept it before client connects, but again may not be aware of this if it does not run application by himself.

Hi @ndarilek ,
Thanks for reporting this issue, maybe related to GDPR consent dialog.
Benoît

Yes I guess last time it was after the GDPR upgrade, but as I said in the original report, it may as well be during server certificate change.


Richer CellReference commands

Once you can do multiple references, the signature should change to:
{
  "cells": [
    ["value", 1.0, [
      [3,4],
      [],
      [1, 2, 4, 4]
    ]]
  ]
}

... Or is that more confusing?


ARM64e

Describe the bug
JB-app  crashed when installed in device.
Device (please complete the following information):
ios12.1.1 bata3
iphone xs
Logs
/private/var/containers/Bundle/Application/34B83DCF-F1DD-4B40-94D8-B472BBC22303/Un.app/Frameworks/libUnDylib.dylib: arm64 dylibs cannot be loaded into arm64e processes

Using latest version?

YES，bata 45

Using latest version?

Yes,Bata 45

Using latest version?

I have solved it,because my cydia impactor is not the latest version.😓


Allow Plugins to Mutate XP for Block Breaks & Entity Deaths

There are currently no good ways to mutate the amount of experience dropped when a block is broken, or an entity dies.
Note: This may also affect other areas of the API.

@DarkArc
Entity XP orbs are just, well, entities. IF the cause/contextual data is correct and you know WHY the XP orb is spawning, this is a non-issue (as you'd grab the Entity and adjust its data).
Is that not the case or?

@Zidane It's not quite the case, as increasing/decreasing the XP may result in more XP orbs in Vanilla which is a different effect than increasing the XP value of the orb.


[New API Request] Private IP Manager to support IP address replacement

Context
When Port Manager updates a port's private IP address, it currently triggers two REST calls to Private IP Manger, one to release the original IP address and then to allocate a new IP address. These two operations are not "atomic" and may cause undesirably complicated fallback mechanisms in Port Manager.
Requirements

Create a new replacement API in Private IP Manager that allows release an existing IP address and allocate a new one to replace it.
The operations of releasing and allocation should either succeed together, or fail together. It is undesirable to have mix of success and failure of the operations.

@chenpiaoping I would expect this issue become more "visible" when we conduct more stressful test. Let us discuss a bit to see how we want to handle it.


cmp() was removed in Python 3

wraptypes needs work.  There is still three instances of cmp() that was removed in Python 3  https://portingguide.readthedocs.io/en/latest/comparisons.html#the-cmp-function
https://travis-ci.org/pyglet/pyglet/builds/567020258#L1776-L1801

Definitely needs work!  wraptypes is still functional under Python2, and does not need to be run often, so unfortunately it has suffered from bitrot. We definitely need to upgrade this with the end of Python 2 right around the corner.
As you can see, wraptypes is using a vendored legacy version of on David Beazley's PLY project. The PLY project itself has been updated to work under Python 2/3 a while ago, but the API also changed along the way.  Perhaps a good short term plan is to attempt to port the existing codebase to Python 3, if it can be made to work without a ton of work.
A longer term plan would be to do a rewrite based on SLY, which seems like a much nicer library to work with.


xilem: Port rest of examples and add Memoize to xilem_core

With this everything should be ported, which was previously available in the "non-coreized" xilem.
(Was also pretty straight-forward btw. I suggest diffing against upstream main, because most of the changes here, is just uncommenting code)
I think only Adapt is missing in new xilem_core when comparing to old xilem_core, but I think this should probably be a separate PR upstream, as it wasn't currently available in xilem anyway.

Thanks for the review, agree with everything.


support string templates

toSource(`test${process.env.HOME}`)

i'm not sure it's possible.
Luckily
toSource(function() {`test${process.env.HOME}`})

already works all right.

Sounds like you figured it out


Convert invalid names to strings to prevent formatting errors

In case user specify incorrect objection for TFLite converter, the code rasise ValueError. Unfortunately, during calculation of error message another error may appear due to invalid string formatting attempt. This PR prevents this situation by forcing string conversion for error messages.
Below is the example of such an error condition.
/usr/local/lib/python3.6/dist-packages/tensorflow/lite/python/util.py in get_tensors_from_tensor_names(graph, tensor_names)
      114   if invalid_tensors:
      115     raise ValueError("Invalid tensors '{}' were found.".format(
  --> 116         ",".join(invalid_tensors)))
      117   return tensors
      118

  TypeError: sequence item 0: expected str instance, Tensor found

This PR appears to mask an issue with the user not passing in the correct types. I don't think this is the proper way to solve this issue.


Revert "Revert "Revert "Custom Menu Rewrite + Fun Fact Moving + Scaled Pixel Constraints + Beta v0.6.2 Prerelease 3"""

Reverts PartlySaneStudios/partly-sane-skies#393

https://media.tenor.com/izTNkY2BgkAAAAAM/fade-away-oooooooooooo.gif


This is a really big annoyance for me as i pretty much only use keyboard commands to navigate etterna.
Suggested fix: While holding down CTRL trigger no input on search, so hitting CTRL + 1 will not trigger the input and take you off the screen normally.

I'm almost sure the current develop branch had a change to only allow numeric input during search while pressing control, and cancel tab-changing while pressing control (More or less the opposite of what you're proposing) so this should not be an issue next release

ah ok; sorry. close this or whatever


Collapse: collapse item can be disabled

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

稍后再补充一个单元测试。


Fixed _winreg OpenKey parameter

Fixing registry key for iSCSI initiatorname (Win64)

Thank you for contributing to openstack/compute-hyperv!
openstack/compute-hyperv uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


add unit-test for ListVolumeName

Signed-off-by<EMAIL_ADDRESS>Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixes part of #1763
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 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.


IndexError: list index out of range

I am encountering an issue while using the google.generativeai.chat library. Whenever I input the phrase "who are you?" as a message, it consistently triggers an error. This behavior seems unexpected and needs to be addressed.
response = palm.chat(messages=["who are you?"])
# error
  ...
  File "/Users/jybaek/work/xxxx/.venv/lib/python3.8/site-packages/google/generativeai/discuss.py", line 413, in reply
    return _generate_response(request=request, client=self._client)
  File "/Users/jybaek/work/xxxx/.venv/lib/python3.8/site-packages/google/generativeai/discuss.py", line 461, in _generate_response
    return _build_chat_response(request, response, client)
  File "/Users/jybaek/work/xxxx/.venv/lib/python3.8/site-packages/google/generativeai/discuss.py", line 443, in _build_chat_response
    request["messages"].append(response["candidates"][0])
IndexError: list index out of range

Is it a known issue? Please check.

I think you're using an older version of the client,  Instead of "List index out of range" this now returns None.
That itself is not helpful.
The API is not returning any candidates for this request:
curl -H 'Content-Type: application/json'        \
    -H "x-goog-api-key: ${PALM_API_KEY}"      \
    -d '{"prompt": {"messages": [{"content": "who are you", "author":"0"}]}}'     \
    "https://generativelanguage.googleapis.com/v1beta2/models/chat-bison-001:generateMessage"

{
  "messages": [
    {
      "author": "0",
      "content": "who are you"
    }
  ],
  "filters": [
    {
      "reason": "OTHER"
    }
  ]
}

The owners are aware of the problem.

I found the cause. I was using Python 3.8. When installing google-generativeai with pip install in Python 3.8, it installs rc1.
Thank you for your help :)

Yeah, I think we've yanked those versions, so you shouldn't see them anymore.


[SE-0253] Minor edits.

Remove grammatically wrong clause: "the naming this API often repeats the type".

The intended writing was probably "the naming of this API often repeats the type". Though the clause still doesn't quite make sense in context: repeated calls to parser.applied(to:) would be syntactically noisy but would not repeat type info.

Minor gardening and renaming for clarity and brevity.

In code examples: rename polynomial(2) to f(2).

Thx dan!


System.Runtime version <IP_ADDRESS> not found

Describe the bug

I have few dll's which are generated from .net 5 preview version
2.I add reference to these dll

Now when I invoke function
I get following error
====================================================================
Error: System .10.FileNotFoundException: Could not load file or assem bly 'System .Runtime, Version= <IP_ADDRESS>, Culture= neutral, PublicKeyToken= b03f5f7f1 1 d50a3aI.
The system cannot find the file specified.
File name: 'System .Runtime, Version= <IP_ADDRESS>, Culture= neutral, PublicKeyToken= b03f5f7f1 1 d50a3aI
at Subm < 0.MoveNext0
at System.Runtime.Com pilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
at Subm ission#5. < Initialize> 0
at Subm ission#5. < Factory > (Object[] subm issionArray)
at Microsoft.CodeAnalysis.Scripting.ScriptExecutionState.RunSubm issionsAsync[TResult](lm mutableArray'1 precedingExecutors, Func'2 currentExecutor,
StrongBox'1 exceptionHolderOpt, Func'2 catchExceptionOpt, CancellationToken cancellationToken)
====================================================================
Version : 1.0.137901
VSCODE Extension

I don't believe .NET 5 is supported yet per #663
I validated this on my local machine (with .NET 5 preview 7 installed) by following these steps:

Run dotnet --version and observe the output:5.0.100-preview.7.20366.15

Run dotnet --list-sdks and observe the output:2.1.503 [c:\program files\dotnet\sdk]
2.1.602 [c:\program files\dotnet\sdk]
2.1.800 [c:\program files\dotnet\sdk]
2.2.103 [c:\program files\dotnet\sdk]
3.0.100 [c:\program files\dotnet\sdk]
3.1.101 [c:\program files\dotnet\sdk]
3.1.203 [c:\program files\dotnet\sdk]
3.1.300-preview-015135 [c:\program files\dotnet\sdk]
3.1.400-preview-015138 [c:\program files\dotnet\sdk]
5.0.100-preview.7.20366.15 [c:\program files\dotnet\sdk]

Open a notebook and add the following C# code:using Runtime = System.Runtime.InteropServices.RuntimeInformation;
using static System.Console;

WriteLine($"Running Version: {Runtime.FrameworkDescription}");

Run the notebook and observe the output:Running Version: .NET Core 3.1.4

Hi @seesharprun we do not support yet .NET 5. At the moment we target .NET core LTS.

Related: #869

I don't believe .NET 5 is supported yet per #663
I validated this on my local machine (with .NET 5 preview 7 installed) by following these steps:

Run dotnet --version and observe the output:5.0.100-preview.7.20366.15

Run dotnet --list-sdks and observe the output:2.1.503 [c:\program files\dotnet\sdk]
2.1.602 [c:\program files\dotnet\sdk]
2.1.800 [c:\program files\dotnet\sdk]
2.2.103 [c:\program files\dotnet\sdk]
3.0.100 [c:\program files\dotnet\sdk]
3.1.101 [c:\program files\dotnet\sdk]
3.1.203 [c:\program files\dotnet\sdk]
3.1.300-preview-015135 [c:\program files\dotnet\sdk]
3.1.400-preview-015138 [c:\program files\dotnet\sdk]
5.0.100-preview.7.20366.15 [c:\program files\dotnet\sdk]

Open a notebook and add the following C# code:using Runtime = System.Runtime.InteropServices.RuntimeInformation;
using static System.Console;

WriteLine($"Running Version: {Runtime.FrameworkDescription}");

Run the notebook and observe the output:Running Version: .NET Core 3.1.4

Sorry, but what is  the notebook and how could you run it?


Unsubscribe form doesn't affect Talk

Currently zooninverse.org/unsubscribe form doesn't change Talk email preferences as it should.

The talk client can disable all subscription preferences for the user via https://github.com/zooniverse/Talk-Api/blob/f87386c9fd4fd2bd67dd95a549d397e7b91a8251/app/schemas/subscription_preference_schema.rb#L15
If they are disabled then all existing / active subscriptions are disabled as well
https://github.com/zooniverse/Talk-Api/blob/f87386c9fd4fd2bd67dd95a549d397e7b91a8251/app/models/subscription_preference.rb#L49


Create requirements.txt

not sure if this is standard for OSS projects, but I think it makes it clear that there are external dependencies.

(unless of course i set it up all wrong!)

hmm, just saw setup.py...


Guidance About Step 6

Hi @azesmway ,
I hope you're doing great.
I'm new to this plugin and having issues in understanding 6th step of setting things up. Can you please elaborate a bit that which folder will be copied to which destination? It's confusing me and I'm not able to copy paste the desired folder on required destination.

Thanks in advance

I've these files. Are these correct?
If these are correct, where I'll have to move them?

@jaferi512 That looks good, why cant you copy over the folder?


git plugin on OS X: gwip results in "xargs: illegaloption --r"

Using the git plugin on OS X (Mavericks) and invoking gwip results in the following:
xargs: illegal option -- r
usage: xargs [-0opt] [-E eofstr] [-I replstr [-R replacements]] [-J replstr]
             [-L number] [-n number [-x]] [-P maxprocs] [-s size]
             [utility [argument ...]]
[US74142 53ebf4d] --wip--
 3 files changed, 415 insertions(+), 18 deletions(-)
 create mode 100644 ...

Seems to work though.

Im happy to inform that this issue is going to celebrate anniversary.

Not if #2790 gets merged before May 15.


Request permission to badge the app

Why
Unclear why this works for me without allowing this permission - maybe there's a system default? - but Expo docs say you need to request permission to badge the app: https://docs.expo.dev/versions/latest/sdk/notifications/#setbadgecountasyncbadgecount-options
Test Plan
@gaearon sees if he receives iOS badges after this PR.

Wasn't necessary it seems.


Selenium file upload on TestContainer

What's the best way to test file upload on Selenium? Should I add a volume for the BrowserWebDriverContainer and use the path from there?
@ClassRule
public static BrowserWebDriverContainer chrome = (BrowserWebDriverContainer) new BrowserWebDriverContainer()
            .withDesiredCapabilities(DesiredCapabilities.chrome())
            .withRecordingMode(BrowserWebDriverContainer.VncRecordingMode.SKIP, null)
            .withFileSystemBind("/tmp", "/tmp", BindMode.READ_WRITE);

It works, but I just would like to know if is there a better way to do it.

@pavelpp should have really good experience with that ;)


General Next Steps And Enhancements to Prioritize

This repository hasn't really been touched since November and needs some love. Here is my priority list for us to talk about and ticket further with @j08lue and @ividito:

Write IaC and documentation about using cloud provider storage solutions (RDS and GCD):

add optionally-enabled cloud provider storage to Terraform
documentation is already started here but needs to be totally redone
make sure service charts and configs take extraSecret and extraConfig to be mounted into the pods like JH does for extra* things
relevant to https://github.com/developmentseed/eoapi-k8s/discussions/65#discussioncomment-9447443

It's typical for most demo k8s installs to default to using an in-memory database but I think our default should just install Crunchy's Postgres Operator . Then folks have high-available k8s alternative already set up to go

move CI/CD tests to using k3's. Even though we currently can have parallel tests deploy parallel eoapi-k8s and run against a real cloud provider k3's would be more reliable and just use the CI runners

We need to automate releases using GH Actions for releases:

releases should also auto-generate a documentation site on GH pages

The support helm chart is crucial to set up Prometheus and certain auto-scaling and needs to be it's own separate chart release:

we need to make sure by default this seamlessly gives us the Grafana dashboard and ability to auto-scale by request

Write documentation about generating certs for TLS using Let'sEncrypt. There are manual steps for this like running let's encrypt CLI and generating secrets. The current helm addition PR also conflates the idea of staging and prod

Finally got around to ticketing these:

https://github.com/developmentseed/eoapi-k8s/issues/77
https://github.com/developmentseed/eoapi-k8s/issues/80
https://github.com/developmentseed/eoapi-k8s/issues/39
https://github.com/developmentseed/eoapi-k8s/issues/81

@ividito: let me know which ones you're interested in and we can start jamming on them

I can pick up 2 and 3 - possibly overlapping with 4, if we start expanding on the eoapi-support CI.


Update to Jetty 9.4

This is a meta-issue.

Pushed my changes in jetty-9.4 branch. Tests are passing again. Review please :)

Works for me. I will merge the branch into master to continue further work on 1.1.0

Merged in bdc834b7bb661db5b9bdb7d5c9b0561f29485a86


Mister clean

Just some general cleanup

Coverage increased (+1.9%) to 87.619% when pulling 895c2c2c1056a988023b9063df286f133f8f5e36 on mister-clean into e3de45f7c5ffda413123ec15fc05c8f73f668081 on master.


Controllers specs

I did the following in Controllers specs:

Make sure that there are no linter errors
Make sure that you used correct flow Gitflow
Make sure that you documented your work in a professional way
Add RSpec as a gem to your project.
Create a Request spec file for all your controllers.
Make sure that for each action you check: If the response status was correct, If a correct template was rendered, and if the response body includes correct placeholder text.

@TedLivist Thanks


Problem/Suggestion : file protocol

Hi,
Skip Redirect 1.05 on Firefox 44.0.1 64-BIT running flawlessly. True pleasure.
One suggestion:
I see that Skip Redirect handles the "ftp", "http", "https", "mms", "rtsp" protocols.
Could it handle the "file" protocol as well?
Example:
I'm using Sage  (Firefox add-on) as my RSS Feed Reader. It displays feeds in the file format.
Some links will include redirects, for instance,

https://www.bing.com/news/apiclick.aspx?ref=xxxxx&url=LINK

In that case Skip Redirect is not programmed to interfere.
T

Is Sage still working? The last couple of reviews are not very encouraging. So I rather not try myself. So what you are saying is that LINK will start with file://? I that's the only thing, it would be a one line addition here: https://github.com/sblask/firefox-skip-redirect/blob/master/lib/url.js#L25 But I don't quite understand why a bing.com link would have a file:// in it, does Sage do some magic there? (Again, I didn't try.)

I'm using Sage++ (Higmmer's Edition) Installation Page which has no AMO page, available at

http://blog-imgs-30-origin.fc2.com/h/i/m/himag/sagepp-higmmersedition-install.html

The add-on is signed and runs great, always updated.
Concerning the file issue, perhaps I haven't been precise enough.
Bing News RSS is :

https://www.bing.com/news/search?q=&format=RSS

Sage++ displays this url here on Firefox in a tab:

file:///[MY PROFILE]/chrome/sagepp/sagepp.html

The links don't change, they are of the form:

https://www.bing.com/news/apiclick.aspx?ref=xxxxx&url=LINK

But, as I understand it, the link being on a file: page, Skip Redirect doesn't handle it, or am I missing somethin

Fixed with version 1.0.6, currently under review by Mozilla


[spark] Support cross partition write

Purpose
spark support Cross Partitions Upsert Dynamic Bucket Mode

Tests

API and Format

Documentation

How is this PR going?

@waywtdcc cross partition write needs to be refactored, will rebase after refactoring

Now, How is this PR going?

@waywtdcc sorry for delay,  we choose to  closed it, The current implementation is friendly to streaming scenarios, but the batch cost is too high. We need to find a more efficient implementation in batch scenarios.

Doesn't flink batch also support it? The logic of the two should be the same, right?


fix: ensure we do not rewrite external Origin headers in wrangler dev

In https://github.com/cloudflare/workers-sdk/pull/4812 we tried to fix the Origin headers to match the Host header but were overzealous and rewrote Origin headers for external origins (outside of the proxy server's origin).
This is now fixed, and moreover we rewrite any headers that refer to the proxy server on the request with the configured host and vice versa on the response.
This should ensure that CORS is not broken in browsers when a different host is being simulated based on routes in the Wrangler configuration.
Author has addressed the following:

Tests

[x] Included
[ ] Not necessary because:

Changeset (Changeset guidelines)

[x] Included
[ ] Not necessary because:

Associated docs

[ ] Issue(s)/PR(s):
[x] Not necessary because: bug fix

Happy to merge. Might want to hold off merging since I believe we want to get a hot fix out today.
One remaining point (you decide whether to address or not): should we print something to the console to let the user know the origin/host is being rewritten across all request/response headers inc Set-Cookie (and not the body) and let them know they can opt-out by...?

Yeah good question... we have not previously had a message for rewriting the request.url, and prior to 3.18, I believe that we would have been rewriting response headers in this way anyway.
Perhaps we could put some debug logging messages in, so at least if the user is getting confused it could be analysed via increasing the logging level?

In remote mode, yeah no, but locally workerd logs now reach the log file (not the terminal but that's what you were going for anyway)

Hmm I tried adding console.debug() logs to the ProxyWorker but these always appeared in the output of wrangler dev (whatever the logging level). So I think this is too much noise. Perhaps we need to tweak the Proxy Worker's Miniflare configuration?
I think we can leave this to another time, or if someone gets confused by this rewriting.

@petebacondarwin Can you please take a look at
https://github.com/cloudflare/workers-sdk/issues/5221
I think this PR is related, I had couple of notes:
https://github.com/cloudflare/workers-sdk/issues/5221#issuecomment-1996147061


Layout column widths

Is your feature request related to a problem? Please describe.
I want to define different widths for different columns. Right now I can't find anything that seems to indicate I can do anything other than evenly wide columns. I could do weird workarounds, but I'd rather just define some % widths.
Describe the solution you'd like
Some % width definitions, I could see fixed width columns being useful too and a combination of the two would be nice as well.
Describe alternatives you've considered
I could just use it like it is now, but some of my columns have information that really wants more space than the neighbouring columns.

I think you want the columns parameter when constructing your Layout.  See https://asciimatics.readthedocs.io/en/stable/asciimatics.widgets.html#asciimatics.widgets.layout.Layout


[BEAM-13932][Playground] Container's user privileges

Update proxy and permissions for the container's user
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @KhaninArtur @ilya-kozyrev

@ilya-kozyrev @andreykus could you please take a look at this PR?

retest this please

LGTM


feat: add support for neovim v0.9

This PR vendors vim.system for neovim < v0.10 to work.
Notes:

vim.uv is not available in <v0.10, however, it seems vim.system is only using standard libuv functions
I didn't touch the version check code. Please feel free to modify

Also I didn't do thorough tests as I'm only just adopting this awesome tool! Kudos!
for #58

thanks! is this working for you locally? if so what version of Neovim are you on?
I'll circle back to this later today or sometime tomorrow. Thanks for the contribution 👍

thanks! is this working for you locally? if so what version of Neovim are you on?
I'll circle back to this later today or sometime tomorrow. Thanks for the contribution 👍

I'm on v0.9.4, yes it's working smoothly so far!

hey @jackielii I think this PR is looking pretty good. I modified things a bit. Could you run KittyScrollbackCheckHealth on your v0.9.4 nvim version and let me know if it runs successfully? Thanks! I need test a couple more things but think this should be good in the next day or two to merge.

Nice! Just tested it and everything looks good! thanks!

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

closes #58


Rename web5.did.register

The word register is confusing, should be renamed to something more descriptive and less overloaded like setController or setAgent etc.

Hey @chris-tbd -- closing as a duplicate for an issue that Angie already opened
Welcome your feedback and ideas -- join the convo here #27


Does not work with 1.18.2

Currently your mod targets 1.18.1 as can be seen in your gradle.properties file. Nearly all mods built for 1.18.1 don't work for 1.18.2. That includes yours. This is an issue because you've incorrectly labeled your mod as compatible with 1.18.2 on CurseForge releases when it is not.
All of your mods that target 1.18.1 or 1.18.0 should be properly checked to see if they run on 1.18.2 (BuildableCampfire doesn't, the game crashes).
I would submit a crashlog, but it is pretty self explanatory why it isn't working.
Have a good day. :)

Thank you for the warning. I haven't played Minecraft for nearly a month, so I didn't even knew there was a new version until today. I actually added the 1.18.2 version to the "fabric.mod.json" file but it seems like I forgot to rebuild it, so I probably just posted the old build.
Thank you for the warning again and have a nice day :]


course v2 learning resource types icons

Steps to Reproduce

Either:

Build any course on new theme locally (How to do it? Check the description of this PR)
View this course deployed on netlify

Go to the course homepage
View learning resource types icons

Expected Behavior

Learning resource types icons should be the same as what we have in the design.

Actual Behavior

Old icons are being used.

Screenshot or Screencast
(Ignore the other issues in the screenshot. will have separate tickets for them)

@pdpinch In invisionapp, we have only 4 learning resource types icons whereas currently, in old theme, we have around 12 material icons in total:

however, the total options of learning resource types are more than that but I assume we are using 1 icon for multiple types that are similar.
So I think we'll need to have the replacement/alternate of all these 12 icons so I can replace them all.
ps: I also think that svgs will be better but we can get the designer's thoughts on this.

I started a google sheet so we can review the mapping of resource types to icons: https://docs.google.com/spreadsheets/d/1rlyOUMHHEJ9hOBwZ8QwZo8SQTA-eC0eLGqmaFIt3V78/edit#gid=0
Can you help me fill it in? Is there someplace in the code where I can see the current mapping?

I started a google sheet so we can review the mapping of resource types to icons

Thanks

Can you help me fill it in

Sure

Is there someplace in the code where I can see the current mapping?

Yeah, the mapping is in learning_resource_type.html but not sure if it's easy to understand.
Strings between Hugo's if else are the material icons:
e.g:
{{- if eq $context "Course Introduction" -}}
      groups
{{- else if in $context "Audio" -}}
      equalizer
....

Course Introduction type has the icon groups.
Audio type has the icon equalizer and so on

Got closed mistakenly, so reopening this

@MAbdurrehman12 we completed the course-v2 redesign somehow without resolving this issue. Can you update this with what we currently have and what remains to be done (if anything) ?

@pdpinch We had new icons for learning resource types that can be seen in the design, however, we need all icons for all types which are pending at the moment.
Type to icon mapping can be seen here

@MAbdurrehman12 what's the state of this currently? If we don't have an icon for a course's learning resource type, what do we display?

what's the state of this currently?

@pdpinch The state of this ticket is the same as described in this comment

If we don't have an icon for a course's learning resource type, what do we display?

I think all learning resource types are being displayed from this file.
We just need to replace the old icons with the new ones shown in this design

We've been living with this as is for some time now. If we want to update these icons, we will align it with Learn designs.


Release 8.7.0

Summary of the Pull Request
Release 8.7.0
Added
Changed
Fixed

Codecov Report

Merging #3403 (c5c2b99) into main (a364051) will decrease coverage by 1.40%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3403      +/-   ##
==========================================
- Coverage   34.23%   32.84%   -1.40%
==========================================
  Files         297      269      -28
  Lines       36366    31386    -4980
==========================================
- Hits        12451    10309    -2142
+ Misses      23915    21077    -2838

see 59 files with indirect coverage changes


Can we find a vulnerability scanner that works with pnpm?

Discussed in https://github.com/guardian/csnx/discussions/449

Originally posted by joecowton1 February 15, 2023
Currently across the Guardian we use Snyk to check for dependency vulnerabilities, and Dependabot to check for dependency updates.
Neither of these works with pnpm, which we use here in csnx.
We currently use Renovate to update dependencies, we're awaiting full approval from the tech council for wider use, but as a test it seems to be working well.
We'd like to find a vulnerability scanner to suggest for approval by Infosec. There was talk at the recent Dev X / Infosec catchup that it would be good to think holistically about a department wide solution to dependency vulnerability, perhaps we can use csnx as a testbed for this.
Suggestions so far have been:

Google osv scanner
Mend Bolt

Please add any thoughts below.

I added this to the WebX rota project board because I was testing org-wide projects, but I can't find a way to remove it from that project again, sorry! I don't think it's messed with any of the settings in the CSNX project though 🤞

No worries!


Can't enable vibrate.

com.google.zxing.client.android.BeepManager#vibrateEnabled is always false, and it can't be changed

BeepManager beepManager = new BeepManager(getActivity());
beepManager.setVibrateEnabled(true);
and play vibrate in result callback
beepManager.playBeepSoundAndVibrate();
and close on onPause()
beepManager.close();

You need to add permission to VIBRATE in the AndroidManifest also