YDR/YFT file size increase after import-export

Vanilla drawable geometries only use 2 bytes of float's 4 bytes for normals, sollumz exported geometries use full 4 bytes, so packed and deflated file size increased, maybe can processing normals when exporting to avoid it, this CW fork solve this problem https://github.com/fhsjxk/CodeWalker/tree/float16-normal

Vanilla drawable geometries only use 2 bytes of float's 4 bytes for normals

No, they don't. Check the vertex data of any model. Some rare ones use Dec3N (packed in 32-bits, 10 bits per X,Y,Z component), but the majority simply uses 32-bit floats, 4 bytes each component.

You cannot just add new entries to VertexComponentType and expect it to work, this enum comes from the game engine, the engine just won't know about any new entries.
https://github.com/fhsjxk/CodeWalker/blob/bc2c7ed762ee39ed51fc2dba942282684c142892/CodeWalker.Core/GameFiles/Resources/VertexType.cs#L26-L30
Similarly, you cannot just add a special case in GetComponentType and expect the game to know about it. The game uses that Types field to know the type of each component. The game will still expect 32-bit floats.
https://github.com/fhsjxk/CodeWalker/blob/bc2c7ed762ee39ed51fc2dba942282684c142892/CodeWalker.Core/GameFiles/Resources/Drawable.cs#L4290-L4299
And that's just not how you get a half value from a float value, you don't just copy 2 bytes of the float.
https://github.com/fhsjxk/CodeWalker/blob/bc2c7ed762ee39ed51fc2dba942282684c142892/CodeWalker.Core/GameFiles/Resources/Drawable.cs#L3974-L3979

Vanilla drawable geometries only use 2 bytes of float's 4 bytes for normals

No, they don't. Check the vertex data of any model. Some rare ones use Dec3N (packed in 32-bits, 10 bits per X,Y,Z component), but the majority simply uses 32-bit floats, 4 bytes each component.
You cannot just add new entries to and expect it to work, this enum comes from the game engine, the engine just won't know about any new entries. https://github.com/fhsjxk/CodeWalker/blob/bc2c7ed762ee39ed51fc2dba942282684c142892/CodeWalker.Core/GameFiles/Resources/VertexType.cs#L26-L30VertexComponentType
Similarly, you cannot just add a special case in and expect the game to know about it. The game uses that field to know the type of each component. The game will still expect 32-bit floats. https://github.com/fhsjxk/CodeWalker/blob/bc2c7ed762ee39ed51fc2dba942282684c142892/CodeWalker.Core/GameFiles/Resources/Drawable.cs#L4290-L4299GetComponentType``Types
And that's just not how you get a half value from a float value, you don't just copy 2 bytes of the float. https://github.com/fhsjxk/CodeWalker/blob/bc2c7ed762ee39ed51fc2dba942282684c142892/CodeWalker.Core/GameFiles/Resources/Drawable.cs#L3974-L3979

Maybe my expression is not clear, I mean this

For normal, first 2 bytes of every components are zero
And these are dumped vertices normal data of cs_x_array02.ydr

first 2 bytes are also zero

So the normals are rounded off. Each component still takes 4 bytes in memory once the resource is loaded by the game, guess it might help with compression with all the repetitive 0s, but compressed file size doesn't matter much.


Use same structure for build and dev

Changes:

install the bower dependencies into the src/third_party/bower directory
move the code to be built from build/dev/uproxy to build/src

The directory structure when developing becomes the same as when building, making it easier to reason about it, and getting proper editor support without the need for workarounds.

This change is

:100: to pushing build/dev/uproxy up a level! Something about build/src looks a little bit funny to me vs. /build/dev...do you reckon build/ will basically contain a full mirror of the top-level folder? FWIW, I do think we'll get rid of build/dist which would fit in with that model.

Just chatted about it, let's go with build/src :-)
:+1:


Add publish operation to iotdata

Adding very small feature.

Coverage increased (+0.003%) to 92.234% when pulling e2f14440003033fdcef8b4e0034b00bc11a81837 on toshitanian:add-publish-operation-to-iotdata into e227a30aa9269585ef854d0a9a27552014829266 on spulec:master.

Coverage increased (+0.003%) to 92.234% when pulling 3526b9097f40253feef3a5f1428843e7397a13a3 on toshitanian:add-publish-operation-to-iotdata into e227a30aa9269585ef854d0a9a27552014829266 on spulec:master.


3 tags showing as one through MQTT integration

I have a really weird issue. Could be related to the new version of discovery (0.2.1) or not because I had only one ruuvitag before updating and now I have three. So can't know for sure. Anyway, I'm seeing only one device on the MQTT integration and it's contents are shifting between the three RuuviTags and values are always shown as Unknown.
When I click open the device list of the MQTT integration it randomly has one of the Ruuvis shown and if I click on that one the entities inside might be the same Ruuvi's or another one's. Also if I then click one of the entities it might show different Ruuvi's information. See below screenshot (I've named them Ruuvi-1, Ruuvi-2 and Ruuvi-3):

Any ideas?
BR,
Petteri

I agree, what a weird issue :/
If you only use MQTT integration with this addon, you can remove the MQTT integration to clean up and re-install the MQTT. It's not an elegant solution, but it should works.
Can you give me the hardware where Home Assistant run? On MacOS, for example, tag "ID" is obfucate and can change. But not on RaspberryPi (they never change). And Home Assistant need a fixed id. Maybe a lead?

Release 0.2.3 fix some bugs that might be related to yours. Tell me if an update fix your problems.

That fixed this! Everything is in order now. Thank you 👍

Great, good news! :)


bpf: add missing drop notifications

Fix up a few more missing drop notifications.
Add drop notifications from various error paths in the BPF datapath.

/test

note: for the 1.12 / 1.13 backports, the last patch in the series isn't required - hs-ipcache was only introduced in 1.14.


Resize Route not Working

I have tried the /resize route but it is cropping the image. I have found the /size route is working fine. is it an expected behavior ?

Yes, all the image transformantions implies cropping.

Actually for me the /resize route /crop route is doing the same purpose , where as /size is working as expected.

Try passing 'force=true' query param and /transform endpoint. Cropping should not be enable in that case.


Fix 500 Server error for some CSVs

For some CSVs the following error occurs:
500 Server Error: Internal Server Error for url: https://nassgeodata.gmu.edu/axis2/services/CDLService/GetCDLStat

I think the issue is we need to slow down our requests to the cropspace API to reduce the load on that server.

Looking into this further and the issue is with CSV files that have empty values for latitude and longitude.
This results in nan,nan,nan,nan for an albers bounding box which cause the cropspace 500 error.


File type from vichuploadbundle

Hello, I've an entity called "MediaObject" and an Entity called "MediaObjectImage" that extends MediaObject.
In MediaObjectImage I've this property:
/**
     * @var File|null
     *
     * @Assert\NotNull(groups={"media_object_create"})
     * @Assert\File(
     *     maxSize="5242880",
     *     mimeTypes = {
     *          "application/jpg"
     *      }
     * )
     * @Vich\UploadableField(mapping="media_object_image", fileNameProperty="filePath", mimeType="mimeType")
     */
    private $file;

How can I "pass" a file from Fixtures yaml? I've tried with the path like:
App\Entity\MediaObjectImage:
media_financial_image_1:
file<EMAIL_ADDRESS>id : '0d1253f2-ebb6-4194-a22c-57c4b2e10b94'
But the error is:
  Expected argument of type "Symfony\Component\HttpFoundation\File\File or null", "string" given at property path "file".

Can you help me? Thanks

I think you would either need to create an instance of the file first:
Symfony\Component\HttpFoundation\File\File:
  image_1:
    __construct: ['./build/dummy/image/image.jpg']

(this is pseudo code you would need to check how that object needs to be instantiated)
Or rely on a custom data provider

Thanks 😄


Add TrackPoint extensions

Hi,
I would like to use track point extensions in generated xml files so I can add heart rate, candence and other metrics. It should look like this:
<trkpt lat="54.8591470" lon="-1.5754310">
<ele>29.2</ele>
<time>2015-07-26T07:43:42Z</time>
<extensions>
<gpxtpx:TrackPointExtension>
<gpxtpx:atemp>11</gpxtpx:atemp>
<gpxtpx:hr>92</gpxtpx:hr>
<gpxtpx:cad>0</gpxtpx:cad>
</gpxtpx:TrackPointExtension>
</extensions>
</trkpt>

It is simple to implement:
if (point.extensions) {
      const trackExtension = trkpt.ele('extensions').ele('gpxtpx:TrackPointExtension');
      Object.keys(point.extensions).forEach((key) => {
        trackExtension.ele(`gpxtpx:${key}`, point.extensions[key]);
      });
    }

Could you add it to this library?

Hi there,
I've just released v1.2.0 which should let you add extensions as you've requested (hopefully 🙂). Let me know if something doesn't work as expected.
Thanks for the request!

@pdhoopr I tried it and it works great! Thanks.


chore-setup reduxtoolkit#42

what were done here  on that PR

install redux/toolkit and react-redux dependancies
reaate states store
create reducer function and define events right there
demostrate who things work on user interface finished

how this can be tested

clone https://github.com/atlp-rwanda/ecommerce-app-legends-fe and switch to ch-Setup-Redux#42
run npm i install in your terminal
run npm run dev to start app on web browser
click the buttons there to see how they are dynamically render the contents

thanks all get fixed


Remove warning that VERSION is redefined

I noticed the tests were complaining that VERSION was redefined

Thanks! Good catch.


Results of pre-training stage 1

Thank you for using the Issue submission template. Please follow the steps below to provide relevant information. We will prioritize issues with relatively complete information. Your cooperation is appreciated.
Hint: Fill in the [ ] with an x to mark it as checked. Delete any option that is not related to this issue.
Please check the following before asking

[x] Due to frequent dependency updates, please ensure you have followed the steps in our Wiki
[x] I have read the FAQ section AND searched for similar issues and did not find a similar problem or solution
[x] Third-party plugin issues: e.g., llama.cpp, text-generation-webui, LlamaChat, we recommend checking the corresponding project for solutions

Choose the issue type
Base model:

[x] LLaMA
[ ] Alpaca

Issue type:

[ ] Download issue
[ ] Model conversion and merging issue
[ ] Model inference issue (🤗 transformers)
[ ]  Model quantization and deployment issue (llama.cpp, text-generation-webui, LlamaChat)
[ ] Performance issue
[x] Other issues

Describe the issue in detail
Great work! Can I ask how were the results of pre-training stage 1? Can you elaborate more on why you chose to do both stages (instead of only stage 1 or only stage 2)? Thank you for your time!

Provide a screenshot or log of the issue
(If necessary) Please provide a text log or screenshot to help us better understand the issue details.

After PT stage-1, the CLM loss was about 5~6.
(We forgot to train LM head in PT stage-1. We expect the loss can be much lower with a trainable LM head).
The two-phase training was our initial exploratory plan, however it is not guaranteed to be optimal.
As for the 13B model, we directly apply the PT stage 2 without PT stage 1, since training a 13B model is more expensive and we find the PT stage 2 is more training-efficient.

@airaria Can you please tell me how to perform the Pre-training stahe 1 using the file run_clm_pt_with_peft.py?
Do we have to comment out the PEFT code, i.e., comment out:

PeftModel.from_pretrained(model, training_args.peft_path) and
get_peft_model(model, peft_config)

Do I understand correctly?


Initial implementation of Remove

Adds an initial implementation of the remove directive

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


Increase the timeout for kustomize apply to give certmanager more time  kubeflow/manifests#806

Certmanager appears to take a long time to start see kubeflow/manifests#806
and  jetstack/cert-manager#2537

We are seeing Kubeflow deployment failures because deployments
fail because they can't invoke the certmanager webhooks.

I suspect the error is being triggered when applications try to create
cert-manager resources and fail because the webhooks for cert-manager
are unavailable.

Add logging to clarify which application was being deployed when
an error occurred.

Bump the timeout from 5 to 10 minutes to give cert-manager time to start

This change is

/assign @richardsliu
/assign @kunmingg

/lgtm
/approve
/hold
Adding a hold in case you want to fix the typo, otherwise you can remove it and merge.

Here are the logs from the presubmit.
The log messages now bracket the deploy with the name of the application being deployed.
e.g.
2020-02-02T20:13:58.780220317Z  main            util.py                     72 INFO     time="2020-02-02T20:13:58Z" level=info msg="Deploying application kubeflow-roles" filename="kustomize/kustomize.go:172"
2020-02-02T20:13:58.903482552Z  main            util.py                     72 INFO     clusterrole.rbac.authorization.k8s.io/kubeflow-admin created
2020-02-02T20:13:58.981223772Z  main            util.py                     72 INFO     clusterrole.rbac.authorization.k8s.io/kubeflow-edit created
2020-02-02T20:13:59.058537460Z  main            util.py                     72 INFO     clusterrole.rbac.authorization.k8s.io/kubeflow-kubernetes-admin created
2020-02-02T20:13:59.140292315Z  main            util.py                     72 INFO     clusterrole.rbac.authorization.k8s.io/kubeflow-kubernetes-edit created
2020-02-02T20:13:59.221192455Z  main            util.py                     72 INFO     clusterrole.rbac.authorization.k8s.io/kubeflow-kubernetes-view created
2020-02-02T20:13:59.297529142Z  main            util.py                     72 INFO     clusterrole.rbac.authorization.k8s.io/kubeflow-view created
2020-02-02T20:13:59.297745760Z  main            util.py                     72 INFO     time="2020-02-02T20:13:59Z" level=info msg="Successfully applied application kubeflow-roles"

Hopefully this will make troubleshooting easier.
kubeflow-kfctl-presubmit-kfctl-go-iap-211-5f27b3e-7488-1cc0-1476697611.log.txt

/lgtm
/approve

/hold cancel


Branding Site: Administrative Plugin

Recommendation
If we add the Administration Plugin and the Git-Sync plugin, we can create a self-sustaining and backed-up site using GitHub webhooks. In addition, updates and changes to content can be made entirely in-browser, which means only our Server Administrators will need access to the AWS instance via SSH. The *Site Administrators can simply have a User created for them and be given access.
Admin Plugin
https://learn.getgrav.org/admin-panel
Git-Sync Plugin
https://github.com/trilbymedia/grav-plugin-git-sync
Status
@mmilutinovic1313 already explored this option a little bit on the Server, but it appears he made have had some Permissions difficulties.
Tested locally and the Admin Plugin works perfectly.

I can confirm that admin plugin is working locally - should be able to get this working on the server once we get merges taken care of tonight/tomorrow morning :)

Update: This has not been successfully ported to the server.


Chapter 11

Page 372, Equation 11.25 and 11.26
"T" -> transpose symbol

Page 378, line above Equation 11.58
"emulalate" -> "emulate"

fixed, thanks


Add Manufacturing Solution page to website

Story
No response
Description
Navigation:
Add Solution to top navigation bar
Add Manufacturing to the submenu
Other solutions will be added over time
Title: Node-RED for Manufacturing
URL: https://flowforge.com/solution/manufacturing
Header Graphic: ideally a picture of a manufacturing equipment line
Text: https://docs.google.com/document/d/1EBpmv4sD68xMcDd0_tryfbGJ8yrMV2Ln7psd5fMmHww/edit?usp=sharing
Competitor pages:
https://www.crosser.io/solutions/industry-40/industrial-iot-data-integration/
https://www.crosser.io/solutions/industry-40/iot-edge-stream-analytics/
https://www.losant.com/solutions/industrial-equipment-monitoring

Rough layout agreed with Ian:

Mark-itechture diagram first drafts:

feedback please @iskerrett

in context of the page too - think it needs some header/text between the 4 segments and the actual diagram?

will have different icons for each piece of equipment - and thinking about how I style the "Factory" box to look a bit more like a factory too...

end of my day, but a quick Factory layout idea:

I am liking how this is coming together. I am not sure the Node-RED Dashboard really works. I think we will need more of a dashboard screen shot with the word 'dashboard'. It also doesn't need to be a Node-RED dashboard.
FWIW, I wouldn't worry about trying to stylize a factory. The word factory is sufficient. I worry if we add too much it will get too busy.

in context of the page too - think it needs some header/text between the 4 segments and the actual diagram?

I am not sure this is needed. I hope it would be self-evident.

updated to show an actual dashboard, didn't add text "dashboard" as not sure it's needed:

fatory variant

as a future iteration, would love to make this a Lottie Animation

Equipment icons in & slight shadow added:

Equipment icons in & slight shadow added:

Looking great. My preference is without the factory imagery. I think it makes it too busy.

as a future iteration, would love to make this a Lottie Animation

fwiw, I really don't like these types of animations on website. Unless they are adding specific value in the communication, I find it is just adding noise and distractions.

fwiw, I really don't like these types of animations on website.

We'll have to agree to disagree on that one :)

@iskerrett any update on what you'd like the CTA to be here?

Still awaiting confirmation on the CTAs for this page

How about 'Contact to discover how FlowForge can help Connect, Integrate, Visualize & Automate your industrial data.'
I know this is a bit long for a CTA. We could put this at the end with a Contact Us button.
At the top, we could have a Contact Us button and the hover help would be the sentence above?

Need to replace the "Edge" pictogram with something more manufacturing specific
Refinement of content to be done

Can you remind me what refinement is required?

Polish the diagram, make it svg ideally so that it renders crisper on screen
Really think we need a  element to introduce the FlowForge selling points, doesn't look right just going straight into a paragraph.

In the original document I have a title 'FlowForge Adds DevOps to Node-RED for Manufacturing'

Not convinced the 2x2 layout works in the FlowForge section due to the longer headers, could try a similar format to what we have on the home page?

Sure give it a try.
Have you decided to note use images for the text the describes Node-RED?

Can you remind me what refinement is required?

It wasn't anything in particular, but you mentioned that you'd do a review of the content once it was in place, to see if any areas could do with a trim.

In the original document I have a title 'FlowForge Adds DevOps to Node-RED for Manufacturing'

So you did, apologies, not sure how I missed that.

Have you decided to note use images for the text the describes Node-RED?

I think so, it feels clean enough as it is, and with the markitecthure diagram, feels like it has enough visual engagement in that section.

Update:
https://user-images.githubusercontent.com/99246719/222516116-37eece9e-6690-4978-9c3f-8ae6ae01e154.mov
Happier with the FF section than I was, still feels like it needs a bit more polish, but content is in. Next on the list is:

[ ] Update the top Pictogram to be more Manufacturing-based
[ ] Update the Markitechtecture diagram

How about 'Contact to discover how FlowForge can help Connect, Integrate, Visualize & Automate your industrial data.'

@iskerrett would you like an inline, purpose-built form - or redirect to contact form from here?

Can you remind me what refinement is required?

It wasn't anything in particular, but you mentioned that you'd do a review of the content once it was in place, to see if any areas could do with a trim.

I think the content amount looks fine so I don't think a refinement is required.

Adding in the CTA at the moment, have this for the bottom of the page:

Was researching other Solutions pages, and found that HiveMQ use "Book a Meeting" as a CTA instead, which I quite like? Feels more premium? Could link to a Calendly?

Please leave it with the Contact Us form for now.

now live: https://flowforge.com/solutions/manufacturing/


storage incentives: add stamper putter to api

#873
changes that were made outside the scope of this PR are due to complaints from deepsource.

@zelig but the CI is failing. I don't know any way around that, particularly that deepsource scans different parts of the codebase randomly on different PRs :man_shrugging:


Emitting on activity start

Hello 👋
After some significant use of SerilogTracings, I have found that I keep falling into the following pattern when debugging long-running sequential activities.
using (LoggerActivity _ = logger.StartActivity("Do the thing"))
{
    logger.Information("Do the thing");
    // Do the thing
}

Which then gives me the desired
Do the thing
Do the thing completed in (2)ms

Ideally we could just use
using (LoggerActivity _ = logger.StartActivity("Do the thing"))
{
    // Do the thing
}

And get the desired output.
Would it be reasonable to suggest that we could investigate making this default functionality? Emitting on start and completion would assist with some sequential debugging and make the tracings library significantly more well-rounded.

Thanks for the suggestion! I've used a similar approach in the past, but I think it's too opinionated to be implemented in the base library. Creating an extension method on ILogger to do this is the way to do this for now.

Since the original post, I have been considering this further, and I agree that making it the default functionality is quite an opinionated approach.
But I also thought of an alternative which, if it's interesting for the folks here, might be worth considering;
If there was an implementation on LoggerActivity which spits out an early emit without the span and returns a LoggerActivity via a fluent interface, you could achieve something like this:
using (LoggerActivity _ = logger.StartActivity("Do the thing").Announce()
{
    // Do the thing
}

And get
Do the thing
Do the thing completed in (5)ms

I'm not exactly sure about the terminology yet, but I would be happy to give it a go. It looks plausible to me from my initial tinkerings.

The API and terminology work well; the feature overall is still something I think might be too opinionated for the library at this point, though.
Here's a quick extension method version:
static class LoggerActivityExtensions
{
    [MessageTemplateFormatMethod(nameof(messageTemplate))]
    public static LoggerActivity AnnounceActivity(this ILogger logger, string messageTemplate, params object?[] args)
    {
        logger.Information(messageTemplate, args);
        return logger.StartActivity(messageTemplate, args);
    }
}

Writing it helps surface questions like: What level should the "announcement" be at? Should the activity still be announced even when the span is suppressed via sampling? Would it be nicer to include some indication in the announcement event that it's related to an activity (such as with the text " starting"? (The last one important because the " completed in x" text is only added in text outputs, leaving the two events with identical messages in other sinks.)
All are pretty straightforward to tweak via the extension method implementation, but would need either baked-in opinions or configuration if we made this a base library feature.


exporter fails to start - FileTransferCouldNotStart error

FileTransferCouldNotStart error thrown when starting a data transfer job.
This blocks all exports.
errors:
HttpError 404 when requesting https://storagetransfer.googleapis.com/v1/transferJobs?alt=json
returned \"Failed to obtain the location of the source S3 bucket. Additional details: HTTP/1.1 404 Not Found\". Details:
Failed to obtain the location of the source S3 bucket. Additional details: HTTP/1.1 404 Not Found

Attempted solutions:

restart exporter - did not work. This error occurred also on a recently started exporter
check s3 bucket exisstence - bucket did not exist, because dataset was a dcp/1 dataset.

see log excerpt
2023-10-23 14:35:07,615 - TerraSubmissionExporter - INFO - submission_uuid:fce97270-fce0-4744-8a4e-a93d95521852 - export_job_id:65368494fb47305198b95cf5 - project_uuid:005d611a-14d5-4fbf-846e-571a1f874f70 - Starting data transfer
2023-10-23 14:35:08,333 - TerraSubmissionExporter - ERROR - submission_uuid:fce97270-fce0-4744-8a4e-a93d95521852 - export_job_id:65368494fb47305198b95cf5 - project_uuid:005d611a-14d5-4fbf-846e-571a1f874f70 - Rejecting message: {\"exportJobId\":\"65368494fb47305198b95cf5\",\"submissionUuid\":\"fce97270-fce0-4744-8a4e-a93d95521852\",\"projectUuid\":\"005d611a-14d5-4fbf-846e-571a1f874f70\",\"callbackLink\":\"/exportJobs/65368494fb47305198b95cf5\",\"context\":{}} due to error: FileTransferCouldNotStart
2023-10-23 14:35:08,334 - TerraSubmissionExporter - ERROR - submission_uuid:fce97270-fce0-4744-8a4e-a93d95521852 - export_job_id:65368494fb47305198b95cf5 - project_uuid:005d611a-14d5-4fbf-846e-571a1f874f70 -
Traceback (most recent call last):\n","stream":"stderr","time":"2023-10-23T14:35:08.336754558Z"}
2 File \"/app/exporter/terra/gcs/transfer.py\", line 25, in start_job
 client.transferJobs().create(body=transfer_job.to_dict()).execute()
 File \"/usr/local/lib/python3.10/site-packages/googleapiclient/_helpers.py\", line 130, in positional_wrapper
 return wrapped(*args, **kwargs)
 File \"/usr/local/lib/python3.10/site-packages/googleapiclient/http.py\", line 938, in execute
 raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: \u003cHttpError 404 when requesting https://storagetransfer.googleapis.com/v1/transferJobs?alt=json returned \"Failed to obtain the location of the source S3 bucket. Additional details: HTTP/1.1 404 Not Found\". Details: \"Failed to obtain the location of the source S3 bucket. Additional details: HTTP/1.1 404 Not Found\"\u003e
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
 File \"/app/exporter/queue/listener.py\", line 39, in try_handle_or_reject
 self.handler.handle_message(json_body, msg)
 File \"/app/exporter/terra/submission/handler.py\", line 38, in handle_message
 self.submission_exporter.start_data_file_transfer(export.job_id, export.submission_uuid, export.project_uuid)
 File \"/app/exporter/terra/submission/exporter.py\", line 26, in start_data_file_transfer
 self.terra_client.transfer_data_files(upload_area, project_uuid, job_id)
 File \"/app/exporter/terra/transfer.py\", line 32, in transfer_data_files\
 self.gcs_transfer.start_job(transfer_job)
 File \"/app/exporter/terra/gcs/transfer.py\", line 30, in start_job
 raise FileTransferCouldNotStart() from e
exporter.terra.gcs.exceptions.FileTransferCouldNotStart

happened becuase the dataset was a dcp1 datgaset and the s3 uri was pointing to a non existent bucket.


Ability to Compile Dart Code and Distribute it as libraries

Every Code distributed as library is visible for user, like c/c++ if there is an ability to distribute as bin libraries is better feature

Moving this to the SDK repo because I think it's slightly more of an implementation concern than a language design issue.

Or is their a way to dart code as minifed and somewhat obfuscated format ? The goal is to be able to use this dart code as a dependency in a flutter application, but without exposing all the source code in an extremely easy to read format.

Is there any update on the matter?

The agora_rtc_engine seems to be doing it. Any insight on how the package https://pub.dev/packages/agora_rtc_engine is able to implement it - hide the source code, yet allow users to use it?

@munificent Not really. All they have done is, not share the URL for the source on the pub. Once you add it as a dependency, all the code, in that package at-least, can be browsed. And even the source can be download from https://pub.dev/packages/agora_rtc_engine/versions

@mncmeditech Not really. All they have done is, not share the URL for the source on the pub. Once you add it as a dependency, all the code, in that package at-least, can be browsed. And even the source can be download from https://pub.dev/packages/agora_rtc_engine/versions

Thanks a lot for the clarification.


derive(Format): Support more than 256 variants

Closes #63

Nice! Thanks for the PR.
bors r+

Thanks! (and sorry for the repeated rebase work)
bors r+

No problem, these were minor anyways. :)


Test emoji

Thanks for this repo.
----Test-----
:smile:
:laughing:
:sweat_smile:

test too :grinning:


Tracking Issue: 2nd beta inversion

This is an extremely important issue on the difference in expressivity between Stitch and classic dreamcoder. Our goal is to close the gap between the two.
For the map example from original dreamcoder you need true higher order inventions instead of just context threading (context threading: #36), since you need to have #0 (car $0) in the invention instead of just #0 $0.
I suspect this might incur a decently large overhead and my hope is that we can run our dreamcoder benchmarks and find that it's unnecessary for most domains anyways. However for McCarthy lists we would need it to discover map of course and probably other recursive functions. I suspect this doesn't come up a ton but we will see when we properly benchmark against dreamcoder.
If we do need to implement it I think that we could, and we'd want it to be guarded by a commandline flag like --higher-order or --no-higher-order in case it slows it down a lot.
Ideas:

if we did do this I think only doing one level of higher-orderness is good
We have the first_mergeable_zid_of_zid which also serves to tell us all of the suffix zippers
Importantly multiuse (and multiarg) can show up in this, so it really is an inner search
Thankfully it's highly constrained because I believe youd only ever want a higher order function if you're trying to abstract something that has a variable (otherwise you just would abstract it directly?). Or wait is that not true, would you sometimes want to do this to effectively punch out a scaffold in order to make your function? Like if instead of car $0 it was car empty it would still be more compressive to create a higher order function. important point

is there some way you could reuse search work you already did since truly these inner functions are inside? We dont keep old results around but idk maybe think about it.
hm essentially we want to recursively run compression or AU among the set of arguments, and its ok if it doesnt work for all arguments but that will cause subsetting issues.

This search problem might not be super bad bc it's such a narrow subset of nodes and small tree? Hopefully at least though also this might be attempted for big things
important to ask if the subset property still holds with these kinds of suffix merges. As long as we start from the whole thing punched out then yes it's fine
If you added this your upper bound would probably have to weaken since arguments could be partilly sent back into the body

2022-03-16 more ideas:

I think there are some constraints you could put on the inner search to make it much more tractable:

limit the arity to 1 or at most 2
forbid multiuse, or perhaps just do arity 1 + multiuse

I think this recursive inner search could benefit a lot from optimizations like #76 because the more general speedups we get the more we're okay with putting compute into this inner search
If the node subsets are small enough the inner search might really not be big!
and I suppose youd have to weaken your upper bounds on the args since they might get refined in this way. Or maybe just do that for the very last of the args
wait what if you left partial invs on the worklist and either right then or in a second pass you explore doing the inner search

2022-03-16:

important I think this applies beyond just cases where the argument to the inner function is a variable. Even without any $i involved I think this is useful. First we antiunify to get the initial var of #i, then we antiunify across the arguments to put some shared chunk of the arguments back into the body. Thats a beneficial thing to do even if $i vars arent forcing you to do it or anything.
I wrote this up on notebook paper but basically I think non-HOFs can capture patterns in a single contiguous chunk of tree, while HOFs let you capture this PLUS having an arbitrary varying gap followed by another contiguous chunk of matching tree below. So it allows for patterns with gaps in them.

closing with #121


Why use express in client folder

Hi, i'm using express and apollo server to build server side, react for client but i switch to nextjs for seo frienly. In your project, i dont understand why use express in client folder. Can we use it in server folder and seperate with client (for nextjs and client dependency)

Hello! Yes i actually have an express server on the server (because apollo server is built on express). Then i also have an express server on the client. The express server on the client is a custom server that belongs to the server side rendering for Next.js.
You don't need a custom express server for Next.js, but if you want to use other features/customize the server side rendering Next.js give you the option to use a customized express server.

Got it, many thanks


psn.xposed not loading correctly

psn.xposed website just gives me the ps4-exploit-host rather than a database. how do I access the database that's collected and submit my entitlements?

Shoot me a PM on Twitter or Discord


Question about Trident Module Valid Range

Anybody know if the valid ranges correspond to the object size in the original image or the resized image, say 800x1200?
Also, in class RpnParam, what does the image_anchor mean?
Many Thx!

if valid_ranges_on_origin = True, then it is correspond to the object size in the original image.


FR: Show online status

Hi
THis is mainly an Android request but it would be nice if this is also possible in other platforms.
It would be very nice if the online status of a person can be seen somewhere in the main ui without clicking in the user's room then opening the room details.
I would say most messanging apps provides that kind of detail and is useful when conversing.
thanks

Unfortunatly, the presence API has been disabled server side for performance reason.

Thanks, that makes sense. Would it possible to at least see if the person read/saw the message or not?
The problem is that it is hard to tell if the person interacted with the message or the room, that seems a bit better on the desktop app.
Thanks

The read receipt are here for that, and should work on Android. Can't you see them?

not sure, I do not see receipts. I need to what that is


readme example contains error

If i try to run the example from the demo: talib.SMA([23,34,12,34,23], 12) it will yield an error:
TypeError: Reduce of empty array with no initial value
    at Array.reduce (native)
    at Object.SMA (/Users/h/git/makers_demo/node_modules/ta-lib/index.js:142:24)
    at repl:1:11
    at ContextifyScript.Script.runInThisContext (vm.js:25:33)
    at Object.exports.runInThisContext (vm.js:77:17)
    at runInContext (/Users/h/.nvm/versions/node/v6.9.5/lib/node_modules/coffee-script/lib/coffee-script/repl.js:58:17)
    at REPLServer.eval (/Users/h/.nvm/versions/node/v6.9.5/lib/node_modules/coffee-script/lib/coffee-script/repl.js:47:25)
    at bound (domain.js:280:14)
....

Hi Hems... thanks for pointing that out. You need to passt an array which has a greater length than the provided sample length (12). I will fix the example as soon as possible ;)

yeah i got that! was just doing a demo the other day and noticed that copy+paste from readme doesn't work ( :

I extended the Readme ;)


Error: ENOENT: no such file or directory, stat<EMAIL_ADDRESS>

Hi! After upgrading to 0.3.0 I have now this error message:
Error: ENOENT: no such file or directory, stat<EMAIL_ADDRESS>It looks like app folder hasn't been published:
https://www.npmjs.com/package/@cap-js/graphql?activeTab=explore

Hi @ThePlenkov, you're right, the problem is that unfortunately the app/ folder wasn't included when publishing the package to npm (needs to be included in the package.json). Thank you for making us aware of the issue!

Hi @ThePlenkov, we just released a new version that includes this fix


Added get_gas_all as part of the plugin.

Stack:

#1857
#1855
#1854
#1853
#1852 ⬅
#1851

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

crates/cairo-lang-starknet/src/plugin_test_data/hello_starknet line 112 at r1 (raw file):
Previously, orizi wrote…
stability is the constant case - good or not good.
if it enabled us to bind the initial gas cost at a lower cost, it would be good (otherwise, we need to always collect for the get_gas_all worst case, which can get quite large)

If you start with a value that is too high, you can get a refund in the good case right?


[Feature] Thread-safe implementation

To support multi-threading in QCOR, we need to audit and fix the usage of some static global variables.
This may include:
(1) the qcor runtime (qrt_impl): building up circuit IR for quantum kernels, optimization, submission, etc.
(2) the qpu instance cached in the internal_compiler namespace (which will be used by the qrt for execution)
When pursuing this feature, we could also look into moving the qpu shared pointer as a member of the qrt implementation for future maintainability.
Note: For JIT compilation (using QJIT, which is non-copyable), we need to make the entire class thread-safe (which I think we've done in https://github.com/ORNL-QCI/qcor/pull/157)

Hi @1tnguyen, I'm now thinking of how to fix this. Essentially, my idea is to create a std::map of thread ID and the corresponding instance when ::quantum::initialize() is called. The map will be like std::map<std::thread::id, std::shared_ptr<QuantumRuntime>>
, and std::map<std::thread::id, std::shared_ptr<Accelerator>>.
For now, I'm assuming that ::quantum::initialize() is manually called at the very beginning of a function that is executed by a std::thread. For example:
void foo() {
    ::quantum::initialize("qpp", "empty");
    ...
}

int main() {
  std::thread t0(foo);
  std::thread t1(foo);
  t0.join();
  t1.join();
}

However, we'll need to discuss how to facilitate this. For example, we could develop a bit higher-level API that tells the runtime where the entry point of each "threaded" part is. Or we could create a compiler pass that automatically inserts such an initialization call.
Going back to my original point. For (1) (qrt_impl), I'm thinking of 1) creating a thread-qrt_impl map after this line (https://github.com/ORNL-QCI/qcor/blob/master/runtime/qrt/qrt.cpp#L154) and preparing a getter function that returns an appropriate qrt_impl that corresponds to a current thread id, and 2) replacing all the occurrence of qrt_impl with the getter function.
For (2), I'll do a similar thing in setAccelerator(), for example, around this line (https://github.com/ORNL-QCI/qcor/blob/master/runtime/qrt/internal_compiler/xacc_internal_compiler.cpp#L121). We'll also have to update the function so it can return a newly cloned instance every time it is called by a newly created thread though.
I also agree with you that we can move the qpu into the qrt_impl, which we could discuss more.
Please let me know if you have questions or suggestions.
Thanks!
Akihiro


Export all rows via export_visual_data API

Please provide the options to export all rows from visual data without having to feed number to rows keyword.
export_visual_data(page_name, visual_name, rows=all_rows, export_data_type=models.ExportDataType.SUMMARIZED.value)
The documentation  doesnt explain about it.

Thank you for your input, we updated the method to return all rows by default, see new documentation.


context labelled differently

pytest --cov=test_foo --cov-context=test; coverage report --contexts=test_foo
environ identical to #900
setting dynamic_context = test_function in .coveragerc, i get context reported as
test_foo.test_foo
but without that setting, the context is reported as
test_foo.py::test_foo|run

There's no single definition of what the context labels should be.  The built-in "test_function" uses one style, and the pytest plugin uses another more elaborate style.
I hadn't thought much about the situation where two different mechanisms are trying to set contexts. Maybe there could be some error handling or warnings in that case?

a warning sounds reasonable to me. happy to have a first pass at a pr (warning if setting a context means overriding on already set). does coverage already have a warnings system you could point me to?

There is a warnings facility, for example: https://github.com/nedbat/coveragepy/blob/cecc5ddc2d1a95223ad708a3dcc27f7a7f4d54b9/coverage/control.py#L701-L702
But off the top of my head, I'm not sure how this will work.  pytest-cov sets a context by calling Coverage.switch_context, which I guess could warn if the coverage config already had a setting? We have to be careful of unintended interactions.

ah, i thought the api might have been enter/exit context
perhaps this is more of a documentation issue, e.g. recommending test runnings/plugins over dynamic_context = test_function
maybe most of my confusion is from following the alphas, and picking up the dynamic_context setting before there was support in pytest-cov

@nedbat having re-read the docs i'm happy to close this unless you think any action is needed

I added a warning for this case in 16ef35f9.

This is now available in coverage 5.0.2


Performance vs Exposed for simple CRUD

Describe the bug
Hi, this is not necessarily a bug, but just something that I've noticed running experiments vs other Kotlin ORMs.  I was just doing a simple load test on my local machine to a postgres instance running in docker.
Essentially, all I'm doing is spamming inserts for a specific table, and have noticed that Komapper is significantly slower than Exposed.
Komapper result

Exposed

As you can see, the http_req_duration is nearly 5x higher for komapper vs exposed.  Curious if you guys have any idea why this may be (using JDBC for both, same repo, simply swapped out the persistence layer).
Here is the K6 test if curious.
import http from 'k6/http';

export const options = {
    stages: [
        { target: 100, duration: '10s' },
        { target: 500, duration: '10s' },
        { target: 1000, duration: '10s' },
        { target: 500, duration: '10s' },
        { target: 100, duration: '10s' },
    ],
    thresholds: {
        http_req_failed: ['rate<0.001'],
        http_req_duration: ['p(95)<100'],
    },
};

export default function () {
    const url = 'http://localhost:8080/author';
    const payload = JSON.stringify({
        name: 'name',
    });

    const params = {
        headers: {
            'Content-Type': 'application/json',
        },
    };

    http.post(url, payload, params);
}

Just does a simple ramp up to 1000 virtual users over 10 second stages.

Thanks for your post.
Can you share your test code (including the persistence layer)?
I want to investigate why Komapper is slower than Exposed.

sure thing, here ya go https://github.com/unredundant/exposed-komapper-comp
Let me know if you have any problems running it, instructions are in the readme.
I'm wondering if it could be that for Komapper I am not using a connection pool? I poked around in the JdbcDatabase but did not see a way to inject a connection pool the way you can using Exposed + Hikari.

Thanks for sharing.

I'm wondering if it could be that for Komapper I am not using a connection pool? I poked around in the JdbcDatabase but did not see a way to inject a connection pool the way you can using Exposed + Hikari.

You can rewrite ConnectionManager.komapperDatabase as follows:
    val komapperDatabase: JdbcDatabase by lazy {
        val dataSource = HikariDataSource(HikariConfig().apply {
            jdbcUrl = PostgresConfig.CONNECTION_URI
            username = PostgresConfig.USER
            password = PostgresConfig.PASSWORD
            maximumPoolSize = PostgresConfig.DEFAULT_MAX_POOL_SIZE
            initializationFailTimeout = PostgresConfig.DEFAULT_INIT_FAIL_TIMEOUT
            isAutoCommit = false
            transactionIsolation = "TRANSACTION_REPEATABLE_READ"
            validate()
        })
        JdbcDatabase(dataSource = dataSource, dialect = PostgreSqlJdbcDialect())
    }

Could you rerun the performance test with this configuration?

yep, that was it, thanks! komapper is actually 35% faster when the connection pool is properly configured


Make 'make check' more consistent across projects

Makefile:
Add 'build-test-image' target, (possibly) distinct from
'build' target.
README.md:
Add mention of 'make build-test-image'
Add CONTRIBUTING.md stub with pointer back to README.md.
Signed-off-by: Ben Crocker<EMAIL_ADDRESS>

@bcrocker15 I've made a few changes on this before marking it to be merged.


Fixed width navigation making it difficult to read

Hi,
Maybe it is the wrong place to report, but I couldn't find the right place. Also,  this is the first issue that I created as a developer.
The API documentation has a fixed width side navigation. Considering the long class names, it can be a pain to differentiate submodules. I searched a list view of the API modules, but I couldn't find it. Maybe you can consider adding a page (not a sidebar view) that lists API modules or a sidebar with a proportional width.  Thanks.

This is no longer relevant, we have switched over our documentation to cloud.google.com


OS display value for SUSE Linux Enterprise Server

Description
Due to the length of string output for distro plus arch - "OS: SUSE Linux Enterprise Server 12 SP3 x86_64" this gets unfortunately cropped/trimmed (on an 80x24 default terminal) to "OS: SUSE Linux Enterprise Server 12 SP4". This is confusing, given the release naming by SUSE of incrementing SP#s over time for a given major release (12, in this case). This can currently be worked around with a wider terminal or by turning off os_arch.
Neofetch version
neofetch --version : 3.2.1-git
(used from packaged at https://build.opensuse.org/package/show/utilities/neofetch as neofetch-3.2.0+git20170813)
Screenshot

Config file
no changes from default
Verbose log

Run neofetch -vv 2> neofetchlog
Upload the contents of neofetchlog to pastebin, gist or equivalent.
https://pastebin.com/aXfHjH0x

I've pushed a change to remove Enterprise Server from the output. The line should now fit.


Admin impersonating user1 cannot access Publications page of user2, for whom user1 is a proxy

I'm not quite sure what's not working here. The problem might just be that I'm trying to test this from my admin account. Here's what happens:

While acting as Brandy (bak25), I sent a request to Seth (sre53) to be Brandy's proxy.
While acting as Seth, I confirmed that request.
Still acting as Seth, I went to Brandy's profile page. My only option there is a button to "Unbecome this user." (This was unexpected. I expected to have to click "Become this user" on Brandy's profile.) There is no "manage profile" option. I also don't see a header on this page indicating I'm acting as Seth (or Brandy).
If I try going directly to the URL for the Manage Profile Publications page, it is the page for Seth, not Brandy.
I tried signing out and then signing back in. Then I went to Seth and Brandy's profile pages. On each page, there was a "Become this user" button, as I expected. On Seth's page, I clicked that button.
I entered the URL again for Brandy's profile page. The button there reads "Unbecome this user," just like I described above.
I entered the URL for the Manage Profile Publications page, and it took me to Seth's page again.

@anaelizabethenriquez Being an admin is partially the problem here.  You cannot proxy for someone while you are admin proxying for someone else.  You are viewing Brandy's public profile while proxying as Seth, so you must unbecome Seth before you become Brandy.  The wording "Unbecome this user" is a bit misleading too.  What happens when you click that button is you are signed out of whoever you are already proxying for.  So in this case, if you clicked that button you would unbecome Seth.  Then, the button to become Brandy would appear.  It may be more clear if the wording on that button indicated who you are still signed in as.  So it might say "Unbecome Seth Erickson".  Also, it would probably be helpful if the banner was still at the top of the screen that has the 'Stop being sre53' link.


fix: unused warning on cx in server functions

When running cargo clippy on server functions that use cx: Scope it has an unused variable error.
It appears that the logic for adding an #[allow(unused)] notation is inverted.

Thanks!


Remove Config#key_prefix

This was clearly an over-engineering.

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

Comparison is base (c22c334) 100.00% compared to head (4eac15c) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main        #3   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          190       179   -11
  Branches        17        14    -3
=========================================
- Hits           190       179   -11

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


Toyota RAV4 XLE hybrid 2024 error my vehicle is not recognized. #32179

Describe the bug
Good afternoon. I am writing for the second time after Shane Smiskol's (https://github.com/commaai/openpilot/issues/32179)

reply as he recommended to write again to the support. The problem, according to him, is in some part of the equipment. Please follow the link and familiarize yourself with the detailed information on my problem. Thank you. Have a good day.
Which car does this affect?
Toyota RAV4 XLE hybrid 2024 (VIN: JTME6RFV6RD557044)
Provide a route where the issue occurs
results: google_104000190100648345810
openpilot version
0.9.6
Additional info
This is where I took the picture Shane Smiskol's asked me to take. https://github.com/commaai/openpilot/issues/32179

Duplicate of https://github.com/commaai/openpilot/issues/32179


Adds Sentry extension (WikiTeq fork) [WIK-1209]

https://github.com/WikiTeq/mediawiki-extensions-Sentry

Since this already was pushed to the WikiTeq branch this can probably be closed. I don't think this will be considered helpful by general users.

I agree with @tosfos.


Force and forbid respawning

This pr enables to forbid respawning with --no-respawning cli flag, and enables to respawn with specified node flags.
This pr will help to solve js-liftoff's issue #89, gulp-cli's issue #109 comment, and issue #4.

I'm sorry but there is a problem that mocha return exit code 0 in spite of error. This cause is seemed to be in my test code. Please wait.

The cause of the problem above is original two test cases which execute flaggedRespawn so that respawn is needed.
I created the pr #9 for this problem, and rebased this pr on #6, #9 and #8 clearly.

Cherry-picked and amended as 7ca348671e060f22bf541b59d78bc3ff9cc0a4d8
@sttk this series of PRs have been amazing and I'm glad we're moving to a solution on this stuff. Thanks so much!!

Thank you for merging! I'm glad, too.


[Doc] AD Anywhere

See https://github.com/opensearch-project/anomaly-detection/issues/521
To help address this disconnect between Visualizations and AD, we propose an integrated solution that allows users to quickly create, run and view anomaly results from a Visualization. For certain Visualizations, users can utilize the effort they've already provided to visualize the data they’re interested in (source data, metrics to analyze), and create an anomaly detector on that same data with just a few extra clicks.

@vagimeli: Since this is related to Alerting and Observability as well, I'm to go ahead and assign this to Alice.

This feature has been delayed to 2.4 or later. Updating labels and milestones now.

Created outline of tasks to document in the Dashboards docs to share with @vagimeli as writer for OpenSearch Dashboards.
Assigning back to Melissa for the Dashboards docs.

This issue is moving to 2.5. Updating milestone now.

Thanks Nate

This is a brief list of the tasks that need to be documented, based on the detailed list within the [META eng ticket #521](http:// https://github.com/opensearch-project/anomaly-detection/issues/521).
What you can do in the Dashboards to detect anomalies in your data visualizations:

Create the anomaly detector in the Dashboards.
View the anomaly and get a detailed list.
Set up notifications when detections are triggered in the Dashboards.
Administer the detector (turn on/off).
Set up predefined roles for access.

this appears to be a duplicate ticket to  META 2075.
Feature description - same as 2075
Resource ticket - same: META 521
is it ok to close and reference other ticket?

@alicejw-aws is working specifically on the alerts, @vagimeli is working on the detectors and dashboard. However, due to the prioritization of #2294, this will be delayed to 2.6

Closing this in favor of #937

Closing this in favor of #2075


Enable script in setup.py

remove script/colorprint.py
check if command colorprint exists in shell search patch after installation

script/colorprint.py is just a synbolick link and only script/colorprint would be setup, so it's OK


Update srs_app_log.cpp

issue#207

I have merge this request from master to develop, then from develop to 1.0release. Please submit pull requests to develop in future.


Compute Shader Skinning

What problem does this solve or what need does it fill?
Currently Skinning is recomputed in many passes (prepass, main pass, shadow passes, etc...).
What solution would you like?
Utilize compute shader skinning where skinning is done beforehand in a compute shader which writes out skinned vertices to a buffer. There are a few advantages to doing this:

Skinning is only computed once for all passes
Results in fewer shader permutations since vertex shaders are not specialized on skinning.
The previous skinned vertex buffers can be kept around for computing the velocity pass (used in TAA, motion blur, etc...)

The use of compute shaders means that WebGL would require a separate skinning implementation.
Additional context
Article about wicked engine's implementation: https://wickedengine.net/2017/09/09/skinning-in-compute-shader/
Doom Eternal uses this: https://simoncoenen.com/blog/programming/graphics/DoomEternalStudy

Should include computation of morph targets in the compute pass as well.

Unity does this too.


Fix GetSpeed & GetPosition in Wheel class

Currently GetSpeed() and GetPosition() return local member variables that are the values of the passed desired speed and homing offset angle.
Instead return GetFeedback().position and GetFeedback().speed for their respective functions as this is what we are actually looking for.

discovered that these calls can be very expensive and are better off using the programatically set values established from the lerp functionality that will be implemented soon.


Swipe Navigation

Hey guys,
Since this isnt officially supported yet i thought I'd ask if there is a branch for the ReadiumSDK launcher for IOS that implements swipe navigation. Ive been looking through the code and I cant seem to find where the navigation is controlled. I know there is a UINavigation controller in there somewhere I think if I find where that code is located I can get started on implementing swipe navigation.
Cheers

Update:
EpubViewController.m has the controls for the navigation. The functions invoke a javascript that utilises: @"ReadiumSDK.reader.openPageNext()" and @"ReadiumSDK.reader.openPagePrev()".
Im gonna first try and add a swift controller for the swipe navigation.

I apologize, I put this in the wrong place. Was meant to go on the readiumSDK section.


SharePoint files

It would be useful to be able to open SharePoint and other files served over HTTP.

Yes. I believe the target endpoint needs to serve the xlsx. For example, GitHub doesn't, or it does, and I have not figured out how. The xlsx is a zip, and the libraries the crack it open need to do so it can process the xml.
If that can be figured out with the constraints of EPPlus. Happy to implement.
I have a Read-Csv function that knows how to read a string, file, or url https://github.com/dfinke/PSKit/blob/master/ReadCsv.ps1
Would love to add that to this module.


[html] Don't mark quoted strings as links

VSCode Version: 1.3.0-insider
OS Version: Windows 10

-- Sorry for the English, I used Google Translator --
Problems with src = "' '" and href = "' '"
Can not find file.
Example: <ng-include src = "'./view/template/header.html'"></ng-include>
When using single quotes within double quotes.
Do not allow it to be considered link when something comes after the extension.
Example: <script scr = "js/angular/angular.min.js?"></script>
<link rel = "stylesheet" href = "css/animate.css?"/>
Similar as in Eclipse.
---Suggestion---
*I did not find these options in VSCode

Open in a new tab when you click this link, as currently happens when we use with ".html"
For when image, example: <link rel = "shortcut icon" href = "img/ico.png"> or <img src = "img/ico.png" /> have the option to choose whether you want to open with VSCode or my standard program.

In Eclipse, do not open a ".html" that is within the same folder (not considered link), when you put the full path. If possible do VSCode accept the way complete and short.

@Tekbr is this about detecting links or what happens when you click a link?

@bpasero,  if I understand correctly, the situation mentioned can not consider link when I press Ctrl and move the mouse over the line.

Do not allow it to be considered link when something comes after the extension.

Example:

<script scr = "js/angular/angular.min.js?"></script>

<link rel = "stylesheet" href = "css/animate.css?"/>

Now, about the rest, it is when I click on the link.
Example:
1- Take this situation: <ng-include src = "'./view/template/header.html'"> </ng-include>
2- Now click "Ctrl + Click left mouse button."
3- The VSCode not find the file.
On the suggestion to open pictures with the default program, you got any questions?
Any questions, just ask.

Ok from the description I think we cannot open double quoted links from the editor. But I would also not assume to see a link in that case since that seems like a weird link to have.

The link detection code for html now belongs to  @aeschli

Technically, single quotes are valid characters in hrefs, see https://tools.ietf.org/html/rfc3986#section-2. We can't just ignore such links, but will optimistically unline them. It's the user's decision to open them (or not)

-- Sorry for the English, I used Google Translator --
@aeschli I think the translation was confusing ... besides that at the time I was a new user on Github. In addition to putting many suggestions in a single case. 😊
Sure, since it's a link the same could point you to the correct path. Even if we are single quotation marks or not.
Which would lead to a new issue (I think 🤔 ), for VSCode to map, from where the folder opened by it is located.
1- It is possible?
2- And on the invalid link. Is it possible to block? (As in Eclipse)

If a link is syntactically correct (and, surprisingly, ' ' is) we can't block it.
If it really exists we only know when we try to open the link. But at parse time we can't do that, it would be too expensive (and potentially make web requests).


Add config option for setting taskmanager user name

Passsing the mesos taskamanager user as flink config

I think you've missed updating the docs as part of adding the config option, which has caused test failures:
16:26:11.208 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.22.1:test (integration-tests) on project flink-docs: There are test failures.
16:26:11.208 [ERROR]
16:26:11.208 [ERROR] Please refer to /home/travis/build/criteo-forks/flink/flink-docs/target/surefire-reports for the individual test results.
16:26:11.208 [ERROR] Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.
16:26:11.208 [ERROR] -> [Help 1]
16:26:11.208 [ERROR]
16:26:11.209 [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
16:26:11.209 [ERROR] Re-run Maven using the -X switch to enable full debug logging.
16:26:11.209 [ERROR]
16:26:11.209 [ERROR] For more information about the errors and possible solutions, please read the following articles:
16:26:11.209 [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
16:26:11.209 [ERROR]
16:26:11.209 [ERROR] After correcting the problems, you can resume the build with the command
16:26:11.209 [ERROR]   mvn <goals> -rf :flink-docs


Allow "identity" to be returned from branch.init()

@Kirkkt  This is to allow the identity to be returned in branch.init() for Cordova.  Is this the right way to do this, and should this data be returned there?

would you kindly provide some background on why we need identity in cordova?

This was for SDK parity, and a partner request

nitpick - we return it as 'identity_id' (and not 'identity') inside the other SDKs. 'identity' is reserved for developer identities, and i see that the key from the response is identity_id.
Just a bit of feedback

but identity and identity_id are two distinct concepts:
identity is set by the developer to whatever string they desire
identity_id is what branch generates for them: see branch.setIdentity() for more details.

@Kirkkt - yes, exactly. we're returning the branch generated 'identity_id'. however, this code sets the branch generated 'identity_id' to the key 'identity', whereas I think, in order to maintain feature parity w/ the native SDKs, it should be set to the key 'identity_id' (since that's what it is - not the developer set identity)

In that case, my change is bad, I need to revert it.  Plus, it sounds like there's a separate 'identity' field that I need to find, and figure out why it isn't available.  Thanks!

@Sarkar The identity_id isn't being returned from branch.init() either, should that field be added as well?  And if so, @Kirkkt would it make sense for me to add that to utils.whiteListSessionData()?

so it seems to me identity_id is some UUID type of thing doesn't make much sense to human
while identity is something that, as a developer, I could read, like<EMAIL_ADDRESS>I think we should refrain from returning identity_id (e.g. 4310574) unless there is a good justification.
if branch.init() is not returning identity (e.g<EMAIL_ADDRESS>we have a bug. Fixing this bug should be the scope of this PR; if we need to expose identity_id, that is a somewhat unrelated discussion.
Let me know if you disagree.

if it's a new user that we don't recognize, identity being null is normal - my understanding.


[BUG] the application did not respond

Describe the bug
For some reason whenever I use /configure with the valid credentials it just says "the application did not respond." But I'm not sure if it is an aternos issue or discord issue because I used /ping and it was able to respond.
Expected behavior
I expected the bot to configure correctly so I could start my aternos server through discord.
To Reproduce
Steps to reproduce the behavior:
(MIGHT NOT WORK FOR EVERYONE)

I did /configure
I filled in the credentials
it responded with "the application did not respond."

Screenshots

Other information
Add any other information or context about the problem here.

UPDATE: everything except /ping is saying "The application did not respond"

Sounds like a discord or VPS issue. The public bot is in ~ 1000 servers so some downtime is to be expected. Host it yourself to avoid issues like this.

Read https://github.com/sleeyax/aternos-discord-bot/discussions/61. Create a new issue if you're experiencing the same issue with a self-hosted bot.


[CX_HARDENING] Evaluate and Remove Issue Links in HotShot

Closes #3169

This PR:
There's lots of dead issues in the codebase, this PR removes them or, if they're easy enough, fixes them. It also fixes two other small issues with type complexity and general format.
This PR does not:
Update any tests or docs, that is coming later.
Key places to review:

I agree with your reasoning for everything on this, looks like a nice cleanup to me


Save rdb sometimes will failed on hash type

Haven't reproduced this problem every time.
Make ardb a slave of redis, and sync successfully.
Try  save redis
[8034] 02-23 10:26:41,966 INFO Close write latch for snapshot save preparing start.
[8034] 02-23 10:26:41,966 INFO Open write latch for snapshot save preparing complete.
[8034] 02-23 10:26:41,966 INFO Start to save snapshot file:/mnt/ardb/backup/save-redis-snapshot with type:redis.
[8034] 02-23 10:27:11,822 ERROR Previous key:KVUSER-1.0:3913588 with type:3 is not complete dump, 1 missing in 2 elements
[8034] 02-23 10:27:11,822 ERROR Failed to write dump file for reason code:-2

KVUSER-1.0:3004  is a hash type, on redis master, hkeys can see two subkeys.  on ardb slave, value is same as master(both subkeys and value).
I assume the 2 elements in error log means  2 subkeys for hash type. But seems they're all here.
Not always happen after slaveof redis master immediately, sometimes after one night, when I try to save redis on slave ardb, will come to this error. And once it happen, save redis won't work anymore on this ardb slave.
An example value for of hash is:
    visit_count:  "1"
    trail_start: "\xff"      #  \xff is msgpack encode of number -1

If I turn off slaveof redis and do save redis again. It will success

I'm testing the possibility of replace current redis with ardb.
So once I switch to ardb, and all function works, I'll  make original redis slave of ardb as backup. If I come to any unexpected problem, I can switch back to redis.
Really appreciate your work on ardb, bug fix so quick 👍


Master preserved

This patch reverts 6fda0bc1..4b2c963b, and then re-merges dev with a merge commit in order to preserve history

@rianquinn @JaredWright I belive this is good to go.


successfulJobsHistoryLimit, failedJobsHistoryLimit with cron workflow is not working

Summary
The successfulJobsHistoryLimit, failedJobsHistoryLimit with cron workflow is not working for us. I tested with simple cron workflows and works fine. But, in our cron, we are calling workflow templates using templateRef which in turn calls for another workflow template using templateRef that has the container definition. Could this nesting be causing the issue? Here's the snippet for cron:

What happened/what you expected to happen?
Completed workflows should be deleted as per their status and parameters successfulJobsHistoryLimit, failedJobsHistoryLimit
What version of Argo Workflows are you running?
2.9.5. Similar, issue was there with 2.9.3 too, which we're using

Could this nesting be causing the issue?

Not likely. If it works for one CronWorkflow, it should work for the rest. Can you dig through your workflow-controller logs and see if you find something relevant for this? Searching for this string will help: Enforcing history limit for '<CWF_NAME>'

Thanks @simster7 for commenting, I can see still issue despite seeing the string you mentioned in the workflow controller logs. Closing the issue as don't have time right now to dig more, will re-open if I get to see again in future

it still not working in 3.3.10

I can confirm this is not working on 3.4.3. I'm also using a CronWorkflow that calls WorkflowTemplate like the OP.
In my case I have two different cronjobs:

Hourly ( with a history of the last 24 jobs)
Daily (with a history of the last 7 jobs)

I can only see the workflow of the last executed job in the case of the hourly jobs, I don't get any workflow of the daily jobs.

Hourly

apiVersion: argoproj.io/v1alpha1
kind: CronWorkflow
metadata:
  name: dremel-druid-cron-hourly-historical-reindex-prod
  labels:
    project: dremel
    app: druid
    env: production
spec:

  schedule: "1 * * * *"
  suspend: false
  # Allowed to start within the next 58 minutes
  startingDeadlineSeconds: 3480
  concurrencyPolicy: "Forbid"
  successfulJobsHistoryLimit: 24
  failedJobsHistoryLimit: 24

  workflowMetadata:
    labels:
      project: dremel
      app: druid
      pipeline-type: cron
      env: production

  workflowSpec:

    archiveLogs: true

    ttlStrategy:
      secondsAfterCompletion: 86400  # 24 hours
      secondsAfterSuccess: 86400     # 24 hours
      secondsAfterFailure: 86400     # 24 hours

    arguments:
      parameters:
        # Repo
        - name: target_env
          value: "production"
        # Docker image
        - name: docker_image
          value: ""
        # Docker version tag
        - name: docker_tag
          value: "4.1.1"

    workflowTemplateRef:
      name: dremel-druid-cron-hourly-historical-data-reindex
      clusterScope: false

Daily (pretty much the same)

apiVersion: argoproj.io/v1alpha1
kind: CronWorkflow
metadata:
  name: dremel-druid-cron-daily-historical-reindex-prod
  labels:
    project: dremel
    app: druid
    env: production
spec:

  schedule: "0 2 * * *"
  suspend: false
  # Allowed to start within the next 20 hours
  startingDeadlineSeconds: 72000
  concurrencyPolicy: "Forbid"
  successfulJobsHistoryLimit: 7
  failedJobsHistoryLimit: 7

  workflowMetadata:
    labels:
      project: dremel
      app: druid
      pipeline-type: cron
      env: production

  workflowSpec:

    archiveLogs: true

    ttlStrategy:
      secondsAfterCompletion: 604800 # 7 days
      secondsAfterSuccess: 604800    # 7 days
      secondsAfterFailure: 604800    # 7 days

    arguments:
      parameters:
        # Repo
        - name: target_env
          value: "production"
        # Docker image
        - name: docker_image
          value: ""
        # Docker version tag
        - name: docker_tag
          value: "4.1.1"

    workflowTemplateRef:
      name: dremel-druid-cron-daily-historical-data-reindex
      clusterScope: false


Generate multiple css file

Is it possible to generate multiple css file? To set multiple entry point (maybe by folder) along with specific name. So we can import it separately on different page, like for frontend side and admin side.

Are you talking about using unocss/cli to do that？

Ohh yes, that right, sorry didn't know such thing is exist. Thank you.


Unable to trigger skip_forward or skip_backward as described

Setting up a remote, most functions work as described. Attempting to set up the skip_backward and skip_forward functions to go back.forward 10 seconds, seems to not be functioning with chromecast.
custom_actions:
skip-backward:
icon: mdi:rewind-10
tap_action:
action: key
key: skip_backward
skip-forward:
icon: mdi:fast-forward-10
action: key
key: skip_forward
Error: Failed to call service remote.send_command. Enum RemoteKeyCode has no value defined for name 'KEYCODE_skip_backward'

The skip backwards/forwards commands are from the Apple TV integration example and are not valid for Android TV. Android TV does not have separate skip backwards/forwards commands. You can sort of skip by adding hold_secs to the service call data to a fast forward or rewind command.
custom_actions:
  skip_forward:
    tap_action:
      action: call-service
      service: remote.send_command
      data:
        entity_id: remote.android_tv
        command: fast_forward
        hold_secs: 2

You can also now use momentary button mode to make the skip time scale with how long you've held the button.
custom_actions:
  fast_forward:
    momentary_end_action:
      action: call-service
      service: remote.send_command
      data:
        entity_id: remote.google_tv
        command: MEDIA_FAST_FORWARD
        hold_secs: HOLD_SECS

Thanks for the info on the skip buttons.
I'm trying your workaround but I can't seem to figure it out. I broke it down into a one button remote to make it simpler.
type: custom:android-tv-card
remote_id: remote.living_room_tv_2
slider_id: media_player.living_room_tv_2
rows:
  - - skip-forward
custom_actions:
  skip-forward:
    icon: mdi:fast-forward-10
    momentary_end_action:
      action: call-service
      service: remote.send_command
      data:
        entity_id: remote.living_room_tv_2
        command: fast_forward
        hold_secs: 2

Thanks for the help!

If you're not going to use HOLD_SECS for the value of hold_secs or momentary_start_action you should change momentary_end_action to tap_action. That way it'll trigger on a regular tap rather than release.

I see, I misunderstood it as a place holder for the amount of time I want it held. I think I have the code right now, but still having major problems.

skip-forward: not getting any response. Returns the same enum RemoteKeyCode error from the original post.
fast_forward: technically works, but it holds fast forward down regardless of how long I hold the button (one tap will hold down fast forward).

type: custom:android-tv-card
remote_id: remote.living_room_tv_2
slider_id: media_player.living_room_tv_2
rows:
  - - play_pause
  - - skip-forward
  - - fast_forward
custom_actions:
  skip-forward:
    icon: mdi:fast-forward-10
    tap_action:
      action: call-service
      service: remote.send_command
      data:
        entity_id: remote.living_room_tv_2
        command: fast_forward
        hold_secs: 2
  fast_forward:
    momentary_end_action:
      action: call-service
      service: remote.send_command
      data:
        entity_id: remote.living_room_tv_2
        command: MEDIA_FAST_FORWARD
        hold_secs: HOLD_SECS


iOS App Review Issue: misleading content

From the beta review, in case you plan on pushing to the apple store:
Design - 4.1
Your app or its metadata contains misleading content. Specifically, your app creates a misleading association with Pokemon Go.
Next Steps
Please remove or revise any misleading content in your app and its metadata.
In addition, we recommend adjusting your app name so that the misleading element is used as a descriptor, not as part of the product name.
For example, the following style is an acceptable format for names:
GreatApp - with tagline "for Pokemon Go"

Just user PokeRadar or something :)

Do we have to remove pokemon, team, gym... (pokemon go icons) from app? @jankcat

I'm waiting for a new app icon. When I have it I will try to submit.

how do you select those options @jankcat ?

@hoangnn89 Actually, the app doesn't contains all license file for custom libs.

but if we check NO then they definitely ask about those pokemon icons stuff, and we're doom

Do you guys have a version that is totally independent that you're trying to submit to the Apple App Store?

I have one built.... I just need to change my data source to a local trainer account I guess.
https://m.facebook.com/story.php?story_fbid=1061613040574683&id=100001779903579&comment_id=1061748810561106&notif_t=feed_comment&notif_id=1469972088409686&ref=m_notif

I have this same problem ....
Apple said
"
Next Steps
Please remove or revise any misleading content in your app and its metadata.
In addition, we recommend adjusting your app name so that the misleading element is used as a descriptor, not as part of the product name. For example, the following style is an acceptable format for names
"
they also attached my screenshots and icon
What I'm trying to understand ...

Should I just change the name of the in iTunes connect or
Should I change bundle id, profile, and have to rebuild with new name in all the codes?
Do they mean like just change the name so that the icon and screenshot will be fine, meaning
I don't have to change the icon and screenshot again( as the new name will be fit for the images).
Please any other explanation

Did they say nothing about the background hack or your submitted versions didn't include it ?

it's mine, but I dont know how to change it, just change the app name?
Design - 4.1
Your app or its metadata contains misleading content. Specifically, your app creates a misleading association with iPhone￼.
Next Steps
Please remove or revise any misleading content in your app and its metadata.
In addition, we recommend adjusting your app name so that the misleading element is used as a descriptor, not as part of the product name. For example, the following style is an acceptable format for names:
PutYourUniqueAppNameHere - with tagline "for iPhone￼”

Try to change the name and screenshots and re upload it
On Mon, Nov 7, 2016 at 5:58 PM, yuedongkui<EMAIL_ADDRESS>wrote:

it's mine, but I dont know how to change it, just change the app name?
Design - 4.1
Your app or its metadata contains misleading content. Specifically, your
app creates a misleading association with iPhone￼.
Next Steps
Please remove or revise any misleading content in your app and its
metadata.
In addition, we recommend adjusting your app name so that the misleading
element is used as a descriptor, not as part of the product name. For
example, the following style is an acceptable format for names:
PutYourUniqueAppNameHere - with tagline "for iPhone￼”
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/istornz/iPokeGo/issues/49#issuecomment-259023029, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKiRTwnpq-8VM6uKbp-cgPiwzssGDCQQks5q79csgaJpZM4JVmAX
.


Add TF_Log for gcs

@mihaimaruseac
This PR add loggging for gcs filesystem

As part of the final evaluation, I must provide a link to the work I have done. Since the blog post isn't finished yet. I've created a small gist and once we finish the blog post, we could add its link to the gist. Could you take a look at it ? Here is the link: https://gist.github.com/vnvo2409/dc1e3d98c8958efe8ebac2773f9a33fd Thank you!

Sorry for the delay, have been out of GitHub in the past few days.
The Gist looks good so far, make sure you fix typos. Under Link, you can say that these are links to the implementation. If possible, you can also add other links for all the PRs you have done or a link to the search page that lists all the PRs.
You also have contributions to Bazel and aws I recall. We should include them too in the gist.


please rebuild nginx and upload to launchpad

After the recent glibc CVE and updates, the nginx package fails to run, complaining about config that requires the upload module. Can you or someone affiliated with the project roll a new nginx package for distribution? The docker-galaxy container also relies on this version of nginx, so it's failing like so:
unknown directive "upload_store" in /etc/nginx/nginx.conf:117

Ongoing in PR #75 / @natefoo can you ping us once it's done ?
In the meantime you can install the old version like so:
apt-get remove nginx-common nginx-extras
apt-get install nginx-common=1.4.6-1ubuntu3.3ppa1 nginx-extras=1.4.6-1ubuntu3.3ppa1

Sorry, I had the PR open in a tab for 2 weeks, just kept getting bumped down. Updated packages have been pushed to the PPA and are successfully built and published.
We need to get Jenkins building these (although there'd be no way to sign them?)

Thanks @natefoo
I'm not too familiar with building .debs ... when you do apt-get source $pkg, you automatically get those security fixes applied?

@natefoo, no easy way to sign. I see a couple options.

Mint a new jenkins key, other groups do this. Use it to sign all galaxy stuff and keep it secure somewhere
Have a jenkins job for signing which emails you a .deb and then polls an email account for a reply from you with a signed deb? Or something.equally contrived ;)

@mvdbeek Yes, apt-get source fetches the upstream source and the distro patches. When you do the debuild step, the patches are applied.

@erasche We could have a general signing key and then add subkeys for Jenkins.

thanks, all. i didn't see that PR earlier. the pkg issue seems resolved for now.

@natefoo I'd +1 a jenkins subkey.


[BUG] Module not found: Error: Can't resolve 'fs' in 'C:\digital-b2c-channel-web-app\node_modules\applicationinsights\out\Library'

Description/Screenshot
Module not found: Error: Can't resolve 'fs' in 'C:\digital-b2c-channel-web-app\node_modules\applicationinsights\out\Library'

Steps to Reproduce

Clone the project or navigate to the project directory.

Run the yarn storybook build command. which internally runs following:
yarn add loader-utils --cmd-install && cross-env NODE_OPTIONS=--openssl-legacy-provider build-storybook -s ./public -o ./storybook-output

Observe the error mentioned above.

SDK Version: Next.js 13.3.1
How you initialized the SDK: I ran the yarn storybook build command.

Expected behavior
I expected the yarn storybook build command to run successfully without any errors.
Actual behavior
I encountered the error message mentioned above, and the build process failed.
Additional context
This error seems to be related to the fs module in the applicationinsights package. It appears to be blocking the build process.

@JacksonWeber / @hectorhdzg ??

@kiran8983 this is usually the kind of error when you try to run Node.js SDK in the browser or some where Node.js native libraries are not available, not super familiar with next.js but maybe this is related to way you are building your app?

@kiran8983 Were you able to determine if you were running the Node.js SDK in a non-server part of your Next.js code?


More CA paths

Hi,
you can find more CA paths in my PEP draft at https://bitbucket.org/tiran/peps/src/5ac9cd851f24554621be1c475a1b8ab2a313208f/pep-9999.txt?at=default#cl-269
You should favor CApath over CAfile if the Python version supports it. A CApath is faster becaues it loads CAs on demand.
Christian

@jayvdb Not quite.
https://www.happyassassin.net/2015/01/12/a-note-about-ssltls-trusted-certificate-stores-and-platforms/
Distros and OSes all have various (and usually more than one) locations for trust stores. The most-canonical location for RHEL, Fedora and Mageia (and derivatives and probably various smaller distros) is /etc/pki/tls/certs/ca-bundle.crt (as a bundle file). Arch uses Fedora's system. Debian (and derivatives, inc. Ubuntu) provides /etc/ssl/certs as a hashed directory and /etc/ssl/certs/ca-certificates.crt as a bundle file. OpenSUSE provides /etc/ssl/certs as a hashed directory for compatibility purposes, but not /etc/ssl/certs/ca-certificates.crt. It has a bundle file at /var/lib/ca-certificates/ca-bundle.pem, but discourages use of any particular location and encourages use of OpenSSL's default (see below) - https://github.com/openSUSE/ca-certificates . OS X has it in /System/Library/OpenSSL, according to the references I could Google, anyway. For Windows builds, your guess is as good as mine.
Really if you're trying to use the 'system' location you should not be specifying a location at all but letting OpenSSL handle it; see e.g. https://docs.python.org/2/library/ssl.html#ssl.SSLContext.load_default_certs . But as I read it, httplib2's design does not really allow for this, it is expecting the 'locater plugin' to provide a filesystem location.

I would propose adopting the approach that go's stdlib does - https://golang.org/src/crypto/x509/root_linux.go


Change output name XXX_centerline_optic --> XXX_centerline

Description
The current convention for output file name is only the "what", not the "how".
Regarding sct_propseg, i suggest we remove the default output centerlines.

@charleygros agreed, i've updated the proposed change here

Please clarify, what sort of action is requested on this?
the change XXX_centerline_optic --> XXX_centerline would result in not having distinct files?
file_ctr = os.path.join(param_test.path_output, sct.add_suffix(param_test.file_input, '_centerline_optic'))

@abtahizadeh

in sct_propseg: the idea is to not output "XX_i_centerline" from sct_propseg anymore (i.e. the one generated from the segmentation), but only output the centerline from OptiC, which we would then call  XX_i_centerline (previously called XX_i_centerline_optic)
in sct_get_centerline: change output name XX_i_centerline_optic --> XX_i_centerline

@charleygros @benjamindeleener: after giving it more thoughts, i'm not sure this is a good idea to remove the "_optic" suffix and output the centerlines without it. The problem is that some people use the output centerline for subsequent tasks (e.g. straightening, smoothing, etc.) where they should be using the segmentation instead (more accurate). So, i suggest we simply don't output any centerlines anymore after sct_propeg or sct_deepseg_sc. If people want them (for debugging purpose), we can add a flag for it. Pls let me know your thoughts.

@jcohenadad : My argument for outputting the centerline after the segmentation tools were to easily see if a segmentation fail is due to the centerline detection or the segmentation module itself. And subsequently, facilitate the found of a way to fix it --> if we observe that the detection fails, we can opt for another -ctr algo (e.g. cnn or svm or viewer).
A flag dedicated to this could be a solution

if we go that route, we also need to find a convention for flags, regarding centerline "method" and centerline "output". In sct_deepseg_sc, the plan was to replace ctr with centerline, but then we need to find a good flag for naming the output centerline. Alternatively, we don't create a specific flag for it, and instead we set -r 0(don't remove temp files). After all, this flag was created for the purpose of debugging.

Just wanted to summarize what we said above to check if we still agree on it, before to (finally) implement it:
-- sct_propseg, sct_deepseg_sc and sct_deepseg_lesion: do not output a centerline file. If the user need it, she/he could use the flag -r 0.
-- sct_get_centerline: output a centerline image with the filename convention: *_centerline.nii.gz. Currently: -method viewer --> *_viewer_centerline.nii.gz and -method optic --> *_centerline_optic.nii.gz
Is it correct?


Include "region" in key names for da.store

[x] Tests added / passed
[ ] Passes flake8 dask

Note that you would still get the same key if you stored data into two different stores, but we can't rightly hash those stores, since, by their nature, they may be massive and/or remote.

cc @jakirkham for review

Looking again, I think I've got myself confused over this one. Is regions or indeed multiple sources and/or stores compatible with return_stored?
The return value there is a single Array, so some trickery would be required to make a single name for it and decide how the parts are to be assigned.

@jakirkham , like so?

Thanks @martindurant. Yes the gist is good.
Would say how the UUIDs are being managed currently is a bit convoluted, but this almost certainly my fault because of how the code is structured here currently. Feel free to change things around as needed to get the right behavior from this.

@jakirkham , do you think that this can be considered ready as-is? I don't have much appetite to unravel the graph making.

Yeah, I think that is reasonable.
The bugs it fixes are pretty important. The issue noted above is pretty minor and is internal to Dask. We can revisit it later.
Thanks @martindurant. :)

Cool.


Creating duplicate account logs error but doesn't display to user

Logged Error

But page keeps loading with nothing happening.

I need to know the details, where does it occur and stuff like that.

I need to know the details, where does it occur and stuff like that.

On the register page.
I already have an Admin user then I tried creating another user with the same username. The front page keeps loading until it fails without any info. Meanwhile, on the console we get the UserExistsError.

@thebishaldeb  Didn't notice any PR that fixed this issue.
If it'll need some fix, I think I can see into it

Coding period is over.  You can work but wont get any points.


[bug]  doesn't calculate mobile layout on initial load

Describe the bug
When footer first loads on the page it doesn't calculate if it is in mobile mode in order to put the footer links into accordion mode.
  override render() {
    this.#compact = !this.screenSize.matches.has('md');
   ...

Setting the private #compact  variable with a check on the render() will set the inital view.  After that then the onChange event would kick in as well.  This is how we are currently doing this in navigation-secondary.
Which Component?
rh-footer
Steps to reproduce

Load ux.redhat.com in a mobile view.  The footer links are in the desktop format.

Expected behaviour
Footer links should load in accordion view at mobile width
Screenshots

Element HTML
No response
Operating System (OS)
iOS
Browser
No response
More browser information
No response
Additional Info
No response

This was fixed in https://github.com/RedHat-UX/red-hat-design-system/pull/961


[Swiftype] Add meta tags to details pages

Summary
Add the following <meta> tags to the <head> of all quick start detail pages so we can use that in swiftype search queries.
The theme's SEO component does this automatically by looking at the domain of a given site's siteUrl (set in gatsby-config.js). We will need to include a way to pass the type for quick starts (because they live on developer subdomain) as a prop to override this default behavior.
<meta class="swiftype" name="type" data-type="enum" content="io">

Add a <meta> tag for each keyword associated with a quick start:
<meta class="swiftype" name="tags" data-type="string" content="TAG_NAME_1">
<meta class="swiftype" name="tags" data-type="string" content="TAG_NAME_2">

Add a <meta> tag for the quick start name:
<meta class="swiftype" name="quick_start_name" data-type="string" content="QUICK_START_NAME">

the content should populate with quickstart.title field.
Notes
In order to see those meta tags in development, I think you may have to run a build.
Acceptance criteria

[ ] On each quick start details page, the above meta tags are in the <head> element
[ ] On each quick start page, the only type field is quick_starts (and not developer)

Relavant link
Theme's SEO component

@roadlittledawn i see the site type here as both "io" and "quick_starts" and somehow my change ended up being "quickstarts" 😂 which one of these would you prefer?

my bad. didn't get a chance to update all the tickets to use same value. i believe we said we'll go with quickstarts


BLE communication

First of all, thank you very much for all your work, AlexxIT!
I’ve been dabbling around with FW and SW here and I have a couple of questions that I couldn’t find answers for:
I am trying to get notifications from a N100 lock, and for now I am only using the BLE part of Gateway 3, and I’m struggling a bit. Started of with v1.4.7_043(factory default) which gave me all kinds of problems.
I therefore installed the custom FW 1.4.6_0012 and reinstalled everything in HA, getting a new token etc.
I saw this under known problems afterwards :
“Gateway on firmware v1.4.6_0012 won't setup without connection to Xiaomi Cloud”
Does this mean that the BLE functionality won’t work without the cloud or is it only during setup?
I probably should mention that I am blocking WAN on the gateway and I’m getting a (maybe related error) in HA log:
Logger: custom_components.xiaomi_gateway3.core.mini_miio
Source: custom_components/xiaomi_gateway3/core/mini_miio.py:157
Integration: Xiaomi Gateway 3 (documentation, issues)
First occurred: 4:03:44 PM (2 occurrences)
Last logged: 4:34:14 PM
<IP_ADDRESS> | Device offline

Does this mean that the BLE functionality won’t work without the cloud or is it only during setup?

In usual case yes. But I fix this in 0012 and 0160 firmwares.

Great! Thank you very much. Would it be possible to make a “Local control” option that skips the WAN connection without breaking the functionality? Might be a pull request? (I have a “trust issue” with Xiaomi after poking around in the FW of one of their lamps. They removed “LAN control” support rather suddenly without notifying their customers)

The BLE device database is loaded from the cloud. If the gate does not see a device for a long time, it can delete it from the database


Added company names and sorted alphabetically

Looking good, dont forget to follow https://twitter.com/NextBigThingBot


About constraints related to reg_preimage

Regarding the evm/keccak implementation, it appears that the constraints related to reg_preimage are only the following one.
https://github.com/mir-protocol/plonky2/blob/00579850be0fc9caeae31ec21815b923d985187c/evm/src/keccak/keccak_stark.rs#L267-L273
This means that reg_preimage can be chosen independently of reg_a and others.
I think this would cause a soundness issue when reg_preimage is subject to cross-table lookups.
Am I overlooking something? I appreciate your feedback. Thank you.

Shouldn't this also be constraining the higher bits?

Thanks for reporting, @npwardberkeley could you take a look later?

I'd like to create a PR for this if that's fine

Oh yes, go for it!

I created a PR to constrain the higher bits of the preimage. Should I also add a constraint that reg_a and reg_preimage should be the same at the start of each input?

Closing as #1201 and #1202 have been merged.


Handle the tie bar for affricates in IPA segmentation

IPA segmentation doesn't appear to correctly handle the tie bar for affricates. For instance, from https://github.com/kylebgorman/wikipron/pull/76#discussion_r339324885, I saw the following for Brazilian Portuguese:
adição a d͡ ʒ i s ɐ̃ w

Instead, we would want the following for this example:
adição a d͡ʒ i s ɐ̃ w

this seems like a job for the parser i wrote! happy to submit a pull request for it to resolve the issue.