Update the RTC-S1 SAS for the cal/val delivery

This PR updates the RTC-S1 SAS for the cal/val delivery:

Add coverage check for ancillary inputs (DEM);
Provide three mosaicking options: average, first, and bursts_center;
Reproject burst product before mosaicking if needed.

Minor updates:

Substitute metadata field ISCEVersion to isce3Version;
Update some comments, fix typos, indentation, and bugs;
Update dem_description to dem_file_description;
Update default compression algorithm from ZSTD to DEFLATE;
Separate runconfig burst and mosaic geogrids;
Update functions to compute burst and mosaic geogrids;
Update metadata field orbitType;
Remove requirement of save_nlooks = True;
Add test runconfig at tests/runconfigs/s1b_los_angeles.yaml and use it for workflow testing rather than using the runconfig  downloaded from zenodo (during workflow test).

@gshiroma This is a huge PR as you mentioned. Please give me the highlights about what feature(s) are added and changed. That will save a lot of time on my end.
You mentioned "Add coverage check for ancillary inputs (DEM)". One of things I remember is the mosaicking function. Anything else?

@gshiroma This is a huge PR as you mentioned. Please give me the highlights about what feature(s) are added and changed. That will save a lot of time on my end. You mentioned "Add coverage check for ancillary inputs (DEM)". One of things I remember is the mosaicking function. Anything else?

Just updated the PR description. Thank you, @seongsujeong

Here is my comments about the changes in core.py - Seongsu

Thank you, @seongsujeong ! Just addressed them.

Thank you for your review, @seongsujeong . I think I addressed all your comments/concerns so far. Please let me know if I missed anything or if you'd have any further comments.

I think this PR LGTM to merge. There is one last comment here about how to index the invalid pixels, but I'll leave it to you to decide. Thanks for your hard work @gshiroma!

Thank you, @seongsujeong . Just addressed that comment in this commit. Should I wait for you to merge your PR in this one?


support scanning and iterating with a range of vnodes using merge sort

Could I ask what's the conclusion on limiting the vnode to be some small factor * max parallelism?
If the scan result is large, we may have to buffer a lot of data for merge sort?


[BUG]Error loading other people gist

Describe the bug
Error loading other people gist after network error
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
The gist I own:

GistPad show load gist:

Desktop (please complete the following information):

VS Code Version: 1.47.3
GistPad Version: 0.1.8

Additional context
Add any other context about the problem here.

I'm not able to repro an issue here, so I'm going to close it. Please re-open this if you're still running into an issue. Thanks!


Break circular reference

These services could be marked as lazy, this change is solving problem with DI circular reference chain.
About lazy services
Before this change we was not able to inject \League\Tactician\CommandBus to our event listeners due to DI circular reference detection.
Exception message:
Circular reference detected for service "tactician.commandbus.default", path: "tactician.commandbus.default -> tactician_domain_events.middleware.release_recorded_events -> tactician_domain_events.dispatcher -> app.subscriber.domain_object_was_created". (Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException)

@borNfreee What do you think about this simple solution?

Thanks for your time and creating this PR @uro.
Could you please elaborate why do you need CommandBus inside EventListener or EventSubscriber? What is the use case?
I'm not against your proposal, just want to understand the problem you are trying to solve.

Thanks for your fast reply.
At this moment I'm trying to implement simple saga with tactician and your bundle. Till this moment it works great but now it doesn't allow me to execute commands from domain events.
A saga is basically an simple event handler that issues new commands from domain events.
It's also called as Process manager.
Theoretically we could think about eventListeners like about eventHandlers which are working in same way as commandHandlers, but then we are loosing tactician commandHandlers functionalities like middlewares etc.

I see your point, that you for the explanation.
Since users who don't have ocramius/proxy-manager installed won't be affected, I think it's a safe update.
Thank you for the contribution.

If you don't install the ProxyManager bridge and the ocramius/proxy-manager, the container will just skip over the lazy flag and simply instantiate the service as it would normally do.


Allow adding server metadata to the client leave message

The ClientJoin message currently has this capability, but the leave message doesn't.

Curious: What type of info you want to put?

We want to pass some user info from the frontend, all the way to the storage persist logic. Mainly for auditing purposes


homescreen widget development

I don't see a clean way of developing a flexible app alongside a homescreen widget.  Am I missing something?

I don't believe there's anything about the Cloud SDK which would prevent one from creating a homescreen widget in one's app.  Was there something in particular you ran into, some roadblock or something specific that's missing?

Closing this for now.  @bfiegel if you can provide the info requested, please feel free to re-open!


How can mmdetection obtain images with only masks（mmdetection 怎样可以获得只有掩码的图像）？

I have obtained a model using mmdetection training. Now I use it to predict images with only masks. How should I achieve this?

I have the same problem, have you solved it yet?（我也有同样的问题，请问你解决了吗）
Looking forward to your reply.

I also wish I could have extracted the mask separately, have you solved it? Looking forward to a reply

Maybe this blog will help.
https://blog.csdn.net/m0_46246301/article/details/130561039

I also wish I could have extracted the mask separately, have you solved it? Looking forward to a reply

Maybe this blog will help.
https://blog.csdn.net/m0_46246301/article/details/130561039


Discrete field operator overloads

Adds the remaining relevant operator overloads for nicer syntax in forms.

I'm not sure if addition and subtraction are really sensible operators unless the 'other' is also a DiscreteField made with the same quadrature.
Should I also add a test for these?

Should I also add a test for these?

Sure, at least to have an idea what kind of use cases you had in mind for these. Probably very useful when defining different @Functionals? I recall sometimes having to use an explicit u.value and this probably circumvents that in many cases.
Some calculations currently done inside the forms are dealt by __array__ which I think NumPy uses internally to transform an object into an array.

Yes that sounds like a good use case.
A test could simply be something like
@Functional
def sum(w):
    return w.x[0] + w.x[1]
sum.assemble(basis)

which should evaluate to the integral of f(x,y) = x + y. I think w.x[0] have the type DiscreteField.

I'll merge this, I can add some tests in #556.


Newest version clobbers window manager keyboard shortcuts

Issue Type: Bug
Windows manager ShiftIt no longer controls the window size/position after the most recent version update. Windows of other applications continue to work properly. Changing keyboard shortcuts appears to have no effect. Controlling directly through window manager menu not controlling either. Either shortcuts or menu based control works with common apps such as Chrome, Finder, Firefox, Outlook, Photoshop.
VS Code version: Code 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520, 2019-11-25T14:52:45.129Z)
OS version: Darwin x64 18.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz (4 x 3400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (1.88GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

/cc @deepak1556 can it be related to updated Electron?

@bjnsn can you test with 1.40.1 https://az764295.vo.msecnd.net/stable/8795a9889db74563ddd43eb0a897a2384129a619/VSCode-darwin-stable.zip to confirm if its a regression from electron update. Thanks!

@deepak1556 1.40.1 works correctly unlike 1.40.2.

Thanks for confirming!

@deepak1556 maybe same cause as https://github.com/microsoft/vscode/issues/84195 ?

@bpasero they are not same, #84195 has been since electron 6.0.0 upgrade and this only started with 1.40.2 and electron 6.1.5 so something in this bump broke.

@deepak1556 @bpasero should we close this, seems to be fixed?


The example project not working.

Great work, but please check this issue.
https://github.com/calebji123/WordleInIdris/issues/3
The example project WordleInIdris seems out-dated. It uses an old vscode extension, throwing error when opening in vscode devcontainer, I think it is not  "ready-to-use" anymore.

Hey @weihongliang233, thanks for letting me know! I added an example project in examples/ to replace the WordleInIdris project. Let me know if it works for you!

Yes I have tried this example. It works well on my machine! 🎉


#1517: display the actual Exception when a property check fails

The Exception being generated for this case had two distinct problems:

First, it wasn't recording the inner Exception as the cause of the outer Exception. So the -oS and -oF flags weren't showing the inner stack.
Second, the error message was hard-coded to use the outer Exception, and completely ignored the inner one. This needed some careful rewriting.

To make the problem more manageable, merged together the two (identical) clauses that were creating the error message, into a single common function. (In general, PropCheckAsserting desperately needs a refactor.)

Closing this because reworking it in this branch: https://github.com/scalatest/scalatest/tree/feature-implement-shrink-for-forall-with-one-arg


send minimap to pulsar-mel2 and slurm9

Due to queues for pulsar-mel3_big..
Up to 500Mb to pulsar-mel_mid, 500Mb to 5G to pulsar-mel_big, default dest slurm_9slots

it really depends what day it is whether this is a good choice.. next week maybe it will be slurm cluster that is overloaded


Insert extra symbol, error occurs.

Input code
KeyEvent {
                code: code @ (KeyCode::Up | KeyCode::Down | KeyCode::Left | KeyCode::Right | KeyCode::End | KeyCode::Home),
                modifiers: event::KeyModifiers::NONE,
            } => {
                self.output.move_cursor(code);
            }

Output code
KeyEvent {
                code: code @ (
                    | KeyCode::Up
                    | KeyCode::Down
                    | KeyCode::Left
                    | KeyCode::Right
                    | KeyCode::End
                    | KeyCode::Home
                ),
                modifiers: event::KeyModifiers::NONE,
            } => {
                self.output.move_cursor(code);
            }

Additional context
Insert extra symbols after format.

.

It looks like rust-analyzer's custom parser does not support leading | in patterns. I filed an issue here.
In the meantime I recommend removing the leading | and ignoring that code by placing // prettier-ignore above it.

This issue has been fixed in rust-analyzer.
From what I understand the fix should be part of next week's release, or tomorrow if you use rust-analyzer nightly.


Update kaiwei's Project Portfolio Page

Update kaiwei's Project Portfolio Page

Codecov Report

Merging #26 (ffd6458) into master (482eb68) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #26   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


[React Query] option to allow responses with both { data: TData, errors?: Array }

Loving the react-query codegen plugin, thank you very much!
As we know, graphql returns { data, errors } -- the custom fetcher seems to force us to resolve just data or throw just errors, but we could have some successful data, and maybe one field resolver error.
Is it possible for this to be an option so we can see successful data as well as errors?  I only throw if !!errors && data === null , or maybe i never even want to throw if its a 200 from graphql (aside from bad input, invalid auth, timeout, etc.)
Or is it even possible for the generated code to determine the return types from the custom fetcher?

@dotansimha Sorry for bothering you again, but is this something that can be considered?


Auto close when user choose the date?

How can I make the calendar auto close after user is already chosen the date?
    flatpickr(".flatpickr", {
      minDate: new Date().fp_incr(-30),
      maxDate: new Date(),
      enableTime: true,
     altInput: true,
     altFormat: "F j, Y h:i K"
    });

A normal date picker will close after picking the date, unless

time picker is configured
date range is configured

You have time picker enabled, which is why it is not closing.
This is intentional, because if you're including a time picker the assumption is after the user picks a date they also need to pick a time (or vice versa because you have know way of knowing what the user will select first).

A normal date picker will close after picking the date, unless
time picker is configured
date range is configured
You have time picker enabled, which is why it is not closing.

Precisely.
If you'd still like to close the calendar on date selection, just fp.close() inside an onChange


This test will not run

This test refuses to run although I have followed the instructions.  How do I go to the next challenge?
Challenge Name

https://www.freecodecamp.com/challenges/change-the-font-size-of-an-element
Issue Description

Browser Information

Browser Name, Version: IE
Operating System: Windows
Mobile, Desktop, or Tablet: Laptop

Your Code

Screenshot

Click "Map" on the green bar at the top of the page, then find the next
challenge in the series.
On Thu, Nov 17, 2016 at 6:43 AM, Doris Ofori<EMAIL_ADDRESS>wrote:

This test refuses to run although I have followed the instructions. How do
I go to the next challenge?
Challenge Name
https://www.freecodecamp.com/challenges/change-the-font-size-of-an-element
Issue Description Browser Information

Browser Name, Version: IE
Operating System: Windows
Mobile, Desktop, or Tablet: Laptop

Your Code
Screenshot
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/11683, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AMDHkkksQypoMXEjgg_xtWaeHpSY9Cxmks5q_D3TgaJpZM4K1NEF
.

--
Zobel, Gregory

hi @dayofori and @OingoBoingo1984,
The issue tracker is for reporting bugs only. Please use the help chat room or try looking through the forum for help with a specific challenge.
Happy Coding.


fix issues with always rendering

Lots of  unecessary expensive re-renders are happening because of always using () =>

Lots of unecessary expensive re-renders are happening because of always using () =>

@brandonseydel
Could you explain a little more about why?


framebuffer for e-ink display. Added new IOCTL param FBIO_REDRAW

Kconfig changes
SSD1680 driver changes
ioctl(..., FBIO_REDRAW, ...) added
Summary
It is possible to ask E-ink driver to redraw. In case of frame buffer modification, the screen is not updated. We want to reduce number of redraw operation. It is time and energy consuming without obligatory implementation of callback function putarea (struct lcd_planeinfo_s).
Impact
New callback function redraw in struct lcd_planeinfo_s (lsd.h file). Implementation of this function is optional.
Testing
Yes on ssd1680 e-ink display.

@adamkaliszan the title need update too.


Language support in template service

Describe the solution you'd like
I need the template service to support multiple languages. Having customers in multiple non-English speaking countries, the service in its current form, is of limited use to us. If the service supported language specific versions of a template, the client could provide a preferred language (either as part of the url or possibly through the accepted-language header), and get a response back with a template, a confidence (unaffected by the language) and a language indicator of some sort (either the requested language or a fallback language).
Normally all labels will need to be localized. If the service returns bound data, any dates/times and separator characters must be localized as well

Hey @palktonderAtpowelno ,
while this does not exist we had similar issues, There's a workaround that might help you and might be even better than what you ask for depending on your use case.
Have in mind, you can data-bind literarily everything in the JSON template with placeholders. Not only values. You could have even use a placeholder for like sizes, widths, and colors.
I wrote one template where not only actual data but also the labels etc where all placeholders, on data-bind I just bind the labels from my resource dict for whatever language I need, same for date/time formatting, etc.
Here's an example for the data you could use:
 {
    "resources": {
        "CAPTION_INPUT_NAME": "Family Name",
        "CAPTION_INPUT_FIRSTNAME": "First Name",
    },
    "data": {
       "name": "Matt Hidinger",
        "profileImage": "https://pbs.twimg.com/profile_images/3647943215/d7f12830b3c17a5a9e4afcc370e3a37e_400x400.jpeg"

    }
}

Sample template here:
 {
    "type": "AdaptiveCard",
    "version": "1.0",
    "body": [
        {
            "type": "TextBlock",
            "text": "{$root.resources[\"CAPTION_INPUT_NAME\"]}"
        },
        {
            "type": "Image",
            "altText": "",
            "url": "{$root.data.profileImage}",
            "size": "Small"
        },
        {
            "type": "TextBlock",
            "text": "{$root.data.name}"
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json"
}

HI @DeeJayTC,
I've considered this approach, and it might solve the problem for me personally. At the same time I'm not sure this is a good approach going forward. An important question is whether having all the labels as bound data makes the templates too difficult to create and edit.
More important though is that I'm not sure this would support the natural flow for how templates are created. Especially for the public version of the template service where people might donate templates, I foresee that many will supply an "standard" English version. To support data bound labels, somebody would have to transform these templates before they could be made available. Since resources are separate from the "real" data, it would be natural that they are maintained within the template service, not each template user. This again would mean that you'd have to support a new process for adding resources for new languages.
By using a separate template for each language, there might be less work to verify each contributed template. Also, adding a new language would just mean contributing a new template using tools already established.

hmm, an interesting idea could be to think about how to make templates as such localizable in general. I mean AdaptiveCards itself or the template service could have a resource dictionary for common terms such as properties related to persons, addresses, etc and maybe there's a way to just say "give me that template with en_US labels" and the service itself handles translation.
Maybe even in some way, however that might look, for people to add new resources?
It could be as simple as having resource files per language in the Github repo which the service can use and a new param &lang=de_de or &lang=en_UK ..etc and if nothing for the language is found it falls back to whatever is in the template itself.
hen you retrieve the template the service scans the JSON to check if there are common terms in it.
EG if I have the First Name as the label somewhere and I request DE the service would replace the English term with the German term. Using some AI in the background or something like Louis.ai could actually make that a pretty powerful feature especially thinking of power automate etc. That means nothing is lost during design time, you just design the template in English and at runtime, its translated to whatever language you're requesting.

I see that a support for common terms could potentially solve some of the challenges related to this, but there are still some pitfalls. One is the obvious one of people having to use the right label key words, the other that the localization support would be somewhat limited.
Maybe I'm just too pessimistic by nature, but I worry that many potential template contributors would shy away from the extra burden of making a template localize-able. That would mean that this burden would be put on the template service maintainers. Some (AI-)logic could be created to replace any terms found in the resource dictionary, but any terms outside of the common terms would have to be handled as well.
I just think it would be easier for people to contribute whole templates. Instead of having resource files in a folder structure based on language, actual localized templates could be found in folders base on the same kind of logic. One potential benefit of this is that localized templates can have a slightly or even vastly different layout if properties of a given language warrants that. This might be things like left to right text or languages with lengthy words or even symbols.
If you look at another issue I've reported concerning support for templates for editing, it has the same challenges on how to support a different template for the same data. Using conditionals, it might be possible to use a single template, but that would really be complex. Always using complete templates located in appropriate folders (for instance an edit folder with localized sub folders), you would have a simple and coherent way of handling all the permutations. Yes, it would definitely end up in very many templates, but it would be easy to contribute a single template.
That said, if the card/template editor made it easy to create localized versions of a template to support the flow you are suggesting, I would be fine with that.


FileModuleResolver is disabled for all wasm, not just wasm-unknown

Hi there!
I'm using Rhai in a wasm32-wasi environment (which has file access) and was hoping to use the FileModuleResolver, but it appears that the public export is disabled for all WASM:
https://github.com/rhaiscript/rhai/blob/84fd228281b1ccc4b062c5caa98d8acb14db3d42/src/module/resolvers/mod.rs#L15-L16
I'd suggest changing this to checking for target_family = wasm and target_os = unknown.

Hhmmm....  target_os = unknown is probably not correct also, because not all known OS for WASM will have file support.
I wonder how std::fs does it for wasm targets... let me research a bit...

Do you know if wasm32-wasi exposes target_os = wasi?  If so, we can probably put a special flag on it.

Ah yeah, fair enough. Yes, I believe the wasi part refers to the OS (the information on this is a little conflicting).
https://doc.rust-lang.org/rustc/platform-support.html mentions only -unknown, -wasi and -emscripten; the latter two should support files. I think checking for -unknown should be sufficient - I think that's what I've seen in other parts of the ecosystem.

The fix has landed in this repo.  You can pull from here to test it out.
If you need it urgently, I can release a point release.  Otherwise, it'll go live in the next release.
Thanks!

Thanks! No rush, I was just experimenting. It seems like you've pushed to your local repo instead of this one, though? (https://github.com/schungx/rhai/commit/12a536c2a42e34784b1299a0ec66ee72c9130747)

Ah yes.  I'll port that through.  Sorry!

It has now landed in this repo.

@philpax  did you get a chance to test this out?

Hi, sorry, I haven't had a chance to try it out yet - will let you know if I do!

Closing for now.  Feel free to reopen if you have any more questions.


Add GitHub Action Workflow

Because we're cool 😎
But this time for realsies

Codecov Report

Merging #28 into master will decrease coverage by 0.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
- Coverage     100%   99.52%   -0.48%
==========================================
  Files           9       18       +9
  Lines         456     1056     +600
==========================================
+ Hits          456     1051     +595
- Misses          0        5       +5

Impacted Files
Coverage Δ

src/BusInOut.cpp
100% <0%> (ø)
:arrow_up:

tests/test_utils.cpp
100% <0%> (ø)

tests/test_businout.cpp
100% <0%> (ø)

tests/test_motor.cpp
100% <0%> (ø)

catchlib/catch.hpp
89.36% <0%> (ø)

tests/test_quadrature_encoder.cpp
100% <0%> (ø)

tests/test_stepper_motor.cpp
100% <0%> (ø)

tests/test_pid.cpp
100% <0%> (ø)

tests/test_hall_encoder.cpp
100% <0%> (ø)

tests/test_tracking_loop.cpp
100% <0%> (ø)

Continue to review full report at Codecov.

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


Google Cloud Logging Logback Appender Client for Java.

PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.

@Mikksrund, can you please clarify what is this issue is about? Thanks!


Tag remplacement improvement

Would it be possible to make tag_remplacement act different on block or inline tag, or maybe allow tag_replacement to be a function called with current tag returning a string (like \n or  )

Hey @JSteunou, I really don't feel like that belongs in this library. Understanding inline vs block tags would involve larger changes and make this more of an HTML parser than a tag stripping state machine. We talked about this in #15, and my opinion hasn't changed from then.

Ahaha I forget I already did bring this topic. But if it comes again it is because it's a strong need.
I totally understand that this lib is not an HTML parser and awareness of tag type is not its job, but, you could delegate the job by accepting tag_replacement to be a function returning a String.
No big changes for you, a simple || condition https://github.com/ericnorris/striptags/blob/master/src/striptags.js#L108 to use it or call it & use it and it could make some developer happy and save white bears (not sure about that last one though ;) )

Just pushing a rapid example, did not check if it works, but it's so we can see what I meant ;)


Question: Pixelated Hero Image on People page

The image we're using is huge: https://master-7rqtwti-ouiqvb5juucvu.us-4.platformsh.site/wp-content/uploads/2022/10/container-19.jpg
However, it is quite pixelated on the people page: https://master-7rqtwti-ouiqvb5juucvu.us-4.platformsh.site/people/
Do you know why?

@bwaltz I'm gonna boost the resolution used for heros sitewide. This is 100% a result of a too finely cropped image sizing in functions.php. Although we are aiming for lightning fast load times, the 1280px width I have set here is entirely too small. Expect a commit shortly amending this, and a few other sizes. We'll probably boost it to 2400px wide.

@bwaltz Update: I've pushed a commit here that should do the trick. We may need to reupload or toggle the image in the CMS after running your next composer.

@gregoryspragginsjr It's still pixelated even after a re-upload of that image.
https://master-7rqtwti-ouiqvb5juucvu.us-4.platformsh.site/people/
I wonder if the template's not using the correct size?

@bwaltz that could indeed be it. Let me take a look then.

@bwaltz Update: Try now. I forgot that we updated these overlay-hero to support full video. That video component was what needed to be updated to support larger thumbnail sizes.


Music continues to play but UI freezes

OS: Windows 7 x64
GPMDP Version: 4.6.1
Issue Descriptions: I can see from another post that there may be an issue with lyrics or something? I'm not sure so I'm posting my own thread to deal with my issue. It happens randomly. Sometimes 2 minutes after opening the program, and sometimes after hours of playing music. I have a new, fresh install of Windows 7 x64, and on my instance of Windows before my refresh, I had a previous version of GPMDP that did not lock-up or crash, whatsoever.
This latest string of lock-ups made me post here, since I do not want to give up and just stop using this program. The last three lock-ups were caused by one song in particular, "The Appeal & The Mindsweep (Metrik remix)" by Enter Shikari. I reproduced this lock-up three times in a row. Not only that, but the song would finish, and the audio would cut off somewhere in the middle of the second song.
 I tried changing other settings with no avail. The SSD this operating system and program is installed on is fine, no errors, 100% life. No other programs are crashing or locking up.
Steps to Reproduce: End task in Task Manager, restart program, look for specific song, UI locks up a few seconds after pressing play.

This is definitely a duplicate of #3483.


[moved] Table.drop sounds like it will mutate and has an out-of-date docstring

Prof. DeNero:

When I see drop, I think it's going to change the table like del.

Looks like pandas calls it drop too, so let's leave it. I fixed the docstring.


ci: Add a gcc 13 job

Neat. libstdc++ was cleaned up to the point where we got compilation errors where we used uint8_t instead of std::uint8_t (with the <cstdint>-include.)


Have keep alive option when get the file lock

If A get the file lock for quite a long time, A should always touch the file to make sure the lock is not stale

+1 patch welcome.  Should use the touch module.

hi,
i was going to write this patch, but i notice that at lines 10-12 of lockfile.js has platform specific code:
var os = require('os')
exports.filetime = 'ctime'
if (os.platform() == "win32") {
  exports.filetime = 'mtime'
}

it would appear that under windows, you check modification time, whereas everywhere else, you use creation time. why doesn't it use modification time everywhere? this will make touching the file easier.
with thanks

@jonathan-love
On windows, ctime is "created time".
On posix systems, ctime is "changed time".  That is, the last time any file attribute (including atime) was changed.  Since atime changes ctime, any modification is an access, it's more like Windows' modified-time or mtime.

chuckles ... righto.

what should we call this function?
renew()
renewLock()
touch()
i don't think touch() is good, because it requires you to be familiar with the touch command.

I would suggest refresh()

+1

I'm touching this thread to make sure it's not stale

i think it might be stale :P


Move Jenkins build process to Pipelines & make templates configurable.

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I added multiple integration tests (Spring) related to the features (with a high test coverage)
[x] Server: I documented the Java code using JavaDoc style.

Motivation and Context
As discussed in #2169, the current Jenkins build configuration is deprecated:

Usage of outdated plugins (Multiple SCMs: https://plugins.jenkins.io/multiple-scms/)
no support for docker builds
increased setup complexity as all dependencies for all supported programming languages must be installed within the Jenkins master
Bad scalability: All builds are executed on the Jenkins master and adding more agents is rather expensive as all dependencies must be installed on all agents (and also updated).

This PR moves the Jenkins configuration to so-called Pipelines which define the build process, not through UI settings but declarations in a file.
Exercises that use the old setup should still be supported (e.g. you should still be able to import them into other exercises or start these exercises as student).
Description
The build process in Jenkins is moved to a config.xml-file that shares common steps for all programming languages (e.g. triggering a build when a push to GitLab occurred).
It is extended by a Jenkinsfile that is dependent on the used programming language which will be included in the generic config.xml file.
The builds steps (including used docker images, the checkout process, the actual build steps, and the reporting of the results to Artemis) is included in the Jenkinsfile.
Another improvement is that files from the src/main/resources/templates/jenkins-folder can now be modified. This allows (especially non-TUM-institutions) to override settings that might fit well into our deployment, but not so well in different deployments (e.g. a setup where build agents are not static like in our setup, but managed through a Kubernetes cluster).
The ResourceLoaderService manages this: Files are first loaded from a template folder next to the artemis.war-file and if no results are found, the files are loaded from the classpath (e.g. files included in the .war.file).
This avoids additional setup steps for all users, but allows experienced users to adjust the jenkins-templates to their own setup.
Note: This PR requires at least version 1.1.1 of the Jenkins notification plugin: https://github.com/ls1intum/jenkins-server-notification-plugin. It is already installed at our Jenkins instance at TUM.
Also, some other Jenkins plugins are required for this setup, please refer to the changes in the documentation included in this PR.
The updated documentation also contains steps for setting up remote agents.
Possible Questions related to the default Jenkinsfile
Q: Why did you use checkout([$class: 'GitSCM', branches: [[name: '*/master']], doGenerateSubmoduleConfigurations: false, extensions: [], submoduleCfg: [], userRemoteConfigs: [[credentialsId: '#gitCredentials', name: 'tests', url: '#testRepository']]]) instead of git url: '#testRepository' credentialsId: '#gitCredentials'?
A: The git step might be sufficient for the most cases, but it is recommended to use the checkout step instead (see https://www.jenkins.io/doc/pipeline/steps/git/). I decided to use the checkout step as the Jenkinsfile might be adjusted by other universities and this shows them more of the available options.
Q: Why must every step of the Build stage be within a timestamps block?
A: Artemis expects the timestamp to parse the logs.
Q: But why did you not activate it for all pipeline builds? There even is an option to do this.
A: This would also activate the timestamps for builds that are not related to Artemis but might be executed on the Jenkins instance.
Q: Why did you use the cleanup action in the step to send the results and not the always option?
A: There are multiple stages available that can be used to execute commands after the build completed. However, we can not change the order of execution of these steps and cleanup is always executed last.
Q: Why is there a || true next to cp target/surefire-reports/*.xml $WORKSPACE/results/?
A: If the build fails (e.g. due to a compilation error), no test-results are generated (and thus they can not be copied). The cp command then exits with a non-zero exit code and Jenkins will then stop the build (and will not sent the test results to Artemis.
To prevent this, we ignore the return value of the cp command.
Steps for Testing
As this affects the Jenkins/GitLab-setup (and not the Bamboo/Bitbucket-setup), please use TS2 to test these changes.

Make sure an exercise exists that was created before deploying this PR. Make sure that you have not started it yet.
Deploy this PR.
Create a new Java or Python exercise (for Java, also try static code analysis) and verify that the solution and base repository are executed correctly (i.e. go to Edit in Editor and validate that the solution repository passes all test cases whilst the base repository passes none).
Start the exercise as a student and participate: The normal behaviour, including the detection of failed and successful test cases, should work. Also validate that build logs are shown if a build error occured.
Import the exercise into a new exercise and validate that the repositories are updated correctly (e.g. by validating that pushing to the new solution repository actually affects the correct exercise)
Start the imported exercise as students and repeat the testing from step 4.
Repeat steps 4-6 but use the programming exercise that was created before this PR was deployed instead.

Please also test that creating programming exercises still works on the Bamboo/Bitbucket-setup (TS1/TS3).
Test Coverage

ExerciseService.java: 85%
programming-exercise.component.ts 95%

I tested on test server 2. All of the tests passed until this part "Repeat steps 4-6 but use the programming exercise that was created before this PR was deployed instead."
* when I imported a programming exercise that was created before the deployment, I found a few errors. Importing an SCA programming exercise worked fine, but non-SCA had some errors.

First: The error is "no tests found" for template and solution repostiory
* https://artemistest2.ase.in.tum.de/#/course-management/1439/programming-exercises/1928
  Second: I re-imported it, and then the template repository build failed, but the solution repository is fine.
  https://artemistest2.ase.in.tum.de/#/course-management/1439/programming-exercises/1929

Steps to reproduce (test server 2):
1. Import a non-SCA programming exercise that was created before the pr was deployed

2. go to view

3. build the template and solution repository

4. notice the error in the template/solution repository, either build failed or no tests found

@tiffca Thanks for testing! I made an error during the configuration of the build agents but I fixed it now. I retriggered the failing build plans and it seems to work now. Can you validate?

@MaisiKoleni Thanks, I will investigate this tomorrow.

@MaisiKoleni I fixed the issue now:

The problem was that the path to the workspace on the remote-agent and the local-agent differed.

Ok so I am a bit confused here about the change in documentation:
My assumptions:
If we merge this PR then, all new exercises are gonna work with the pipeline and docker.
What I would urgently change in the docs then:

[ ]  There is no need to make the user install the old plugins needed, because the docs are for new users.
[ ]  The docs still describe how to set up jenkins in a docker container, but with out docker being available there, which would be ok if you had remote agents but if not a problem.
[ ]   Installing remote build agents sounds like it is optional but with the setup that is now described in the docs such a remote agent would be necessary.
[ ]   Since the docs probably describe a small setup maybe the guide should tell you how to set up one jenkins instance, but in a way that it has docker installed etc.
[ ]  Tell users in the docs how to setup caching, because without that you are really wasting a lot of resources(network, disk and cpu).

I suggest we improve the docs in a follow up PR

@krusche I tested the merged code from develop and in our environment it worked with all recent changes (using Jenkins Notification v1.2, Artemis Dev branch, external build agent with docker). I can now try to change the template to try to use the K8s Jenkins build plugin in a next step.


[Feature] Work with the Regions Feature

Converting some of the Module's Features to region as so (in my thoughts):

Cloning

Most important unique feature to copy, worth doing it for sure

Teleport

Unnecessary to port, already included in core

Levels

Would be a nice holdover until levels becomes core

You'll probably want to files this here instead now: https://github.com/BeneosBattlemaps/Beneos-Multilevel-Tokens


feat(protocol-engine): Start a Protocol-Engine–backed implementation of InstrumentContext

Overview
Closes #7324.
Changelog

Add opentrons.protocols.implementations.engine.InstrumentContext to implement opentrons.protocols.implementations.interfaces.instrument_context.InstrumentContextInterface.
Return NotImplementedError from all methods.
Return an instance of this class from ProtocolEngineContext.load_instrument(), though it doesn't do anything yet.

Review requests
By my count, the default arguments for these methods are now duplicated in 3 places:

implementations.interfaces.instrument_context
implementations.instrument_context
implementations.engine.instrument_context

Neither our typechecker nor linter appear to warn us if they get out of sync with each other.
Is it worth doing something about this? For example, one possible solution:
class InstrumentContextInterface(ABC):
    def touch_tip(self, location, radius=1.0, v_offset=-1.0, speed=60.0):
        self._defaulted_touch_tip(location, radius, v_offset, speed)

    @abstractmethod
    def _defaulted_touch_tip(self, location, radius, v_offset, speed):
        ...  # To be overriden by subclass.

Edit: It's actually worse than this—opentrons.protocol_api.instrument_context duplicates them, too. Maybe we should remove default args from these inner layers, and rely on the top-level protocol_api layer to provide them?
Risk assessment
None. Adds new code that isn't used anywhere.

Great observation about the default parameters. I think the correct path is to remove non-None default parameters.
InstrumentContext already defines the default values. We're not doing ourselves any favors by redefining.
This was just an oversight on my part.

As discussed in chat:

This PR now omits default arguments from the new file that it creates.
Another PR will remove default arguments from the files that already exist in edge and close #7395.

Linting on this PR will fail until that second PR is merged. (Although the linter doesn't complain about default argument values not matching between an ABC and its implementation, it does apparently complain about default arguments being missing from the implementation.)


WM5 | Marcus Zagorski | Module-HTML-CSS | Week 1

Here is my article based on questions relating to Github, where I have completed all the tasks asked from me.
I have made the main article have a different background compared to the other 2 articles, I have used a consistent style throughout my stylesheet and I have also fixed a footer to the bottom of my webpage.

Super, GREAT work bud. nice design. I like it :100:

@SallyMcGrath I've now managed to fix the issue I was having regarding the marcusmain branch, but there is something I wanted to ask. Am I right in saying that when you commit changes that you need to commit both your stylesheet and html file together? Because when I made a change to just my stylesheet and then recommitted, it wouldn't allow me to preview the website anymore.


How to create a two dimension tensor like (1000,5) (row,col)

I don't know whether I'm wrong or not, it seems that the cppflow::fill and cppflow::tensor can only support one dimension tensor in the example.
If I'm wrong, could you please tell me where to or how to generate a 2d tensor like (1000,5) (row,col) and pass it into the model?
Many thanks!

cppflow::tensor t = cppflow::fill({ 1000, 5 }, float(1));


Forked Blocks inside new synced empty wallet.

I loaded/synced the walletchain now over 5 times (in different wallets) from Zero, even with empty Paycoin folder.....allways have wrong Blocks from 380237 - 380984 in my Chain which not matching with ledger!
"getblockhash" between 380237 and 380984 not matching ledger!
17:39:46 getblockhash 380240
17:39:46 d59a2fcec6e6b1420699eef17e9b009374656ea74c518f2e56dbb3fbe2a803be
17:40:40 getblockhash 380870
17:40:40 b2b3456812822d72d4fcac88e0f3166202d284898524b0064eff32248cab1e40
https://paycointalk.org/topic/1028/resolved-wrong-chain-again/52
https://paycointalk.org/topic/1028/resolved-wrong-chain-again/55

They match up just fine so far as I can tell....
I just synced a wallet yesterday:

The first is the block height at the time of my looking now. As you can see all 3 hashes match on the ledger:
https://ledger.paycoin.com/block/805228a0902c464b3eff1adeb4e592083315a3e37612648e9100d9cc8c6fff43
https://ledger.paycoin.com/block/d59a2fcec6e6b1420699eef17e9b009374656ea74c518f2e56dbb3fbe2a803be
https://ledger.paycoin.com/block/b2b3456812822d72d4fcac88e0f3166202d284898524b0064eff32248cab1e40

Yes, now after ledger rebuild this "forked" Blocks match!
But not before.
After this rebulid today I lost many of debits that was confirmed and matching with ledger before.
How can it be, that the "officiall" ledger was the incorrect chain?

Being that the ledger is managed by Team Paycoin rather than the development team (XPY Dev) I have no direct information on this. However since it's resolved I'm closing this issue.


Add LiveReload

LiveReload updates the browser when files change.
guard-livereload pulls in the eventmachine dependency but it claims to work well on Windows.

Let's have this for the next lab. Rebasing the branches again will be a headache for those who have already started. I'll test it ASAP.


Combinato container

Runs and passes installation test
Will probably need some tweaks to run with spikeinterface

Please add test here

Will do

From today's test, I get this error:

Yep, can you trtmy to generate the toy data with num_channels=1?

It seems to work with num_channels=1 :)

@alejoe91 The container works for num_channels=1


⚠ SkyfallMC Minecraft Server has degraded performance

In 30174f9, SkyfallMC Minecraft Server (play.skyfallmc.ga) experienced degraded performance:

HTTP code: 200
Response time: 2671 ms

Resolved: SkyfallMC Minecraft Server performance has improved in 05434c8.


App preview page: Hide app name and icon

Feature proposal
https://firebase.google.com/docs/dynamic-links/link-previews
On app preview pages, is there a way to hide the app name and icon?

In the image above, it would be the Google Photos logo and name.

Now that Firebase DynamicLinks is planned to shut down, closing its feature requests.


Header is broken when Composer is resized.

At the exteme end of the spectrum it looks like this:

At a medium width (typical if docking two windows side-by-side)

@sangwoohaan the team will need responsive designs for this header to properly address this.

Looks this has been fixed. On Windows:


Updated to latest supported versions

@nadenf
Need to verify akka http license is free to use.
ci.yaml should be generated with sbt github actions. Have you used that?


bug: noice.util.is_blocking from "noice.nvim' breaks search UI

Did you check docs and existing issues?

[X] I have read all the flash.nvim docs
[X] I have searched the existing issues of flash.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0 LuaJIT 2.1.1716656478
Operating system/version
macOS 14.5
Describe the bug
Since 4e1f9f198226aea5b82a6df75e9913b49796cdda, the new noice.util.is_blocking check breaks search UI when we have flash.nvim and noice.nvim
When we do forward search, matched keywords and labels are highlighted, but the string in the search box is missing the last character.
Steps To Reproduce

Run nvim -u init.lua init.lua
Type /opt

Expected Behavior
Search box contains the string "opt", and all matched strings are highlighted.
Repro
local root = vim.fn.fnamemodify("./.repro", ":p")
for _, name in ipairs({ "config", "data", "state", "cache" }) do
	vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)
local plugins = {
	"folke/tokyonight.nvim",
	{
		"folke/noice.nvim",
		dependencies = "MunifTanjim/nui.nvim",
		opts = {},
	},
	{
		"folke/flash.nvim",
		keys = { "/" },
		opts = {
			modes = {
				search = { enabled = true },
			},
		},
	},
}
require("lazy").setup(plugins, {
	root = root .. "/plugins",
})
vim.cmd.colorscheme("tokyonight")

Fixed in noice. Thank you for reporting!

Thanks! However, the fix https://github.com/folke/noice.nvim/commit/b3f08e6cf0fd30847f299cc94707563920fd2139 seems to introduce a new UI issue. The \t character in front of the fake cursor got highlighted when we enter forward search mode.
As shown in the screenshot below, the \t character turns blue. I think this is related to redrawing strategy in noice.message.router.check_redraw.
To reproduce with the config above:

Run nvim -u init.lua init.lua
Type 11G
Type /

I just updated the flash.nvim code to use real cursors on Neovim >= 0.10


Cannot print on Android 4.4.2

As an Android user
I want to be able to print my pdf documents from PDF.js
So I can have a hard copy
PDF file is not applicable but screenshots were taken from the example pdf https://mozilla.github.io/pdf.js/web/viewer.html
Configuration

Samsung Galaxy Ace 3
Android 4.2.2 (latest version the Ace 3 can get)
Chrome 52
PDF.js 0.8.0 (latest)

Steps to reproduce

Click the print button on a document.

Note the 'preparing for document printing' dialog comes up

Expected Behaviour

After the 'preparing for document printing' dialog gets to 100%
The print dialog box opens up with the document present
And I can print the document

Actual Behaviour

Nothing happens. no print dialog box comes up at all.
Therefore I cannot print the document

I can see various similar issues have been raised about this but were closed due to lack of detail.

Closing, since Android 4 is explicitly unsupported in PDF.js 2.0.


Uncaught TypeError: this.web3.eth.contract is not a function

I get this error when trying to instantiate the library.
It seems to be a type on this line:
https://github.com/ensdomains/ensjs/blob/master/index.js#L460
should it actually be?:
var registryContract = this.web3.eth.Contract(registryInterface);

What version of web3 are you using? web3.eth.contract is the correct syntax, at least in <1.0.

1.0.0-beta.34


add Time Year Month migrations

@rochar I think the tests are broken there is nothing here codewise that should make the fail or be related ?

@rochar I think the tests are broken there is nothing here codewise that should make the fail or be related ?

@DanielHodgeGH  main is green... there should not exist any flaky test. Can y re run the pipeline and lets see the results

@rochar I think the tests are broken there is nothing here codewise that should make the fail or be related ?

@DanielHodgeGH main is green... there should not exist any flaky test. Can y re run the pipeline and lets see the results

Yeah it works on rerun - but that failed twice before! :)

@rochar I think the tests are broken there is nothing here codewise that should make the fail or be related ?

@DanielHodgeGH main is green... there should not exist any flaky test. Can y re run the pipeline and lets see the results

Yeah it works on rerun - but that failed twice before! :)

yup they are flaky, we need to fix those


Escaping UploadSymbolsToCrashlyticsAction binary_path

Uploading dSYMs with the UploadSymbolsToCrashlyticsAction action can cause failures with a custom binary_path where the binary_path contains spaces. This proposed change shellescape's the binary_path.

Hi @OliverLetterer - thanks for the contribution! :+1:


Fixed mismatch API description

Fixed mismatch API description between Microsoft.AspNetCore.Session and Microsoft.AspNetCore.StaticFiles.

Hi @janaks09, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

:x: Validation status: errors

File
Status
Preview URL
Details

docs/index.md
:warning:Warning

Details

docs/index.md

Line unspecified: [Warning] Ignore metadata: ms.locale. They are generated by open publish.

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.

@janaks09  see readme.md
API PR's ( api/autoapi/... ) should not be made here. Changes to API documentation should be made in the source repository against the triple slash /// comments.

@Rick-Anderson I changed https://github.com/aspnet/ApiDocs/blob/master/docs/index.md file, is that count as API documentation. Confused..

@janaks09  yes, that's also autogenerated.

Hi @janaks09, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@janaks09  actually we're making an exception here. Thanks for fixing this.
@danroth27  I'll push master to live.

Thanks. Happy to contribute. 👍


Tst add travis

Attn @dchabot this .travis.yml file with epics will be a useful starting point for doing the same to ophyd.


(Fix) Scrollable list of crowdsales

Closes #533

Pull Request Test Coverage Report for Build 1788

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

Totals

Change from base Build 1786:
0.0%

Covered Lines:
401

Relevant Lines:
1943

💛  - Coveralls

This looks a little broken with my laptop's resolution (1366x768).


Logging elasticssearch with HTTPS

Hello.
Would it be possible to add the possibility to switch on and off FLUENT_ELASTICSEARCH_SSL_VERIFY with the built-in fluentd logging?
This is since there would be many times where you use self-signed certs for the elassticscluster
This is for rancher 2.0

rancher/rancher:master 9/10
SSL_VERIFY can be disabled for elasticsearch.
Setup ES cluster
Enabled TLS on transport / http layers with self-signed cert
Enable SSL verify = unable to connect
Disable SSL verify = connection successful


Auction page doesn't redirect to live interface when the auction opens

The web reloads to show the user a "Open Live Auction" button, but we push them directly into the interface. We should do one or the other when the live bidding starts, since currently the user has to hit back then go back into the auction to see the live interface. Reported by @dblock.

Yup closing this one as dupe.


Indentation before #include causes undefined var error

Description
OpenDream fails to parse first object definition in the file, when file's include line begins with indentation.
Steps to reproduce
Add folowing line starting with indentation to TestGame .dme file:
    #include "exodus_announcements.dm"

Example "exodus_announcements.dm" file content:
/obj/TestObj
	a = "The Emergency Shuttle has docked with the station. You have approximately %ETD% to board the Emergency Shuttle."

OpenDream compiles with error:
Error OD0405 at exodus_announcements.dm:2:4: Cannot override undefined var "a"

At the same time, BYOND compiles it without any issues.
Initial research
I've taken a look to this problem in the debugger and found that problem is in DMLexer. DMObjectBuilder tries to process /obj/TestObj/a override statement, and encounters wrong statement's ObjectPath:

ObjectPath should be /obj/TestObj instead of /datum/map which is simply previous processed block's ObjectPath.
Workaround
Removing indentation before #include file's line solves the problem.

The indentation before the #include persists after the preprocessor inserts the included DM file's contents, so the first line of the included file has the same indentation.
The example becomes:
	/obj/TestObj
	a = "The Emergency Shuttle has docked with the station. You have approximately %ETD% to board the Emergency Shuttle."


Stage 3 updates v6

Sorry for cutting things so close. On this branch, I think the flow from reading in stored data should mostly be ok. There is a chance that if students have previously entered freeresponses and they change them, the program may wig out. (I saw a bunch of error messages in the notebook that seemed like they may have been due to that, but it kept running locally).
There are a couple of logic issues - one I introduced and don't have time to fix, so I put a note to the students how to get around the issue.
The other is in John's pro_dat6 guideline. His logic looks correct to me, but the app is giving the opposite feedback, so I put in a note that there's a possibility the feedback is wrong. (Our class is teal; HST is lime green; Hubble is purple)

@patudom I've made a change so that updating the guess text from the state won't throw an error.
I'm not quite sure what's up with the pro_dat6 guideline - on my computer, the line colors are flipped. At least this isn't really a critical bug.


Fix wrong reference to intlMessages attribute

Checklist:

[x] Test your work and double check you didn't break anything
[x] Update any relevant READMEs and docs

Description:
Fix broken reference which caused email attestation to break (blank screen and console errors) after a correct verification code is submitted.

Thanks @tomlinton 😄 I swear that I fixed this somewhere else but it must not have been merged yet.

No worries @micahalcorn, sorry I didn't really look around to see if it had already been fixed in other branches or pull requests.

@tomlinton no, thank you - too many branches, not enough merges 🙃

@tomlinton I'm not crazy. It was in The Release that Never Was. 😄


feat: build time esbuild deps optimization (using proxies)

Description
At https://github.com/vitejs/vite/discussions/4921, Evan proposed:

Use esbuild to also pre-bundle dependencies for prod, then alias deps to the pre-bundled chunks during the Rollup build phase. This would

allow us to drop @rollup/plugin-commonjs and reduce prod/dev inconsistency due to difference in handling CJS between esbuild & @rollup/plugin-commonjs
allow us to cache the pre-bundled deps like the dev deps. Although they would still need to be processed by Rollup, it now needs to process fewer files and no longer need to do CJS transforms (essentially these are done by esbuild instead). Could potentially speed up the build to some extent.

This PR is an initial exploration of the idea. Code isn't abstracted at this point (especially in importAnalysisBuild) to be able to play with the POC more easily.
POC
There is a new option config.build.optimizeDeps that is true by default. If it is enabled, the @rollup/plugin-commonjs isn't included in the pipeline. And CJS compat is achieved with esbuild + es interop in the same way as we have now during dev. If we end up implementing this feature, I think we should have an option to let people keep using the commonjs plugin, maybe until v4?
The dev deps cache can't be used during build as env variables could be different, so we now have a new depsBuild folder that caches the deps during build time. At this point dev and build caches are completely separated but we can use the dev metadata during build to avoid a scan for example (or the other way around).
I had to modify the react plugin because it is currently doing some tricks to optimize bundle size that aren't compatible with the optimize cache. But something similar could probably be added back. See discussion here for more context.
CI
All tests are using esbuild optimization during build, except for:

nested-deps: the importAnalysisPlugin uses the module graph, we have to do something equivalent (@bluwy maybe you could help me here?)
worker: we need to check how to properly handle workers bundling, I didn't spent much time but is esbuild optimization currently disabled for workers

We also need to check build --watch. Right now deps optimization is disabled for watch mode. It may be a matter of watching the buildDeps folder for changes.
SSR builds are also avoiding the use of esbuild optimized deps. We should also explore this.
Possible issues
One of the biggest issues with this scheme is how to deal with missing dependencies. In the PR, what I end up doing for the moment is to call rollup again if a missing dependency is discovered. It doesn't look that bad because the generation is only performed once, but this means that if scanning doesn't catch every dep, build time will be increased (+75%?) the first time is run (then the cache kicks in). But for CI, we will never have the cache. We talked a bit with Evan in the last meeting about this problem. Some initial ideas:

Push users to add missing deps to optimizeDeps.include. Log a message explaining why it is important, or even explore doing it automatically when a missing dep is discovered.
We may need a optimizeDeps.lazyInclude? At least for laddle, they preferred having missing deps because optimizing all the deps of a design system was too slow for them (it may also be that they were using flow instead of typescript, so maybe this is a non-issue)
Ask users to commit a part of the cache to the repo. This may add more complexity, I think the config option is better.
Is there a better way to re-create the bundle when there is a missing dep? Maybe rollup watch could be used to get some incremental gains?

Another possible problem is increased bundle size. The way we handle es interop for CJS isn't as compact as the one used in rollup. I think that when minified and gziped, the bundles shouldn't be that different but we need to review this. A possible improvement is to avoid checking for __esModule when we know the dep is CJS, we have this information and we aren't using it in the interop code. Another possibility is to use the name of the default if it exists instead of creating a new __vite__cjsExportDefault_xxx safe name. I tried both and they help, but I didn't include them in the PR as they aren't important enough right now.

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

nested-deps: the importAnalysisPlugin uses the module graph, we have to do something equivalent (@bluwy maybe you could help me here?)

We could maybe get rid of nested deps in v3 if @bluwy's prebundling of Svelte libraries is able to be enabled by default? I'm not sure if that's the long-term plan or what's left to do there

@benmccann I think there isn't an issue with supporting nested deps, so I think at least in the scope of this PR we should try to support them.

About how to deal with missing dependencies, maybe there is a way to avoid the issue by waiting until every user code has been processed. So:

We don't perform any scanning during build
We need to block because of the CJS interop logic in importAnalysisBuild. So we avoid blocking user code processing with one of these:

Merging https://github.com/vitejs/vite/pull/7568
Using this.load in importAnalysisBuild transform to do a trick similar to what we do during dev with eager processing. See the second example here https://rollupjs.org/guide/en/#thisload
In importAnalysisBuild we start attaching a ?optimized-interop={importExpressionHash}, and then we can do the interop logic at load time through an intermediate proxy. Since we are building we don't have the dev time issue of exploding the requests the browser needs to perform.

We detect somehow that rollup is only awaiting for the optimized deps promises, and then do a single optimization of all deps (if needed, all deps may already be cached)

In importAnalysisBuild we start attaching a ?optimized-interop={importExpressionHash}, and then we can do the interop logic at load time through an intermediate proxy. Since we are building we don't have the dev time issue of exploding the requests the browser needs to perform.

This is now implemented by: https://github.com/vitejs/vite/pull/8265/commits/fbd4c0faeae80170383522f71606315adcf5c67b. If we merge #7568, is possible that the implementation could be simplified and proxies aren't needed.
What I'm doing is issuing a delay in a transform hook, so if there are still user files being processed processing of optimized dependencies will be avoided and only once performed at the end.

The error is due to dynamic import interop. I tried to port it from importAnalysis but we need to do it in place, I don't think it is possible to do the interop in a proxy. But this means waiting for the needsInterop flag so this may be a dead end.
I think we should merge https://github.com/vitejs/vite/pull/7568, as it will simplify a lot the implementation and we can go back to doing the interop in place as needsInterop is known without the need to wait for the optimization.

I haven't checked #8280 but

nested-deps: the importAnalysisPlugin uses the module graph, we have to do something equivalent (@bluwy maybe you could help me here?)

It seems like the config.optimizeDeps.include isn't used in build. IIUC this PR continuously registers missing imports as it goes, when the include wasn't taken into account? Perhaps we should also incorporate it

We could maybe get rid of nested deps in v3 if @bluwy's prebundling of Svelte libraries is able to be enabled by default? I'm not sure if that's the long-term plan or what's left to do there

There's one more thing dominik raised, that's a blessed way to trigger re-optimization on server restart. Currently it's using  a small hack to trigger that. I haven't got to it yet.
We'd also need optimizeDeps.extensions out of experimental stage for it to be relied on in Vite I think.

Closing in favor of

https://github.com/vitejs/vite/pull/8280


Time Series Widget Shows Wrong Date Range - Enterprise Client

Context
This is an issue for an Enterprise client that I have replicated locally in my team account. The time series widget is showing the wrong date range.
User: chandra
Map: chicago_crimes_1
Dataset: chicago_crimes_1
Steps to Reproduce
Using a dataset with date values, create a map.
Select the "animated" aggregation option.
Choose the date column for the aggregation.

Select the beginning of the time series widget onward using the manual selector. The wrong dates will be shown. The time series widget says the dataset starts December 1, 2011.

However, the earliest date values in the dataset are December 31, 2012.

Current Result
When manually selecting to show a certain range of data, the wrong date value is shown in the time series widget if you select data from the beginning of the dataset onward.
Expected result
The time series widget date range should show the correct values in the dataset for the selected time period.
S/B: 10970892
cc @CartoDB/support

I have already reported the same issue here https://github.com/CartoDB/cartodb/issues/9746 I think it is related to deep-insights: https://github.com/CartoDB/deep-insights.js/issues/476

I'm the original bug reporter (enterprise client) here.
I think @zingbot 's issue is identical to this bug, though it should be 1 year off, not 2 years.
The issue mentioned by @ramiroaznar looks like a different issue about bins, which dated from Sep, 2016. I can confirm the bug reported here doesn't exist yet at least until Nov, 2016.

Hey @michellechandra, about your example, I feel the description is wrong. The date column starts from 2012-1-1 to 2012-12-31.

That is correct, I mistyped and the date values in the dataset are January 2012 - December 2012. However, if I select the beginning portion of the timeseries widget, it shows the starting date value as December 2011.

link please?

https://team.carto.com/u/chandra/builder/9ac43b36-b8d9-11e6-a710-0e233c30368f/embed

For me it is being selected correctly, do you have applied any browser zoom?

Tested your public embed in 4 different browsers and same result:

I feel we have found the problem, PR here.

Closed by https://github.com/CartoDB/deep-insights.js/issues/524 and #11634.


Preparing data for inferencing

Hi there,
I have been using the load_data function to create my datasets for HuggingFace's Trainer class, which has worked really well so far. However, I am not sure how I should approach inferencing without a label column which load_data requires. Would really appreciate some advice as I am rather new to this. Thank you!

Hi @insdrs480, for creating datasets we expect labeled data since it follows a train-eval-test loop. If you don't have labels, you can assign dummy labels to that particular column and discard it later. Please let me know if you need any other help!

Thank you for the response! Are we able to specify the model's behaviour when it encounters unknown categorical data?
I'm getting the following error, which I believe is due to my test dataset containing values that are different from what I trained the model on: RuntimeError: mat1 and mat2 shapes cannot be multiplied (256x22 and 28x768)

Thank you for the response! Are we able to specify the model's behaviour when it encounters unknown categorical data?
I'm getting the following error, which I believe is due to my test dataset containing values that are different from what I trained the model on: RuntimeError: mat1 and mat2 shapes cannot be multiplied (256x22 and 28x768)

Not at the moment unfortunately. This is something we'd like to add in though. Do you have any suggestions as to what you would like to see?

Hi folks, I had the same issue and I suggested a solution in #61 to fix it in the load_data() function.

Thanks Doug!

I'm getting the following error, which I believe is due to my test dataset containing values that are different from what I trained the model on: RuntimeError: mat1 and mat2 shapes cannot be multiplied (256x22 and 28x768)

@insdrs480 Apologies I missed this bit! The model will expect the same set of columns that were used for training. You might be getting this error if they don't match. I would advice either setting up default values for missing columns or removing them from the training process entirely.
Closing this issue as Doug's fix solves this. Feel free to re-open/create a new one if you run into any issues!


Makefile: Fix the releases target

THe dep has changed now that we build tar.gz file all architectures

LGTM


[6.0.0][CMake] Explicitly link Testing to Foundation

Cherry-pick #693 into release/6.0.0

Explanation: Previously in CMake builds, when Foundation was not in the regular search directory (e.g. resource directory or SDK search paths) #if canImport(Foundation) used to fail, and the functionalities are not included. This patch provides a way to provide Foundation_DIR  for find_packgage(Foundation CONFIG), so that clients can correctly link Testing to Foundation
Scope: CMake builds
Risk: Low. No actual code changes.
Testing: Passes current test suite Also manually tested the build toolchain
Issues: N/A
Reviewer: TBA

At this point we are not taking this into 6.0.1 release, if we want this in 6.0.2 please target release/6.0.2


Added requirements.txt for sample packages specified in README

Description
Added requirements.txt for sample packages specified in README for 5 AI Kit samples.
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Sample Migration (Moving sample from old repository after completing checklist established)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Command Line
[ ] oneapi-cli
[ ] Visual Studio
[ ] Eclipse IDE
[ ] VSCode

@JoeOster Please approve for testing, thank you.


Auto-generated. Updating Vectara public protos. (8e9e564b1ada0580d2f0…

Adding streaming changes

@pwoznic You can approve and merge whenever you're ready to publish the docs.

@pwoznic go ahead and merge if these look good to you


bug(Mat-Form-Field): Angular Material form field Long label overlapping with input text

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Angular material form field label overlaps with user entered text, Also long labels with fields have issue with responsiveness.
Reproduction
Check stackblitz link: https://stackblitz.com/edit/angular-ar2ulq
Expected Behavior
Complete label should be visible, it should not get shrink.
Actual Behavior
Label is getting shrink or not appearing on the screen
Environment

Angular: 9.2.4
CDK/Material: 9.2.4
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

This custom style is breaking it:
.mat-form-field-label {
  white-space: normal !important;
}

Closing the issue since generally we don't encourage overrides like this.

Even if we remove the override what would be other option?

I don't follow. Removing the override fixes the issue because the text doesn't wrap anymore.

Removing override fixes the overlap issue, but it doesn't fix the label issue as well. Complete label should appear on screen. If not, How end user will read the complete  field label? (if it long to appear on screen). check stackblitz: https://stackblitz.com/edit/angular-ar2ulq

The Material Design spec doesn't recommend splitting it across multiple lines or truncating, but rather keeping the label short to begin with. The truncation is a fallback in case a really tall label does come in.


Update External_Plugins.md

Clapper js scroll text, marquee overlay plugin added.

Coverage increased (+0.02%) to 74.412% when pulling 31dd091770eb71d3ae6d5cc9e4bddaf26544803b on Lethea:patch-2 into b51d9938a4dca1018c14193cac418d9c9fa4d04d on clappr:master.

Cool, thank you @Lethea but where are the dependencies? package.json ( I think you relied on jQuery isn't?) where is the build/minified version?

Hi @leandromoreira , there is no need to build anything, I developed this by following first plugin guide. Therefore there is no build etc script generated . If needed, I will download the plugin generator and develop this with this generator by the way the skeloton will be appear as you ask ( dependicies build script src etc )  :) However fordeveloper  this sample help them. if you accept this , the next time I'll develop it with plugin-generator.see you

Thank you very much !!!


New Project: Markdown Previewer

I would like to add a web application that is built using HTML, CSS and  JavaScript along with some external libraries.
This comes with a text editor which provides a place to type the lines of text. then the previewer will translate those lines of text along with the syntax and semantics of markdown language and display it dynamically as the user is typing.

@urvashi-code1255  I have added the necessary files and made a Pull Request. please review it from here

@urvashi-code1255 I have added the necessary files and made a Pull Request. please review it from here

This project already exist.


Chrisda to Master

OPS Build status updates of commit 3db269c:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit 3db269c:
:clock10: Incremental building: average incremental building time is 12 min(s) 24 sec(s)

OPS Build status updates of commit 3db269c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/policy-and-compliance-content-search/Invoke-ComplianceSearchActionStep.md
:white_check_mark:Succeeded
View (exchange-ps)

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.


[Validation] Verify fix for EnsureBuffering + Json.Net formatter

Issue: https://github.com/aspnet/AspNetCore/issues/14396

Done. Follow up filed here: https://github.com/aspnet/AspNetCore/issues/16615


add ascender and descender to Font documentation

Add missing docs for ascender and descender properties of the Font object.

Thanks @joshmarinacci — sorry it took so long.


How to put the sun in the center?

hi how can i put the sun in the center?

clone the repo, change the line inside the SunRefreshView.java
 mSunLeftOffset = 0.3f * (float) mScreenWidth;

to
 mSunLeftOffset = 0.5f * (float) mScreenWidth;

use it :)

Thankyouuu so much! =)
On Aug 23, 2015 3:09 PM, "Oleksii"<EMAIL_ADDRESS>wrote:

clone the repo, change the line inside the SunRefreshView.java
mSunLeftOffset = 0.3f * (float) mScreenWidth;
to
mSunLeftOffset = 0.5f * (float) mScreenWidth;
use it :)
—
Reply to this email directly or view it on GitHub
https://github.com/Yalantis/Phoenix/issues/18#issuecomment-133795992.


Don't close a modal if a click that ended outside the modal started inside it

@keybase/picnicsquad CC @keybase/react-hackers
When we're showing a modal, we stop onClicks from inside the modal bubbling up to the "cover" opaque background.  Clicking on that background closes the modal.
But if a click starts inside the modal and ends on that background element, the onClick for the background fires.  So if you click and drag to select some text inside a modal but end outside of it, the modal closes.
Since onClick's not going to work, I went with using onMouseDown and onMouseUp and only treating it as a modal close if the click both started and ended inside the background element.
Am open to better ways of doing it, I was surprised I couldn't find anything more obvious to do.

Doesn't onclick have the location?

@chrisnojima It has the end location, not the start location. We want to avoid firing the onClose depending on the start location.

relative-popup-hoc also needs to be fixed

here is my alternative implementation: https://github.com/keybase/client/pull/18930

alternative merged


influx cli sometimes frozes on FUTEX wait when -execute 'drop measurements...' is used

Hi,
I created (due to mistake in graphite template). about 10000 measurements like this one:
internet.asr2.snetwork.AS63859.inbytes
Since I found no way how to delete them using influx SQL commands (seems no wildcard in 'DROP measurement name' is allowed I used 'influx -execute' to list all measurements into a file and then to delete them via a simple bash script using this command line:
influx -database graphite -execute 'DROP MEASUREMENT "internet.asr2.snetwork.AS7679.inbytes"'
The problem is that the influx cli utility several times stopped responding.
When I was using 0.13.0 from your rpm another influx cli instance showed that there is running DROP MEASUREMENTS query (40minutes+ for example). Using KILL QUERY xy caused the query disappeared from SHOW QUERIES but influx with -exec remained blocked.
I upgraded to nightly build (version 0.14.0~n201605230800) in hope the problem is solved here. But later I observed the same problem several times. The only difference is that SHOW QUERIES does't  show the running DROP MEASUREMENT query.
I tried to run strace to see where the influx cli is hanging and it shows:
strace -p 5970
Process 5970 attached
futex(0x9d76a8, FUTEX_WAIT, 0, NULL
There are writes into other measurements simultaneous to the DROPs in the database
Note:
the DROP MEASUREMENT is terribly slow (deleting one measurement each several seconds). The task is running for 24hours and still not finished. Although I am testing influxdb on a virtual machine whith rather poor disk subsystem.
Regards
Dalibor Toman

Is #6563 related? It seems like the inability to use Ctrl+C is one of two things here. The other seems to be that we should offer a way to perform this kind of action asynchronously.
Kill query also only works for SELECT queries at the moment. @rkuchan can add a caveat to the documentation so that this limitation is at least documented while we're working on trying to allow KILL QUERY to be used for more than just SELECT queries.
Does this sound correct?


Improve cli configuration and output

Add new option notruncate. If set true, cli logs will not be truncated
Add new option noreverse. If set true, CLI failure steps will not be in reversed order (see https://github.com/Codeception/CodeceptJS/pull/778)
Add new option outputStyle. Supports types: actor and pageObject.
So now there are several configuration options:
a. Steps: true, outputStyle: actor. Then in console you will see only I methods calls without pageObject history (as before 1.3.0 release)
b. Steps: true, Debug: false, outputStyle: pageObject. Then in console you will see pageObjects methods with arguments, but only top level (that are used in Scenario)
c. Steps: true, Debug: true, outputStyle: pageObject. Then in console you will see pageObjects methods with arguments, with internal calls of pageObjects and I method

so. lets take a look on example:
Test:
Scenario.only('Should get network info', async (I, blockchain) => {
  const resp = await blockchain.getBlockchainInfo(200, coin);

  const fee = await blockchain.getRippleFees();

  I.assertEqual(resp.fees, fee);
  I.assertOk(resp.height);
  I.assertOk(resp.height > 12801200);
});

a version output:
@ripple --
  Should get network info
    I send rabbit request "blockchain.get-blockchain-info", {"network":"XRP"}, {}
    I assert equal "200", 200, "Request 'blockchain.get-blockchain-info' return 200 status, but expected 200"
    I send request "https://s.altnet.rippletest.net:51234/", "POST", {"headers":{"Content-Type":"application/json"},"json":{"method":"server_state"}}
    I assert status code 200, 200
    I assert equal "10", "10"
    I assert ok "12886351"
    I assert ok true
  ✔ OK in 505ms

b version output:
@ripple --
  Should get network info
    blockchain get blockchain info 200, "XRP"
    blockchain get ripple fees
    I assert equal "10", "10"
    I assert ok "12886360"
    I assert ok true
  ✔ OK in 355ms

c version output:
@ripple --
  Should get network info
    blockchain get blockchain info 200, "XRP"
      sbus request body 200, "blockchain.get-blockchain-info", {"network":"XRP"}
          I send rabbit request "blockchain.get-blockchain-info", {"network":"XRP"}, {}
          I assert equal "200", 200, "Request 'blockchain.get-blockchain-info' return 200 status, but expected 200"
    blockchain get ripple fees
        I send request "https://s.altnet.rippletest.net:51234/", "POST", {"headers":{"Content-Type":"application/json"},"json":{"method":"server_state"}}
        I assert status code 200, 200
    I assert equal "10", "10"
        I assert equal "10", "10"
    I assert ok "12886368"
        I assert ok "12886368"
    I assert ok true
        I assert ok true
  ✔ OK in 1351ms

Sorry I need some more time to get through this )

Ok, my opinion is in this setup cli reporter is doing too much.
The code looks over complicated and I don't think I will be able to support it.
I think instead of providing 3 output modes it's better to provide different reporters and the ability to switch between them.


[fixes #3] made changes to fix output flag

This fixes the error when tried to give an output file path. Now, the given output file path is not checked if it exists, simply the extension of the given filepath is checked.

LGTM.


Java binary compilation instruction is incorrect.

Hi,
I noticed that after installing the java binary the instructions present both on the README and on the web page to compile the example class is wrong. When compiling it should also include the lib/protobuf.jar in the classpath but currently this is missing hence it is causing some headaches when getting started. Correct command to compile should be like this:
javac -d objs -cp lib/com.google.ortools.jar:lib/protobuf.jar \ examples/com/google/ortools/samples/RabbitsPheasants.java

Hello cemKoc,
Thank you for your remark. We updated the README with the right command.
Regards.


bots/minions

How to make our bots a copy of our cell .. ie with the same name and the same skin ..

Duplicate
@MastaCoder @Megabyte918 Close


Disable XR server when compiling without 3D

This PR disables XRServer and all XR classes when compiling without 3D. Also, this PR organizes servers/SCsub's SConscript lines by alphabetizing them (except with physics and XR at the bottom). The XR modules already disable themselves when compiling without 3D, but in master the classes are still compiled and referenced by the rendering code, from the main.cpp file, and exposed to scripting.
Note: There is a code order change in register_server_singletons's add_singleton calls. I don't think this will change behavior but please look at this.

Thanks!


Add policies to enforce or add Istio ambient mode

Description
Add two polices related to Istio ambient mode: enforce and add.
Ambient mode is an alternative to sidecar mode, for which Kyverno already has policies to enforce or add. Ambient mode is applied using a different namespace label, so these policies enforce or add that label.
Checklist

[x] I have read the policy contribution guidelines.
[x] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[x] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

Thanks for the contribution @StevenSmiley!


Update env_checker.py

Fix typo about Box maximum limit. Add return type annotations.

Thanks for spotting that, could you fix the tests. You should just need to update the IGNORE_WARNINGS list to include the new case

@pseudo-rnd-thoughts: I have updated the CHECK_ENV_IGNORE_WARNINGS list

Updated the CHECK_ENV_IGNORE_WARNINGS in two more test files. This should fix the failing tests.


AWS data download not working

Hello,
Thank you for the wonderful work you've done!
I’m trying to run the code from the repository, but I’m encountering an issue with downloading the data. When I run the commands below, they execute almost instantly without actually downloading any files.
Here are the commands I used:
aws s3 ls s3://diva360/raw_data/synced/ --no-sign-request
aws s3 cp s3://diva360/raw_data/synced/2023-05-03_session_blue_car_synced.tar.gz ./ --no-sign-request

aws s3 cp s3://diva360/processed_data/blue_car/ . --recursive --no-sign-request --exclude "*" --include "transforms*"
aws s3 cp s3://diva360/processed_data/blue_car/frames_1.tar.gz ./ --no-sign-request

Could you help me where I'm missing?

Thank you so much!


[zh] Resync certificate-signing-requests.md

[zh] Resync certificate-signing-requests.md with en page referring to #39180.

/lgtm
/approve


Fix Id handling

Fixes #1187

To not mess up the Spring Data Mongodb serialization I have changed now only the MorphiaSerializer in this port of this fix.


[Documentation] Python Automation script and workflow for github wiki

Description
Motivation
Create a GitHub wiki (a separate git repo relative to this repo) for the documentation across this GitHub repo. GitHub wikis have better visibility and navigation for users.
Background
GitHub wikis lack version control and are entirely separate relative to this source repository. To solve both of these issues, we use the documentation/markdown files from the main repo to update the Github wiki repository.
To maximize the benefit of the wiki, a hierarchical view is created in a sidebar. However, the mapping of the markdowns files to the sidebar poses some challenges.
The PR is an attempt at a very simple solution. A GitHub action that tracks changes to a set of files and triggers a python script. The script picks up pre-written comments in the set of files to generate a sidebar. From the sidebars linking reference, it copies over the files from the main repo to the wiki repo.
Pre-Requisites to Merge this in

[ ] Create a repository secret. Based on the legacy access rules, tick the repo permission box.

Outstanding issues

[ ] Improve the format of the sidebar, so categories don't repeat. Rolling the main README of a category into the category heading so that the heading is also a link and not just text.
[ ] Change the internal links of the Markdown files in the wiki to point to other paths in the wiki by either:

A regex change to the current wiki file names (aka wiki titles)
Changing the wiki file mapping to align with the internal links.

Issue
Fixes #133
Remaining/new issues
Type of change
Please mark the relevant option(s):

[x] New feature, functionality or library
[ ] Bug fix
[x] Code health or cleanup
[ ] Major breaking change
[x] Documentation
[ ] Other

List of changes

(Apologies in advance) Some formatting changes from auto linting of markdowns files. I don't like this as it has blown out the diff. I can try to remove if it's too messy.
Added a GitHub action that tracks change to .md files, sets env variables and runs a python script
Added a python script in a new directory called tools. Finds the path of .md files, maps them to the specified format described in the files, generates a GitHub wiki sidebar file and respective wiki files. Pulls the current wiki, syncs the files and pushes them.
Added comments at the bottom of .md files found from find . -name "*.md" | grep -v -e "vendor" -e "app" in the form of <!-- GITHUB_WIKI: category/subcategory/.../file_name -->.
Brief description of maintaining the wiki format comment in the docs/development/readme
Brief mention and linking to the wiki in the main readme.

Testing

[ ] make develop_test
[ ] LocalNet w/ all of the steps outlined in the README
d

Screenshot of output from example wiki on a personal fork: example wiki

Required Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have tested my changes using the available tooling
[ ] I have updated the corresponding CHANGELOG

If Applicable Checklist

[x] I have updated the corresponding README(s); local and/or global
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added, or updated, mermaid.js diagrams in the corresponding README(s)
[ ] I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

Just FYI @Olshansk; I am not able to set the Assignee, label, priority, etc. while creating the pull request during or after submitting the draft PR.
This is an example of a cross repo PR that has the buttons:

This is what I see while trying to creating a cross repo PR for pocket:

This is what it looked like after creating the draft PR, usually the words become clickable links to set the assignee etc.:

Maybe there is a different way, but those are the ones I was familiar with.

@okdas Could you help configure the secrets in our github repo to enable this via automation? See .github/workflows/wiki_sync_process.yml

@okdas Could you help configure the secrets in our github repo to enable this via automation? See .github/workflows/wiki_sync_process.yml

I somehow lost access to this functionality, I'll work on either getting access or setting the secrets with the infra team.

@okdas Could you help configure the secrets in our github repo to enable this via automation? See .github/workflows/wiki_sync_process.yml

I somehow lost access to this functionality, I'll work on either getting access or setting the secrets with the infra team.

Updated the settings. Let me know if you've taken another look

@profishional Let us know if you have time/capacity to tend to the changes we mentioned so we can get this PR over the finish line. If not, @okdas and I can push it ourselves

Ok I've solved the conflict. But can't merge "only those with write access to this repo can merge PR".

@profishional I can merge it for you. Can you update the changelogs first please? Again, I would do it on your behalf, but this branch isn't in our main repo.

I don't follow. You've merged it but you'd like me to update the Changelogs as another PR?

@profishional Sorry about that. I wrote the original message but then added an EDIT at the bottom:

EDIT: Ignore. I will make an exception because of how minimal the changes in each repo are.

We'd be updating every changelog saying "Added a GITHUB WIKI tag", so the changelog diff would be greater than the actual diff, which I saw as unnecessary overhead. I made an exception and merged the changes as is.
I also just created a followup ticket that's adding it in the missing READMEs here: #488.
I think this is really awesome btw!! https://github.com/pokt-network/pocket/wiki


How to implement dispatchTouchEvent() in fragment?

Hello,
This library works very well in activity but not getting touch detection in the fragment.
So how to implement in the fragment. And also don't get dispatchTouchEvent() method in a fragment.

any news here?

So it's not possible at the moment to implement touch detection inside a fragment? I would need put 2 square/rectangle inside an activity and put 2 different touch detector.

Hi, inside my mainactivity I put a FrameLayout in which there is another layout. Inside that layout I initialized sensey while in mainactivity i set dispatchTouchEvent like this:
    @Override
    public boolean dispatchTouchEvent(MotionEvent event) {
        // Setup onTouchEvent for detecting type of touch gesture
        //Sensey.getInstance().setupDispatchTouchEvent(event);
        rightFragment.sensey.getInstance().setupDispatchTouchEvent(event);
        return super.dispatchTouchEvent(event);
    }

However when I touch the screen, the touch event is being recognized in everywhere while it should recognized only inside the layout inside the fragment.
Can anyone please help me?


Icons for submenu

With Firefox 56, Firefox will support icons for context menus: https://blog.mozilla.org/addons/2017/08/10/webextensions-firefox-56/
I want to support this and actually have a working version:

(maybe another icon for custom search engine, don't know yet)
But I need to add compatibility checks, because it's only available for Firefox >56 and not Chrome/Opera. Should be fairly simple with runtime.getBrowserInfo().

Well, that was easy... only a try statement :)


RL with baseline: logsoftmax + metrics

This branch is outdated. This number of scalars is too big no feasable. Closing.


Update v3-1-migration-guide.md

layout analysis (../prebuilt-layout) ---> layout analysis
Fixed the link of "layout analysis".

Looks good. Thanks @shigeooooooooooo


Return validator errors in default response

This returns the validation errors as part of the default response.
For my use-case, including the errors in the response is valuable, but I'm not sure if this would be better as a configurable option.
One alternative would be to refactor responseValidationFn ad requestValidationFn, so they could be used to modify the response.

Hi @iadknet! I'm super glad you are contributing more :) Can't even express how much motivation you've got into me!
As for this PR, I think the best unobtrusive place to handle validation errors would be in your custom requestValidationFn. This way the application just executes as usual without interrupting usual flow. For my personal use case I do something like this:
const responseValidationFn = (validationReq, data, validation) => {
  logger.warn(`schema definition validation failed for ${validationReq.method} ${validationReq.originalUrl}`);
  logger.warn(util.inspect(data));
  logger.warn(util.inspect(validation, { depth: 7 }));
};

and the function itself is used as a part of middleware parameters as so:
  const opts = {
    schema: apiSchema,
    validateRequest: false,
    validateResponse: true,
    responseValidationFn,
  };
  app.use(validator(opts));

The reason I was using a very general default error handler, was to prevent exposure of application internals to API consumer by accident.
But with your scenario I guess you just want more data to be passed into next(), so you could do eventual handling there. Let's go with the second option you proposed and add a configurable option.

Btw, let's keep the commit style/naming consistent and try to follow this guide https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#type . I found this way of naming very maintainable, and keeping things tidy is usually a good idea.
Also would be good if we squash all the commits once the development on the branch is done. When I contributed to Ghost this was how they handled commit history - https://docs.ghost.org/v1.0.0/docs/git-workflow#section-clean-up-history, let's try this method as well, if you don't mind :)

@gargol Thanks!  For my use case, I am building a swagger validation proxy, so that I can run automated tests on our APIs (through the proxy) and have the tests fail if the requests or responses do not conform to the swagger doc - without having to integrate swagger validation with the services themselves, which may be Java, Node, Python, etc.
The problem I had using requestValidationFn, like you mentioned, was that I could not use it to modify the response, and when running tests through the proxy, I found having easy access to the validation errors in the response to be helpful.  Also, since requestValidationFn does not modify the response at all, the external tests may pass, even if swagger validation fails.

hey @iadknet! will you have time in a coming week to implement this feature behind the flag? will have half a day commute this Saturday, so was planning to do some work on this package :smiley:

I'll pick this up and add option flag for it.

Implemented in #34