ELM-1463 Refactor sftp server user code generation

Now instead of hardcoding in a dev account and a user account, the landing_zone module takes a list of maps where each map is a unique user give access to the server and contains all details needed. This change allows scaling to giving n unique users access to each server (instead of 1 + dev account previously).
To achieve this have made the following changes:

abstracted code for creating the "supplier" and "dev" accounts into a generalised landing_zone_user module
abstracted code for creating the "supplier" and "dev" allow listed ip security groups into generalised server_security_group module
moved all access credential info for devs and suppliers into locals.tf and updated module variables in main.tf

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@dms1981 Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@dms1981 Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@dms1981 Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform

@pricemg Terraform plan evalaution detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform


Introduce DataContext

this PR introduces new data structures to decouple the runtime from the data layer. Doing so solves some flaws that the current design presented:

the DataContext represents a transactional unit. It is created when a transaction is started by typescript and lives until it is committed or discarded later. It contains all the necessary data for the data layer to operate.
The JobContext is a deno resource that spans the lifetime of a job and contains the data that a job will need to access in its lifetime. This simplifies the management of resources.
The SQL transaction is not leaked into the runtime anymore, and the concept of the transaction is abstracted away.
Implicit transactions are not supported anymore. It is always to the consumer of the data layer APIs to provide one, in the form of a data context, and then commit it.
by grouping together all the resources for a single job, we make a step toward allowing multiple requests to be handled concurrently by a worker.

Not to the reviewer: I reckon this PR is slightly bigger than I would have liked, but 2 weeks of rebasing got the better of me, and I had to squash it. I tried to cut it into more parts, but ultimately that would make a lot of code changes to just revert it in the next PR, and I need to move on to the following PRs in this series.

@honzasp You are missing the bigger picture here. Let me address your points.

Why couldn't we have multiple data contexts per job?

It is completely acceptable that a job has multiple DataContexts, just not possible to hold multiple contexts at the same time. Furthermore, even if you allowed it, then I see no issue with the current approach. Notice that allowing a job to accept multiple data contexts means that we also need to tie the objects to the data contexts they come from.

One reason is that policies need information from the HTTP...

This is because the current policies are more or less stateless. They only need some information from the job, and that information is available from the beginning of the job. What you propose is that we have two separate resources, one for the policy context, and another for the current data context for the job (notice that these are already 2 resources that you need to manage). Now, the new policies are stateful. They cache some computation, mark some object dirty etc... In that sense, the logical lifespan of the PolicyContext is the DataContext, since some of the state held by the context is only valid in the context of the transaction it originates from.
FIY, the new policies do have a PolicyContext. It is created and passed to the DataContext at its creation, and having more design freedom, it is more decoupled.
Please see the current inclusion of the JobInfo in the DataContext as a hack, rather than a design decision. The scope of this PR is not to refactor legacy policies, and this appeared to be the least intrusive way to glue things together for now.

The concept of a "policy context" also makes it obvious that policies assume that the data operations are only invoked from HTTP requests, but this is false:

I'm not sure I understand this part, is the PolicyContext a good thing or a bad thing? 😄

There is no need to couple data context to job creation. (Some jobs don't need a data context, and some jobs may need multiple data contexts.)
But that is not, the data context is created when a transaction is requested. When we have jobs that need multiple simultaneous DataContext, we'll accommodate the JobContext to do so. The only reason they are still tied is because of legacy policies, but again it is not the scope of this PR to rework that.

Create and close the data contexts from JS; this replaces the beginning and committing/rollbacking of transactions. There is no need to couple data context to job creation. (Some jobs don't need a data context, and some jobs may need multiple data contexts.)

this is more or less what is happening now

Pass the "policy context" as a resource in the AcceptedJob structure from Rust to JS, and require it to create a new "data context".

I guess this is where we really disagree on the essence of the design. One of my design goals is to minimize the number of resources that we need to manage. This is error-prone, and hard to debug. Hence the job context encloses all the resources for a job in one single resource and simplifies resource management. You may be able to reason about all the resources a job is using in its lifetime, when to deallocate them, and from which side (rust, or js) to do it, but unfortunately, I'm not as confident as you and thinking about how this code will age, I'd much rather refactor a rust struct than keep adding new resources to a job.

Also, notice that js is completely oblivious to what the ctxId is, for it is just some indirection to some context that ties together a bunch of syscall. Say implement a new thing, that is not a job, then it could have its own context, and it is a resource, and pass its id to js, and everything would work the same, as far as js is concerned.


Allow Permission to read the log file from different process

With seelog, log written to file is unable to open for read from different process under Windows. We could just expand the permission to OpenFile to add FILE_SHARE_WRITE. I assume this need to be done in writers_filewriter.go.

No reply for a long time. Closing for now, reopen if needed.


inputs attribute of Step is not being updated

The attribute self.inputs in the Step class is never touched beyond initialization, and therefore the step has an empty list of input files. When I wrap a for loop over the input files in a step in polaris.run.serial._run_step():
missing_files = list()
print('DEBUG: listing all input files in step.inputs:')
for input_file in step.inputs:
    print(f'DEBUG: {input_file}')
    if not os.path.exists(input_file):
        missing_files.append(input_file)
print('DEBUG: finished listing all input files in step.inputs.')

I get these results:
  * step: analysis
DEBUG: listing all input files in step.inputs:
DEBUG: finished listing all input files in step.inputs.

(despite the analysis step requiring the outputs of all previous steps as its inputs).
This causes missed files to go unnoticed, resulting in unhandled exceptions that we intended to capture before the step begins executing.

@altheaden, thanks very much for posting this issue!
It looks like we have these lines in compass:
https://github.com/MPAS-Dev/compass/blob/8440b418533f1e1a3e78cef9cc03f4535f1b6b93/compass/step.py#L780-L781
And it seems like the equivalent is missing in polaris:
https://github.com/E3SM-Project/polaris/blob/0ce1674dd4c8350c16fb366df6a0584446f6e052/polaris/step.py#L500-L507
I believe what happened is that converting the inputs to absolute paths got moved to a helper function here (partly to make the linter happy):
https://github.com/E3SM-Project/polaris/blob/0ce1674dd4c8350c16fb366df6a0584446f6e052/polaris/step.py#L587
But then we are missing the final line here following the for loop defining inputs:
        inputs = []
        databases_with_downloads = set()
        for entry in self.input_data:
            input_file, database_subdirs = Step._process_input(
                entry, config, base_work_dir, component, step_dir)
            if database_subdirs is not None:
                databases_with_downloads.update(database_subdirs)
            inputs.append(input_file)
        self.inputs = inputs

Could you see if that 1-line fix changes what you see in polaris.run.serial?  You will need to set up the test cases or suite again to see the change (i.e. the pickle files will need to be modified to have the missing inputs attributes for each step.)

This is the fix I tried last night, but I was getting some weird output in one of the log files. Let me make a fresh run right now and see if I'm still seeing that behavior, and if so, I'll post it here for you to look at.

The output you pointed out to me on Slack looked normal to me.  That output is what comparison with the baseline looks like -- a variable and some time levels where they get compares.  (Maybe that's not what you're talking about, though.)

Oh, I see! That makes sense then, I was comparing it to the output from the actual baseline run. In that case, it seems to be working fine, all the baseline checks ran as expected. I'll go ahead and put together a PR for the branch I've got.

Sounds good!


Ready to accept the pull requests

Hello Everyone we are ready to take part in Hactoberfest

Hello can we connect on LinkedIn if possible to connect on -Goutam Popli

Hello Can we add android app project

Yes
But make sure to add new folder inside projects
On Fri, Sep 29, 2023, 00:08 ROHIT KUMAR @.***> wrote:

Hello Can we add android app project
—
Reply to this email directly, view it on GitHub
https://github.com/TammyAlok2/Hactoberfest2023_ADSC2/issues/1#issuecomment-1739828106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2EBLSMMSMNT66UTSDY2O3TX4W73DANCNFSM6AAAAAA5J2XC34
.
You are receiving this because you authored the thread.Message ID:
@.***>

I wish to contribute. Kindly assign this to me.

Hello, Can you please assign this task to me.Thank you.


Using vswhere with Cygwin can't find latest installation

I'm trying to update a build system that relies on using Cygwin for some build scripts. Since I can't use env variables anymore, I decided to give vswhere.exe at which point I ran into an issue.
The syntax I use to find latest VS2017 installation is as follows:
vswhere -latest -products * -property installationPath

This works fine when called from within a Windows cmd, but running the exact same command in Cygwin produces no results. What's even weirder is that using the -legacy switch works in both cases, so I'm not sure what I'm missing here.
Is there a secret env variable to set or is vswhere looking for latest installation in some odd way?
A workaround I'm looking into right now is using cmd /C but for the life of me I can't understand why I have to resort to this.

For future reference: turns out, the problem was the asterisk, so proper syntax for Cygwin would be:
vswhere -latest -products \* -property installationPath
I hope this helps someone as frustrated as I was with it!


⚠ Bofrost.de has degraded performance

In c15dda7, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1682 ms

Resolved: Bofrost.de performance has improved in c8146f7 after 1 hour, 53 minutes.


[Update] Unread messages intent

The updated read unread messages intent in this PR can read messages from both public and private channels.
This intent uses simple resolution of channel names. i.e. it will resolve the input to the best matching channel or group name without any intent confirmation.

Should updating model with previous intents be in separate PR?

pls resolve conflict thx


Errors with running examples on a fresh clone/build

DrawSprite
seems like there are some problems with loading the resources:
$ ./rizz --run ./lib03-drawsprite.so
version: 0.4-17-gf9b8182
(init) vfs
(init) temp memory: 8x5120 kb
(init) jobs: threads=7, max_fibers=64, stack_size=1024kb
(init) asset system: hot-loading=1
(init) profiler (cpu): port=17815
(init) graphics: OpenGL 3.3
(init) coroutines: max_fibers=64, stack_size=2048kb
(init) http client
vfs: mounted '/cache' on '/d/code/game/rizz/rizz/bin/.cache'
(init) plugin: imgui (libimgui.so) - dear-imgui plugin - v1.77.0
(init) plugin: 2dtools (lib2dtools.so) - 2dtools plugin - v1.0.0
vfs: mounted '/assets' on '/d/code/game/rizz/rizz/examples/assets'
(init) plugin: drawsprite (lib03-drawsprite.so) -  - v0.0.0
ERROR: sg_image_desc.width must be > 0
ERROR: sg_image_desc.height must be > 0
ERROR: ^^^^  VALIDATION FAILED, TERMINATING ^^^^
rizz: /d/code/game/rizz/rizz/src/rizz/../../3rdparty/sokol/sokol_gfx.h:12819: _sg_validate_end: Assertion `0' failed.

NBody
Seems like is not built by default ?
Draw3d
$ ./rizz --run ./lib08-draw3d.so
version: 0.4-17-gf9b8182
(init) vfs
(init) temp memory: 8x5120 kb
(init) jobs: threads=7, max_fibers=64, stack_size=1024kb
(init) asset system: hot-loading=1
(init) profiler (cpu): port=17815
(init) graphics: OpenGL 3.3
(init) coroutines: max_fibers=64, stack_size=2048kb
(init) http client
vfs: mounted '/cache' on '/d/code/game/rizz/rizz/bin/.cache'
(init) plugin: imgui (libimgui.so) - dear-imgui plugin - v1.77.0
ERROR: error: vertex shader output `f_color' specifies flat interpolation qualifier, but fragment shader input specifies no interpolation qualifier

ERROR: sg_pipeline_desc.shader missing or invalid
ERROR: ^^^^  VALIDATION FAILED, TERMINATING ^^^^
rizz: /d/code/game/rizz/rizz/src/rizz/../../3rdparty/sokol/sokol_gfx.h:12819: _sg_validate_end: Assertion `0' failed.
ERROR: plugin init failed: ./lib3dtools.so
ERROR: initializing plugins failed

Btw,  I couldn't see the imgui windows in any of the examples (unlike the screenshots)

Thanks !
DrawSprite didn't showed any errors for me. could you debug it yourself and give me a callstack or more details ? (or just submit a PR)
NBody isn't available on linux yet. because I added compute-shader support for D3d11 backend only, so OpenGL/metal compute-shader support is currently lacking.
Draw3d is fixed with the latest commit
btw, what linux distro are you using ?

Thank you for the response. I'm using latest version of Kubuntu

DrawSprite didn't showed any errors for me. could you debug it yourself and give me a callstack or more details ? (or just submit a PR)
will try to fix and send a PR :+1:


London10-AfshaHossain-FullStackProjectAssessment-Week2-Level 200

Great work! 😊 Keep in mind to remove those console.logs() from the codebase before pushing, as they no longer serve the purpose, once you've effectively debugged your program. Otherwise, amazing stuff and keep up the good work!

Thank you very much for taking the time to review my code!


[Separator] Separator component in vertical orientation do not take up any height in the DOM when contained in flex containers

Bug report
Current Behavior

Vertical separators within flex containers don't show up (they don't take any height in the DOM) and therefore don't show up. Horizontal works as expected

Expected behavior

Vertical divider should take up 100% of the height
Reproducible example
CodeSandbox Template
Example: https://codesandbox.io/s/reverent-dust-pl1b5j?file=/App.js
Suggested solution

Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
Separator
1.0.1

React
n/a
17.0.2

Browser
Chrome
109.0.5414.119

Assistive tech

Node
n/a

npm/yarn
npm
8.3.1

Operating System
Mac OS X
12.6

FWIW I've tried fixing this exact issue in an "in-house" UI component, before trying to move to Radix for the separator component, but wasn't able to figure out the solution

@scottwarren You need to drop the height: 100%, with that property the separators are attempting to size to the parent of which is not explicitly provided.
https://codesandbox.io/s/vertical-separator-fill-parent-f1ysyt?file=/App.js

Let me know if that solution works for you, I'm going to close this out as it's more of a styling consideration than an issue with the library.

Thank you very much!


oap can't review data after  about 8 hours  in 6.0.0-beta-SNAPSHOT

Please answer these questions before submitting your issue.

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

Question

What do you want to know?
I am running skywalking in docker,the storate use es, Everything works fine at the beginning, I can collect all the monitoring data. But after about 8 hours, I can't collect all the data.  The final solution is to restart oap. and then it is back to normal. But I don't know why this happens ?  Can you help me solve this problem?
skywalking-oap-server.log

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

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

Requirement or improvement

Please describe about your requirements or improvement suggestions.

6.0.0-alpha , agent add -Dskywalking.agent.service_name=spring-boot-demo, skywalking ui dashboard can not display service, service‘s ’count is 0. Endpoint and DB & Cache display count.

@qdlake Your agent mechanic OS time or time zone is not right. Your issue doesn't relate to this.
We have no clue what is happening in this issue.

this time, I deploy oap with a different configuration. first I use h2 as the storage. after 8 hours , It is normal.
second, I use es as the storage, after 8 hours , It is normal. third,I use es as the storage, and config the namespace in system environment with SW_STORAGE_ES_NAMESPACE,after 8 hours ,It no longer collects a bit of data. So I can only restart the oap component to make it back to normal.

@wu-sheng @peng-yongsheng  Can you give some ideas to consider what is the trigger?

If the namespace, there is a bug fixed in beta milestone, the delete mechanism doesn't work right. So, based on the things you provided, it may be caused by ElasticSearch overload. That is the only thing I could relate with.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 801 ms

Resolved: Bofrost.lu performance has improved in 632e2eb after 6 minutes.


Validate Modifier for Predicates

Validate Modifier for Predicates
:recycle: Current situation & Problem
SpeziValidation currently provides the validate(input:rules:) modifier to validate String-based inputs using SpeziValidation. This PR extends this functionality to Bool expressions. The new validate(_:message:) modifier can be used to validate all sorts of boolean conditions and supply a validation failure reason with it.
:gear: Release Notes

New validate(_:message:) modifier to validate bool expressions.

:books: Documentation
Documentation was updated to highlight the new modifier.
:white_check_mark: Testing
TBA
:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

CC: @AdritRao, this might be helpful for https://github.com/StanfordSpezi/SpeziOnboarding/pull/54


Navbar flag input keys off of challenge id instead of challenge index

What?
The navbar flag input's previous and next buttons can sometimes show the wrong challenge if the current challenge was created recently, but shows up earlier in the challenge listing:

The line in question appears to be this one: https://github.com/pwncollege/dojo/blob/master/dojo_plugin/utils/dojo.py#L504
It probably should be using this: https://github.com/pwncollege/dojo/blob/master/dojo_plugin/models/init.py#L437
cc @ConnorNelson

I'll Fix it. But this also means my tests are useless 🤣 sorry mate

Haha! I was just working on this myself! I'll let the expert handle it though!

Sorry, I lied! 😅  While learning how the dojo worked, I also saw a way I could reduce the number of queries sent to the DB, so I took a stab at it too!


request: custom method to skip encoding

Is your feature request related to a problem? Please describe.
Some hevc 10bit videos are tried to be encoded, but vaapi doesn't support it, failing with:
[Parsed_scale_vaapi_2 @ 0x55822b8a5680] Hardware does not support output format yuv420p10le.
[Parsed_scale_vaapi_2 @ 0x55822b8a5680] Failed to configure output pad on Parsed_scale_vaapi_2

In:
    "video": {
        "index": 0,
        "codec": "hevc",
        "bitrate": 8066898,
        "pix_fmt": "yuv420p10le",
        "profile": "2",
        "fps": 23.976023976023978,
        "framedata": {
            "pix_fmt": "yuv420p10le",
            "color_space": "bt709",
            "color_primaries": "bt709",
            "color_transfer": "bt709"
        },
        "dimensions": "1920x1080",
        "level": 4.0,
        "field_order": "unknown"
    },

Out:
    "video": {
        "codec": "h265vaapi",
        "map": 0,
        "bitrate": 6000,
        "crf": 23,
        "maxrate": "10m",
        "bufsize": "24m",
        "level": 0.0,
        "profile": null,
        "preset": null,
        "pix_fmt": "yuv420p10le",
        "field_order": "unknown",
        "width": null,
        "filter": null,
        "params": null,
        "framedata": null,
        "title": "FHD",
        "debug": "video.max-bitrate",
        "device": "sma"
    },

autoProcess.ini
[Converter]
postopts =
hwaccel-decoders = h264_vaapi, hevc_vaapi, mjpeg_vaapi, mpeg2_vaapi, vp8_vaapi, vp9_vaapi
post-process = False
ignored-extensions = nfo, ds_store
temp-extension =
ffmpeg = /usr/local/bin/ffmpeg
preopts =
minimum-size = 0
delete-original = True
ffprobe = /usr/local/bin/ffprobe
force-convert = False
output-extension = mp4
hwaccels = vaapi
output-directory =
process-same-extensions = False
threads = 0
move-to =
copy-to =
output-format = mp4
sort-streams = True
detailed-progress = True
hwdevices = vaapi:/dev/dri/renderD128
hwaccel-output-format = vaapi:vaapi
wait-post-process = False
bypass-if-copying-all = False
opts-separator = ,
regex-directory-replace = [^\w\-_\. ]
[Video]
profile =
crf-profiles = 0:23:5M:15M,4000:23:10M:24M,8000:23:20M:60M
codec = h265vaapi, hevc, h265, x265, x264, h264
crf = -1
max-width = 0
max-bitrate = 6000
pix-fmt = nv12, yuv420p, yuv420p10le
max-level = 0.0
filter =
force-filter = False
codec-parameters =
preset =
dynamic-parameters = False
bitrate-ratio =

Describe the solution you'd like
In #1385 you introduced the custom function blockVideoCopy so that I could force encoding of x264 10bit videos.
Maybe a solution to this is a blockVideoTranscode option, working exactly the other way?
Describe alternatives you've considered
I could remove 10bit alltogether (yuv420p10le) bit since my TV supports playing those (10bit HEVC at least) I'd rather not. Not sure if that would even work.
I could also ser max-bitrate to something way higher so the file would not need to be encoded. But for other formats that can be encoded I enjoy having smaller files.
An alternative could also be to disable vaapi for just those files and use cpu transcoding instead. Not sure how this would be implemented here.

Maybe transcoding does not need to be skipped but use another ffmpeg command? I just found out that
... -vf "format=yuv420p10le|vaapi,hwupload,scale_vaapi=format=yuv420p10le" ...
results in the error while
... -vf "format=nv12|vaapi,hwupload" ...
seems to work (
in: Video: HVC1 1920x1080 23.976fps [V: hevc main 10 L4.0, yuv420p10le, 1920x1080 [default]]
out: Video: HVC1 1920x1080 23.976fps 5594kbps [V: FHD (hevc main 10 L4.0, yuv420p10le, 1920x1080, 5594 kb/s)]
)
But honestly I have no idea about ffmpeg filters and was just trying to get rid of the "yuv420p10le" I saw in the error message ;-)

Seems like you need to just remove that pixfmt from your allowed list and use a different one.
A lot of this can also be handled via the HDR options
Looks like you're just moving things from hardware to software shuffling those codecs around and then reuploading to hardware sort of negating your VAAPI benefits

In https://github.com/mdhiggins/sickbeard_mp4_automator/issues/1300#issuecomment-674547729 you wrote that both formats should be on the list to prevent reencoding. Sind I don't want to reencode HEVC 10bit (unless the bitrate is too high), I have yuv420p10le on that list (at the end). Is that not correct any more?

In general the script attempts to maintain the existing pixfmt as converting between different formats can cause problems and color issues and it's gets tricky automating conversion between them. It behaves differently than codecs.
What is the end goal for your media conversion? Having a pixfmt that your hardware encoder doesn't support will definitely introduce errors it seems like you're trying to reduce bitrate mostly
I suppose I could include an option to attempt to preserve the source pixfmt versus using the first on the list. That would let you include it but not use it for conversion.
I also might be able to read the supported pixfmt options by querying FFMPEG though not sure how helpful that would be since I don't think it reports encoder specifics I'll have to research that.

What is the end goal for your media conversion?

That's a very good question and got me thinking. It could previously probably be summed up like this:
"Just use whatever source there is as long as it's supported by my TV (264 8bit, 265 8bit, 265 10bit). If the source has a ridicoulous bitrate, convert to hevc with a good but not over-the-top quality."
I defined "ridiculous bitrate" as anything >6000 for the video part.
But thinking of it, it usually doesn't really matter as I have only used half of the 20TB my NAS is equipped with and my devices have at least 100Mbit capabilities. So probably I can just go with max-bitrate = 0 and be done with it. Will try how that works for me.

My personal strategy is to avoid transcoding video at all costs. Unless I'm encountering an unplayable format, I try to remux as much as possible. Any sort of conversion is going to degrade quality to some degree even without capping bitrate (and more so if capping bitrate) and I aim to try to preserve the quality of the original source as best as possible especially for things like 4K HDR content where I'm using more storage space because I want that higher quality. It's always a good rule of thumb to try and get things in a source format that's close to what you want. Transcoding audio is generally not as big of a deal and creating an extra audio stream for compatibility and keeping an 'original' is a more viable strategy without wasting a ton of space unlike for video
I also have 2 separate instances of Radarr running in parallel, one for standard stuff and one for my 4K bluray rips so I can have varying degrees of quality. On the 4K Radarr instance I keep the MKV container, remux everything, and just filter out languages I don't need.  These are for things I intended to play on my home theater with ATMOS/4KHDR playback
I have a 2nd Radarr instance that contains most of my other movies in lower quality that I convert to more streaming friendly formats for watching on lesser quality setups/mobile etc where I convert to mp4 h264/ac3
Anyway
db52f28708f4e20db8addd42cc245a12071df813
That includes a new option prioritize-source-pix-fmt which defaults to True
If you set that to False then the script will use the first option on the list more like codecs and should give you the control you want
Another option available:
The custom "validation" method (https://github.com/mdhiggins/sickbeard_mp4_automator/wiki/Custom-Functions) can be used to intercept the script settings and run custom logic and change settings before the FFMPEG options are generated
You could for instance here detect some aspect of the codec/pix_fmt and make dynamic changes to your settings to achieve more granular control
Here is an example where if trueHD is present the output format/extension are changed to mkv
https://github.com/mdhiggins/sickbeard_mp4_automator/issues/1541

Thank you again for your amazing support and the insights of how you use radarr personally. I've been using the max-bitrate=0 setting now for a week and I think I'll stick with it, converting only the few files that my TV unfortunately doesn't support. Closing this now :-)


multiple permit generation fix

Resolves #446
this issue is made to fix multiple permits being generated in a single issue

@0xcodercrane can you review this

@me505 you might need to import UserType


minor changes

Reasons for making this change:

TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

updated .md file


(360tse) Version <IP_ADDRESS>9 advertises connection with BitDefender and Avira.

The description for the above package claims a connection with BitDefender and Avira software. The package website makes no mention of this connection. The package in the Chocolatey repository was last updated in 2018, which may make it badly out-of-date, the "false advertising" notwithstanding.
Expected Behavior

That the description omits the words Avira and BitDefender to correctly reflect that there is no relationship between these 3 vendors.
Current Behavior
Searching for BitDefender returns additional unrelated results (360 Total Security and 360 Total Security Essentials)

Possible Solution

Edit the description to remove the words Avira and BitDefender
Remove the out-of-date package

Steps to Reproduce (for bugs)

Search for avira or bitdefender in the main repository
Note that 360tse Version <IP_ADDRESS>9 is among the returned results.

Context

360tse package appears to claim a connection with both BitDefender and Avira, The 360ts website makes no mention of this connection. Searches for Avira and BitDefender return unrelated results.
Your Environment

Package Version used:  Not installed.
Operating System and version: Windows 10 Pro 21H2 19043.1237
Chocolatey version: 0.11.2
Install/uninstall gist: Not sure what this is asking.

@wotacs I appreciate you raising this off the back of the email discussion.

The 360 Total Security Essential application has been discontinued since 2019 and had also been removed from AU checking around the same time.
@wotacs
I have to wonder why you are asking about an application that has been discontinued for almost 2 years. Since people don't know history. Allow me please to link to this  page
This is how the page looked back in the day. The first paragraph lists the multi engine approach that 360 Total Security Essential used.
The references that are made about Avira and BitDefender not being on the current 360 Total Security page are correct, but both packages 360ts & 360tse had at one time used these additional scanning databases in their applications. The 360 Total Security application has undergone recent updates and removed its ability to use the BitDefender and Avira databases. The description for the 360ts package should probably be updated to mention this change. Please, feel free to submit a PR if you feel so inclined.
Thank you for the feedback.

I raised this issue because the product is so out-of-date and yet it comes up in a search for two other well-known and current AV products. It may be time to totally remove it from the repository, or mark it prominently as Out-Of-Date.
From: RB @.>
Sent: Tuesday, 28 September 2021 05:42
To: chocolatey-community/chocolatey-coreteampackages @.>
Cc: wotacs @.>; Mention @.>
Subject: Re: [chocolatey-community/chocolatey-coreteampackages] 360tse Version <IP_ADDRESS>9 advertises connection with BitDefender and Avira. (#1720)
The 360 Total Security Essential application has been discontinued since 2019 and had also been removed from AU checking around the same time.
@wotacs https://github.com/wotacs
I have to wonder why you are asking about an application that has been discontinued for almost 2 years. Since people don't know history. Allow me please to link to this page https://web.archive.org/web/20190529084331/http:/www.360totalsecurity.com/en/features/360-total-security-essential/
This is how the page looked back in the day. The first paragraph lists the multi engine approach that 360 Total Security Essential used.
The references that are made about Avira and BitDefender not being on the current 360 Total Security page are correct, but both packages 360ts & 360tse had at one time used these additional scanning databases in their applications. The 360 Total Security application has undergone recent updates and removed its ability to use the BitDefender and Avira databases. The description for the 360ts package should probably be updated to mention this change. Please, feel free to submit a PR if you feel so inclined.
Thank you for the feedback.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/chocolatey-community/chocolatey-coreteampackages/issues/1720#issuecomment-928212421 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AVS42ZPUEERAOD7LUJJXGDDUEDCJ7ANCNFSM5E2UGLIA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/AVS42ZIJPWCQGFDSNNGBP43UEDCJ7A5CNFSM5E2UGLIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOG5JWLRI.gif

Please, @wotacs next time edit your replies for charity.
The process for updating a package as outdated is mentioned on the package page. This usually requires submitting a PR that changes the metadata to outdated/discontinued.
@majkinetor or @pauby can help you with this process. I will second the vote to move the 360tse package as outdated.
Thanks for the reply.

I apologise for any confusion caused, @RedBaron2, This is my first experience with GitHub Issue Tracking. I'd be happy to fix any mistakes made, if I could get a better idea where I have erred.  My uneducated assumption would be that I should have pruned the incoming email text - happy to go back in this email chain and do that, if this is my main failure.
I'll ask a bit of leeway while I learn (as fast as I can) how to interact here in the best possible manner.

If the final release of the package is on the community repository then my suggestion would be to submit one last version, mark it as EOL, change the description to remove the lines that @wotacs mentioned and also to update the description to confirm it's EOL.
If the package has been deprecated, then follow that process instead.

As there has been no update on this issue and nobody has picked it up, I'm going to go ahead and close it. We can always re-open it if necessary.


Solution to #9 "Find the largest element of the array"

#9 Find the largest element of array

@SamriddhiMishra
Hey I think the solution is intended to be added in the existing files under the basic directory of the application. Like for this issue, you've to make change in the function in the basic/easy.py file's largest_of_three function on line 59.
Happy Hacktober!

Tha


Report: current state of the guide

The Wagtail user guide documentation is the primary source of information for editors, moderators, and administrators of Wagtail websites. These roles don't require any prior technical knowledge. Due to this, the user guide documentation must be simple, accurate, complete, and easy to navigate.
The issue of structure
The structure of the user guide documentation follows the principles encouraged by the Diataxis framework. While this is a step in the right direction, strict adherence to the principles raises some concerns.
The writers of the user guide documentation should see the Diataxis framework as a guide with flexible principles. The writers should apply the principles based on the subjective needs and purpose of the documentation. For instance, the audience and purpose of API documentation are different from those of programming language documentation. The documentation should reflect these differences, however small they may be.
In the case of the Wagtail user guide documentation, it's safer to assume that most of the users of the user guide documentation may not have any technical background. Hence, the writers of the user guide should give priority to the arrangement of the main sections and also the words used as their titles. For instance, it would be a great idea to replace the title Tutorial with Getting started and Explanation with Concepts.
It would also be great to have the Concepts section before the explanation section. This way, new users of the documentation can get used to the technical language used to describe the features of the Admin interface before learning how to use them.
The issue of inconsistency with the writing style
The existing content of the user guide is great and easy to understand. However, they seem to be scattered across the pages.
One way to improve this is to adopt and implement a writing style to bring about consistency in the tone, language, and grammar of the user guide documentation.
The issue of undocumented Wagtail features
The user guide documentation needs to include the new features added to the Admin interface. This is essential in promoting Wagtail's capabilities.


Saving an empty setting results in errors

https://github.com/area17/twill/blob/a7fd4573fcae840980b53977a0573576da0e1ce6/src/Repositories/SettingRepository.php#L49
The ->filter() portion of the above line results in the $settingsTranslated variable not being set when I save a settings page with one setting and I try to leave the setting blank. This results in a php "undefined variable" error.
Removeing the ->filter() fixes it but not sure if this is the best solution

This should be fixed in 1.2.2, thanks @ferpetrelli !


Add placements into verilog json file

@parijatm @soneryaldiz This should do it. It was a little more complex than I thought. I had to convert from a flat view to a hierarchical view. Currently a JSON view of the stitched verilog to printed to stdout.

@parijatm Improved this some. I now generate the 'leaves' list in placement_verilog_json that includes bounding boxes of the leaf cells. The intermediate hierarchy bounding box are computed correctly now, and the visualization (-rp) uses this json file as its only input.

@parijatm It also seems that placements are messed up for n>1 , which should give you an opportunity to use your checking scheme to show this.

Fixed the overlap issue for n>1 on cascode_current_mirror_ota. It was a problem on my end.


HDDS-6497. S3gateway throw exception when checking jmx or prom pages from browser

What changes were proposed in this pull request?
When checking the JMX or Prometheus or other pages of S3g from browser, the following error was throw from S3g. (From docker container, might be different from local service)
2022-03-23 15:32:57,281 [qtp124888672-18] INFO ozone.OmUtils: ozone.om.internal.service.id is not defined, falling back to ozone.om.service.ids to find serviceID for OzoneManager if it is HA enabled cluster
2022-03-23 15:32:57,282 [qtp124888672-18] INFO ozone.OmUtils: No OzoneManager ServiceID configured.
Mar 23, 2022 3:32:57 PM org.glassfish.jersey.internal.Errors logErrors
WARNING: The following warnings have been detected: WARNING: Unknown HK2 failure detected:
MultiException stack 1 of 1
javax.ws.rs.WebApplicationException: User doesn't have the right to access this resource.

The reason for this issue is when trying to send request from browser, the browser will send another request like "http://localhost:9878/favicon.ico", which would be treated as a request to bucket of "favicon.ico", as a result a new OzoneClient was to inject for this request.
This ticket is to fix this issue.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6497
How was this patch tested?
Manually test: Error logs disappears when calling "http://localhost:9878/jmx".

@adoroszlai @elek Could you help to check this PR?

@adoroszlai Thanks for the review.
I updated the commit a bit to return an actual icon, could you help to check?


Add \smash, laps, spaces, and phantoms

Support \smash, including the optional argument from AMS.

Change \llap and \rlap so that they render in text style. Repeat: This changes KaTeX behavior.

Add \mathllap and \mathrlap. These will act as they do  in mathtools and as in previous KaTeX versions of \llap and \rlap.

Add \mathclap and \clap.

Add \hphantom and \vphantom.

Add \thinspace, \medspace, \thickspace

Add \hspace.

This work will resolve issue #270 and parts of #50 and #164.
A. Perlis has written a concise description of items 1 thru 5. Except for \smash's optional argument. It's described in the AMS User's Guide.
Item 6 also comes from the AMS User's Guide.

My apologies for the disconnectedness in this PR. And @edemaine thank you the help in getting restarted.
@kohler I haven't yet looked into your comment about calling makeVList from groupTypes.smash. I'll do that later this evening.

@ronkok I've updated the \llap, \clap, and \rlap to use textrm instead of mathrm.  The unit tests are passing now.  I'll regenerate the Lap screenshots.

@kevinbarabash, It just occurred to me that there is another breaking change in release 0.9.0-alpha, one that is not yet listed in the release notes.
The \llap and \rlap functions now render in text style, which follows LaTeX behavior. Also, KaTeX now has functions \mathllap and \mathrlap. These will act as they do  in mathtools and as in previous KaTeX versions of \llap and \rlap.
My current draft of the function support page has been so updated. But I did not emphasize the change as much as I did when \color changed. If you want that draft revised, let me know.

Good call.  I can up date the release notes.

I've updated the release notes.


I found a problem, that is, the download progress has a large span, 0->30->80->100. If it increases from 0-100 at a constant speed, the experience will be much better.

I found a problem, that is, the download progress has a large span, 0->30->80->100. If it increases from 0-100 at a constant speed, the experience will be much better.

Hello @xiongzhaoxu
Unfortunately, I can't play with the download progress speed, It depends on the network speed for the user, server speed, etc..
also the DownloadManager which is currently used in the native code, only gives total size & downloaded size
I only attach a listener Broadcast to the DownloadManager instance and it fires the progress event only when it gets a chunk of the file, this chunk might differ during the download progress, specially if the server is facing heavy traffic or your connection is not stable
but I guess that the gap between progress steps you are facing as follow:
0 -> 30 because at first, you need to establish the request itself to the server
30 - 80 -> here, you got the request established and the stream is active, so a bigger chuck of the file has been received
80 -> 100 this is not considered as a big jump unless the file is very large

If you have any questions or notes please feel free to reopen this issue or file a new one


Mxevent optimisation

Improved processing of HS requests responses that contain matrix events.
The improvement mainly consists in removing the Mantle modelling which appears to be slow on arrays with thousands of items.
Even if such processing is done on a separated thread, not on the UI one, it still worths to save CPU time
Here is the result of the improvement on a /initialSync response processing:

before: 2.4s
![before] (https://matrix.org/_matrix/media/v1/download/matrix.org/aBuYmahzTwEtMYwQhpbKdaHM)

after: 0.2s
![after] (https://matrix.org/_matrix/media/v1/download/matrix.org/StcNllSHowSkCLczMeTfLUUT)

lgtm


Track ETH deposits on Fuel V1

The Fuel V1 adapter doesn't track the ETH deposited into Fuel, which is around 0.25 ETH

@krzkaczor @antooni I still don't see it updated on L2Beat.com, is there a delay until the data gets ingested?

Yes, there is a delay. When the staging backend will be promoted to production, then updated ETH balance will be visible.


Add documentation on Terraform recipe providers configuration

What content needs to be created or modified?
Terraform recipe tab in https://docs.radapp.io/guides/recipes/howto-author-recipes/
Describe the solution you'd like
For Terraform recipes Radius uses required_providers block from Terraform module to determine if configuration for AWS/Azure should be added. However this is not a requirement enforced by Terraform so we should explicitly document this requirement for specific providers to be configured by Radius.
This issue in the Radius repo has more context on the confusion caused by this: https://github.com/radius-project/radius/issues/7109
Where should the new material be placed?
Terraform recipe tab in https://docs.radapp.io/guides/recipes/howto-author-recipes/
The associated pull request from the Radius code repo, if applicable
Not a PR, but we have an open issue: https://github.com/radius-project/radius/issues/7109
AB#11126

Thanks for taking this up @jasonviviano!

PR merged


3.3.0 Release Prep

Coverage remained the same at 97.253% when pulling 78367c906c041dc8b7ba5a4bc3590abec059f564 on trevor-vaughan:3.3.0_release_prep into ae9f1e14ee488adb282cacc4de0e35c2325868a3 on hercules-team:master.


[macOS] Kitematic destroys existing vm named "default" managed by docker-machine when using other driver than virtualbox.

Expected behavior
Kitematic use existing vm (created via docker-machine) named "default".
Actual behavior
Kitematic removes/destroys the existing vm named default when using other driver than virtualbox.
Information about the Issue
This only happens when using other docker-machine drivers like xhyve driver.
docker version:
$ docker version
Client:
 Version:      17.05.0-ce
 API version:  1.29
 Go version:   go1.8.1
 Git commit:   89658be
 Built:
 OS/Arch:      darwin/amd64

Server:
 Version:      17.05.0-ce
 API version:  1.29 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   89658be
 Built:        Thu May  4 21:43:09 2017
 OS/Arch:      linux/amd64
 Experimental: false

docker-machine version:
$ docker-machine version
docker-machine version 0.11.0, build 5b27455

docker-machine-driver-xhyve version:
$ brew info docker-machine-driver-xhyve
docker-machine-driver-xhyve: stable 0.3.2 (bottled), HEAD

kitematic version: 0.17.0
Kitematic checks whether a virtualbox vm exists named "default and since I'm using the xhyve driver, the default vm managed by docker-machine gets destroyed. Kitematic should not use virtualbox directly when probing existing vms but rather ask docker-machine.
The logic for destroying/creating vms when starting happens here.
Steps to reproduce the behavior

Install docker, docker-machine via brew

$ brew install docker docker-machine

Install xhyve driver for docker-machine by following instructions

$ brew install docker-machine-driver-xhyve

# docker-machine-driver-xhyve need root owner and uid
$ sudo chown root:wheel $(brew --prefix)/opt/docker-machine-driver-xhyve/bin/docker-machine-driver-xhyve
$ sudo chmod u+s $(brew --prefix)/opt/docker-machine-driver-xhyve/bin/docker-machine-driver-xhyve

Create default machine

$ docker-machine create --driver xhyve default

docker-machine reports a vm named default using driver xhyve

$ docker-machine ls
NAME      ACTIVE   DRIVER   STATE     URL                       SWARM   DOCKER        ERRORS
default   -        xhyve    Running   tcp://<IP_ADDRESS>:2376           v17.05.0-ce

Install kitematic via brew cask

$ brew cask install kitematic

Start kitematic
docker-machine reports a vm named default using driver virtualbox

$ docker-machine ls
NAME      ACTIVE   DRIVER       STATE     URL                         SWARM   DOCKER        ERRORS
default   -        virtualbox   Running   tcp://<IP_ADDRESS>:2376           v17.05.0-ce

It's been documented that Kitematic will use/create a VM called default - I would suggest naming your other vm something else than default.
For example:
docker-machine create --driver xhyve xdefault will solve all of your issues


chore(webpack): add jest-cli dependencies

fixes #825

 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.Andrew Camilleri seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

thanks!


Fix Directory make_dir and make_dir_recursive

Fix #40509
Thanks @Relintai for the report

To make sure there aren't any more issues with this, here is a list of all the Directory functions which require the Directory to be open
list_dir_begin
get_next
current_is_dir
list_dir_end
get_drive_count
get_drive
get_current_drive
get_current_dir
get_space_left
* copy
rename
* remove
* - Maybe Remove This Requirement?

Needs a rebase on current master to fix CI.

copy
remove

Given #40146, makes sense to require a directory to be opened explicitly first before remove (everything which modifies the filesystem should just require this then).

Thanks!

Cherry-picked for 3.2.3.


Review new security addition to domains process

From: Adrian Warman
Links: https://security-guidance.service.justice.gov.uk/domain-names-policy/ https://github.com/ministryofjustice/security-guidance/blob/verification0406/docs/domain-names-policy.md#domain-operations-operations-engineering-team
Good morning
Over in security policy, we're tackling a modest update to the information on domain names. The current published material is here. The proposed update is to add an extra policy statement (POL.DOM.010-220406 ) saying that a first step in a domain name request is to seek security approval (to make sure that there are no security issues with proposed domain), and then pass to OpsEng for technical approval and implementation. You can see a draft of the update here.
The overall intent is to make sure that 3rd parties don't try and set up a domain that would or might cause a security problem.
More than happy to discuss and revisit (ie change if required) the proposed update accordingly.

@AntonyBishop Needs refinement just copied their message over

Ben Ashton
8 minutes ago
Morning Adrian - will add a ticket to our board to pass on feedback to yourself/update runbooks where relevant.
Would we be given any freedom or does this include very simple record additions e.g. a new record for a preapproved top level domain/zone that very obviously fits standards/has existing records that are very similar?
Adrian Warman
2 minutes ago
Thank you
@Ben Ashton
:slightly_smiling_face:
We definitely don't want to get in the way of things, and certainly not cause problems for BAU. This is aimed much more at new domains, especially (primarily) those requested by/for 3rd parties. Perhaps if you could identify some scenarios that we could explore?

Related to https://github.com/ministryofjustice/security-guidance/issues/331

Awaiting security response.


Update data-api version to 0.0.25

Automatic PR raised by GitHub Actions to update data-api version to 0.0.25

Already up to date in main


Sync actions documentation

Issues:

It is not clear that sync action should return json in specific format to respond with "success"
it is not clear that status can be one of "success" or "ok"

https://github.com/keboola/kbc-ui/blob/a61f9c89dae82155ac35fef6958607a1a68c0e37/src/scripts/react/common/TestCredentialsButtonGroup.coffee#L69
sample (encoded to json in RunCommand):
https://github.com/keboola/dynamodb-extractor/blob/8cacf3db2ffdd38c9d9f7cd8a90940b4fa29bb77/src/Keboola/DynamoDbExtractor/Extractor.php#L52
cc: @Actimel

this is UI thing, nothing to do with component development, component can return anything

bych to zavrel @ujovlado

Ok, got it. But components are working with our UI ... it I create component which is not "compliant" with UI (not returning proper structure) it will not work.

if I create component which is not "compliant" with UI ... it will not work.

it will not work either way, because currently, the UI has to be enabled for that component manually

Yep, I agree ... also there's process for approving in devportal-ui. ok then. :+1:


Error: dlopen failed: library "libstdc++.so.6" not found

Tried out the node js for mobile samples, specifically, use native module with node folder. Built the project in the Android studio on mac machine by following the steps, and add the sqlite3 reference in the main.js. The sqlite3 is downloaded from npm. The app failed with the error  Error: dlopen failed: library "libstdc++.so.6" not found.
How to resolve the issue?

Hi @dannygt2003 ,
Here's the response from gitter.
Modules containing native code are supported by default in the plugins, but outside the plugins there's not set method to build the native modules for the target platforms. The method from the plugins is not supported on Windows 10:  https://code.janeasystems.com/nodejs-mobile/faq#are-nodejs-native-modules-supported
There's some discussion for WSL in: https://github.com/JaneaSystems/nodejs-mobile/issues/4
There are samples which use sqlite3 in one of the plugins in: https://github.com/JaneaSystems/nodejs-mobile-samples/tree/651937e419bf1c82fda1f06e8f9c828d5df86fbc/cordova/UseNativeModules and https://github.com/JaneaSystems/nodejs-mobile-samples/tree/651937e419bf1c82fda1f06e8f9c828d5df86fbc/react-native/UseNativeModules


Convert Spaces to Tabs for repo

@oblodgett You may want to ensure to use tabs for indentation, but space for (positional) alignments, otherwise your alignments get skewed when changing the tab size (different developers will use a different default tab size in their IDEs, terminals, ...)

With this change we will need to get rid of the tabspace and tabspace2 in our ~/.gitconfig files otherwise git will revert this change.


react-router v6 upgrade

Description
Migration to react-router v6. Necessary for future upgrades to React18 as react-router v5 is incompatible with React18.
Summary of major changes:

partial migration to react-router v6.4 loader APIs (i.e. createBrowserRouter)

hoisted some routes to the top level of the application
refer to comments left in routerConfig.tsx

refactor routing and routes to v6
deprecate usage of history object, which is incompatible with v6 where router handles the navigation (abstraction over history and loader APIs)
refactor usage of <Prompt /> component, which was deprecated
update miscellaneous APIs to v6 (e.g. Navlink active styling, etc.)

Other changes:

Removed legacy link redirect from SA Rook (/interactive-sicp/ → /sicpjs/)
Refactored some components from class-based to functional components

NOTE: Pending updating of tests
Type of change

Dependency update

How to test

manual testing of all routes to ensure that the new routing works, including all navigation buttons

in all 3 deployments (disabled, playground only, full academy)

Checklist

[ ] I have tested this code
[ ] I have updated the documentation

Thanks for working on this! Be sure to test against #2337, we don't want to repeat what happened there. Also relevant: #2348.

Note that I have not tested against this as well, as no one is (heavily) using Source Academy right now, I think it's safe to merge first.


Compile failed

Please answer these questions before submitting your issue.

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

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?
COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /usr/local/src/java/skywalking/apm-protocol/apm-network/src/main/java/org/apache/skywalking/apm/network/trace/component/command/BaseCommand.java:[21,1] package org.apache.skywalking.apm.network.common does not exist
[ERROR] /usr/local/src/java/skywalking/apm-protocol/apm-network/src/main/java/org/apache/skywalking/apm/network/trace/component/command/BaseCommand.java:[30,26] package Command does not exist
[ERROR] /usr/local/src/java/skywalking/apm-protocol/apm-network/src/main/java/org/apache/skywalking/apm/network/trace/component/command/BaseCommand.java:[45,12] package Command does not exist
[ERROR] /usr/local/src/java/skywalking/apm-protocol/apm-network/src/main/java/org/apache/skywalking/apm/network/trace/component/command/CommandDeserializer.java:[20,48] package org.apache.skywalking.apm.network.common does not exist

Which company or project?

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

Requirement or improvement

Please describe about your requirements or improvement suggestions.
I hope to be able to compile by sourecode in ubuntu. Could you help solve this problem?

The compiling doc is required. Read it and follow it.

I also made this mistake
My environment: jdk 1.8.0_291, maven 3.8.3, os: windows
my step:
step1: git clone -b v7.0.0 --recurse-submodules https://github.com/apache/skywalking.git
step2: mvnw package -Pagent，dist


Error: "Admin channel unreachable" after update to 2.0.17

Describe the bug
I had been running Accessbot in ECS, and it was working well in a channel named "#demobot". A colleague reported that it stopped responding to access commands, but it does respond to status and plugin commands.
I checked uptime and saw that it was only up for a few minutes, so I checked version. It seems that ECS automatically grabbed the new version, 2.0.17.
To Reproduce
Deploy 2.0.17 using Slack Bolt, and specify an admin channel.
Expected behavior
Access command should work.
Actual behavior
They give error below.
Screenshots

Attachments
Config:
demobot config.txt
Logs:
demobog logs.txt

I tested here and it worked like a charm. Please verify your plugin config and test again. I'm telling this because the syntax of the var ADMINS_CHANNEL looks wrong and needs to follow the # + name of the channel model, e.g. #privatechannel or #admins-channel.

Thanks for the update. I just reverted to 2.0.16 and it's working again, with the same channel configured:
           {
            "name" : "SDM_ADMINS_CHANNEL",
            "value" : "#demobot"
          },

I did add a second admin with this update, too, but I wouldn't think that should have an effect.
Some other bit of interest: this is running in ECS, and I'm grabbing the Slack and SDM tokens from AWS Secrets Manager.

@wrenhunter you're right! It was a bug that came with a feature that made some changes in webclient channels fetch. We fixed it and made PR's, we're just waiting to merge. Thank's for the issue!

Thank you very much!
On Fri, Jan 28, 2022 at 1:05 PM Wallisson Rony @.***>
wrote:

@wrenhunter https://github.com/wrenhunter you're right! It was a bug
that came with a feature that made some changes in webclient channels
fetch. We fixed it and made PR's, we're just waiting to merge. Thank's for
the issue!
—
Reply to this email directly, view it on GitHub
https://github.com/strongdm/accessbot/issues/199#issuecomment-1024479559,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOU6XYE7MR4LT5X7B6V4A6LUYLLHNANCNFSM5NBITRFA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
David Petersen
Customer Engineer
StrongDM

Thank you very much team, we hope to close this bug tomorrow ASAP. We'll keep you informed via this issue.


Fix service selector in PGVector deployment

Service selector does not match the deployment's app label so the service is not able to find the postgres pod.

Thanks a lot!


Roll Flutter Engine from 2560e1af6fbd to 847a6d06d0d1 (1 revision)

https://github.com/flutter/engine/compare/2560e1af6fbd...847a6d06d0d1
2023-05-14<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from hXcPXU_V5nVeGkEbt... to qnNU5eQaox06ejpSE... (flutter/engine#42023)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from hXcPXU_V5nVe to qnNU5eQaox06
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Use an array for path matching?

I was unsure wether to file this as an issue or a feature request. Currently the API proposes that the path attribute must be a String object. This is quiet hindering, when you think of the following case:
    { path: '/participant', component: ParticipantFilter, name: 'participant' },
    { path: '/participant/(.*)', component: ParticipantFilter, name: 'participant' },

This does produce a lot of DRY code
Named routes are bugging this way

It would be more elegant with:
    { path: ['/participant', '/participant/(.*)'], component: ParticipantFilter, name: 'participant' },

Are there any plans to bring Array path attributes?

Can you explain the use case for this, in comparison to an alias?

Just use an alias (it also supports arrays)


Need to know which OpenSSL version supports TLS 2.0 and build procedure for building it on VS 2010  x64bit windows 2008 R2

Hello,
I have two quetions

I have built my application with OpenSSL 1.0.1.g but its read me file says it supports TLS v1.
I need to know which version of OpenSSL supports TLS 2.0, so I can take code from Git hub and built?
what are the steps for building that code with Microsoft Visual Studio 2010 X 64 bit on Windows Servers 2008 R2 Operating system?
is their configuration to make openSSL to use TLS instead of SSL?

I have two quetions

That's three ;-)

I have built my application with OpenSSL 1.0.1.g but its read me file says it supports TLS v1.
I need to know which version of OpenSSL supports TLS 2.0, so I can take code from Git hub and built?

There is no such thing as TLS 2.0. SSL/TLS versions in order of oldest to newest are:
SSLv2
SSLv3
TLS1.0
TLS1.1
TLS1.2
TLS1.3 (still in development - spec not yet finalised)
OpenSSL 1.0.1g supports all of the above except TLS1.3. However the 1.0.1 series is now out of support and no longer receiving security updates. Unless you got your version of OpenSSL from an O/S vendor (in which case the O/S vendor will be supporting it), you should upgrade to a newer version.

what are the steps for building that code with Microsoft Visual Studio 2010 X 64 bit on Windows Servers 2008 R2 Operating system?

Windows build instructions are included in the INSTALL.W64 file in 1.0.1 and 1.0.2. If you upgrade to 1.1.0 then instructions are in INSTALL.

is their configuration to make openSSL to use TLS instead of SSL?

OpenSSL will use TLS by default wherever possible.

I have already built my application with OpenSSL 1.0.1g and it connects to the web service with TLS 1.2 and our initial communication handshake still fails and there is no response from web service ? what could be wrong?
where to find INSTALL.W64 this file? on git hub or OpenSSL.org site? on git hub what is directory hierarchy?

I have already built my application with OpenSSL 1.0.1g and it connects to the web service with TLS 1.2 and our initial communication handshake still fails and there is no response from web service ? what could be wrong?

It could be any number of things. You could try using the openssl s_client application to connect to your server to see if that succeeds or not. If you can't figure it out then you should send a question to the openssl-users email list (details at https://mta.openssl.org). Provide as much information as you can, e.g. what you were trying to do, any error messages that you saw, what the output from s_client was etc.

where to find INSTALL.W64 this file? on git hub or OpenSSL.org site? on git hub what is directory hierarchy?

It's in the top level directory of the download. The 1.0.1 version is also viewable here:
https://github.com/openssl/openssl/blob/OpenSSL_1_0_1-stable/INSTALL.W64

Hello mattcaswell,
Thanks for quick answer to my previous questions.
We are connecting to one web service by extracting SSL client  certificate  using OpenSSL client but it only shows SSL certificate information in debug logs. There is no way to identify which protocol we are using to connect to webservice SSL or TLS?  is there a way to know which protocol is being used to connect to web service?

Please go to the mailing list as Matt said.

Hey rochsalz,
When I tried to subscribe to openssl-users mailing list, I dint receive any mail to confirm.
Not sure how to get information needed by me?

email<EMAIL_ADDRESS>and say you want to join


[PE-D][Tester C] Incorrect Example in UG

For the following example to add a patient, I copy pasted it into the app. However, an INVALID COMMAND FORMAT error message appeared. It seems like the example is wrong.

I believe that it could be because the emergency contact ec/ parameter is missing in the example!

Labels: severity.Low type.DocumentationBug
original: maypfv/ped#2

duplicate of #195


Adding support for normalcy/cancun flags

Description
Adding some support to run with the normalcy flag. This should align somewhat with the goals of testing the pessimistic proof setup as well.
Currently I'm testing with:
kurtosis run --enclave cdk . '{"args": {"enable_normalcy": true, "cdk_erigon_node_image": "hermeznetwork/cdk-erigon:v2.60.0-beta2", "erigon_strict_mode": false}}'

fyi @Nashtare


Not able to deploy function with .zip archive

I do understand that the issue is not related to the functions-framework directly, but maybe someone from the team can reach the GCF team and ask them to check it out.
So, the issue is the following. I'm not able to deploy a GCF source as .zip archive via Cloud Console UI or using terraform. But if I unpack the function and go with gcloud functions deploy it works just fine. Here is the error message I got both from Cloud Console UI and from terraform:
Error: Error waiting for Creating CloudFunctions Function: Error code 3, message: Build failed: {"cacheStats": [{"status": "MISS", "hash": "8e51312351ee02e20ebdcdcefb100908ee770a1c81810601ddfcd599221c617b", "type": "docker_layer_cac
he", "level": "global"}, {"status": "MISS", "hash": "8e51312351ee02e20ebdcdcefb100908ee770a1c81810601ddfcd599221c617b", "type": "docker_layer_cache", "level": "project"}]}

Does anyone have some ideas about what could be wrong? Unfortunately, I cannot share to source code as it is under the NDA. I'll try to create a reproducible example and share it.
Meanwhile, I have also created issue in the public tracker.

Hi @yuri-sergiichuk, the public tracker is the right place for this issue. I'll make sure the team sees it.


PathCalc for removing file extensions from webpages

It would be useful to have a PathCalc that turned paths like projects/anchovy.html into projects/anchovy/index.html. This would then effectively hide the .html from the URL, while otherwise maintaining the same structure.

Would benefit from #63 but does not require it.


[7.x] Add new fromString constructor to collections

This allows you to lazily iterate over the characters in a string:
foreach (LazyCollection::fromString($hugeString) as $character) {
    //
}

I don't think we merge this without the contract changes (which you reverted). I was more meaning that we'd have to move this to master.

I don't think we merge this without the contract changes (which you reverted).

Totally disagree. It's very useful as is. No need to wait.

I was more meaning that we'd have to move this to master.

After this is merged, I'll sent a PR to master adding it to the contract :+1:

It is possible we can make a BC exemption since this is so early into the 7.0 release series. We could maybe treat it as a bug fix?

Up to @taylorotwell.

I'm gonna be honest and I know you'll probably ping me about this but I don't think this is a very common use case. Macro it in maybe?


Macro F1 score depends on the size of the minibatch

I find our current implementation of the macro F1 score is dependent to the size of the minibatch. Our strategy is to average the F1 scores of each minibatch. However, the result would be different if we choose different batch size and order of the samples (https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/metric.py#L623-L626). This will certainly confuse the users. I suggest marking the macro F1 as "Not Recommended" and set micro F1 as the default. @sethah @szha

Fixing this requires api change, so we should do that in 2.0. #9686


feat(components): typography

Draft for design review, missing component tests

@fampinheiro can we merge a separate PR with the certificate fix to master?

Ready when https://github.com/yldio/asap-hub/pull/46 is merged and this is rebased


How to define \pmod?

Hello,
Could anybody please tell me how to define the function \pmod?
\pmod a is displayed as (mod a)
Thank you in advance.

It doesn't seem like our current function support has a good way to do this. You could add a function in src/functions.js which returns a new type entirely (like pmod) and then handle it in both src/buildHTML.js and src/buildMathML.js.
However, \pmod sounds like something that would be easily solved using macros. We don't support those yet, however. See #250

@xymostech :+1: Thanks!


[1.0.0 Roadmap] Remove all SQL state from the library (and schema).

The library should not care anything about what the provider is using in the background. This also allows for each sql provider to be more customized to that individual database, rather than trying to find a set of sql syntax/types common to all sql databases. As well as reduction of core code (bloat) and finally massive simplification of SettingGateway.
Potentially add a SQL QueryBuilder Util with overridable defaults, to aid in reduction of duplicate code/making SQL Providers more maintainable.

There is already a pr for this, this doesn't need an issue made.


Add link on subsequent post messages as well

Hi there,
Is there a way to put the post link into every Slack message rather than just the first post on a topic?
It seems it's hard-coded that way, would you accept a PR changing it?

That is a bug that should be fixed in the latest version! Are you self
hosting? If so, can you go to /admin/upgrade and tell me what version you
see?
On Tue, Dec 13, 2016, 9:04 PM Bradley Priest<EMAIL_ADDRESS>wrote:

Hi there,
Is there a way to put the post link into every Slack message rather than
just the first post on a topic?
It seems it's hard-coded that way, would you accept a PR changing it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/discourse/discourse-slack-official/issues/9, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABUno_D7pjSvli99RIuQFYFudEgWizwwks5rH008gaJpZM4LMcoE
.

Hi Nick,
We have a hosted Discourse instance, and the plugin is v1.0.0, but the settings link actually links to your fork. https://github.com/nicksahler/discourse-slack-official

This has been fixed in
https://github.com/discourse/discourse-slack-official/commit/d1d2d1c9966f339a33d543c5255a6d505da9878d

but the settings link actually links to your fork. https://github.com/nicksahler/discourse-slack-official

Oops that was a mistake. Fixed in
https://github.com/discourse/discourse-slack-official/commit/b148f4b7d489da521c306a61c2848ad502811ac7


submit_batch_jobs.sh not working in GPU environment

I am working on deploying GPU jobs with submit_batch_jobs.sh in SNANA, but in the GPU environment submit_batch_jobs.sh is not working because of some import errors, for example ModuleNotFoundError: No module named 'coloredlogs'. I'm sure there will be more things down the line, but in the first instance could all the packages be added so that submit_batch_jobs.sh -h can run successfully?

Hi @mattgrayling
The GPU environment indeed has a smaller subset of packages than what we have in the CPU-only environment.  I am not sure what list of packages submit_batch_jobs.sh needs explicitly (though I can imagine coloredlogs might be one of them)  I can also ask Rick, but here are the full lists of what we install in the CPU environment:
the packages from conda-forge
and others from PyPI we pull in astrodash
Do you know which packages we really need?  I'm hesitating to put everything into the GPU-enabled environment for fear that there will be some inevitable conflicts.  Though we could try it and see what happens.


Add support for specifying remote tasks using resolvers

Tekton v1 has deprecated ClusterTasks and uses the resolver feature instead (documented at https://docs.openshift.com/pipelines/1.15/create/remote-pipelines-tasks-resolvers.html). cdk8s-pipelines should support references to remote tasks in pipelines using this new syntax

Closed by https://github.com/cloud-native-toolkit/cdk8s-pipelines/pull/51 and https://github.com/cloud-native-toolkit/cdk8s-pipelines/pull/53


Document first_non_empty attribute function

Noticed there's a first_non_empty attribute function that isn't mentioned in the docs, just making a note here.

Also the pre-commit validation isn't passing.

https://batch.openaddresses.io/job/353102 failed with:
ok - updating job: 353102 with {"status":"Running","version":"1.1.0","output":{"cache":false,"output":false,"preview":false,"validated":false},"loglink":"batch-prod-job/default/04db52cc23f0491aabf4e537f94ff5f9"}
[{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/0/oneOf/0/type","keyword":"type","params":{"type":"string"},"message":"must be string"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/0/oneOf/1/type","keyword":"type","params":{"type":"array"},"message":"must be array"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/0/oneOf","keyword":"oneOf","params":{"passingSchemas":null},"message":"must match exactly one schema in oneOf"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/1/required","keyword":"required","params":{"missingProperty":"field"},"message":"must have required property 'field'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/1/required","keyword":"required","params":{"missingProperty":"pattern"},"message":"must have required property 'pattern'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/1/additionalProperties","keyword":"additionalProperties","params":{"additionalProperty":"fields"},"message":"must NOT have additional properties"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/1/properties/function/enum","keyword":"enum","params":{"allowedValues":["regexp"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/2/required","keyword":"required","params":{"missingProperty":"field"},"message":"must have required property 'field'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/2/additionalProperties","keyword":"additionalProperties","params":{"additionalProperty":"fields"},"message":"must NOT have additional properties"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/2/properties/function/enum","keyword":"enum","params":{"allowedValues":["postfixed_street"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/3/required","keyword":"required","params":{"missingProperty":"field"},"message":"must have required property 'field'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/3/required","keyword":"required","params":{"missingProperty":"field_to_remove"},"message":"must have required property 'field_to_remove'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/3/additionalProperties","keyword":"additionalProperties","params":{"additionalProperty":"fields"},"message":"must NOT have additional properties"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/3/properties/function/enum","keyword":"enum","params":{"allowedValues":["remove_prefix"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/4/required","keyword":"required","params":{"missingProperty":"field"},"message":"must have required property 'field'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/4/required","keyword":"required","params":{"missingProperty":"field_to_remove"},"message":"must have required property 'field_to_remove'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/4/additionalProperties","keyword":"additionalProperties","params":{"additionalProperty":"fields"},"message":"must NOT have additional properties"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/4/properties/function/enum","keyword":"enum","params":{"allowedValues":["remove_postfix"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/5/properties/function/enum","keyword":"enum","params":{"allowedValues":["join"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/6/required","keyword":"required","params":{"missingProperty":"format"},"message":"must have required property 'format'"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/6/properties/function/enum","keyword":"enum","params":{"allowedValues":["format"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/7/required","keyword":"required","params":{"missingProperty":"variable"},"message":"must have required property 'variable'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/7/required","keyword":"required","params":{"missingProperty":"functions"},"message":"must have required property 'functions'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/7/additionalProperties","keyword":"additionalProperties","params":{"additionalProperty":"fields"},"message":"must NOT have additional properties"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/7/properties/function/enum","keyword":"enum","params":{"allowedValues":["chain"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/8/required","keyword":"required","params":{"missingProperty":"field"},"message":"must have required property 'field'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/8/required","keyword":"required","params":{"missingProperty":"index"},"message":"must have required property 'index'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/8/additionalProperties","keyword":"additionalProperties","params":{"additionalProperty":"fields"},"message":"must NOT have additional properties"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/8/properties/function/enum","keyword":"enum","params":{"allowedValues":["get"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/9/required","keyword":"required","params":{"missingProperty":"value"},"message":"must have required property 'value'"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/9/additionalProperties","keyword":"additionalProperties","params":{"additionalProperty":"fields"},"message":"must NOT have additional properties"},{"instancePath":"/layers/addresses/0/conform/street/function","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf/9/properties/function/enum","keyword":"enum","params":{"allowedValues":["static"]},"message":"must be equal to one of the allowed values"},{"instancePath":"/layers/addresses/0/conform/street","schemaPath":"#/properties/layers/properties/addresses/items/properties/conform/properties/street/oneOf","keyword":"oneOf","params":{"passingSchemas":null},"message":"must match exactly one schema in oneOf"}]
Error: Source does not conform to V2 schema
    at Job.fetch (file:///usr/local/src/batch/lib/job.js:87:19)
    at async flow (file:///usr/local/src/batch/task.js:83:9)
ok - updating job: 353102 with {"status":"Fail"}
false

This is documented and in the schema now.

The pre-commit validation succeeds, but batch is still failing, see: https://batch.openaddresses.io/run/2989

After adding it to the schema in this repo, we needed to (1) rev the package.json (https://github.com/openaddresses/openaddresses/commit/2a2dfb81ec8c39998904bcfae59fbf41f6251727) and (2) update the dependency in batch https://github.com/openaddresses/batch/pull/381. Those should be done now (thanks @ingalls!), so a rebuild of the PR should work.


Fix example cube fbo

This pull request fixed two issues with the cubemap framebuffer example:

In glsl reflect(I, N) should take an incident vector that is opposite to the
eyeDir (so -eyeDir) and a surface normal
More https://www.opengl.org/sdk/docs/man4/html/reflect.xhtml

Cubemap was rendered upside down because FBO 0,0 is at the bottom left while for cubemaps 0,0 at the top left. The solution is to flip the UP vector in our lookAt matrix while rendering to the FBO. More here http://www.nvidia.com/object/cube_map_ogl_tutorial.html and here http://www.mbroecker.com/project_dynamic_cubemaps.html

This change is

example/cube-fbo.js, line 26 at r1 (raw file):

const teapotFBO = regl.framebufferCube(CUBE_MAP_SIZE)

const CUBEMAP_SIDES = [

nice!  this is way better than that dumb switch statement

Comments from Reviewable

example/cube-fbo.js, line 126 at r1 (raw file):

  void main () {
    vec4 env = textureCube(envMap, reflect(-eyeDir, fragNormal));

:+1:

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable