More Quest particles customization

What does your feature do?
Reworks how "new-quest" setting works, basically it will only appear for when the NPC has a new quest, after the quest is picked up the particle will stop to indicate that the NPC has no more quests, instead a new "active-quest" particle would be used indicating that you have an active quest from that NPC. Another new particle called "finished-quest" would introduce a particle that shows up when you've finished reached a certain point in the quest preferably this setting should be set within a stage.
Each setting has a priority meaning one will show over the other if there's more than one active.
"active-quest" = 0
"redo-quest" = 1
"new-quest" = 2
"finished-quest" = 3
Also another setting to set the amount & speed of particles that are shown per option.
How would people use it?
It would make users understand more what's going on, usually MMORPG's quests have something similar to this, I had a few users get confused on why the NPC wasn't giving more quests because it just showed "new-quest" for when there was a new quest and for when it was active.

Thanks, keep up the awesome work!
Edit: Had another interesting idea, perhaps all this can be done via "Actions" that change the particles displayed on the NPC. Thought I'd also put it on here as it could be worth it.

Related to #455
I will consider shading EffectLib for extended particle support.

https://lectern.browsit.org/resources/resource/45/


spec: update cli sign spec for tag to digest translation

This PR is mainly to improve the output message of notation sign command for tag to digest translation.

Codecov Report

Merging #439 (4754adb) into main (40973f9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #439   +/-   ##
=======================================
  Coverage   32.82%   32.82%
=======================================
  Files          25       25
  Lines        1237     1237
=======================================
  Hits          406      406
  Misses        819      819
  Partials       12       12

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Remove the changes for debug option, which will be covered by separated issue.

LGTM


How to add wifi driver RT2870 on Goke GK7205V200 board

Hi everyone,
Yesterday arrived my ipcam with a Goke GK7205V200 chip. I flashed without any troubles OpenIPC. But i don't get the wifi working.  I read that i maybe need to copy the wifi driver from the original firmware to OpenIPC but i don't have any idea how. Does someone have  steo by step instructions to get the correct driver and integrate in OpenIPC?
Thank you very much!

I have made a lot of progress! Can connect to wifi manually if i run the following commands:
setnetwork -i wlan0 -s SSID -p PASSWORD -m dhcp echo 9 > /sys/class/gpio/export echo out > /sys/class/gpio/gpio9/direction echo 0 > /sys/class/gpio/gpio9/value modprobe mt7601u ifup wlan0
So i think gpio9 enables the wifi module. I found the configuration in the network settings so i added the commands what i found here: https://github.com/OpenIPC/wiki/blob/master/en/network-settings.md#wlan0--hi3516ev200gk7205v200gk7205v210--camhixin
And whas hoping wifi would automatic connect, but sadly it doesn't.  I see if i run lsusb after a reboot the wlan module is not detected. But with the manual running comands aboven it does. What goes wrong?????
auto wlan0 iface wlan0 inet dhcp     pre-up echo 9 > /sys/class/gpio/export     pre-up echo out > /sys/class/gpio/gpio9/direction     pre-up echo 0 > /sys/class/gpio/gpio9/value     pre-up modprobe mt7601u     pre-up wpa_passphrase "SSID" "password" >/tmp/wpa_supplicant.conf     pre-up sed -i '2i \\tscan_ssid=1' /tmp/wpa_supplicant.conf     pre-up sleep 3     pre-up wpa_supplicant -B -D nl80211 -i wlan0 -c/tmp/wpa_supplicant.conf     post-down killall -q wpa_supplicant     post-down echo 1 > /sys/class/gpio/gpio9/value     post-down echo 9 > /sys/class/gpio/unexport


Built Native failed

1: Built LLVM.sln with vs2022
2: cmake -DCMAKE_INSTALL_PREFIX=../install -DPATH_TO_LLVM=../../../../llvm-project/artifacts/install -G "Visual Studio 17 2022" -A x64 -Thost=x64 ../../..
I will get the output:
Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19045.
CMake Error at sources/libClangSharp/CMakeLists.txt:36 (find_package):
Could not find a package configuration file provided by "Clang" with any of
the following names:
ClangConfig.cmake
clang-config.cmake

Add the installation prefix of "Clang" to CMAKE_PREFIX_PATH or set
"Clang_DIR" to a directory containing one of the above files.  If "Clang"
provides a separate development package or SDK, be sure it has been
installed.
-- Configuring incomplete, errors occurred!
When i set CMAKE_PREFIX_PATH = llvm-project/artifacts/install, That output is same

CMAKE_PREFIX_PATH is not important
set
-DClang_DIR=artifacts\install\lib\cmake\clang
everythis is well

Sorry I didn't get to answering this directly, I was busy helping out at Microsoft Build conference the past week.
ClangDir should have been auto resolved relative to PATH_TO_LLVM, but it sounds like there was some machine specific issue that caused that to fail and you found a workaround by resolving the path manually.

For anyone who might find my comment in the future: I encountered this issue and solved it by using an absolute path for PATH_TO_LLVM instead of the relative path shown in the directions.


net: lib: http_server: enable application to set response headers

Is your enhancement proposal related to a problem? Please describe.
I am currently evaluating the Zephyr HTTP Server for a new Project we will soon be starting. The problem I stumbled upon was that there currently is no possibility to set response headers. Specifically I would need to set the status code as well as the content type dynamically within the dynamic HTTP resource callback.
Describe the solution you'd like
Ideally we would add an additional http_data_status called HTTP_SERVER_HEADER in which the callback can put header data into the buffer and return the byte-count written. The API could then depending on HTTP1/2 take those headers and send them to the client. If 0 is returned on the first call with status HTTP_SERVER_HEADER the default headers would be used.
Describe alternatives you've considered
One alternative would be to not provide this functionality at all. Another would be to add the status code header option to the http_resource_detail_dynamic struct like the content_type. But this would not address the issue of dynamically changing headers as well as other custom headers that might be needed in the future.
Additional context
-

This is already implemented and merged.


Doesn't work

hi yes this doesnt work

Yes it does!

just use 1.19.4 and bukkit

incorrect, doesn't work @tnn1nja


Update Developer Guide for AddCommand

Let's help future developers understand the main flow of how a new Internship added is sorted according to dates.
Let's also share with them our design considerations for this command.
Fix some user story table formatting issues too.

Codecov Report

Merging #85 (77b9797) into master (5d378ba) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #85   +/-   ##
=========================================
  Coverage     72.23%   72.23%
  Complexity      435      435
=========================================
  Files            74       74
  Lines          1365     1365
  Branches        143      143
=========================================
  Hits            986      986
  Misses          328      328
  Partials         51       51

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Feature] Normal Noteblock Intergration

simaler to the 1.7.10 OpenPeripherals I was wondering if you could make it so that the computers for CCtweeked will work with the Note blocks that are part of the base game. yes we have speaker blocks but i would love to use actual note blocks to use a program that i used in an older version and see if i can get the thing to work in 1.16.5. I would really apreciate this since OpenPeripherals most likely wont be getting updated to this version

Mh, because our integration system is made for tile entities(A noteblock is not a tile entity), I will for now clear the milestone until I have the time to improve our system.

Added in 0.7.4r


Moving deploy.sh into image

The current way of deploying openvpn is that cloning project and running commands. If deploy.sh would be moved into the image, there will be no need to clone project. This is might be considered as a temporary improvement before moving into a chart.

I want the image itself to stay as barebones as possible so i don't think i'll do this.

The problem is that cloning repo is tedious and error-prone. Using commands would be cooler. Given the fact that this image is providing k8s related implementation, having a file producing necessary manifests is not problem.
This also makes more consistent across repo and other commands. Repo or image might be outdated. In case such an inconsistency might create a problem.

I'm re-evaluating openvpn PKI initialization and will take your comments into account. I will not cover this proposed workflow however. (temporary stuff tends to stay forever)


[HINTS]Add implementation for hints on functions ec_negate and compute_doubling_slope (cairo_secp/ec.cairo)

[HINTS]Add implementation for hints on functions ec_negate and compute_doubling_slope (cairo_secp/ec.cairo)
Description
Description of the pull request changes and motivation.
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #341 (b93ee86) into ec-negative-and-doubling-slope-hints (a0cb4fd) will decrease coverage by 0.01%.
The diff coverage is 94.66%.

@@                           Coverage Diff                            @@
##           ec-negative-and-doubling-slope-hints     #341      +/-   ##
========================================================================
- Coverage                                 97.06%   97.05%   -0.02%
========================================================================
  Files                                        40       40
  Lines                                     14055    14130      +75
========================================================================
+ Hits                                      13643    13714      +71
- Misses                                      412      416       +4

Impacted Files
Coverage Δ

src/vm/hints/secp/ec_utils.rs
97.27% <94.44%> (-2.73%)
:arrow_down:

src/vm/hints/execute_hint.rs
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Support for no console in the web UI.

As an end user, I want to be able to turn off console access on the server, but still be able to use the web UI to browse dashboards, manipulate app mode tables.
To test this, set deephaven.console.disable to true on the server, and try to connect from the web UI.
Presently the web UI has an error if no console types are supported, and the tables cannot be used, nor the dashboard loaded.

@niloc132 maybe a dumb question, but how/where do we set the deephaven.console.disable? Would that be defined in our docker-compose file or something?

I think client could get this information from a "Web" ObjectType plugin. Potentially the same object as expressed in https://github.com/deephaven/web-client-ui/issues/703

For this release, UI can check that getConsoleTypes returns an empty list, and just show an empty dashboard in that case.
We can work on general configuration for #703 later.

@niloc132 right now the only end point to retrieve an object through the JS API is via IdeSession.getObject. With console disabled, we're unable to retrieve a session object.
Should I just expose getObject through IdeConnection as well? Might mean moving some of the logic into WorkerConnection (such as IdeSession.getVariableDefintion)

Blocked on https://github.com/deephaven/deephaven-core/issues/2779


Pre-processing new data set for BertSum

Can you please let me know that how I can pre-process a new data set, like a list of paper abstracts as an input of BertSum.

Have you read README yet?
There is an option so you can process your own dataset. You need to download the stories to see its format, then you can process your data to have the same format.
Basically, each story has 2 parts: the source and the target. Each line separated by a blank line, then there is a @highlight line before each target line. Anyway, just follow the steps in README and look at data, code. Then you will figure it out yourself.

According to your explanation, for prediction samples that do not have target line, I should not have @highlight line in my .story files. Is it right?

According to your explanation, for prediction samples that do not have target line, I should not have @highlight line in my .story files. Is it right?

Sure, if you have already trained a model, so in the prediction step, you don't need to provide the label for it. Target lines are the ground truth (or as I called it "label") that you expect the model to output as similar as possible. In short, yes, you don't need to provide target lines when it comes to prediction.

Thank you for your reply. I am having another strange problem. When I follow the Option2 for pre-processing data, I can tokenize .story files successfully and generate .story.json file. But When I run the step-5 I always get three empty square bracket:
[nhassanp@uc1f-bioinfocloud-assembly-base src]$ /data/conda_envs/20200204/miniconda3/bin/python preprocess.py -mode format_to_bert -raw_path ../merged_stories_tokenized/ -save_path ../bert_cnn/ -oracle_mode greedy -log_file ../logs/preprocess.log
[]
[]
[]
(Since I don't have url for my data, I skip step-4)

@binhna Thank you for your reply. I am having another strange problem. When I follow the Option2 for pre-processing data, I can tokenize .story files successfully and generate .story.json file. But When I run the step-5 I always get three empty square bracket:
[nhassanp@uc1f-bioinfocloud-assembly-base src]$ /data/conda_envs/20200204/miniconda3/bin/python preprocess.py -mode format_to_bert -raw_path ../merged_stories_tokenized/ -save_path ../bert_cnn/ -oracle_mode greedy -log_file ../logs/preprocess.log
[]
[]
[]
(Since I don't have url for my data, I skip step-4)

You should check the format_to_bert function in data_builder.py.
Empty square bracket means that they found didn't find any file JSON in step 4.
You should check glob.glob(os.path.join('../merged_stories_tokenized/', f'*{corpus_type}.*.json'))
Note that name of the json file from step 4 should look something like this abcd.test.0.json --> {name}.{corpus_type}.{number I supposed}.json

@binhna Thank you for your reply. I am having another strange problem. When I follow the Option2 for pre-processing data, I can tokenize .story files successfully and generate .story.json file. But When I run the step-5 I always get three empty square bracket:
[nhassanp@uc1f-bioinfocloud-assembly-base src]$ /data/conda_envs/20200204/miniconda3/bin/python preprocess.py -mode format_to_bert -raw_path ../merged_stories_tokenized/ -save_path ../bert_cnn/ -oracle_mode greedy -log_file ../logs/preprocess.log
[]
[]
[]
(Since I don't have url for my data, I skip step-4)

You should check the format_to_bert function in data_builder.py.
Empty square bracket means that they didn't find any file JSON from step 4.
You should check glob.glob(os.path.join('../merged_stories_tokenized/', f'*{corpus_type}.*.json'))
Note that name of the json file from step 4 should look something like this abcd.test.0.json --> {name}.{corpus_type}.{number I supposed}.json

@binhna Thank you! I appreciate you help.
I modified the saving path and now it works. But another issue happened!
In _format_to_bert function there is line which I do not understand it:
name = re.search('Files.(.*).test.json', json_file).group(1)
in my case json_file = "/data/examples/nhassanp/PreSumm-master/bert_cnn/train.6.bert.pt"
and the result for name is "AttributeError: 'NoneType' object has no attribute 'group'". Because of that I get the following error when I run step5 of Option2:

First of all, AttributeError: 'NoneType' object has no attribute 'group' means that the re.search('Files.(.*).test.json', json_file) supposed to always return some value in the json_file, but it doesn't, so it returns None, and then you can't call group() from Novetype. You should change the code to compatible with your filename tho.
For the second question, at the line you show me, there is no tgt in it, because these values are from b_data, maybe he changed the name of tgt to something else. Look at the first line in the for loop, source and target are from 'src' and 'tgt'. So it makes sense

@binhna You are right. It seems that he updated his code but did not update his pre-processed data sets.


No longer works after Home Assistant .12 update

Since upgrading to the .2 update. The HACS integration no longer works. In settings>devices & services, I get this error: Retrying setup: Start Listening was not completed in timely manner.
The Addon part, appears to be working fine… Anyone else?

Same problem here

Genius! easy fix, thank you...

Genius! easy fix, thank you...

Where is this easy fix? I don't see it.

Genius! easy fix, thank you...

Where is this easy fix? I don't see it.

@maypo52
https://github.com/fuatakgun/eufy_security/issues/525#issuecomment-1315143963

Long story short: set the value of TRUSTED_DEVICE_NAME in the configuration of Eufy Security Add-on to any value that you like (e.g. HomeAssistant). Helped me out to fix this.

Since upgrading to the .2 update. The HACS integration no longer works. In settings>devices & services, I get this error: Retrying setup: Start Listening was not completed in timely manner.
The Addon part, appears to be working fine… Anyone else?

Can Confirm that Changing the TRUSTED_DEVICE_NAME and restarting the Eufy Security Add ON, did actually restore access.

I had the same problem after the last update. changing the TRUSTED_DEVICE_NAME did help me a little but now i get this in my log. Connected to station (...) on host (ip) and port (...) INFO  Disconnected from station (...) this keeps happening over and over with a different port each time.


[Improvement] Creating a role in the Gravitino CLi with a missing role name give an unexpected error

What would you like to be improved?
The command gcli role create --metalake metalake_demo gives the unexpected error "name" field is required and cannot be empty.
How should we improve?
Display a better error message with the right missing field.

The commands gcli role delete -metalake metalake_demo give Invalid string to encode: null which also could be improved and be more user-friendly.

Note the error is saying that the name of the of the role is empty so it not incorrect it's just not user-friendly.


Strapi + Digital Ocean Spaces on localhost (CORS issue)

Hello,
In order to make the strapi-provider-upload-do work on localhost I had to customize the strapi::security in middlewares.js. Otherwise, all the links in the preview would be broken.
I used this config:
  {
    name: 'strapi::security',
    config: {
      contentSecurityPolicy: {
        useDefaults: true,
        directives: {
          'connect-src': ["'self'", 'https:'],
          'img-src': [
            "'self'",
            'data:',
            'blob:',
            'dl.airtable.com',
            'MY-DO-SPACES-URL.digitaloceanspaces.com',
          ],
          'media-src': [
            "'self'",
            'data:',
            'blob:',
            'dl.airtable.com',
            'MY-DO-SPACES-URL.digitaloceanspaces.com',
          ],
          upgradeInsecureRequests: null,
        },
      },
    },
  },

Questions:

Should this be added to the docs?
Is this the correct config?
Is this a security concern?
Is this needed only locally or also for production? (Haven't been able to deploy yet, so I'm not sure. Will update when I do.)

I'm not sure I understand where the issue is comming from or even what is the precise issue you are facing.
Feel free to provide a log, stack trace or better yet a codesandbox / jsfiddle example; And I will look into it.
Regards,
Stanley.

I think we can close this. After further investigation, I've found this in the strapi docs: https://docs.strapi.io/developer-docs/latest/setup-deployment-guides/configurations/required/middlewares.html#internal-middlewares-configuration-reference
Example from the docs:
// path: ./config/middlewares.js

module.exports = [
  // ...
  {
    name: 'strapi::security',
    config: {
      contentSecurityPolicy: {
        useDefaults: true,
        directives: {
          'connect-src': ["'self'", 'https:'],
          'img-src': [
            "'self'",
            'data:',
            'blob:',
            'dl.airtable.com',
            'yourBucketName.s3.yourRegion.amazonaws.com',
          ],
          'media-src': [
            "'self'",
            'data:',
            'blob:',
            'dl.airtable.com',
            'yourBucketName.s3.yourRegion.amazonaws.com',
          ],
          upgradeInsecureRequests: null,
        },
      },
    },
  },
  // ...
]

So looks like it's safe to use.
Btw. for anyone encountering this in the future, I also had to customize strapi::query in order to get it working (per the example in docs).

Noted,
I'm closing the issue. Feel free to reopen if it was indeed an issue with the package.


Route53 ARecord Override props not being used

Discussed in https://github.com/jetbridge/cdk-nextjs/discussions/204
props.overrides.aRecordProps is never actually passed into the creation of the ARecord (ditto for AAAARecord). Seems to me like it should though 🤔
https://github.com/jetbridge/cdk-nextjs/blob/v4.0.0-beta.25/src/NextjsDomain.ts#L168-L169

@patrickufer, you're correct. Thanks for raising this issue. Could you create a PR? Thanks!

Any chance you could release this under v4.0.0-beta.26 for us? 🙏


New float layout partly blocks play-button sooner in playlist

This seems to be a cosmetic only problem on mobile devices, but I really have more trouble then before clicking on play-button with longer named files (and will hit more often with new subfolder feature). It still works but you have to press left outside the range you would usually press.
Still just a small annoyance, so no quickfix needed. Tried with CSS clear:both; but didn't find a good solution so that the break of filename-DIV in new line would only occur on very small viewport.
Could of course also be fixed by adding the a href="?playpos=x" on top to the filename (and perhaps style it a bit with nounderline:link ... ) to get a bigger area to click.
Version 1.0.0 seems to have more margin before overlapping still occurs. Simulated using FF in very small window for both 1.0.0 and 1.1.6:

Should be solved in actual master branch.

In Version 2.1.8 with Spotify I have no problems on my 5" mobile phone. Pressing the play buttons works fine for me and the text starts on the right site of the button.
When I reduze the browser window to the minimum size everything is fine too

Hi @gizmo21 @SpookyFishes
I made a new version using "tables" which seems to work. Please have a look and give feedback.
All the best, micz

I just updated to 1.1.9-rc7 and it looks fine on my mobile phone :-)

Thx for new version, I still had no time checking as I'm still on 1.0 as it's the fastest and most reliable for me (or the endusers ;) ). Hope to upgrade an DEVcard soon™ .


Remove features dictionary as input to calc_feature

The calculation of each feature is independent of the rest, so having "features_compute" as an input argument to calc_feature suggested otherwise and was not necessary. It might also pose problems if we decide to multi-thread the feature calculation at the DataProcessor or FeatureProcessors level.

I see, then let's wait on it until the project develops in a way that we need to go one way or the other. There might be other ways to handle the interdependency as well. Right now this can wait. Just to know, which feature is currently calculating "ratio between beta and gamma"?

Currently there is no feature that is doing that. But I've seen that many times that it was very relevant for decoding. For example Zixiao's paper used those: https://www.nature.com/articles/s41746-024-01115-7


encoding problems on windows

Please see this question on StackOverflow:
http://stackoverflow.com/questions/29830563/encoding-problems-with-gulp-on-windows

Add <meta charset="utf-8"> to your <head>.

That doesn't solve the problem. The page is shown the same way. Additionally, we use gulp on js and css files. No meta's are allowed there

+1
I think this should be part of gulp.src. It uses readFile underneath which accepts an encoding option. I think it's reasonable to expose this option to user instead of swallowing it up. I wonder if it has ever been discussed before. Otherwise I'd love to create a PR to vinyl-fs to expose this option.

@limdauto I think there is an issue open for it that nobody ever got around to

@contra hey mate could you take a look? https://github.com/wearefractal/vinyl-fs/pull/66

Closing as https://github.com/wearefractal/vinyl-fs/pull/66 were closed.


Donut Image Processor

What does this PR do?
Adds image processor for Donut model
Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Tiers

If any changes are made to the tier file "basic, lucky, titan" it says no tiers can be found please delete the tries fold so it can be remade

I found the bug, you can't mix hours and minutes in the envoy time


UI: Cron workflows filter not working as expected

Summary
when I click “Cron Workflows”, the app navigates  to
https://argo.curation-ppd-usw2.iks.a.intuit.com/cron-workflows/undefined
The search field is empty and no cron workflows are listed.
When I click the X in the empty search field, the search fields remains empty but the url changes to: https://argo.curation-ppd-usw2.iks.a.intuit.com/cron-workflows/
And now all cron workflows are listed.
If you navigate to https://argo.curation-ppd-usw2.iks.a.intuit.com/cron-workflows/  directly, it will redirect to https://argo.curation-ppd-usw2.iks.a.intuit.com/cron-workflows/undefined
Diagnostics
v2.10.0-rc1

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Does this only happen for cron workflows?

Looks like it is actually happening for every view that has a filter

I feared as much


2 only work sometimes, it was flaky as :tx-id kept increasing

I was having a similar problem with :tx-time, my local time was different than the CI time for hardcoded string times


add log compare test for blinky example

Add a log compare test for this example for enable example CI testing

Question: how do I maintain this log (generate a new one, just need to edit it as I understand right) - any change to example needs log update.

Question: how do I maintain this log (generate a new one, just need to edit it as I understand right) - any change to example needs log update.

the detailed instruction is in that fixed link, the log will like to be manually maintained, they just standard RegEx. if any one change the example the log will need to be updated.

I am thinking that this does not test at all that the blinking happens. What you think about adding "led on" and "led off" prints to the file and check that at least couple of those are there? Then we know at least that board is not in stuck. For sure we don't know if led is on really but at least logic works.

I am thinking that this does not test at all that the blinking happens. What you think about adding "led on" and "led off" prints to the file and check that at least couple of those are there? Then we know at least that board is not in stuck. For sure we don't know if led is on really but at least logic works.

@jamesbeyond please review, and send separate PR

I am thinking that this does not test at all that the blinking happens. What you think about adding "led on" and "led off" prints to the file and check that at least couple of those are there? Then we know at least that board is not in stuck. For sure we don't know if led is on really but at least logic works.

I think that wouldn't be necessary. printing led on/led off are not equivalent to the LED is physically on or off. that will require manual checks. also the on and off logic is fairly simple. in this test we just want to make sure the example wasn't crash/broken badly from beginning


Annotate forecast recent events for post-process filtering

The UMASEP model is essentially a triggered nowcasting model in that it will only produce a positive forecast in response to either a sufficiently strong solar flare or a detected rise in proton flux.  Nonetheless, it issues a forecast at a regular (e.g. 3 min) cadence, which naturally results in an enormous number of true negative (TN) in the (inverted) All Clear contingency table.  This renders all metrics that include TN mostly useless, as the statistic is dominated by forecasts in which the only evaluation the model has done is that conditions are quiet.
A more useful TN count could be obtained if only forecasts that should have been the result of a model trigger are considered, i.e. only select the UMASEP forecasts that were derived from a solar flare of sufficient magnitude, or by the onset of a SEP event (or sub-threshold non-event).  This would allow metrics such as Specificity {TN/(TN + FP), P(Forecast:No|Event:No)}, False Alarm Rate {FP/(TN + FP), P(Forecast:Yes|Event:No)}, Accuracy (TP / N), and Rate Correct ( (TP + TN) / N ) to have their intended meaning.
Since the details of the nature of a forecast (triggered vs. quiet) are not stored, we can only select the "should have triggered" forecasts by pre-processing the X-ray and proton time series and extracting timestamps.  Many groups have done this, and we should leverage existing flare lists.
To incorporate the lists in the analysis, SPHINX outputs should include a kind of per-forecast "recent events" datum.  For every forecast issue time, look into a 2 hour window preceeding the issue time and record:

pre_max_flare_class : largest flare flux (in GOES class notation)
pre_max_flare_time : largest flare peak time
pre_max_gt10_flux : max proton flux (per integral flux channel – 10, 30, 50, 100)
in_onset : (boolean) True if issue_time is after SEP event start time and before onset max time

These annotated quantities can be used to make post-processing selections (e.g. pre_max_flare_class >= C5) to pare down the contingency tables and make the TN statistic take on new meaning.

Note that this feature request is designed to implement Steve's requested analysis from his email of Aug 28, 2023.  I copy it below for reference.

With all the push to be ready for SEPVAL I almost hate to mention this because I don’t want to detract from the effort.  But if I don’t write while I’m thinking about it, I’ll forget.  Forgive me if I have this wrong.
I was troubled by the discussion of scoring UMASEP a couple weeks back.   If I understood, the number of forecast events was the cadence the model is polled or posts a forecast, which I believe is 3 minutes.   This creates an incredible number of forecast cases. (175 kforecasts/year at 1per3 minutes).  Positive forecasts are very few.  This make the forecast matrix
Actual
Forecast       Yes      No
Yes               5
No                         175 K * several years

It would seem to be better to score it based on when it should be triggered, which is principally C5 flares or larger.  This would make the number of forecast events:
Total events  = Number of C5
+ number of non-C5 instances it should trigger
+ number of Non-C5 triggers that produced a forecast.
Maybe it still is a large number but would produce a better representation of performance.
In general, grading on the number of forecasts posted (X times a day) *  (days)  as  the number of forecasts,  seems excessive when there are specific circumstances to trigger a model’s forecast.  Maybe a framework more like below would make more sense
the number of times it should have triggered + the number of times it triggered when it shouldn’t have
Rather than every forecast that a model posts (especially for models that post forecasts at high cadence).  Daily forecasts would use the number of days for the forecast (e.g. MAG4).
I think this is a little different than we were discussing and thought I would bring it up for consideration.
Thanks
SJ


docs: add EduLinks site to showcase

Motivation
The website https://edulinks.app, which I created and maintain, is built with Docusaurus. I'd like to add it to the showcase page.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Run yarn run start:v2 and check that the page http://localhost:3000/showcase is rendered properly.
Related PRs
No.

Thanks!

Cool page! :)
Sadly the pages source is not public, so it's not possible to either contribute nor to see your page as a reference...

Thanks👍


Extract bitcoin-transaction-monitor crate

See commit messages for details.
The overall motivation for this is to improve compile-times across our workspace, test things in isolation, allow for a narrower focus when debugging issues and potential reuse in other / future projects (we have re-written transaction monitoring a couple of times now).

I don't understand why CI is failing, I tested opening a position locally and it worked like a charm.

It seems to be failing in the smoke test, it might be just the case that it takes a bit of time to set up all the counters in the daemon. It should work if you increase the timeout on the smoke test!

I am doing a binary search of why CI is failing by force-pushing different sets of commits. Please ignore until I have found the problem!

@thomaseizinger It seems to be failing in the smoke test, it might be just the case that it takes a bit of time to set up all the counters in the daemon. It should work if you increase the timeout on the smoke test!

But there are no counters in this PR :D

@thomaseizinger It seems to be failing in the smoke test, it might be just the case that it takes a bit of time to set up all the counters in the daemon. It should work if you increase the timeout on the smoke test!

But there are no counters in this PR :D

it still is just the smoke test, not the real tests ;D

@thomaseizinger It seems to be failing in the smoke test, it might be just the case that it takes a bit of time to set up all the counters in the daemon. It should work if you increase the timeout on the smoke test!

But there are no counters in this PR :D

it still is just the smoke test, not the real tests ;D

And yet it is reproducible on CI! I don't understand :(

I am doing a binary search of why CI is failing by force-pushing different sets of commits. Please ignore until I have found the problem!

So I could reproduce that it is literally the last commit that introduces the problem ..

Issue seems to be resolved with the changes I am making in #1554. Will leave this as draft until the other on is fully-rebased and ready.

bors r+


⚠ Secret Website 2 has degraded performance

In 4d67b9a, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 3790 ms

Resolved: Secret Website 2 performance has improved in ff85c46.


Segfault (core dumped) when running on Linux (manjaro)

Godot version:
3.1.1.stable.official
OS/device including version:

Issue description:
Trying to start godot, and i get this error.

It shows the initial splash screen, but crashes before it reaches the project manager.
Interestingly, if i cd to /usr/share/applications or /usr/bin then it works. But trying to launch from /home or from my taskbar doesn't work.
Steps to reproduce:

Install godot, be it through the AUR or from the self-contained executable on the website.
Try to launch it.
You'll get this error

Minimal reproduction project:
None to be had

A permission problem I guess? I have the exact same chipset and it works, although I'm on Ubuntu not Manjaro.

sudo godot seems to work, but it doesn't load the same editor preferences, and opening the terminal first is a bit cumbersome anyways

Could you install strace then run the Godot binary using strace?

The output from strace is too long for the terminal (it seems to discard it and I can't scroll up to it), and I'm not able to get it to print out to a file. I'll try compiling it and running gdb later and let you know

Do some of those help? https://askubuntu.com/questions/690703/save-current-terminal-scrollback-to-file

Here's the log: https://bin.privacytools.io/?ebc11f3f9b4caa11#J7VL7zf6AndnjoQgxB80YynRzD1Zxurgpb1gcCXeS+A=
It seems to be same thing over and over again (not even sure why it's calling the epic games store). I don't think the issue I was talking about got resolved though @Zireael07 some of the output (a lot of it actually im guessing) gets cut out because the terminal just discards it since it's too much. However, I figured some log was better than none so there you go. Not sure if the log will help much but oh well

@Sajo8 It might be better to post the log here as a comment (in a code block, surround it with ```) so it doesn't expire over time :slightly_smiling_face:

You sure? It's really long. Here's a version which doesn't expire: https://bin.privacytools.io/?e74e6ee947360c42#o5BIwSBB9CgPmK0pCJn/YmxtsVxnmJQfbvDI+NUP33Q=

You sure? It's really long. Here's a link which doesn't expire: https://bin.privacytools.io/?6cea05757ebd84e0#JfMEWoKrSb/2JS2GA2GAxz3BBVZ/gvPu90BX4uMi6lw=

So running /usr/bin/godot works but godot doesn't? Could you post the output of alias godot and **cat ~/.config/godot/editor_settings-3.tres | grep projects
** from your terminal?

No, /usr/bin/godot and godot both fail (when running it from the home dir). cd /usr/bin/ and then godot works. It also works if you cd to /usr/share/applications and run godot.
I don't have any alias set for godot so alias godot just returns a "not found" error (bash: alias: godot: not found)
output of the other command:
projects/::home::sajo::Desktop::saj::godot::chukwas-labyrinth = "/home/sajo/Desktop/saj/godot/chukwas-labyrinth"
favorite_projects/::home::sajo::Desktop::saj::godot::chukwas-labyrinth = "/home/sajo/Desktop/saj/godot/chukwas-labyrinth"
projects/::home::sajo::Desktop::saj::godot::demoplatformer = "/home/sajo/Desktop/saj/godot/demoplatformer"
projects/::home::sajo::Desktop::saj::godot::demomultiplatformer = "/home/sajo/Desktop/saj/godot/demomultiplatformer"
projects/::home::sajo::Desktop::saj::godot::fbla = "/home/sajo/Desktop/saj/godot/fbla"
projects/::home::sajo::Desktop::saj::godot::demoplatformerrigid = "/home/sajo/Desktop/saj/godot/demoplatformerrigid"
favorite_projects/::home::sajo::Desktop::saj::godot::fbla = "/home/sajo/Desktop/saj/godot/fbla"

Sorry for the late answer. Can you still reproduce the issue?
Given that it works from /usr/bin and /usr/share/applications, but not from /home/sajo, I suspect that you have a project.godot file in /home/sajo which Godot tries to open, and then it imports your whole home folder as a Godot project.
It then hits stuff like this as shown in your strace;
stat("/home/sajo/Games/epic-games-store/dosdevices/z:/etc/audisp/plugins.d", {st_mode=S_IFDIR|0750, st_size=4096, ...}) = 0

Which seems to be an EGS install with a wine prefix that has a symlink to your root filesystem / and so it tries to load the whole / and eventually loops back to itself and explodes.

Closing due lack of response

Closing due lack of response


nonFatalErrors is always empty in the indexing status API

This was left out from the initial non-fatal errors implementation, but it's of course important for usability.

Was the intention that this would be an array of the observed non-fatal errors?

Exactly.


Minor documentation update

Callout distinction between original and current SQL engines.
Signed-off-by: Max Ksyunz<EMAIL_ADDRESS>Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #718 (09733a5) into main (5c6fd72) will decrease coverage by 31.98%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #718       +/-   ##
=============================================
- Coverage     94.75%   62.76%   -31.99%
=============================================
  Files           283       10      -273
  Lines          7678      658     -7020
  Branches        560      118      -442
=============================================
- Hits           7275      413     -6862
+ Misses          349      192      -157
+ Partials         54       53        -1

Flag
Coverage Δ

query-workbench
62.76% <ø> (ø)

sql-engine
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...earch/sql/opensearch/data/utils/ObjectContent.java

...ecutor/protector/OpenSearchExecutionProtector.java

...org/opensearch/sql/analysis/QualifierAnalyzer.java

...org/opensearch/sql/opensearch/client/MLClient.java

...java/org/opensearch/sql/ppl/parser/AstBuilder.java

...pensearch/sql/planner/physical/RenameOperator.java

.../sql/planner/physical/PhysicalPlanNodeVisitor.java

...arch/storage/system/OpenSearchSystemIndexScan.java

...ript/filter/lucene/relevance/MatchPhraseQuery.java

...arch/storage/script/filter/FilterQueryBuilder.java

... and 263 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

Please see https://github.com/opensearch-project/sql/pull/728 -- same change, but with passing DCO check.


ci(aks+eks): bootstrap juju controller with agent version 3.5.0

Context
https://github.com/canonical/bundle-kubeflow/issues/921 is causing issues when deploying some charms; we need to add the workaround suggested there.
What needs to get done
Edit the deploy-to-eks and deploy-to-aks workflows to bootstrap the juju controller with agent 3.5.0 to avoid the issue.
Definition of Done
The juju controller that runs in those workflows is bootstrapped with agent 3.5.0 and the error is not present on the CI anymore.

Closing this as mentioned in https://github.com/canonical/bundle-kubeflow/pull/941#issuecomment-2183191865

Based on the decisions in #940, this PR won't apply anymore and will be replaced by #945. Feel free to close this PR and the issue it is trying to solve.


Find the package.json in the /mnt/app/bundle directory

Find the package.json in the /mnt/app/bundle directory

Closes #7

👍

👍

👍


SDK needs to be updated to handle Horizon 0.17.0 breaking changes

In Horizon release 0.17.0 (released 26 Feb 2019), several breaking changes were made (changelog), changing or removing fields that had previously been marked as deprecated.

Breaking changes
Fields removed in this version:
    Root > protocol_version, use current_protocol_version and core_supported_protocol_version.
    Ledger > transaction_count, use successful_transaction_count and failed_transaction_count.
    Signer > public_key, use key.
This Horizon version no longer supports Core <10.0.0. Horizon can still ingest version <10 ledgers.
Error event name during streaming changed to error to follow W3C specification.

Here is the relevant Horizon commit, showing the fields that were changed: https://github.com/stellar/go/commit/575707ed074a5672438b0f2b788f8b38584ef18d
This is an urgent issue! Any code using this SDK to access a Horizon 0.17.0 server (which includes the SDF public horizon instance at https://horizon.stellar.org/) will be broken until the SDK is updated to match the new response fields.

I'm going to close this issue in favor of #67
This SDK mainly uses the raw horizon JSON responses so this also might not require any changes.


Exempt scala-actors-migration from version checks.

Fixes #1818.

Can one of the admins verify this patch?

I feel like we should flip the logic and check only for scala-compiler, scala-library, etc - http://mvnrepository.com/artifact/org.scala-lang

Sure, but isn't false positive warnings better than no warning and repercussion failures? I'm not sure, but I'm happy to do it the other way.

yeah, it's safer to have false positives. e.g. scala-reflect showed up one day and had to be included.   THe scala team is now aware of our convention (or at least were at some point) hence everything moving into org.scala-lang.modules.  I'd rather special case the outlier to the rule than prevent flexibility in design in the future.

this LGTM

If we're not flipping the logic, could we also add the "scala-pickling" to the known bad list (even though the latest Pickling has moved to org.scala-lang.modules)?

Absolutely.


Update git ignore to exclude intellij folder

Add arguments to simulator to allow java debugging
Add config to gradle to allow using standard target to publish to maven local repository

I have been working on some updates to the simulator to add more functionality to ctre simulator.  And I found the ability to debug useful.  I thing other teams may also find that handy also to debug their code locally.


Enable Osmosis authz amino exec support

Pending a chain update fixing Authz amino MsgExec support for governance message types. Currently MsgExec throw a signature error and is disabled for Osmosis in REStake after #627

Pending https://github.com/osmosis-labs/cosmos-sdk/pull/342


Fix ODM example

Tests
Ubuntu 18.04
CUDA 10.1
Pytorch 1.4

Train + eval of Direct mode (0.778 IoU) on 'chair' class (30 epochs)
Train + eval of MVD mode (0.760 IoU) on 'chair' class (498 epochs)
Resume training
Visualization

Notes

IoU is good but results don't look as good as they do in paper.
In above test, MVD mode did worse than Direct mode.

Currently re-training to confirm that no regression occurred after these latest fixes.


check type assertions where it makes sense to avoid runtime panics

What this PR does / why we need it:
#22 - This PR adds type assertion checks where it feel sensible in order to avoid nil pointer referencing that causes runtime panics.
There are still a lot of type assertions occurring in the code, but adding checks for many of these either didn't make sense in the context, or would require code refactoring beyond the scope of the ticket.
Which issue(s) this PR fixes:
Fixes #22

@mandelsoft certainly understand where you're coming from. I think, even if we don't expect there to be issues caused by these type assertions, that checking them is still a worthwhile exercise. It adds very little code overall, and we remove an unknown variable when encountering possible runtime panics. Effectively, I don't think it hurts to be sure that these assertions won't cause us issues moving forward.


Recording video on LineageOS gauguin not working

LineageOS 19.1 2022-06-06, gauguin
Secure Camera v43 or 42
Not sure if Secure Camera or LinageOS updates broke video recording. What happens is:

Start Video
image on screen instantly gets stuck while recoding time continues ticking
as soon as the video is stopped message Recording too short to be saved is shown
video is never saved

Here is Logcat (30 days), maybe it is useful to help understand what is going on.

EIS is used by default when available now. When it's available, there's a toggle for disabling it in the settings dialog for video mode. Please check for that toggle and try disabling it. Your OS may have a broken implementation of EIS.

Thanks for the speedy response. Disabled EIS in settings but the problematic behavior persists.

Please post logs that aren't restricted behind client-side decryption and WASM.

Can you be more specific as to what to change?
I stopped Orbot but am still seeing 06-11 17:11:51.002  1322  1885 W ActivityManager: Foreground service started from background can not have location/camera/microphone access: service org.torproject.android/.service.OrbotService in the log.

Hi @foss-,

I stopped Orbot but am still seeing 06-11 17:11:51.002 1322 1885 W ActivityManager: Foreground service started from background can not have **location/camera/microphone** access: service org.torproject.android/.service.OrbotService in the log.

The logs related to Orbot are not related to our camera application. Could be an issue related to Orbot/OS or maybe the service hasn't actually been properly turned off as expected.
Please share the logs in it's decrypted form or if you're concerned about any sensitive data getting via Matrix (@wowowowowowowowowow:matrix.org) to enable us to further help with this issue.
Thanks for trying out our camera application @foss-!

Hi @foss-,

I stopped Orbot but am still seeing 06-11 17:11:51.002 1322 1885 W ActivityManager: Foreground service started from background can not have **location/camera/microphone** access: service org.torproject.android/.service.OrbotService in the log.

The logs related to Orbot are not related to our camera application. Could be an issue related to Orbot/OS or maybe the service hasn't actually been properly turned off as expected.
Please share the logs in it's decrypted form or if you're concerned about any sensitive data getting via Matrix to enable us to further help with this issue.
Thanks for trying out our camera application @foss-!

It has nothing to do with Orbot.

Log is freely accessible, no encryption or required password for access.
Additional testing revealed the issue also happens with v41 which iirc worked fine previously. This would suggest the issue was not introduced by a recent Secure Camera update but instead lies with LineageOS itself.
Will bring this up with the maintainer of gauguin. Sorry for not testing this thoroughly enough before reporting. If help is needed, will re-open.
Thanks for helping me out so kindly.


Add react-admin admin GUI for user management tasks

@elitan Now Hasura support react-admin I think it can be useful if this backed-plus can support user management task (create user, role, reset users password, ...) thought a GUI admin panel.
What is your opinion?

I agree. A GUI would be helpful.
However you can already do most of in the Hasura console. You can:

Create users
Delete users
Edit users (email, role etc)

You can not reset user password in the Hasura Console with this repo tho.

@elitan Thanks man, so are you any plan for add any GUI to this repo?

Exactly.
A GUI for managing users and storage should probably live in another repo. I currently do not have any short term plans on doing that.


Remove old string parsing util and replace them by ParseInt

We remove DecimalStringToNum and OctalStringToNum in this PR, since they are hard to use and too less general than ParseInt (e.g. the parsed integer type cannot be changed), and they are just wrappers of ParseInt after we introduce ParseInt.

Thanks all. Merging...


ORM by itself?

Hi,
So for the time being, I'm working with some Lua scripts requiring database access. I don't need the entire Gin framework, only the Models/ORM as it seems far superior to anything related I've come across so far.
Is there a way to use it without OpenResty and the rest of the Gin (API related) framework?

I don't think so. I believe it relies on the OpenResty database drivers.

Hi, as per @jtarchie 's response I had to rely on internal OpenResty drivers so no additional work has been provided to make the ORMs and migrations work as standalones.
If needed, it is possible to extract the ORM and migration engines from here:
https://github.com/ostinelli/gin/tree/master/gin/db


Not getting all outputs when using kubectl logs with tetragon-cli

As mentioned in the README, I was using kubectl logs -n kube-system ds/tetragon -c export-stdout -f | tetragon observe in order to observe the tetragon events. Unfortunately it seems that kubectl logs is failing to follow the logs (I think in the event that too many logs are being produced at once.
To combat this, I have used stern instead and this seems to be far more reliable:
stern -l app.kubernetes.io/name=tetragon -E tetragon -n kube-system -o raw | tetragon observe

Maybe it would be better to enable the tetragon-cli to not rely on being piped the output of logs from another binary.

I think that currently tetragon-cli only transforms json messages to readable one-line logs, which seems implemented in pkg/encoder/encoder.go.
The log is in fact stored in /run/cilium/tetragon/tetragon.log. You may verify it via
kubectl cp kube-system/"$(kubectl get pods -n kube-system | grep tetragon | awk '{print $1}')":/run/cilium/tetragon/tetragon.log ./tetragon.log -c tetragon.

Hi @Forsworns 😄
Sure this is the case, but I am simply saying that from a usability perspective, there may be a more optimal way of providing the user with a way of observing the events than what is currently proposed here.
@willfindlay, are their any reasons why tetragon observe would not benefit from some extra logic so that it can do its own streaming of the tetragon logs? I know that the Argo Workflows project does this in their CLI (see here), which could be an option.
Alternatively, if this lack of reliability is something that others have observed from using kubectl logs -n, could it be considered to suggest a different command to pipe into tetragon observe?


Wilderwild dosent work properly on 1.21 neoforge sinytra

Mod Name
wilderwild
Mod Homepage
https://modrinth.com/mod/wilder-wild
Minecraft version
1.21.1
Describe the bug
using PCL luncher, with the crash "名为 wilderwild-2.4.8-mc-1.21_moj_1.21.jar 注入失败，导致游戏出错。”
Steps to reproduce
1.im sure that i had installed the lastet version of sinytra connector and forgified fabric api.
2.adding the wilderwild mod to the mods folder.
3.game could not lunch and the crash report comes out.
...
Logs
https://gist.github.com/YaeSlight/90de085d4571da03affd8fb4a619632e
Additional context
No response

Duplicate of #1055


Add Hashing Message Calculator

Fixes Issue🛠
Closes #1369
Description👨💻

Add new calculator - Hashing Message Calculator
Created directory in /Calculators with 4 files - index.html, style.css, script.js and README.md
Created a subdirectory inside /Calculators/Hashing-Message-Calculator, assets with background.jpg
Modify index.html for the main page to include the Hashing Message Calculator in the Calculator Section

Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[X] New feature (non-breaking change which adds functionality)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[X] I am an Open Source contributor
[X] I have performed a self-review of my code
[X] My code follows the style guidelines of this project
[X] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷
Screenshot:

Linked Issue: #1369

@Rakesh9100 This PR can be merged.


fixup! feat: enhanced cron expressions

Order is important in _create_schedule_with_variables.
Cron expression in entity_config must be replaced before it is decomposed into scheduled data.

Refs: #455

@gdubicki, sorry about the new PR 🤕 .  I just tried main on our internal instance and it choked because replacing happened after the final POST data was created.


Password toggle black in dark mode

Description

could you please explain a bit more i want to solve it?

Hey, I'm currently not accepting code contributions to any of my GitHub projects. Thanks for your interest though.
Bug reports, feature requests, and other suggestions for improvement, on the other hand, are always welcome.
I see that the "good first issue" tag on the tickets was misleading, which is why I removed the tags.

ok buddy


embedded.tts.dylib does not support run by Rosetta

Describe the bug
When I run a program with x86_64 architecture under the Apple M1, if there is libMicrosoft.CognitiveServices.Speech.extension.embedded.tts.dylib in the lib directory,
Then the program will crash. Whether I use it or not, it will work normally after deleting it.
The program of the arm64 architecture does not have this problem.
Version of the Cognitive Services Speech SDK
1.24-1.26
Platform, Operating System, and Programming Language

OS: Mac OS 13.2
Hardware: M1
Programming language: C++

Additional context
return information：
zsh: illegal hardware instruction
lldb debug:
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_INSTRUCTION (code=EXC_I386_INVOP, subcode=0x0)
    frame #0: 0x000000010b009070 libMicrosoft.CognitiveServices.Speech.extension.embedded.tts.dylib`___lldb_unnamed_symbol3625 + 32
libMicrosoft.CognitiveServices.Speech.extension.embedded.tts.dylib`___lldb_unnamed_symbol3625:
->  0x10b009070 <+32>: vxorps %xmm0, %xmm0, %xmm0
    0x10b009074 <+36>: vmovups %ymm0, 0x5ec79c(%rip)
    0x10b00907c <+44>: leaq   0x5199c1(%rip), %rbx
    0x10b009083 <+51>: movq   %rbx, %rdi
Target 0: (s) stopped.

Hi @czkoko I'm Darren from the Speech SDK team. Thank you for reporting this. What does your C++ program do? Can you share example code? Are you using the SpeechSynthesizer object?
Please do a run of a single session, while taking Speech SDK logs leading up to the crash, and attach the SDK logs here.
Thanks,
Darren

@czkoko Since you are using SpeechConfig (for cloud services), do you not expect to need embedded speech? In that case your project should have only Microsoft.CognitiveServices.Speech nupkg installed, not any of Microsoft.CognitiveServices.Speech.Extension.Embedded.* packages. We've verified x64 dylibs on native x64 platform, not with Rosetta, so there could be some incompatibility.

@pankopon  Yes, I know how to avoid this problem. I also tested that there is no problem running on Intel CPU, It's just report the potential incompatibility problems. Maybe I will use embedded speech in the future. After all, only compiling x86_64 can reduce part of the program size, and it can run on apple and intel CPU at the same time.

@yulin-li Can you please check.

Hi @czkoko , thanks for reporting this issue and we will investigate and try to fix. We have created an internal work item (id 5264814) to track. We will update this issue once it's addressed.
I am closing this issue for now as it's tracked internally and needs some time.


Add nginx conf and dockerize client

This adds a production build pipeline and nginx server as part of #7
This does not address wiring the application client & server up- I have a few questions and thoughts around doing this, especially since the client prod vs dev differs greatly.

Generally, what is the end expected deliverable? Is this PR overreaching, or do we want a nicely buttoned up production app?
@carrollgt91 as someone with Docker chops- do you have strong opinions on how to deviate dev vs. prod Dockerfile & docker-compose.yml? How about doing:

# Prod
Dockerfile
docker-compose.yml

#Dev
dev.Dockerfile
dev.docker-compose.yml

#Other env (testing, etc)
env.Dockerfile
env.docker-compose.yml

I'm a bit rusty in this category, and quick googling doesn't show a great coalescing.

Do we want to pursue hot reloading on FE & BE at the same time? This is the part I have some ideas on but could see some speed bumps.

I'm also open to feedback on everything- approach, technologies used, etc.

Going to go ahead and merge this, as it will assist Tom in getting started on the backend integration.
Right now, we're just focusing on dev - there are a number of important questions we'll need to ask before really digging into production configuration, and it likely will be more coupled to the initial deployment environment. So, for now, this looks great! Thanks Kevin!

Re: hot reloading both projects when one is saved - not for now, no. I think that's easy enough to work around.


fix: piece_loader to piecer_loader

Replace piece_loader with piecer_loader, as described here https://github.com/explosion/spaCy/discussions/13152#discussioncomment-7670206.
Description
Work in progress.
This trivially fixes the problem, but I would appreciate any direction on what tests you would like. I added a short test that attempts to validate the expected test outputs, but this is currently failing. Also not confident that it is testing what I think it is. Happy to delete/modify, or add further tests as you see fit.
Types of change
Bugfix
Checklist

[x] I confirm that I have the right to submit this contribution under the project's MIT license.
[ ] I ran the tests, and all new and existing tests passed.
[x] My changes don't require a change to the documentation, or if they do, I've added all required information.

Just to give some background, the use of spacy-curated-transformers with the trf v3.7 pipelines was a bit of soft launch to test all the curated package internals in use. So far the internals part has gone quite well, but we're still working on all the user-friendly bits in spacy for creating new pipelines. In particular, spacy init config doesn't yet support curated_transformer in terms of setting up a longer pipeline with listeners or providing language-specific defaults/recommendations.
fill-curated-transformer was intended to bridge the step in between spacy init config and spacy init fill-config, since curated_transformer is written quite generally and doesn't necessarily have to load a model through transformers. But frequently this is what users want to do and so this reads all the settings from a HF model and sets up the model settings and initialize steps to load the model.
spacy init config -p curated_transformer config1.cfg
spacy init fill-curated-transformer --model-name xlm-roberta-base config1.cfg config2.cfg
spacy init fill-config config2.cfg config3.cfg

curated_transformer defaults to spacy-curated-transformers.XlmrTransformer.v1 with defaults for xlm-roberta-base, so fill-curated-transformer primarily adds the initialize sections.
If you edit config1.cfg above to modify the architecture and piece_encoder, fill-curated-transformer will update a number of related settings (vocab_size is the main one, but there are several) along with the initialize settings.
So with these edits:
@architectures = "spacy-curated-transformers.CamembertTransformer.v1"
piece_encoder = {"@architectures":"spacy-curated-transformers.CamembertSentencepieceEncoder.v1"}

and
spacy init fill-curated-transformer --model-name camembert/camembert-base config1.cfg config2.cfg

You end up with a working config for camembert-base.
At this point it looks like you're expected to provide the right architecture and piece encoder. It also doesn't currently check whether the specified piece_encoder is correct for the provided model, and the errors you get beyond this point (e.g. from sentencepiece) are not particularly helpful.

Let me see if I can get this working well enough for the testing side. The user-friendliness side would still be to come...

Our internal testing models weren't configured correctly aside from explosion-testing/xlm-roberta-test. I've also used the hf tiny internal testing models in spacy-transformers, with the known risk that the test suite would break if they are modified/removed.
Longer-term it would be better to have our own versions/clones/copies of the models that we're using.


Support for multiple CVs as posts

Some very simple edits to the original. Instead of having a single index.html, we can use jekyll to its full potential and have multiple CVs as posts in _posts. Then, when serving with Jekyll we get to see all of them.
Waiting for any comments before updating README with this info. 👍
Please note that you will need to add dates to the names of the CVs in the _posts folder for them to appear when you serve with jekyll (just like one would usually do when using the blogging functionality of jekyll). Check the examples in the branch.

Thanks for the PR :) What would be the reason to have multiple CVs? I am not sure I see the use case and I am wondering if it's worth the increase in complexity?

My pleasure! This grew out of applying for tech jobs in the past and having to track what CV I used for different applications. They would often end up looking significantly different, so I thought it would be nice to organize them like this. Also, the increase in complexity is minimal as I'm just tapping more into Jekyll which was already there. I do appreciate that you may want to keep everything clean though. :-)

@kgourgou That's really nice idea 👏
Most of the job-searching platforms recommend the candidates to have a modified version of the CV for each job or at least for a type of job. That's why I believe this PR relevant.
P.S. @elipapa markdown-cv is really useful, great job!


Change decode_length_delimiter to a mutable borrow

While it can make sense in some cases to use a clone here, let's not force users to do it since the function does not need it.

There is a blanket impl Buf for &mut T where T: Buf provided in the bytes crate, so a clone is never necessary.  The API is set up to be maximally flexible in it's current format.


Question: Variable Return List

Hello!
is it possible to define a list variable?
I want to create a new task which must return a variable of string.
Is this possible?
A.Kleine

You can write your result in Hashtable property of the task so it can be
accessible to other tasks.
Le mar. 17 sept. 2019 à 10:07, akleine01<EMAIL_ADDRESS>a
écrit :

Hello!
is it possible to define a list variable?
I want to create a new task which must return a variable of string.
Is this possible?
A.Kleine
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aelassas/Wexflow/issues/115?email_source=notifications&email_token=AAKHB2QZ5XOLHDX5FU22LF3QKCM4NA5CNFSM4IXM5PD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HLZNVKA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKHB2QGUIG2U6OEW62THELQKCM4NANCNFSM4IXM5PDQ
.

Thats great..
Thanks


Update Enketo

What has been done to verify that this works as intended?
EE 4.1.4 is running on the QA server.
Why is this the best possible solution? Were any other approaches considered?
We tried to get an additional improvement to number inputs in but it’s not complete after all. 4.1.4 is the safest version to release.
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
This release mostly has bug fixes and should be relatively safe.
Does this change require updates to documentation? If so, please file an issue here and include the link below.
No.
Before submitting this PR, please make sure you have:

[x] verified that any code or assets from external sources are properly credited in comments.

@lognaturel, are there any changes since 4.1.2 that we should mention in the forum announcement?

I don’t think any really need to be mentioned. A few targeted bug fixes: https://github.com/enketo/enketo-express/releases/tag/4.1.3


direct support for blue / green deployment pattern in helm

The blue / green deployment pattern as described at http://container-solutions.com/kubernetes-deployment-strategies is not supported natively in kubernetes or helm. If it belonged anywhere, would it be helm or kubernetes? Would the helm maintainers potentially accept a blue/green pull request?

This actually happens to be something service meshes handle really well. I highly suggest taking a look at tools like istio if you require the need to perform blue/green deployments for your application as they are able to handle gradual roll-ins and roll-outs because they control the flow of traffic coming into your application.
Helm works more in the sense of a traditional package manager, upgrading charts from one version to the next in a graceful manner (thanks to pod liveness/readiness probes and deployment update strategies), much like how one expects something like apt upgrade to work. Blue/green deployments are a very different beast compared to the package manager style of upgrade workflows; blue/green sits at a level higher in the toolchain because the use cases around these deployments require step-in/step-out policies, gradual traffic migrations and rollbacks. Because of that, we decided that blue/green deployments are something out of scope for Helm, though a tool that utilizes Helm under the covers (or something parallel like istio) could more than likely be able to handle that use case.
For more context, see the discussion in these threads/blog posts:

https://github.com/kubernetes/kubernetes/issues/33462
http://dougbtv.com/nfvpe/2017/06/05/istio-deploy/

Hope this helps!

Closing as answered but please re-open if there are further questions about b/g deployments. Thanks!

@bacongobbler I understand where you are coming from, but I think that istio is the low level way of implementing what we would like. What I mean to say is that (at least to me) I describe my entire server deployment with helm charts. If I change a chart from one version to the next, its helm that reads the helm files and deploys all the changes. Now, it does this by interfacing with lower level components described in the chart, but nonetheless I'm not running kubectl or other commands myself directly to change things. Furthermore, I think it would be incorrect to do so because it means that the state of the world isnt within the realm of any one tool which will lead to errors when these tools dont work together. Or am I missing something?


Add pagination feature

Add pagination as starter feature.
eg:
https://github.com/michaelbromley/ng2-pagination

@klihelp Thank you for the suggestion.
We are not adding features that are not core or build related.
If we do, this repo will not help developers but confuse them.
Think about the time it will take for a developer to remove those it does not need.
There are thousands of little feature, datatable, toaster, modals etc... we can't add them all.


Simplify Network/Link/Node inheritance structure

We have (all arrows point to a super-type):
interface Network     < class OtsNetwork
 ^                       ^
interface RoadNetwork < class OtsRoadNetwork

                                / < class CrossSectionLink
interface Link < class OtsLink |  < class Connector
                                \ < class CapacityLink

interface Node < class OtsNode < class Centroid
                  ^
                 class OtsRoadNode

All classes with Ots in the name can be merged with their respective interfaces. Only for OtsRoadNode this is different. But the point of this class eludes me. It has a constructor with a Direction argument. But OtsNode has a heading argument in its constructor. This information seems double. However, OtsNode.getLocation() returns a DirectedPoint without heading, only based on its OtsPoint3D. Just like OtsRoadNode, this could (and it should...) return a DirectedPoint with heading, and then OtsRoadNode would not add anything anymore. Then we would have only Node and Centroid.
Overall we could go from 13 to 8 classes.

OtsRoadNode has a Direction, while OtsNode has an optional double heading. The direction can likely be removed, as this is only required during setup/parsing. It can still be part of XML to derive center lines. There is no need to store direction information in nodes. If needed during setup, a Map<Node, Direction> may be used temporarily.

All above simplifications have been implemented. For the time being, nodes still have a heading/direction, as it is currently used to get the start-line and end-line of a link.


why the card background color  is grey ? how to change it?

I got the same issue when set useMaterial3: true for ThemeData.
The PlayingCard widget uses the Card widget, which background depends on this parameter.

@xvadim is right, this is due to some new material styling. Added the ability to customize color and surface tint color (with white defaults), which should fix this.

Please reopen if any issues, thanks!


test: Add toleration trait e2e tests

Release Note
test: Add toleration trait e2e tests

Let me merge it. OpenShift and Yaks workflow failures are unrelated.


substitute pyramid generation for bioformats2raw in BRT

Ensure pytools rechunking script works

closed in https://github.com/niaid/image_portal_workflows/pull/203


Unclear what input files are needed. - correction of required files commands

Hello Søren and Jacob
I am uncertain of the input files to the tools from the documentation. Is it possible you can give a more detailed documentation.
In the "Required files section":
samtools fastq -T MM,ML {DORADO_BASECALLS.sam} > {READS.fastq}
minimap2 -ax map-ont -y {ASSEMBLY.fasta} {READS.fastq} |
        samtools view -bS |
        samtools sort > {ALIGNMENT.bam}
        samtools index {ALIGNMENT.bam}    <----- What is the point of the bam.bai file?
modkit pileup {ALIGNMENT.bam} --only-tabs

Also as of modkit pileup 0.2.3 the command line is:
Usage: modkit pileup [OPTIONS] <IN_BAM> <OUT_BED>
change to:
modkit pileup --only-tabs {ALIGNMENT.bam} {OUT.bed}
Finally, there needs to be line breaks - the final will look like:
samtools fastq -T MM,ML {DORADO_BASECALLS.sam} > {READS.fastq}
minimap2 -ax map-ont -y {ASSEMBLY.fasta} {READS.fastq} | \ <---
        samtools view -bS | \ <--
        samtools sort > {ALIGNMENT.bam}
samtools index {ALIGNMENT.bam}    <----- What is the point of the bam.bai file?
modkit pileup --only-tabs {ALIGNMENT.bam} {OUT.bed}

I hope that is correct.

That is correct, i have changed to the required files section. Thank you


Duplicate ID in PUT url when editing model in ui-router custom path

Hi, currently I'm setting up restangular to edit my model. Formerly it was ok when I'm not using custom url to show the form, like http://mysite.com/admin/clients (I just load the form there).
But, then I use ui-router to setup the URL like this http://mysite.com/admin/clients/{clientId}/edit.
When I do model.put() or model.save(), the resulting PUT url is wrong. It become like this
PUT http://referral.dev/admin/clients/f2aa1490-5a8f-11e5-b936-b3a54e0c9925/f2aa1490-5a8f-11e5-b936-b3a54e0c9925
It should be like this:
PUT http://referral.dev/admin/clients/f2aa1490-5a8f-11e5-b936-b3a54e0c9925
What I do wrong?
For convenience, here is debug from restangular object:
{
    "restangularCollection": false,
    "$object": {
        "id": "f2aa1490-5a8f-11e5-b936-b3a54e0c9925",
        "referrer_id": "f1529d80-5a8f-11e5-8ca1-21124a5fa590",
        "email"<EMAIL_ADDRESS>        "first_name": "Isabelle",
        "last_name": "Herzog",
        "mobile": "450.357.8579x621",
        "stage": "member",
        "stage_at": "2015-09-09 00:00:00",
        "previous_stages": [
            {
                "stage": "lead",
                "stage_at": "2015-08-30 00:00:00"
            }
        ],
        "total_investment": 190000,
        "email_token": "caf34dddfe00f36b180d459236830934",
        "confirmed": false,
        "unsubscribed": false,
        "transfered_at": null,
        "previous_referrers": [],
        "created_at": "2015-09-14 03:23:20",
        "updated_at": "2015-09-14 03:23:20",
        "referrer": {
            "id": "f1529d80-5a8f-11e5-8ca1-21124a5fa590",
            "email"<EMAIL_ADDRESS>            "name": "Esperanza Aufderhar Sr.",
            "mobile": "658.549.0277",
            "role": "referrer",
            "organization_id": "f0577b10-5a8f-11e5-984f-f120de15e3c1",
            "last_login_at": "0000-00-00 00:00:00",
            "suspended": 0
        },
        "route": "f2aa1490-5a8f-11e5-b936-b3a54e0c9925",
        "reqParams": null,
        "restangularized": true,
        "fromServer": true,
        "parentResource": {
            "route": "admin/clients",
            "parentResource": null
        },
        "restangularCollection": false
    }
}

And, here is the restangular object if don't use custom URL (which working):
{
    "id": "f2aa1490-5a8f-11e5-b936-b3a54e0c9925",
    "referrer_id": "f1529d80-5a8f-11e5-8ca1-21124a5fa590",
    "email"<EMAIL_ADDRESS>    "first_name": "Isabelle",
    "last_name": "Herzog",
    "mobile": "450.357.8579x621",
    "stage": "member",
    "stage_at": "2015-09-09 00:00:00",
    "previous_stages": [
        {
            "stage": "lead",
            "stage_at": "2015-08-30 00:00:00"
        }
    ],
    "total_investment": 190000,
    "email_token": "caf34dddfe00f36b180d459236830934",
    "confirmed": false,
    "unsubscribed": false,
    "transfered_at": null,
    "previous_referrers": [],
    "created_at": "2015-09-14 03:23:20",
    "updated_at": "2015-09-14 03:23:20",
    "referrer": {
        "id": "f1529d80-5a8f-11e5-8ca1-21124a5fa590",
        "email"<EMAIL_ADDRESS>        "name": "Esperanza Aufderhar Sr.",
        "mobile": "658.549.0277",
        "role": "referrer",
        "organization_id": "f0577b10-5a8f-11e5-984f-f120de15e3c1",
        "last_login_at": "0000-00-00 00:00:00",
        "suspended": 0
    },
    "route": "admin/clients",
    "reqParams": null,
    "restangularized": true,
    "fromServer": true,
    "parentResource": null,
    "restangularCollection": false
}

Your library is fantastic. Thanks.. :)

Guys, I've solved it. Formerly I was using:
var models = Restangular.all('admin/clients');

Then use
models.one($stateParams.clientId).get()

to get the model.
The correct way, is use Restangular.one to load model, eg. like this:
vm.model = Restangular.one('admin/clients', $stateParams.clientId).get();


Close sqlite harder

Sometimes Tango hangs after finishing. This may or may not be a fix. I can't be sure, because I can't repro.

@h-vetinary confirms that this fixed a hang in his tests. It's unclear why it doesn't hang the same way when we run it.


Fixed for Guzzle PSR-7

It seems that Guzzle changed the API.
These changes got me the library working again.
I hope this is not an false positive..

Im sorry somehow my composer did install guzzle 6 instead of 4...


[Bug] BetterDiscord wont open during startup

Describe the Bug
I have Open Discord, Start Minimized and Minimized to Tray enabled in my Discord's Window Settings. Whenever I start my PC and all my startup applications such as Steam, Anti-Virus and everything else opens up except Discord. I wait up til 5 minutes long to even confirm it.
So what I did was tried fixing it by repairing it using the installer and reinstalled it but didn't fixed it. I decided to remove the plugins by renaming the plugins folder so there's no plugins running with it and restarted. It doesn't start still during startup.
Yes, I checked my Windows tray at my taskbar to see if Discord is just silently hiding and there's none. Everytime I manually open Discord it boots up the loading screen instead of just opening up so it really didn't started.

To Reproduce

Go to user settings
Click on Windows Settings which is in APP SETTINGS category
Enable Open Discord, Start Minimized, Minimized to Tray. All those 3 must be enabled.
Restart your PC
Discord with BetterDiscord installed shouldn't run during startup even without plugins.

Expected Behavior
I'm expecting my Discord with BetterDiscord installed to startup automatically like any other software that has that same features like Steam.

Screenshots
None

Discord Version
Stable 100054 (69f0658)
BetterDiscord 1.2.1
Host 1.0.9003
Windows 10 64-Bit (10.0.19043)

Additional Context
The startup function works before I installed BetterDiscord so after I have BetterDiscord installed, that's when it happened.

Check if Discord is disabled in the Startup tab in task manager.

Check if Discord is disabled in the Startup tab in task manager.

Discord is not listed there at all. The startup function works when I haven't installed BetterDiscord yet.

Press Win + R type %localappdata%\Discord right click to Update.exe --> Create shortcut then press Win + R again type shell:startup and paste the Update.exe shortcut in here.

Press Win + R type %localappdata%\Discord right click to Update.exe --> Create shortcut then press Win + R again type shell:startup and put the Update.exe shortcut in here.

I did what was written in the instructions and checked the Startup tabs in my Task Manager, to my surprised the app named "Update" got duplicated.

Check if Discord is disabled in the Startup tab in task manager.

Is the app named "Update" the Discord you're talking about? I would assume so.

The thing is it says Publisher as GitHub so at first glance it wasn't Discord. It's been Enabled with or without creating the shortcut if this is the Discord app to begin with.
I restarted my PC to test things out with the instructions of creating a shortcut of that Update.exe and no, it didn't solve the issue. All my startups apps are still working fine except Discord with BetterDiscord.
Does Discord with BetterDiscord startup works for you guys or I'm the only one experiencing this problem at the moment? I want someone to confirm the issue aside from me unless its just me.

Yes, the Update GitHub thing it's Discord, so when you launch Discord the autoupdater launcher itself.
Otherwise if it's doesn't work try uninstalling Discord and reinstalling.

Yes, the Update GitHub thing it's Discord, so when you launch Discord the autoupdater launch itself.
Otherwise if it's doesn't work try uninstalling Discord and reinstalling.

I did that and ran an issue along the way. BetterDiscord won't show up in the settings even when I have it installed many times as I've installed then uninstalled. This time I used repair and it finally got it installed.
Now I restarted my PC with the reinstalled Discord app along with BetterDiscord and it's finally working! Some faulty happened in Discord's side after installing BetterDiscord awhile ago. Thank you everyone for helping me resolve this issue.


Not working on CLion

After installation, the plugin does not work on the CLion IDE. It does not apply code highlighting to any .bb nor .bbappend files.

Maybe the same as in #2 ? I'm using the plugin in CLion, works but I had to configure a python interpreter first.

Closing, no update for a long time and I'm using the plugin in CLion nearly each day. Works.


delete 3d point fails

This issue was imported from CodePlex
keenedge wrote 2010-11-30 at 02:07

Open a 3d point featureset. call RemoveShapesAt() call Save();

_m contains 0 "M" values. and this will cause outof range exception at this line
PointShapeFile.SaveAs()
{
...
if (M != null) shpStream.WriteLe(M[shp]);
...}
Everywhere else it seems t otbe the rule that ifa file has z then it must have M also.
So this code in
FeatureSet.RemoveShapeAt() fails to copy and remove the M values
if (CoordinateType == CoordinateType.M)
{
double[] mShape = new double[sr.NumPoints];
Array.Copy(_m, sr.StartIndex, mShape, 0, sr.NumPoints);
m.AddRange(mShape);
}
is I adjust as follows it SEEMS to work
// remove the z values if necessary
if (CoordinateType == CoordinateType.Z)
{
double[] zShape = new double[sr.NumPoints];
Array.Copy(_z, sr.StartIndex, zShape, 0, sr.NumPoints);
z.AddRange(zShape);
            double[] mShape = new double[sr.NumPoints];
            Array.Copy(_m, sr.StartIndex, mShape, 0, sr.NumPoints);
            m.AddRange(mShape);
}
Who can verify that this is the correct fix?
            }

    foreach (int index in remaining)
        {
            if (index < 0 || index >= _shapeIndices.Count) continue;
            ShapeRange sr = _shapeIndices[index];
            double[] xyShape = new double[sr.NumPoints * 2];
            Array.Copy(_vertices, sr.StartIndex*2, xyShape, 0, sr.NumPoints * 2);
            vertex.AddRange(xyShape);

            // remove the m values if necessary
            if (CoordinateType == CoordinateType.M)
            {
                double[] mShape = new double[sr.NumPoints];
                Array.Copy(_m, sr.StartIndex, mShape, 0, sr.NumPoints);
                m.AddRange(mShape);
            }

            // remove the z values if necessary
            if (CoordinateType == CoordinateType.Z)
            {
                double[] zShape = new double[sr.NumPoints];
                Array.Copy(_z, sr.StartIndex, zShape, 0, sr.NumPoints);
                z.AddRange(zShape);
                double[] mShape = new double[sr.NumPoints];
                Array.Copy(_m, sr.StartIndex, mShape, 0, sr.NumPoints);
                m.AddRange(mShape);
            }
            sr.StartIndex = pointTotal;
            pointTotal += sr.NumPoints;
            ProgressMeter.Next();
        }
Shade1974 wrote 2010-11-30 at 05:33
Good catch keenedge. Yes, the smart way to check is testing the CoordinateType. The null check that is there now predates the coordinate test used in the other shapefile classes and may just be poorly tested since I personally don't have a lot of sample
data for M or Z lying around. In theory, if the coordinate type is correct, the array should be dimensioned and sized appropriately. This makes me realize, however, that the saving code is not very fault tolerant to people meddling with the size and dimensionality
of the arrays. A robust method would pad the XY, M and Z arrays if they are found to be shorter than the expected dimensionality when saving because the values are required in the output format, whether your array has the values or not. I will post that idea
as an issue, but at least in the meantime, your fix should solve the short term crisis ok.
Ted

Not actual.


Make outfile not writable by group/world

Currently based on top #16, because they touch the same code block.
Only packet-sd should need to write this file, and consumers should only
be able to read this file.

Wil be integrated into #16.


Add HTTP error handling

Moving work from databricks/databricks-sql-nodejs#102
PECO-554
Currently, our Node.js connector does not implement retry behaviors when our service is busy. This PR implements retry behaviors in certain cases, and adds clearer error messages for the purpose of helping our users debug (i.e. creating custom errors to reccomend they change their api token in the case of a 403).

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.94 :tada:

Comparison is base (d9c9ce5) 93.02% compared to head (5de2938) 93.96%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
+ Coverage   93.02%   93.96%   +0.94%
==========================================
  Files          52       52
  Lines         788      812      +24
  Branches      121      126       +5
==========================================
+ Hits          733      763      +30
+ Misses         24       20       -4
+ Partials       31       29       -2

Impacted Files
Coverage Δ

lib/globalConfig.ts
100.00% <ø> (ø)

lib/hive/Commands/BaseCommand.ts
100.00% <100.00%> (+46.15%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.87 :tada:

Comparison is base (832078d) 95.29% compared to head (ad7806d) 96.17%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
+ Coverage   95.29%   96.17%   +0.87%
==========================================
  Files          51       51
  Lines         787      811      +24
  Branches      126      131       +5
==========================================
+ Hits          750      780      +30
+ Misses         13        9       -4
+ Partials       24       22       -2

Impacted Files
Coverage Δ

lib/globalConfig.ts
100.00% <ø> (ø)

lib/hive/Commands/BaseCommand.ts
100.00% <100.00%> (+46.15%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Clean up some text

Some small changes:

update the display name for maxmind and currency plugins
remove the extra images (they're in the repos now)
other random tweaks

I reverted one change and added back images for the npm plugins.

I reverted one change and added back images for the npm plugins.


The (column) option

This PR propagates the changes in https://github.com/SpineEventEngine/core-java/pull/1184 to the gcloud storage.
The main change is that now there is no more ColumnTypeRegistry. The column value conversion is instead done with the help of ColumnMapping - a new structure which is similar to the old type registry but conforms with the new column API.
The Spine version advances to 1.1.12.

@armiol, @dmdashenkov PTAL.


Update dependencies.  Use iconv-lite instead.  Reduce number of Buffers.

Should fix https://github.com/dommmel/fill-pdf/issues/20.

This is a great PR, thanks for the migration to iconv-lite and consolidating those buffer objects! :beers:

My pleasure!  Could you please push a new release out to npm?  Thanks.

Done! https://www.npmjs.com/package/fill-pdf


New 0.13.0 release fails to deploy with terraform 0.13.0 because of dependency on terraform-null-label

Found a bug? Maybe our Slack Community can help.

Describe the Bug
The new release pinned to terraform 0.13.0 depends on terrafrom-null-label v0.16.0 which is not currently compatible with 0.13.0 because of version constraints.
Expected Behavior
Module should deploy as before on terraform 0.13.0 same as 0.12.x
Steps to Reproduce
Steps to reproduce the behavior:

Change tag to 0.13.0 for dependency
Change your terraform executable to 0.13.0
Run code
Observe the issue.

Screenshots
Downloading git::https://github.com/cloudposse/terraform-aws-s3-log-storage.git?ref=0.13.0 for log_storage...
- log_storage in .terraform/modules/log_storage
Downloading git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.16.0 for log_storage.default_label...
- log_storage.default_label in .terraform/modules/log_storage.default_label

Error: Unsupported Terraform Core version

  on .terraform/modules/log_storage.default_label/versions.tf line 2, in terraform:
   2:   required_version = "~> 0.12.0"

Module module.log_storage.module.default_label (from
git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.16.0)
does not support Terraform version 0.13.0. To proceed, either choose another
supported Terraform version or update this version constraint. Version
constraints are normally set for good reason, so updating the constraint may
lead to other errors or unexpected behavior.

Environment (please complete the following information):
Anything that will help us triage the bug will help. Here are some ideas:

OS: Linux ubuntu 18.04 LTS
Terraform: 0.13.0

Additional Context
N/A

Appologies I notice this is in hand.


instructions on setting up certificate-based HTTPS

See original at https://github.com/influxdb/influxdb.com/issues/284

Fixed in https://github.com/influxdata/docs.influxdata.com/pull/945/commits/73de499a26a483059abda150a9de25a6a7bc0369


Move to nebula.netflixoss 2.2.5

NetflixOSS » feign » feign-pull-requests #22 SUCCESS
This pull request looks good


[BUG] Instance managers keep terminating and getting created while uninstallation is in progress

Describe the bug
Instance managers keep terminating and getting created while uninstallation is in progress, that delays the uninstallation process.
To Reproduce
Steps to reproduce the behavior:

Deploy Loghorn-master 08/23/2021 on a K8s cluster.
Create few volumes, take 2 backups, create some volumes with backing image.
Deploy the uninstall job to uninstall the Longhorn
Observe the logs of Longhorn-uninstall, there are instance managers getting terminated and created multiple times.

Log
container (23).log
Environment:

Longhorn version: Longhorn-master 08/23/2021
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl

There is some code change, so we still need to backport to 1.1.3.


onRegionChange returns negative LongitudeDelta on Android.

version: 0.15.2
If I zoom out the map and pan the map around the globe, onRegionChange may return a region with negative longitudeDelta. On iOS using Google maps provider, longitudeDelta is always positive.
Here's a demonstration:
https://www.dropbox.com/s/j5b6vevvlq22yie/longitudeDelta.gif
[Edit]
Looks like the value is always correctValue - 360 when negative. So I can work around it by adding the 360 degrees when ever I get a negative value.

Did you know what causes this error?
I found a similar bug where onRegionChange return a region with a longitude >180, when dragging around the 180 longitude line.

This is most likely caused by these lines in RegionChangeEvent.java:
region.putDouble("latitudeDelta", bounds.northeast.latitude - bounds.southwest.latitude);
region.putDouble("longitudeDelta", bounds.northeast.longitude - bounds.southwest.longitude);

i can see a fix for this in
react-native-maps/lib/ios/AirGoogleMaps/AIRGoogleMap.m
makeGMSCameraPositionFromMap
if(bounds.northEast.longitude >= bounds.southWest.longitude) {
    //Standard case
    center = CLLocationCoordinate2DMake((bounds.southWest.latitude + bounds.northEast.latitude) / 2,
                                        (bounds.southWest.longitude + bounds.northEast.longitude) / 2);
    longitudeDelta = bounds.northEast.longitude - bounds.southWest.longitude;
  } else {
    //Region spans the international dateline
    center = CLLocationCoordinate2DMake((bounds.southWest.latitude + bounds.northEast.latitude) / 2,
                                        (bounds.southWest.longitude + bounds.northEast.longitude + 360) / 2);
    longitudeDelta = bounds.northEast.longitude + 360 - bounds.southWest.longitude;
  }

can we please have the same fix for Android?? @christopherdro @IjzerenHein

Have the same issue with react-native-maps 0.27.1
react-native 0.61.0

Have the same issue with react-native-maps 1.7.1
react-native 0.66.5


Sequence Level Semantics Aggregation for Video Object Detection

nice ideas; "conceptual coherence across time" and such


Provide a public method to get an Iterable of active pool objects

It would be very convenient to have a method to fetch the current list of active pool objects. This is accessible in most circumstances by checking the children of the pool's transform, but that's less efficient than having access to the internally tracked records.

We already track the alive objects in ObjectPool._aliveObjects so should be simple enough
Would the following signature seem good?
public IEnumerable GetAllAliveObjects();

Unsure on the best name

Not sure if Alive or Active is better, but yeah, that signature looks good.

Added the following for both ObjectPool and MasterObjectPooler in V1.0.1
public IEnumerable GetAllActiveObjects();


Is this solution compatible with On premise version 7

Is this solution compatible with On premise version 7

Afraid not: https://github.com/microsoft/EasyRepro#crm-versions


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1949 ms

Resolved: Bofrost.lu performance has improved in e5efc75.


Dependency "SignalR-Client-Swift" has no shared framework schemes for any of the platforms

Hi,
I tried to compile the latest versión with carthage using those commands
Carfile
github "moozzyk/SignalR-Client-Swift" "0.9.1"
carthage update
carthage update --use-xcframeworks
Both of them says 'Dependency "SignalR-Client-Swift" has no shared framework schemes for any of the platforms' and its not possible to compile it. Trace
*** Checking out SignalR-Client-Swift at "0.9.1"
*** xcodebuild output can be found in /var/folders/carthage-xcodebuild.WvWrRD.log
*** Skipped building SignalR-Client-Swift due to the error:
Dependency "SignalR-Client-Swift" has no shared framework schemes
If you believe this to be an error, please file an issue with the maintainers at https://github.com/moozzyk/SignalR-Client-Swift/issues/new

Those commands works well when using version 0.9.0.
Cartfile
github "moozzyk/SignalR-Client-Swift" "0.9.0"
Regards

Thanks for the report. I haven't tried Carthage in quite a long time so it is plausible it is not working.