suggested unit test for E6C calculation

[The example notebooks] seem to be perfect candidates for unit tests.  E6C is just one of them.
Originally posted by @mrakitin in https://github.com/bluesky/hklpy/pull/52#discussion_r540362501
Actually, any of the examples could become unit tests.

https://github.com/bluesky/bluesky/issues/1196
We also need integration tests.  This one caused a lot of problems and I had to write a bunch of HKL scans using list scan  that could only be scanned in the positive direction

Copied example from https://github.com/bluesky/bluesky/issues/1196#issuecomment-500994794.  Cell 6 of the notebook (RE(scan([hw.det, tardis],tardis.h, 0.0001, -0.025, tardis.k, 0.0001, 0.51, num=5))) fails with:
gi.repository.GLib.GError: hkl-mode-auto-error-quark: none of the functions were solved !!! (0)

for the second position of the scan:
Transient Scan ID: 2     Time: 2020-12-30 10:55:19
Persistent Unique Scan ID: '642b64f8-698d-4615-93fe-d22892a09721'
New stream: 'primary'
+-----------+------------+------------+------------+------------+--------------+--------------+--------------+------------+
|   seq_num |       time |   tardis_h |   tardis_k |   tardis_l | tardis_theta | tardis_delta | tardis_gamma |        det |
+-----------+------------+------------+------------+------------+--------------+--------------+--------------+------------+
|         1 | 10:55:19.4 |      0.000 |      0.000 |      0.000 |        0.791 |        0.425 |       -0.017 |      1.000 |
+-----------+------------+------------+------------+------------+--------------+--------------+--------------+------------+
generator scan ['642b64f8'] (scan num: 2)

Run aborted
Traceback (most recent call last):
  File "/home/prjemian/Documents/projects/Bluesky/hklpy/hkl/engine.py", line 231, in pseudo_positions
    geometry_list = self._engine.pseudo_axis_values_set(values,
gi.repository.GLib.GError: hkl-mode-auto-error-quark: none of the functions were solved !!! (0)

Created a standalone unit test file for the reports in https://github.com/bluesky/bluesky/issues/1196.  This test also fails but in different ways.
For one of the samples (the ESRF test sample), the unit tests can succeed if the tolerance is raised to unacceptable numbers (0.12 to match computed and experimentally-found reflection angles).
For the tardis2 example, computed hkl values could not be matched when the tolerance for matching computed and expected hkl was 0.05.  The first failure identified was:
hkl/tests/issue62/test_tardis_scan_rounding.py .F                                                                       [100%]

========================================================== FAILURES ===========================================================
________________________________________________________ test_tardis2 _________________________________________________________

tardis = SimTardis(prefix='', name='tardis', settle_time=0.0, timeout=None, egu='', limits=(0, 0), source='computed', read_attr..., 'theta', 'mu', 'chi', 'phi', 'delta', 'gamma'], configuration_attrs=['energy', 'UB', 'h', 'k', 'l'], concurrent=True)
constrain = None

    def test_tardis2(tardis, constrain):
        tardis.calc['gamma'].limits = (-2.81, 183.1)
        sample2(tardis)
        tolerance = 0.01    # goal

        assert round(tardis.calc.energy, 5) == 0.573

        # simulate the scan, computing hkl from angles
        # RE(scan([hw.det, tardis],tardis.theta, 0, 0.3, tardis.delta,0,0.5, num=5))
        # values as reported from LiveTable
        data = """
        +-----------+------------+--------------+--------------+------------+------------+------------+------------+--------------+
        |   seq_num |       time | tardis_theta | tardis_delta |        det |   tardis_h |   tardis_k |   tardis_l | tardis_gamma |
        +-----------+------------+--------------+--------------+------------+------------+------------+------------+--------------+
        |         1 | 15:09:51.6 |        0.000 |        0.000 |      1.000 |      0.000 |      0.000 |      0.000 |        0.000 |
        |         2 | 15:09:52.2 |        0.075 |        0.125 |      1.000 |     -0.006 |      0.013 |      0.000 |        0.000 |
        |         3 | 15:09:52.8 |        0.150 |        0.250 |      1.000 |     -0.013 |      0.026 |      0.000 |        0.000 |
        |         4 | 15:09:53.5 |        0.225 |        0.375 |      1.000 |     -0.019 |      0.038 |      0.000 |        0.000 |
        |         5 | 15:09:54.0 |        0.300 |        0.500 |      1.000 |     -0.025 |      0.051 |      0.000 |        0.000 |
        +-----------+------------+--------------+--------------+------------+------------+------------+------------+--------------+
        """
        a = interpret_LiveTable(data)
        tolerance = 0.05    # empirical
        mu, chi, phi = 0, 0, 0  # constant
        targets = list(zip(a["theta"], a["delta"], a["gamma"], a["h"], a["k"], a["l"]))
        for theta, delta, gamma, h, k, l in targets:
            ref = tardis.inverse(theta, mu, chi, phi, delta, gamma)
            assert abs(ref.h - h) <= tolerance
            assert abs(ref.k - k) <= tolerance
            assert abs(ref.l - l) <= tolerance

        # check values reported by tardis.read()
        h = .1
        k = .51
        l = 0
        theta = 41.996
        delta = 6.410
        gamma = 0
        ref = tardis.inverse(theta, mu, chi, phi, delta, gamma)
>       assert abs(ref.h - h) <= tolerance
E       assert 0.08923309431011328 <= 0.05
E        +  where 0.08923309431011328 = abs((0.010766905689886724 - 0.1))
E        +    where 0.010766905689886724 = SimTardisPseudoPos(h=0.010766905689886724, k=0.011716667748269113, l=0.014207699416448642).h

hkl/tests/issue62/test_tardis_scan_rounding.py:268: AssertionError
=================================================== short test summary info ===================================================
FAILED hkl/tests/issue62/test_tardis_scan_rounding.py::test_tardis2 - assert 0.08923309431011328 <= 0.05
================================================= 1 failed, 1 passed in 2.15s =================================================

Obviously, these tests will need some help.  @ambarb: Can we videoconference about this?

@awalter-bnl : Can you attend a videocall with @ambarb to help make this test successful?


Velodyne vlp 32c driver

Hey,
Does Apollo velodyne driver works with vlp 32c?
I saw HDL32e but maybe I missed it..
Thanks!

vlp-32c has the same data structure with hdl-32e. But the angular spacing between two "lines" are different. So you can use it with different calibration files.

we will adapt the vlp-32c in branch v3.1-dev.

please check the branch master and v3.1dev to use driver. thank you.


(BOLT-1085) Document how nil values are passed to tasks

This commit documents how parameter that have the value nil are passed to tasks that use the environment input-method as the string value null.

CLA signed by all contributors.


Jean review triggered emails

#70 implemented adoption emails. Need to check with Jean about the content.

Jean reviewed these and submitted updates. Links to the updated content are in their related issue. Closing this.

hi all, i'm reopening this because i made slight changes to the 1st email copy to accentuate the safety message, per staff request. sorry and thank you!!!

Updated in #131 -- note that we are still using plain text e-mails so things like bold text, italics, and links aren't possible.
Let me know if it looks good!

@jeanwalshie The copy looked right to me...look okay to you?

@jasonlally @jszwedko looks great! sorry, i forgot about the HTML/plain text issue. thanks!!

No worries! We can definitely switch to HTML e-mails since it seems like there is more of a need for them now (additional styling), but let's get these changes in first.

Closed via #131
Opened #138 to switch to HTML e-mails.


Allow to name a connection

Hi @dominikzogg,
You can name a connection using the title column in the connection grid, does that work for you?

@rafaelthca perfect, did not get it


Add Android TTS demo

You can download pre-built APKs at
https://github.com/k2-fsa/sherpa-onnx/releases/tag/v1.8.4

Output of APK analyzer

Here are some screenshots


pygame.org got a HTTP 500 Internal Server Error

When I go to pygame.org, I got a HTTP 500 error message and the website is not working right now

Me too, and I can't install pygame by Powershell

Still down

Getting a server error like 500 implies the dns and nameservers are all working. I don't have info on what the http server itself is running three days.

Still down, but somehow the documentation page still works. https://www.pygame.org/docs/

Also it can load favicon.ico on the error page, but it seems like it's not the same one as the one on the docs page.


Cascade Selection Up+Down, Cascade Deselection Down

I can't figure out how to do this... I'm hoping you have an easy solution.
For List:
a
a.1
a.1.A
a.1.B
a.2
a.2.A
a.2.B
If nothing is selected, and "a.2" is selected, I want "a" and all children of "a.2" to be selected (a.2.A & a.2.B).
And, if everything is selected, and "a.2" is deselected, I want "a" to remain selected, but all children below "a.2" to be deselected.
I have figured out the override of "Deselection Down" and the "Selection Up".
http://jsfiddle.net/DGAF4/41/
But, I cannot figure out how to do the "Selection Down" -- when I try to cascade the downward selection, the "select" event fires when the parent node is "selected" during the cascade up, which then causes the recursive downward selection event to cause everything in the tree to become selected..
http://jsfiddle.net/DGAF4/42/
It would be great if the cascade functionality had this built in.  I tried to implement with temporary global variables to prevent the cascade down when in the middle of doing an up, but I couldn't get it to work.
Thanks for any assistance!

"Conditional cascade" has a very common use for dependency cases where the tree can't have the last node checked if any of the parents aren't checked.
Thanks savage, this should really be in the core.

I have seen this example  (http://jsfiddle.net/DGAF4/41/) and it ia working as expected but is it possible to select first child node automatically if I select parent node and if already any child node selected then not more than 1 child node can be select?

@krunal1089 you probably need conditionalselect ... and a lot of custom logic, but it should be doable.

Hi vakata, Thanks for reply but I am not able to write custom logic as expected can you please help me with this? and child node should select automatically only fir level 4.
I have also created this example http://jsfiddle.net/56dwbe83/24/ can some one try and provide update on this.

Sorry, I can not help you with the custom logic - this is not something I provide as a part of supporting and maintaining jstree. You might have better luck on stackoverflow.


Thank you

sometimes it tap bombs too

@BornaPexor
change this line in code to:
if (b in range(20, 125)) and (r in range(102, 220)) and (g in range(200, 255)):

@BornaPexor change this line in code to:
if (b in range(20, 125)) and (r in range(102, 220)) and (g in range(200, 255)):

This still tap bombs sometimes


Added a 'strings-no-comments-swift3' template. Fixes #222

@AliSoftware As per your comment in https://github.com/AliSoftware/SwiftGen/issues/222#issuecomment-264660153
Fixes #222.

Added a strings-no-comments-swift3 template
Added a unit test to make sure the output is correct. Output was generated this way:

./build/swiftgen/bin/swiftgen strings -t no-comments-swift3 ./UnitTests/fixtures/Localizable.strings -o ./UnitTests/expected/Strings-File-NoComments-Swift3.swift.out

Added a line in the CHANGELOG

Let me know if you have any question. thanks!

Looks good to me from GitHub at least.
Will merge after the 4.0.1 patch release (as adding a new template is a minor version, not just patch, and will go in upcoming 4.1.0)

@AliSoftware Done.


added pause and resume button

issue #275 solved

working video
https://github.com/user-attachments/assets/113b7786-18bb-4a97-859f-e202386ac795

Bug fix (non-breaking change which fixes an issue)
Chore (refactoring code, technical debt, workflow improvements)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How should this be tested?

[x] Test A
[x] Test B

Mandatory Tasks

[x] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@YadavAkhileshh  level2 is needed not level 3 @MastanSayyad


Fix failed test: test mixed gossipsub

This test sometimes failed, even in local environment:
1) go-libp2p-pubsub gossipsub tests
       test mixed gossipsub:
     Error: Message never received, sender 6, receiver 19, index 67
      at Timeout._onTimeout (file:///Users/tuyennguyen/Projects/workshop/gossipsub-js/dist/test/e2e/go-gossipsub.spec.js:28:16)
      at listOnTimeout (node:internal/timers:557:17)
      at processTimers (node:internal/timers:500:7)

Command failed with exit code 1: mocha ./dist/test/e2e/*.spec.js --ui bdd --require source-map-support/register --timeout=60000

Tests are running currently without issue, closing.


delete user button

Should require two authentication steps and a password check.

Requires a backend endpoint that deletes all user data from the backend database as well as a frontend request to Firebase to delete the user account.


Catch 503 response exception

AllDebrid streaming issues:
It seems that the download_link HEAD request to Alldebrid returns a 503 but the aiohttp client raises an exception which is not caught and therefore execution is stopped.
Fix:
Catch the 503 response exception and continue to set the proxy on the retry attempt.
Also disables the timeout on the httpx client for the stream request. Unsure if this is needed.

is this fixing AllDebrid streaming issues?

Yes that was the goal here.

thanks!


The source code button is missing an alternative text that describes its function

The source code button needs an alternative text that describes its function. The visual impaired user has no indication what happens after pushing this button.
Tested on:
https://q-src.github.io/styleguide/en/
(Chrome - Version 58.0.3029.110)
Tested with:
NVDA screen reader

The source code button "<>" everywhere in the guideline

It will not affect anything if I add an alternativ text to the source code button. The alt attribute can only be used with  (Source: https://www.w3schools.com/TAGS/att_input_alt.asp). Please let me know if you still want this.


Example broken

Hello! i use this example when i'm giving technical interviews (the interview is to replicate the tagger component) but it appears that over the weekend an update broke the example!

@Falieson react, react-dnd,react-dnd-html5-backend,react-dom should be installed as peer dependencies. please make sure you install it.

@ad1992  Sorry I wasn't clear enough, when you go to http://prakhar1989.github.io/react-tags/example/ its broken

@Falieson pls refer to https://stackblitz.com/edit/react-tag-input-1nelrc for the example. Will be updating the link in the description as well

Sweet - thanks


Need to update pathfinder_simd

Due to the recent nightly stdsimd changes the dependency pathfinder_simd needs an update in order to build font-kit
https://github.com/servo/pathfinder/issues/550

Note https://github.com/servo/font-kit/pull/227 did some prep for a new release

pathfinder_simd has now been updated.

Not yet


Map view to set current user location

I took the liberty to add a "Map" tab with a map view
By long-pressing the map view, you set a pin.
By pressing a button, the position in the map is set to current user location in simulator.
Let me know what you think, the wrapper around MKMapView is maybe a little bit naive.

That looks beautiful – great job! Is this functionality documented by Apple somewhere? If not, it might be unsupported, which could be problematic in the future.

Apparently not - I found it here https://github.com/lyft/set-simulator-location.
Folks at Lyft did the reverse-engineering and found out that locations are handled through notification center.
Might be worth pointing out that being this a notification from MacOS NotificationCenter, the worst that could happen might be that nothing could happen when the button is pressed, but this shouldn't lead to crashes. My 2 cents: I'd rather have the feature today and take the risk of seeing it broken someday rather than not having it at all. The critical piece of code is less than 10 lines of simple code.

Yes, I agree: if we find that the feature disappears in the future we can just remove it. Thank you!

I've filed FB7577924 asking for a supported equivalent of this. Until then, let's go for it!


Kotlin for the tests

Summary:
It has been our stated strategy that we would evaluate the use of Kotlin by using it for writing tests.
Expected behavior:
New tests would be written using Kotlin & our test coverage would increase.
Observed behavior:

Very few tests seem to be created, whether Java or Kotlin.
We leave the CI build broken & don't actively maintain the test suite
We don't utilize Kotlin (cf #1371 - we could have / should have offered guidance that New tests be in Kotlin for the Nearby failures)

Device and Android version:
One thing to note: we should update the version of Kotlin in our project.
Would you like to work on the issue?
I would be happy to Port the entire suite to Kotlin, for sure, including new ones for Nearby but this needs to be a team buy-in ... are we "in" or "out" of the Kotlin game?  If we are "in" is there a more effective strategy for adoption?

I ran across a blog post from Brian Button (an Agile leader based in the St Louis area) that talks about "Three Degrees of Unit Testing" that might be interesting to the current topic.
https://blog.brianbutton.io/index.php/2018/04/01/the-degrees-of-unit-testing/
I love his comment

Any amount of unit testing, done almost any way, adds value to a code base. Regardless of how and when the tests were written, as long as they are reliable and quick, and prove the code does what we think it does, I’m in favor of them.

Tests are now done in Kotlin, builds are passing, coverage is increasing and there's a nice CI pipeline to alpha releases. Thanks to everyone who helped with this :tada: :champagne:


Calculating PSI of single-cell data with outrigger

Hi,
Can I calculate the psi of all single-cells at the same time? Or I can only calculate them one-by-one? Even when I build index.

Hi,
I am interested in this as well. I have generated some Smart-seq3 data and want to calculate the psi in individual cells. Can we use outrigger to calculate psi for all the cells simultaneously? Thanks!

Hi, Have you solved this?


Examples code(mnist.py) doesn't work

This Link is result of executing the examples_code(mnist.py).
GoogleColabratory - deep-learning-from-scratch-3/try_mnist.py
Enviroment info
import dezero
dezero.__version__
# '0.0.11'

import sys
sys.version
# '3.6.8 (default, Oct  7 2019, 12:59:55) \n[GCC 8.3.0]'

Execute Command
python examples/mnist.py

Output
Downloading MNIST(train-images-idx3-ubyte.gz) ... Done
Downloading MNIST(train-labels-idx1-ubyte.gz) ... Done
Downloading MNIST(t10k-images-idx3-ubyte.gz) ... Done
Downloading MNIST(t10k-labels-idx1-ubyte.gz) ... Done
Traceback (most recent call last):
  File "examples/mnist.py", line 40, in <module>
    acc = F.accuracy(y, t)
  File "/usr/local/lib/python3.6/dist-packages/dezero-0.0.11-py3.6.egg/dezero/functions.py", line 367, in accuracy
    return Variable(acc)
  File "/usr/local/lib/python3.6/dist-packages/dezero-0.0.11-py3.6.egg/dezero/core.py", line 49, in __init__
    raise TypeError('{} is not supported'.format(type(data)))
TypeError: <class 'numpy.float64'> is not supported

Thanks!
I just fixed the problem.
7ce3f51e0addc24ed606c97e38f0e6e56e8b2f4c


Support newer Terraform versions

Newer terraform syntax - with tf > 0.11 - is not supported.


Tasks metrics are all zero

Howdy,
This library does exactly what we need, but we're getting all zeros for our metrics. The metrics and tasks are listed correctly, so we know the redis connection is working, but the values are just 0 and never change.
We're working on this problem, but if anyone has dealt with this I'd love some help.
We're running Celery 3.x, Python 2.7, and the master branch of celery-prometheus-exporter.

Could it be that you have events disabled in your Celery setup? The readme has some hints about that 🙂
Other than that, how are you using the exporter? Inside Docker, the release version, …? 🙂

On 21.12.2017, at 20:15, Emerson Lackey<EMAIL_ADDRESS>wrote:
Howdy,
This library does exactly what we need, but we're getting all zeros for our metrics. The metrics and tasks are listed correctly, so we know the redis connection is working, but the values are just 0 and never change.
We're working on this problem, but if anyone has dealt with this I'd love some help.
We're running Celery 3.x on Python 2.7.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


FlutterMapboxNavigationPlugin.kt

Hello!
Sorry, but my Engish is bad.
Thanks for share your work. I have problems when I try to compile my project after install your package. I get the next errors from Visual Studio Code.
e: C:\flutter\flutter.pub-cache\hosted\pub.dartlang.org\flutter_mapbox_navigation-0.0.5\android\src\main\kotlin\com\dormmom\flutter_mapbox_navigation\FlutterMapboxNavigationPlugin.kt: (168, 47): Type mismatch: inferred type is (???, ???) -> Unit? but ProgressChangeListener! was expected
e: C:\flutter\flutter.pub-cache\hosted\pub.dartlang.org\flutter_mapbox_navigation-0.0.5\android\src\main\kotlin\com\dormmom\flutter_mapbox_navigation\FlutterMapboxNavigationPlugin.kt: (168, 49): Cannot infer a type for this parameter. Please specify it explicitly.
e: C:\flutter\flutter.pub-cache\hosted\pub.dartlang.org\flutter_mapbox_navigation-0.0.5\android\src\main\kotlin\com\dormmom\flutter_mapbox_navigation\FlutterMapboxNavigationPlugin.kt: (168, 59): Cannot infer a type for this parameter. Please specify it explicitly.Gradle task assembleDebug failed with exit code 1
Exited (sigterm)
Thanks a lot!

try v0.0.6

Good afternoon eopeter!!
Thanks for your fast answer. I have good news, package v 0.0.5 works.
Just delete the folder:  C:\flutter\flutter.pub-cache\hosted\pub.dartlang.org\flutter_mapbox_navigation-0.0.5\
And download the package again and it worked!!!
Thank you very much for your time and help.
Regards!!

Why is this package so big????? @eopeter

Seems like some Kotlin dependencies were added to the Lib folder by Android
Studio. Will Clean up!!
[image: image.png]
On Tue, Nov 26, 2019 at 10:04 PM speedy-coder<EMAIL_ADDRESS>wrote:

Why is this package so big????? @eopeter https://github.com/eopeter
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/eopeter/flutter_mapbox_navigation/issues/6?email_source=notifications&email_token=AAPLOMZRFDGOC73MJF2EN6TQVXPURA5CNFSM4JPZ772KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFIEAWI#issuecomment-558907481,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPLOM4U35FQ2GKOEX267Z3QVXPURANCNFSM4JPZ772A
.

--
Sincerely
Emmanuel Oche

@speedy-coder -- I Removed extraneous jars in lib folder in version 0.0.7. Update your package and let me know if the size is still massive on your end.

Hi @eopeter  it didn't work, the last time I checked my apk was 35.1 MB this time after updating the package it became 46.1 MB. Before and After updating the dependency :
Before:(APK)
✓ Built build/app/outputs/apk/release/app-release.apk (35.1MB)
After:(APK)
✓ Built build/app/outputs/apk/release/app-release.apk (46.1MB)
Before:App Bundle
✓ Built build/app/outputs/bundle/release/app-release.aab (38.5MB).
After:App Bundle
✓ Built build/app/outputs/bundle/release/app-release.aab (46.5MB).

How big is the example apk? Are you sure it is not your code? I am not able to verify the sizes at the moment.

On Dec 2, 2019, at 2:29 PM, speedy-coder<EMAIL_ADDRESS>wrote:

Hi @eopeter it didn't work, the last time I checked my apk was 35.1 MB this time after updating the package it became 46.1 MB. Before and After updating the dependency :
Before:(APK)
✓ Built build/app/outputs/apk/release/app-release.apk (35.1MB)
After:(APK)
✓ Built build/app/outputs/apk/release/app-release.apk (46.1MB)
Before:App Bundle
✓ Built build/app/outputs/bundle/release/app-release.aab (38.5MB).
After:App Bundle
✓ Built build/app/outputs/bundle/release/app-release.aab (46.5MB).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Tabs do not wrap

"Open Pipeline, select any number of scripts and run it. After 8 jobs(tabs) the screen shifts to the left hand side. It becomes difficult to open a new job and select any textboxes and buttons. We need to always scroll it to operate any new job."
cc @rachanasinghg

Ok that's definitely a bug! Let's try this instead.


needed_libraries: Fix cannot find section .dynamic

Fix the error cannot find section .dynamic when needed_libraries is
called on a static ELF executable.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

This successful PR had 71.2% coverage reported.
This failed PR has 64.1% coverage reported.
@MikeMcQuaid What's the target coverage? Any idea why the coverage changed so drastically?
https://coveralls.io/builds/20871988

@sjackman https://github.com/Homebrew/brew/pull/5471/checks?check_run_id=45678909 is reporting the correct coverage so it could be a Coveralls parallelisation/timing issue.

I've rebased onto master to trigger a rebuild.

Reported coverage is now 64.5% from Coveralls: https://coveralls.io/builds/20873278
and 71.2% from GitHub Checks: https://github.com/Homebrew/brew/pull/5471/checks

Alternatively perhaps we could fiddle with Parallel Rspec or Simplecov so that's only send on the final build completion?

I'm afraid that's Ελληνικά to me. =p

I've triggered one more rebuild, just for curiosity.

@sjackman 💚 now.


Update for new version of Dart

Since November 2021, a language server is built-in into the dart executable. So no extra downloads necessary. One drawback is that LSP starts very slow.

This change is

Thanks, tried this out and it works


Magic Storage Recipe Recursion incompatability.

On setting the Recipe Recursion set above 0, and loading into a world, then opening Crafting interface, taking a station out, or putting one back in, it prompts two of the Blood Gold Mold from Calamity Overhaul to spawn in my inventory as if I have crafted it.

Can confirm. Doing almost anything with the interface will spawn two of these items each time.

I don't quite understand, but what is the function of Recipe Recursion Depth?

I did find that issues arise when Recipe Recursion Depth is greater than 0 or set to -1. Therefore, the most straightforward solution is to lock this setting to 0, meaning it won't be enabled.

Recipe recursion depth sets how far Magic Storage will search for craftable components in the creation of the desired item. Rather than needing to round up every component one at a time, it will try to make things for you. Saves an awful lot of time and effort.

Recipe recursion depth sets how far Magic Storage will search for craftable components in the creation of the desired item. Rather than needing to round up every component one at a time, it will try to make things for you. Saves an awful lot of time and effort.

My reply to this message might be a bit late. The issue https://github.com/blushiemagic/MagicStorage/issues/275 is a problem with Magic Storage itself, but it has been fixed. However, the new version has not been released yet

My reply to this message might be a bit late. The issue https://github.com/blushiemagic/MagicStorage/issues/275 is a problem with Magic Storage itself, but it has been fixed. However, the new version has not been released yet


Remove unnecessary console.log statement

See issue #57

@treetrum new release is up and published to NPM!


⚠ Bofrost.at has degraded performance

In 3837f59, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1381 ms

Resolved: Bofrost.at performance has improved in e80aaf3 after 6 minutes.


body issue for mounted apps - SyntaxError: Unexpected end of JSON input

What version of Elysia.JS is running?
0.8.15
What platform is your computer?
Darwin 23.2.0 arm64 arm
What steps can reproduce the bug?
import { Elysia } from "elysia";

const app2 = new Elysia().post("/", ({ body }) => body);

const app1 = new Elysia()
  .mount("/app2", app2)
  .post("/", ({ body }) => body)
  .listen(3000);

console.log(
  `🦊 Elysia is running at ${app1.server?.hostname}:${app1.server?.port}`
);

bun run the app and post to /app2 with some body. here's a quick video
https://github.com/elysiajs/elysia/assets/719763/efb4a562-24f5-41d2-92bf-ad453476293d
What is the expected behavior?
to get the body json parsed correctly
What do you see instead?
SyntaxError: Unexpected end of JSON input
Additional information
this only happens when mounting a secondary app
repo with reproduction
https://github.com/stewones/elysia-body-issue/blob/main/src/index.ts

use is a workaround
const app2 = new Elysia({ prefix: '/app2' }).post('/', ({ body }) => body)
const app1 = new Elysia()
	.use(app2)
	.post('/', ({ body }) => body)
	.listen(3000)

This issue seems to be related to scoped being true. Here's a simple example:
import { edenTreaty } from "@elysiajs/eden"
import { Elysia, t } from "elysia"

const a = new Elysia({ prefix: "/a", scoped: true }).post(
  "/b",
  ({ body }) => body.name,
  {
    body: t.Object({
      name: t.String(),
    }),
  },
)

const app = new Elysia().use(a).listen(3000)

const api = edenTreaty<typeof app>("http://localhost:3000")

const { data, error } = await api.a.b.post({ name: "Elysia" })
console.log("data:", data)
console.log("error:", error)

Running the above code results in:
1 | var c=(t,e,r)=>{if(t.endsWith("/")||(t+="/"),e==="index"&&(e=""),!r||!Object.keys(r).length)return`${t}${e}`;let s="";for(let[i,u]of Object.entries(r))s+=`${i}=${u}&`;return`${t}${e}?${s.slice(0,-1)}`},o=t=>t.trim().length!==0&&!Number.isNaN(Number(t));var n=class extends Error{constructor(r,s){super(s+"");this.status=r;this.value=s}};export{c as a,o as b,n as c};
                                                                                ^
error: {"name":"SyntaxError","message":"Unexpected end of JSON input"}
      at new n (.../node_modules/@elysiajs/eden/dist/chunk-HUZ6GXDC.mjs:1:297)
      at .../node_modules/@elysiajs/eden/dist/chunk-Q3DJMDQ2.mjs:1:2695

But after changing scoped: true to scoped: false, running the above code results in:
data: Elysia
error: null

I am not using the scoped: true at all and still getting the error

@elliotwaite I have the same issue with scoped: true not parsing the JSON correctly. ~I did not spend 2 hours debugging the JSON parse issue.~

this is the line  responsible for this bug. Constructing new Request object discards body stream.

Fixed with ba09081, going to published under 1.1.11.


RFE: support nested allocatable derived-type array components inside derived-type coarrays

The transcript below demonstrates the runtime error message that the OpenCoarrays function libcaf_mpi::caf_get_by_ref() produces in a simple scenario wherein a derived-type coarray has an allocatable, derived-type array component.   A cumbersome workaround is to flatten such coarray data structures so that the components of derived type coarrays are intrinsic types.  Even with this workaround, a source allocation will fail.  A workaround for the lack of source allocation is to use mold allocation followed by element-by-element copying of the component array.
The following are the results of compiling the code below with the gfortran versions cited in bold:

8.3.0: the final allocation generates the runtime error message below.
9.2.0: the allocation completes without error but the subsequent component access generates a segmentation fault - invalid memory reference.
10.0.0: same as 8.3.0.

$ cat opencoarrays-missing-feature.f90

  implicit none

  type package
    integer, allocatable :: surface_fluxes(:)
  end type

  type outbox
    type(package), allocatable :: block_surfaces(:)
  end type

  type(outbox), save :: halo[*]

  integer, parameter :: source_image = 1
  integer, parameter :: message(1)=[99]

  if (this_image() == source_image) then
    allocate(halo%block_surfaces(1))
    allocate(halo%block_surfaces(1)%surface_fluxes, source = message)
  end if

  sync all

  if (this_image() /= source_image) then
    allocate(halo%block_surfaces(1))
    allocate(halo%block_surfaces(1)%surface_fluxes, mold = halo[source_image]%block_surfaces(1)%surface_fluxes)
    halo%block_surfaces(1)%surface_fluxes(1) = halo[source_image]%block_surfaces(1)%surface_fluxes(1)
  end if

end

$ caf opencoarrays-missing-feature.f90

$ cafrun -n 2 ./a.out
Fortran runtime error on image 2: libcaf_mpi::caf_get_by_ref(): two or more array part references are not supported.

Error: Command:
   `/usr/local/Cellar/mpich-3.2-gnu-8.3.0/bin/mpiexec -n 2 ./a.out`
failed to run.

Adding a link to the 76-line example that motivated this issue. It seg faults without producing the useful error message above, but captures more detail from the relevant class hierarchy in the ultimate application (Morfeus).

@neok-m4700 I'm adding the "help wanted" label to this issue.  Thanks for your great contribution to fixing issue #511.  The example in this issue is one step closer to what we want to do and the link to the 76-line example  in my earlier comment is even closer to what the full application does.  If you have any interest in looking this, let us know.  As it stands, any OpenCoarrays contributor is at least several weeks away from working on it so any community contributions are welcome.


How to return Option value from pyo3 getter method?

Yesterday I've spent a considerable ammount of time trying to figure out how to achieve this one, https://stackoverflow.com/questions/55192023/how-to-return-optionstring-value-from-pyo3-getter-method, do you have any idea about it?
Thx!

use pyo3::prelude::*;
use syntect::highlighting::Theme as _Theme;

#[pyclass]
pub struct Theme {
    wrap: _Theme,
}

#[pymethods]
impl Theme {
    #[getter]
    fn name(&self) -> PyResult<Option<String>> {
        Ok(self.wrap.name.clone())
    }
}

#[pymodule]
fn mcve(_py: Python, m: &PyModule) -> PyResult<()> {
    m.add_class::<Theme>()?;
    Ok(())
}

Thanks a lot, it works!

:+1:
(Yeah I know we have only poor documents, but I (and maybe others too) don't have enough time now, sorry

@kngwyu No worries, just to let you know this project is totally awesome!!! Main problem on my side is mainly I've been using rust for 1 week so I'm pretty much a total newcomer and I've got stuck many times.
I'll be posting questions here once in a while, I assume you're one of the main core-devs, right?
Anyway, once again, tyvm! Your help was unvaluable, yesterday I'd wasted quite of time trying to figure out this issue :)

This is the issue tracker for pyo3, please do not use it as general support forum. You can get help on gitter and on stackoverflow. Also please don't delete questions but self-answer them so others can profit from the solution as well.


direct mode: volumes not removed completely

When creating and deleting a volume in direct mode, it's namespace remains in "inactive" state. In that state it is found by  pmemNdctl.getDevice from pkg/pmem-device-manager/pmd-ndctl.go, which prevents creating another volume with that name.
I'm not sure what to expect here: should the namespace be gone completely? Or should it be ignored and overwritten when the name is used again?
To reproduce:

run the pkg/pmem-device-manager test from https://github.com/intel/pmem-csi/pull/420 multiple times on the same host in direct mode
or
kubectl create -f deploy/common/pmem-pvc.yaml
kubectl delete -f deploy/common/pmem-pvc.yaml
for i in 1 2 2; do echo $i; _work/pmem-govm/ssh.$i sudo ndctl list --idle; done

->
1
[
  {
    "dev":"namespace0.0",
    "mode":"raw",
    "size":0,
    "uuid":"00000000-0000-0000-0000-000000000000",
    "sector_size":512,
    "state":"disabled"
  }
]
2
[
  {
    "dev":"namespace0.1",
    "mode":"raw",
    "size":0,
    "uuid":"00000000-0000-0000-0000-000000000000",
    "sector_size":512,
    "state":"disabled"
  },
  {
    "dev":"namespace0.0",
    "mode":"raw",
    "size":0,
    "uuid":"00000000-0000-0000-0000-000000000000",
    "sector_size":512,
    "state":"disabled",
    "name":"test-dev"
  }
]
3
[
  {
    "dev":"namespace0.2",
    "mode":"raw",
    "size":0,
    "uuid":"00000000-0000-0000-0000-000000000000",
    "sector_size":512,
    "state":"disabled"
  },
  {
    "dev":"namespace0.0",
    "mode":"raw",
    "size":0,
    "uuid":"00000000-0000-0000-0000-000000000000",
    "sector_size":512,
    "state":"disabled",
    "name":"88e0e66821229cbc2079a486ac77cf0e33a2f655"
  }
]

While the two example volumes were active, they were listed as:
3
[
  {
    "dev":"namespace0.2",
    "mode":"raw",
    "size":0,
    "uuid":"00000000-0000-0000-0000-000000000000",
    "sector_size":512,
    "state":"disabled"
  },
  {
    "dev":"namespace0.1",
    "mode":"fsdax",
    "map":"dev",
    "size":4294967296,
    "uuid":"c05515e8-0ba3-11ea-b052-0242ac110005",
    "sector_size":512,
    "align":1073741824,
    "blockdev":"pmem0.1",
    "name":"83704ecc919dfe1f22f8600440f79b5bfdfa2a19"
  },
  {
    "dev":"namespace0.0",
    "mode":"fsdax",
    "map":"dev",
    "size":4294967296,
    "uuid":"c043a939-0ba3-11ea-b052-0242ac110005",
    "sector_size":512,
    "align":1073741824,
    "blockdev":"pmem0",
    "name":"88e0e66821229cbc2079a486ac77cf0e33a2f655"
  }
]

Note that only one remains behind as inactive.

I recall having seen explanation about ndctl internals that namespace deletion involves marking namespaces disabled. So a deleted namespace (or part of its metadata) may exist in disabled mode, and ndctl display code has to deal with that so that user-visible state remains as expected. There is also notion of "seed" namespace which remains not deleted even if others are gone. We may see some of such side-effects here.

So it's a bug in pmemNdctl.getDevice that it doesn't filter out the disabled namespace.

I'm not sure what to expect here: should the namespace be gone completely? Or should it be ignored and overwritten when the name is used again?

Yes, we should ignore inactive namesapaces: This change fixes the issue.


Added reasonable values for the missing [Economy] section for #439 startup exception

Doesn't actually fix the null reference exception, that would require another code change, but the default Halcyon.sample.ini file should not ship with the missing section anyway.

I recommend adding a little more commentary, a la issue #70. Other than that, this has been long on my todo list - good to see it happen!

I've added some entries for this section in PR #444 but I see now that my commit was also missing EconomyBaseAccount which should be added.
I could be wrong, but I think most of the others in Vin's comment above (like the UserLevel, Stipend, etc) are old options copied from OpenSim but not used in Halcyon. I think some of the comments are very specific to OpenSim concepts that don't apply here, and I don't think we should be attempting to document any other options in the sample INI file unless they are confirmed to still apply and work correctly in Halcyon.  I think the ObjectCapacity option should not be used from INI file, and may be ignored in some parts of the code in favor of the expected region.xml file option.
My immediate goal for this issue is only to fix the null reference exception in this report, not to complete the documentation for all options. However if there are any glaring exceptions (like the EconomyBaseAccount above), we should add those as well, as soon as possible. So for now in this PR, I've removed ObjectCapacity from the sample file and added EconomyBaseAccount.

Works for me. As long as the options for the prices and bank account UUID are in, I am good to go. That is what the website uses with the features in the simulator. Max prims is actually set with the region settings. So I have no idea why it was in the [Economy] section.

I've added the EconomyBaseAccount entry, but I don't think there's any others that are safe to add without further investigation. That may be necessary for #70 but I don't think it's necessary for this one. Ready to go?

Works for me. One advance at at time as it can be verified. :)


Make grascii.lark compatible with LALR

There are also some minor changes to the strings accepted by the language:

A^' is no longer valid: '-ing' should never be disjoined
A^A^ is now valid to distinguish a word with a prefix from a word with a suffix (A^A)
It is now valid to write bk'-n to distinguish -ing from an aspirate
Spaces are no longer ignored

Note: Adding terminal priorities with Earley requires lark 1.0+ (#12).


traces_sampler receives always "generic ASGI request" as transaction name.

How do you use Sentry?
Sentry Saas (sentry.io)
Version
1.11.1
Steps to Reproduce
I have manually instrumented traces_sampler counted on transaction name in sampling context. At some point with sentry-sdk version upgrade, it stopped work - there is always "generic ASGI request" in transaction name. But at the performance page, transaction has a proper name.
How we initiate sentry FastAPI integration:
sentry_sdk.init(
    dsn=SENTRY_DSN,
    environment="dev",
    traces_sampler=sentry_traces_sampler,
    integrations=[
        starlette.StarletteIntegration(transaction_style="endpoint"),
        fastapi.FastApiIntegration(transaction_style="endpoint"),`
    ]
)

The traces_sampler function:
def sentry_traces_sampler(sampling_context):
    transaction_sampling_mapper = {
        "app.api.api_v1.routers.users.some_name": 0.1,
        "app.api.api_v1.routers.messages.some_name_2": 0.5,
    }
    return transaction_sampling_mapper.get(
        sampling_context["transaction_context"]["name"], 1.0
    )

Expected Result

Actual Result
Possible solution: Give the plain URL of the request instead of "generic ASGI request" as the transaction name to traces_sampler.

Related to #1599

The problem here is, that we give the event to the traces_sampler as soon as possible. At this point in time the Starlette or FastAPI integrations have not yet set the transaction name to the route (because they have not yet run)
To give the traces_sampler something to work with, we will set the transaction name to the URL (which later will be overwritten by the Starlette/FastAPI integrations with the route. (because transaction names need to have low cardinality, we can not leave the URL as the name, because it will mess up grouping)

Oh, I just remembered that @polina-popova commented in this issue that it used to work at some point: https://github.com/getsentry/sentry-python/issues/1599#issuecomment-1335139633


fix: leaderboard scss imports and testnet note background

@nud3l
import 'pages/leaderboard/leaderboard.page.scss';

In this case, we could use as it's a child:
import './leaderboard.page.scss';


about chinese

Did I affect your code? Because I saw that you improved Chinese and then removed,I'm sorry,I once modified the code according to the way Chinese people use it. Thank you for your open source.I want help you to finish your project,but my skills are poor,I will help you translate chinese,I will not modify the code.  https://fmy.gooda.homes/

No problem. At first, I saw your Chinese translation and wanted to integrate it because China has over 1 billion people. But then I decided to not rely on translation services like Google Translate and to only manage languages I speak myself.

Hi Goodahome,
If you want to add the Chinese language (like for instance zh_CN for mainland China), then you could create a Pull Request:

create a folder /lang/zh_CN/
copy all files from /lang/en/ to /lang/zh_CN/
translate all files in /lang/zh_CN/

Please use simplified characters.
Then I will integrate the Chinese language.
Thanks.

okay


Add finalizer for VSR

dataMoverImageFqin: 'quay.io/emcmulla/data-mover:finalizer-vsr'

Can you update
https://github.com/migtools/volume-snapshot-mover/blob/8e12e4d47ecdd07a4860e9a54af4060d2f542bb1/controllers/cleanup.go#L194-L200
and remove the if statement?
I'd think all VSRs should be updated to cleanup status regardless of whether or not it has deletionTimestamp.

Can you update
https://github.com/migtools/volume-snapshot-mover/blob/8e12e4d47ecdd07a4860e9a54af4060d2f542bb1/controllers/cleanup.go#L194-L200
and remove the if statement? I'd think all VSRs should be updated to cleanup status regardless of whether or not it has deletionTimestamp.

@shubham-pampattiwar Is this alright? Making sure there wasn't a specific reason for adding this check before changing anything.


WebApi Increase request timeout and keep alive timeout

After 2 minutes u get 502 error(iis as reverse proxy)

Both were Increased to 10 minutes.


Please setup Dependabot to keep dependencies updated

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.

Steps to reproduce
I keep experiencing issues as a module I own has a dependency on NuGet.Versioning this causes issues since PowerShellGet also has that dependency and the versions mismatch. Please keep your dependencies up to date.
Expected behavior
Modules import without conflict

Actual behavior
Module assembly version conflict

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.3.2
PSEdition                      Core
GitCommitId                    7.3.2
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Name          Version
----          -------
PowerShellGet 3.0.19

Visuals
No response

Thanks @ThomasNieto great suggestion


[Bug] v5.0.9 breaks some caching for Classes

Hello, seems like latest version on this bundle breaks something in Symfony
https://github.com/sensiolabs/SensioDistributionBundle/commit/c956be2f4a0984c6bce41e9f81becb69a41cc350
PHP Fatal error:  Cannot declare interface Symfony\Component\DependencyInjection\ContainerAwareInterface, because the name is already in use in /tmp/var/prod/cache/classes.php on line 7553
exit code 255

ping @nicolas-grekas

@waplet which version of symfony do you have? Did you clear the cache completely (i.e. by doing first a composer install then a app/console cache:clear?

@nicolas-grekas We are using Symfony 2.8.10;
I guess that's something behind bootstrap.cache.php and classes.php files, but can't really figure it out.
It happens when
include_once __DIR__.'/../var/bootstrap.php.cache'; and $kernel->loadClassCache(); is enabled;
Cache clearing is made as hook in composer.json

Seems like the bug was due to fact, that implement ContainerAwareInterface was called before cached class loaded.
And afterwards the cache tried to create new class definition.

Why did you close? Found an issue on your side?

Yes, it was somehow issue with Commands which implemented ContainerAwareInterface;

Great! Thanks @waplet for the feedback.


gridcoinresearch.conf corrupts on hard reset

Seems more and more people are running into this issue. On a hard reset of the host; blue screens, cutting power, etc, the gridcoinresearch.conf file corrupts with empty contents or random characters.
A upload of one user's config file showed one long line of "NUL" values.
This causes the wallet to no longer open and instead spit out a C++ Runtime error.
I don't have details on that error, but I'm pretty sure it can be replicated easily by just filling your config file with spaces.
After deleting the config file or replacing it with the right syntax the wallet will start normally.
The error: (by Schietschijf on IRC)

+1 had to reset my conf after shutting down my pc while gridcoin was still running

I have experienced this a fair bit too. I keep a copy handy just in case the wallet crashes for whatever reason.
This behavior suggests that the file is not being closed after the config parameters are retrieved.

Checking the c++ portion, the ifstream destructor guarantees a close of the file, and according to the boost message boards its not good practice for us to add a manual close after we read the file in, so Im making the assumption this may be due to our vb.net additional feature for tickers or the neural network primary cpid row being written and not flushed.
I added code to our .net function to ensure the file is closed after it adds any key.  Lets see if that fixes the problem.  Will be in the next release.

Please let us know if this appears to have fixed it, and we can close it in 45 days.  Thanks.


[libmediainfo] update to 24.03

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Thanks for the update!


TS-4954: Use TSIOBufferCopy in the passthru example plugin.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/963/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/855/ for details.

👍  - Looks good


Dialog backgroud

Hi searchy i'd like to thank you for this nice work,
my problem is:
some time when the Simple alert and Confirmation alert dialog start, their backgrounds takes the bottom of the screen as background for short time, like you can see it in the picture

This happens because the layout is measured twice and redrawn. To prevent this from happening, the background activity's style must be the same as the library's activity style.
You'll notice that when the popup appears, it enters fullscreen mode. If the background activity isn't in fullscreen mode, then you'll have the background being redrawn.


UI Context stack & 4 character constants

Makes the egui context exposed variable a stack.
This way we can have claymore UI host user scripts containing it's own UI.
Additionally I want to make some additional things available within the new Context struct, mainly to be able to map a region inside egui back into window coordinates

Added a test src/tests/ui-nested.edn and also fixed a case where the nested UI would take UI parents from the outer UI (in case of CentralPanel for example)
Also I believe there was a potential leak/corruption issue with the stack variables because they:

retrieved the seq (by value)
pushed
set it on ParamVar
run some other code
pop from the SAME seq as in the first step (here might be an issue since the memory could've been reallocated/resized, but it won't be reflected here)
set it on ParamVar again

Should be fixed by retrieving from the ParamVar again after running the inner code


Bug : Airflow test picks up new AWS API keys, but Airflow schedulers do not!

How do we invalidate STS tokens when we change AWS API Access Keys that Airflow is using? It seems that airflow test works, but airflow run or anything running via the schedulers fail!
Specifically, airflow test ep_demo purge_DLQ 2015-10-07 works but airflow run ep_demo purge_DLQ 2015-10-07 fails!
We have changed our AWS creds on the airflow webserver, where we also run local executors and schedulers. Unfortunately, the new creds are not being picked up. We can run boto code outside of airflow and it runs correctly, picking up AWS creds from the ENV vars. However, in airflow, there seems to be a token. I think this token is cached and is referring to an invalid API token -- we retired the old API Key. How do I invalidate the token that airflow is using? @mistercrunch
[2015-10-07 16:34:47,325] {connection.py:1206} ERROR - <?xml version="1.0"?><ErrorResponse xmlns="http://queue.amazonaws.com/doc/2012-11-05/"><Error><Type>Sender</Type>InvalidClientTokenId<Message>The security token included in the request is invalid.</Message><Detail/></Error><RequestId>3016929b-9c59-56ca-9e92-dc14d7be036f</RequestId></ErrorResponse>
[2015-10-07 16:34:47,327] {models.py:962} ERROR - 'NoneType' object has no attribute 'count'```

Also, airflow test ep_demo purge_DLQ 2015-10-07 works but airflow run ep_demo purge_DLQ 2015-10-07 fails!

That's odd, in theory, the code at https://github.com/airbnb/airflow/blob/master/airflow/hooks/S3_hook.py#L140-L152 should get a new token each time. I am afraid I might be out of my depth on those internals.
The S3 hook is due for a redesign especially using boto3. If anybody wants to volunteer... :smile:

We don't use the hook. Instead, we just use boto2. We don't have any problem running airflow test, but this is not working via the local executors at all. We have built a python script named reload_data.py that reload a range of days. The script calls into airflow tasks via the CLI using airflow test. Right now, we are exclusively using this script. Our airflow server is completely blocked since it uses executors. Any help you can provide would be helpful.

Also, I don't think I need to mention this, but I am able to make calls via boto in a python shell or regular boto program as well. So, it's just a problem with the local executors. I've restarted them.. etc... so I'm really wondering if there is some token caching or something being done.

Ok.. found the issue. A ./aws/credentials file with a different set of creds. I figured that ENV vars always won.


add rdf/ttl output to run-linkml-examples

Added prefix for https://microbiomedata/schema as in https://microbiomedata/schema/ecosystem, but

File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/submission-schema-LusGX8SJ-py3.10/lib/python3.10/site-packages/linkml_runtime/dumpers/rdflib_dumper.py", line 130, in inject_triples
slot_uri = URIRef(schemaview.get_uri(slot, expand=True))
File "/Users/MAM/Library/Caches/pypoetry/virtualenvs/submission-schema-LusGX8SJ-py3.10/lib/python3.10/site-packages/linkml_runtime/utils/schemaview.py", line 877, in get_uri
schema = next(sc for sc in self.schema_map.values() if sc.id == e.from_schema)

@turbomam a linked PR is merged. Can this be closed?


Review careful IAM strategy

in particular review

root account vs. pp-specific accounts for prod & stage
permissions for those accounts and security review of auth check on destructive calls.

Won't be super relevant once we are off AWS.


Initially collapsed button with radio - error

When initially collapsed button contains radio input, on the load of page button.js triggers error "Uncaught TypeError: Cannot read property 'checked' of null" (line 174). Seems check whether 'input' is undefined at the beginning of if(... could solve it.

Bootstrap version: 4.5.2
Operating system: Windows 10
Chrome 85.0.4183.102 x64
Test case: https://jsbin.com/yofuviv/edit?html,output
(described error appears in console of developer tools)

This isn't a super clear example—there's a good amount of unrelated HTML in there. Recreating this from scratch the way I'd do it, I see no errors. https://codepen.io/emdeoh/pen/yLOQjvo?editors=1100
Some of the problems I see:

Mixing button classes with dropdown classes
Rogue .collapse class in there
Custom classes not found in Bootstrap
Mismatched CSS and JS versions of Bootstrap

If you can reduce it down to completely vanilla Bootstrap code and still have a problem, we'd be happy to look again.


fix dataset.md

fix dataset.md
test=develop

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


Survey of resources on RSE and RSEing

Collect resources on RSE-related policies, surveys, training, certification.

Note: some of these resources do not necessarily reflect RSE or RSEing in their titles. For example, European Commission DGRI 2024 on page 29 section "5.1. National policies on skills and training for open science", they list countries with training programs for RSE/RDM. They don't seem to provide links or citations to the specific training programs that qualified for this table. Some of the resources are present because they are directly cited in RSE-specific resources, like European Commission recommendations or raw survey data.

The other position paper is in lack of some good references: https://github.com/DE-RSE/2023_paper-RSE-groups/issues/21
maybe you have ideas for some of the refs? Thank You!

Hi Jean-Noel, all of this is certainly interesting, but so far it's not connected to the text, no?

My idea was to use this BibTeX file as an evidence bank of teaching-related RSE resources, similar to the Software Sustainability Institute Open Evidence Bank and resource hub (e.g. list of RSE journals), or the UK RSE resource database. It would be complementary to the survey, in the sense that resources that are featured on the teachingRSE website should be curated and approved by more than one person, while this evidence bank would have a lower barrier to entry. I also find it more convenient to proactively store interesting resources here, rather than trying to find them again at a later point in time when the manuscript takes a direction that makes them relevant.
One could argue a separate, dedicated repository should be used for this task to avoid cluttering the manuscript BibTeX. If you find this repository is not the right place, I will refrain from adding entries that are not cited in the manuscripts.

ah, ok I see.

how about moving the bibliography into its own repository? and then bringing the bibliography in, either via a sub-project or some other mechanism?


Context not respected for getting elements by id

In the case where context is another document (think of friendly iframes).
In core.js on line 57 - 60 if the selector is that for an id, doc (rather than context) is explicitly used to find it and return.

I think we should fix this.
Right now contexts, as you pointed out, aren't used at all when searching by id (#foo), I think that was originally intended as a performance optimization actually.

Yes, performance optimization and the fact that element IDs are supposed to be unique, never duplicated. This is also how jQuery handles ID selectors: https://api.jquery.com/id-selector/

I'll investigate this further, I'm not very familiar with using jQuery/cash with iframed pages.
I think if elements within an iframe can be selected by id from the parent page/scope without manually passing the document object then that's what anyone should do. If this isn't the case then I think we should support this.

This is not something well supported, even in jQuery. iframe cross-talk isn't easy.
https://stackoverflow.com/questions/6316979/selecting-an-element-in-iframe-jquery
I'd personally classify this as an edge case but if it can be accomplished without adding numerous bytes or slowing down all ID selections then... ¯_(ツ)_/¯

@fabiospampinato To clarify, the use case which led me to find this was cash being loaded into a friendly-iframe and the document passed in belonged to the top window (Not the otherway around).
@shshaw An alternate approach would be allow instances of cash that can be instantiated with a specific context (i.e. alternate documents)

An update on this:

Turns out @shashank-'s use case was kind of already supported in previous versions of cash, you just had to write something like this: $($('iframe)[0].contentDocument).find ( '#foo' )

In jQuery you can achieve the same in 2 other ways:

$( '#foo', $('iframe').contents () ): this is still not supported as it will slow down all the other, far more common, selections by id. Maybe we should still align cash with jQuery in this regard, I'm not sure, let's keep it this way for now.
$('iframe').contents ().find ( '#foo' ): now this is supported too, I've implemented the $.fn.contents method in v2.3.0.

$( '#foo', $('iframe').contents () ): this is still not supported as it will slow down all the other, far more common, selections by id. Maybe we should still align cash with jQuery in this regard, I'm not sure, let's keep it this way for now.

Turns out this can be implemented without sacrificing performance, so as of v.2.3.3 you can now do:
$( '#foo', $('iframe').contents () )

This aligns cash with jQuery in this regard.


fix: fix bug when querying projects with archived=None

a fix for pyright: add .encode("utf-8") after json.dumps


SvelteKit Demo App $app/* unresolved

Steps to reproduce

npm create<EMAIL_ADDRESS>my-app

SvelteKit demo app
Yes, using TypeScript syntax
empty

npm i
rename vite.config.ts -> vite.config.js
npx<EMAIL_ADDRESS>
Result:
Unused devDependencies (2)
@fontsource/fira-mono  package.json
tslib                  package.json

Unresolved imports (5)
$app/stores       src/routes/Header.svelte:1:9
$app/environment  src/routes/about/+page.ts:1:9
$app/forms        src/routes/sverdle/+page.svelte:2:9
$app/environment  src/routes/sverdle/how-to-play/+page.ts:1:9
$app/environment  src/routes/sverdle/reduced-motion.ts:2:9

Expected Behavior
Resolve $app/* correctly

You can try to add the path to the config like so:
https://github.com/webpro/knip/blob/main/packages/knip/fixtures/plugins/svelte/knip.ts#L3-L5
Please let me know how it goes, should at least be documented better.

You can try to add the path to the config like so:
https://github.com/webpro/knip/blob/main/packages/knip/fixtures/plugins/svelte/knip.ts#L3-L5
Please let me know how it goes, should at least be documented better.

Worked like a charm! Thank you very much.


Remove circular dependency between csharptype and typeprovider

Fixes https://github.com/microsoft/typespec/issues/3775

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Generator.CSharp


franz: Allow to run with grsecurity kernel

Motivation for this change
Fix #20539
Things done

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Thanks! Perhaps https://github.com/nixos/nixpkgs/blob/master/pkgs/applications/networking/instant-messengers/rambox/default.nix affected too?

Probably... Looks like the same thing xD


LeanPub Service Tests Failing

:clock11: When did the problem start?
December ~12th

:camera: Live badge
https://img.shields.io/leanpub/book/pages/hands-on-nodejs

https://img.shields.io/leanpub/book/pages/juice-shop

:wrench: Is the live badge working?
Doesn't appear to be working

:link: CircleCI link
https://circleci.com/gh/badges/daily-tests/787

:beetle: Stack trace
LeanpubBookSummaryService [live] known book pages [ GET /pages/juice-shop.json ] - [ GET /pages/juice-shop.json ]
  AssertionError: label mismatch: expected 'leanpub' to equal 'pages'

LeanpubBookSummaryService [live] known book sold [ GET /sold/juice-shop.json ]
   AssertionError: label mismatch: expected 'leanpub' to equal 'sold'

:bulb: Possible solution

Looks like LeanPub have restricted their API to pro plan users: https://leanpub.com/help/api
A pro plan is $12.99 per month so I suspect this means we probably just have to abandon these badges.

So they have 🤔 That's a bummer!

I agree that we shouldn't purchase a monthly subscription just to get access to the data (that could set a bad precedent and lead to it becoming very expensive to run the Shields service).
I took a look at our metrics though and there was well north of 700 requests for Leanpub badges in the last 24 hours which indicates there's some community interest in these badges and has me wondering if it'd be worth reaching out to them to see whether they'd be interested in offering Shields an API token.
Do you think it's worth it or should we just go ahead and start working on deprecating them?

I think given they've been broken for over a week and nobody has raised an issue about it or commented/reacted here we can probably just remove them, but if you're keen to see if we can get a key, feel free.

SGTM!


Add a section on how to use propagators, samplers, defaultAttributes etc.

This issue is motivated by user inquiries on how to change default values.
The @opentelemetry/core module provides multiple implementations of propagators (B3Format, TraceContext and Binary), samplers(AlwaysSampler, NeverSampler and ProbabilitySampler) etc. I think we should add a section (maybe, "advanced use") to highlight how to use/change other implementations.

For "how to use" I believe that a demo is better than words. Adding examples in the examples folder. Perhaps one called "advanced-use" or multiple examples

I think we better hold off on this for a short bit until the OTEP 66 changes are merged as the propagators are going to change somewhat


Repository cannot be installed?

Hey there, I've been trying to install the repository via NPM both in its registry and with the GIT url and I'm getting the following errors:
npm install --save-dev gulp-modernizr
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm http GET https://registry.npmjs.org/gulp-modernizr
npm http 404 https://registry.npmjs.org/gulp-modernizr
npm ERR! 404 'gulp-modernizr' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, or http url, or git url.

and with the GIT url ...
npm install --save-dev https://github.com/doctyper/gulp-modernizr
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm http GET https://github.com/doctyper/gulp-modernizr
npm http 200 https://github.com/doctyper/gulp-modernizr
npm ERR! not a package /var/folders/v3/7m8n_d617kzdvw3z2m6kttlr0000gn/T/npm-6344-rNguOVd-/1403471157007-0.10342534794472158/tmp.tgz
npm ERR! Error: ENOENT, open '/var/folders/v3/7m8n_d617kzdvw3z2m6kttlr0000gn/T/npm-6344-rNguOVd-/1403471157007-0.10342534794472158/package/package.json'
npm ERR! If you need help, you may report this *entire* log,
npm ERR! including the npm and node versions, at:
npm ERR!     <http://github.com/isaacs/npm/issues>

Perhaps something to do with the package.json?
Hope that helps,
Luke

What is the status of this?

Current version works fine for me on both Windows and OS X.

Tried all of the suggestions above with no luck. Any idea when this will get fixed?

Same issue for me.  I am not able to sudo npm install --save-dev gulp-modernizr!

Hi guys and girls is this still an issue for you with the newest version (2.0.0) or did you already move on?

Moved on prob, this is almost 3 years old for me.


API Endpoints

Could you list some of the API endpoints that you have not done as separate issues individually

Account payments through https://api.flutterwave.com/v3/charges?type=debit_ng_account
UK Account payments through https://api.flutterwave.com/v3/charges?type=debit_uk_account
Rwanda Mobile Money through https://api.flutterwave.com/v3/charges?type=mobile_money_rwanda
Uganda Mobile Money through https://api.flutterwave.com/v3/charges?type=mobile_money_uganda
Zambia Mobile Money through https://api.flutterwave.com/v3/charges?type=mobile_money_zambia
Ghana mobile money through https://api.flutterwave.com/v3/charges?type=mobile_money_ghana

Kindly refer to https://developer.flutterwave.com/docs under the PAYMENT METHODS section.


Wrong interrupt header for STM8L10x

In stm8l10x.h the included interrupt header file refers to the STM8S series, not the STM8L10x.
--- stm8l10x.h.orig     2021-10-18 14:52:40.882365259 +0200
+++ stm8l10x.h  2021-10-18 14:54:44.713340650 +0200
@@ -1439,7 +1439,7 @@

 #if defined(_SDCC_)                      /* SDCC patch: doesn't work like that in SDCC -> skip */
   #define INTERRUPT __interrupt
-  #include "stm8s_it.h"                  /* SDCC patch: ensure inclusion in main.c for interrupts */
+  #include "stm8l10x_it.h"               /* SDCC patch: ensure inclusion in main.c for interrupts */
 #endif

 /*============================== Handling bits ====================================*/

hi Gabbla,
sorry for the late response and thanks for pointing this out! I have fixed it already.
Regards, Georg


Error when deployed from Azure

I got this working fine inside our corporate network, but deployed via Azure Web App, we are getting the below error using Authentication Mode 2:
<Error>
<Message>The communication object, System.ServiceModel.Channels.ServiceChannel, cannot be used for communication because it is in the Faulted state.</Message>
</Error>

Networking guys say everything in Azure is connected fine to our on-prem environment.
Is there any suggestions to consider when deployed to a web app accessing on-prem?

Hi,
You need to setup the Azure Hybrid Connection.
https://docs.microsoft.com/en-us/azure/app-service/app-service-hybrid-connections
Under your Api App in Azure click on Networking and follow the wizard for setting up the Hybrid connection. It will download a service and it will sit on prem.

Thanks, we have Azure VNet, but I assume because AX uses TCP to access the AOS, is this why Azure Hybrid Connection is needed?

hi.
No need for any VNet/VPN.
Azure Hybrid Connection takes care of all that.
As the link from docs says:
"It normally does not require firewall holes. The connections are all outbound over standard web ports."
I have implemented it at a number of customers and it was the case. Very easy to set up.
Just needs the azure global administrator to do (i think my memory is right).


python-flask deploy endpoint not reachable

Describe the bug
Trying an appsody deploy of the python-flask stack and the endpoint is not reachable
To Reproduce
Steps to reproduce the behavior:

setup and configure docker desktop with kubernetes
mkdir python-flask
cd python-flask
appsody init python-flask
appsody deploy -v

Expected behavior
Expected to get the usual endpoint webpage
Actual behaviour
The endpoint URL given http://localhost:31867 is not reachable
[Docker] Successfully built 4911c055ac6c
[Docker] Successfully tagged dev.local/python-flask:latest
Built docker image dev.local/python-flask
Using applicationImage of: dev.local/python-flask
Attempting to apply resource in Kubernetes ...
Running command: kubectl apply -f app-deploy.yaml --namespace test
Deployment succeeded.
Appsody Deployment name is: python-flask
Running command: kubectl get rt python-flask -o jsonpath="{.status.url}" --namespace test
Attempting to get resource from Kubernetes ...
Running command: kubectl get route python-flask -o jsonpath={.status.ingress[0].host} --namespace test
Attempting to get resource from Kubernetes ...
Running command: kubectl get svc python-flask -o jsonpath=http://{.status.loadBalancer.ingress[0].hostname}:{.spec.ports[0].nodePort} --namespace test
Deployed project running at http://localhost:31867

Environment Details (please complete the following information):

OS: [e.g. iOS] mac, tried windows10 also

If applicable please specify:

CLI version: 0.4.8
Stack you are using (including the version): python-flask

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

This happened to me too - it seemed to fix itself after I reset my Kubernetes cluster though...

I'll investigate this to see if I can work out what is going on...

The PR #475 contains a workaround fix for the problem, while I have raised as separate issue for the underlying problem


Chapter 11 - Yahoo-Finance no longer working

Wondering if your code will be updated to a new historical data gathering tool as yahoo-finance doesn't work anymore? Maybe iexfinance or Alpha Vantage. And could the predictive model use .csv files instead of .npy? Thanks! Great work!

@windowshopr thanks for filing this issue. I just executed the code once again, which is working fine. I've uploaded the updated codes (mostly tested on TensorFlow 2.0 about 3 months ago, though) as Notebook in each chapter.
But you're right sometimes Yahoo-finance doesn't work. However, I don't have have any workaround here because I'm pretty overloaded at the moment.
Please try running the corresponding Notebook file and let me know it went.

https://github.com/PacktPublishing/Predictive-Analytics-with-TensorFlow/blob/master/Chapter11/StockPricePrediction.ipynb

Thanks for the suggestion! This repository was the first one I came across in the Packt Publishing suite haha. I will give the notebook a shot and if there are any issues, I will report back. Maybe this weekend when I get some time, I'll see if I can tweak the existing code to pull the historical data from another source and make a pull request, so that both options could be available.
Thanks!

@rezacsedu The notebook worked great. I've made some modifications to convert the .npy to a .csv file and using that for the simulations, but not before running a wavelet transform to denoise the stock data (although probably not super applicable here, it is a feature that I'll want to use in future use). I will also be updating where the historical data is sourced from someday.
I've attached my edited code below here just for reference. One question I did have, could you explain the (avg, std) that gets printed at the end? Is the avg the average ending portfolio size out of all 100 simulations, and what would the std deviation depict? Just a newbie question. Thanks!
import warnings
warnings.filterwarnings("ignore")

from yahoo_finance import Share
from matplotlib import pyplot as plt
import numpy as np
import random
import tensorflow as tf
import random
import pywt
from numpy import genfromtxt

class DecisionPolicy:
    def select_action(self, current_state, step):
        pass

    def update_q(self, state, action, reward, next_state):
        pass

class RandomDecisionPolicy(DecisionPolicy):
    def __init__(self, actions):
        self.actions = actions

    def select_action(self, current_state, step):
        action = self.actions[random.randint(0, len(self.actions) - 1)]
        return action

class QLearningDecisionPolicy(DecisionPolicy):
    def __init__(self, actions, input_dim):
        self.epsilon = 0.5
        self.gamma = 0.001
        self.actions = actions
        output_dim = len(actions)
        h1_dim = 200

        self.x = tf.placeholder(tf.float32, [None, input_dim])
        self.y = tf.placeholder(tf.float32, [output_dim])
        W1 = tf.Variable(tf.random_normal([input_dim, h1_dim]))
        b1 = tf.Variable(tf.constant(0.1, shape=[h1_dim]))
        h1 = tf.nn.relu(tf.matmul(self.x, W1) + b1)
        W2 = tf.Variable(tf.random_normal([h1_dim, output_dim]))
        b2 = tf.Variable(tf.constant(0.1, shape=[output_dim]))
        self.q = tf.nn.relu(tf.matmul(h1, W2) + b2)

        loss = tf.square(self.y - self.q)
        self.train_op = tf.train.GradientDescentOptimizer(0.01).minimize(loss)
        self.sess = tf.Session()
        self.sess.run(tf.initialize_all_variables())

    def select_action(self, current_state, step):
        threshold = min(self.epsilon, step / 1000.)
        if random.random() < threshold:
            # Exploit best option with probability epsilon
            action_q_vals = self.sess.run(self.q, feed_dict={self.x: current_state})
            action_idx = np.argmax(action_q_vals)  # TODO: replace w/ tensorflow's argmax
            action = self.actions[action_idx]
        else:
            # Explore random option with probability 1 - epsilon
            action = self.actions[random.randint(0, len(self.actions) - 1)]
        return action

    def update_q(self, state, action, reward, next_state):
        action_q_vals = self.sess.run(self.q, feed_dict={self.x: state})
        next_action_q_vals = self.sess.run(self.q, feed_dict={self.x: next_state})
        next_action_idx = np.argmax(next_action_q_vals)
        action_q_vals[0, next_action_idx] = reward + self.gamma * next_action_q_vals[0, next_action_idx]
        action_q_vals = np.squeeze(np.asarray(action_q_vals))
        self.sess.run(self.train_op, feed_dict={self.x: state, self.y: action_q_vals})

def run_simulation(policy, initial_budget, initial_num_stocks, my_data, hist, debug=False):
    budget = initial_budget
    num_stocks = initial_num_stocks
    share_value = 0
    transitions = list()
    for i in range(len(my_data) - hist - 1):
        if i % 100 == 0:
            print('progress {:.2f}%'.format(float(100*i) / (len(my_data) - hist - 1)))
        current_state = np.asmatrix(np.hstack((my_data[i:i+hist], budget, num_stocks)))
        current_portfolio = budget + num_stocks * share_value
        action = policy.select_action(current_state, i)
        share_value = float(my_data[i + hist + 1])
        if action == 'Buy' and budget >= share_value:
            budget -= share_value
            num_stocks += 1
        elif action == 'Sell' and num_stocks > 0:
            budget += share_value
            num_stocks -= 1
        else:
            action = 'Hold'
        new_portfolio = budget + num_stocks * share_value
        reward = new_portfolio - current_portfolio
        next_state = np.asmatrix(np.hstack((my_data[i+1:i+hist+1], budget, num_stocks)))
        transitions.append((current_state, action, reward, next_state))
        policy.update_q(current_state, action, reward, next_state)

    portfolio = budget + num_stocks * share_value
    if debug:
        print('${}\t{} shares'.format(budget, num_stocks))
    return portfolio

def run_simulations(policy, budget, num_stocks, my_data, hist):
    num_tries = 100
    final_portfolios = list()
    for i in range(num_tries):
        final_portfolio = run_simulation(policy, budget, num_stocks, my_data, hist)
        final_portfolios.append(final_portfolio)
    avg, std = np.mean(final_portfolios), np.std(final_portfolios)
    return avg, std

def get_prices(share_symbol, start_date, end_date, cache_filename):
    try:
        stock_prices = np.load(cache_filename)
    except IOError:
        share = Share(share_symbol)
        stock_hist = share.get_historical(start_date, end_date)
        stock_prices = [stock_price['Open'] for stock_price in stock_hist]
        np.save(cache_filename, stock_prices)

    return stock_prices

def plot_prices(my_data):
    plt.title('Opening stock prices')
    plt.xlabel('day')
    plt.ylabel('price ($)')
    plt.plot(my_data)
    plt.savefig('my_data.png')

if __name__ == '__main__':
    prices = get_prices('MSFT', '2000-07-01', '2017-07-01', r'C:\Users\windowshopr\Desktop\Python Scripts\Stock Market Prediction\Predictive Analytics with TensorFlow\Chapter11\resource\stock_prices.npy')

    # Perform wavelet transform to preprocess stock data
    x = np.array(prices)
    (ca, cd) = pywt.dwt(x, "haar")
    cat = pywt.threshold(ca, np.std(ca), mode="soft")
    cdt = pywt.threshold(cd, np.std(cd), mode="soft")
    tx = pywt.idwt(cat, cdt, "haar")

    # Save data as .csv file to use for simulation
    np.savetxt('stock_prices.csv', tx, delimiter=",", fmt="%s")

    my_data = genfromtxt('stock_prices.csv', delimiter=',')
    plot_prices(my_data)

    actions = ['Buy', 'Sell', 'Hold']
    hist = 200
    policy = RandomDecisionPolicy(actions)

    budget = 1000.0
    num_stocks = 0
    avg, std = run_simulations(policy, budget, num_stocks, my_data, hist)
    print(avg, std)


Support for LibreJS

Problem
A lot of GNU/Linux YouTubers moved to LBRY or synced with it at least. @Lunduke even deleted his YouTube videos. And now posts on LBRY exclusively. So I was surprised when on FSF.org in their Social Network recommendations they don't list LBRY.
I know that the code of this web site is Free Software. But for most people the easy way to check it is to see whether the site runs under LibreJS. Or at least supports NON-JavaScript mode.
Me personally. I know that the license for the entire site is fine. So I whitelisted LBRY.TV and ODYSEE.COM in my LibreJS. GitHub for example works. But blocks 8 scripts.
On LBRY nothing loads. 1 script is blocked. This one. Which probably loads the rest of the site.
LibreJS works by evaluating the complexity of each JavaScript. And if it's anything more then trivial it checks for a License. And it does it for each script independently. If a script has no license. Or the license is not Free. It blocks that script from running.
By not putting a license into the text of the JavaScripts. We prevent a large amount of people from endorsing the platform. So a thing should be done about it.
How to fix the problem
To fix the problem we need to do next:

Install either GNU IceCat Browser or a LibreJS plugin for Firefox.
Go over the scripts that it refuses to load.
Put a license comment to them.

Another possible solution:

Build an HTML-5 (zero JavaScript) version of the site. That might look terrible but will allow people to use it. It could be cool for people who use Tor and block ALL JavaScripts. Not only non-free ones.

Thank you for the issue, this is filed under the front end at https://github.com/lbryio/lbry-desktop/issues/5327


graphql_flutter: migrate the example

Fixes https://github.com/zino-hofmann/graphql-flutter/issues/1030
Signed-off-by: Vincenzo Palazzo<EMAIL_ADDRESS>

Too boring to review this example :)


Bold button in toolbar not woking.

Bold button in toolbar not working if we use modernizr.js in our project.
I just fix my issue by adding  css.
.ql-editor strong{
     font-weight:bold;
 }

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


dask is broken due to pythonPackages.partd

Issue description
Not sure but I believe this is related to the newest version of pandas. I checked and the current partd testing on travis is using an old pandas version https://travis-ci.org/dask/partd/jobs/441023793#L633
Steps to reproduce
nix-build -A python3Packages.partd

fixed on staging-next


Groot partials

Hello! Can you please check this partials js????

Done, now we'll merge groot into out branch to solve conflicts if you think it's ok

Ok, you could resolve the conflicts and merge.

Hi, we've merge groot-partials with groot through the console because it was complicated doing it with github. So, we've solved the conflicts and it's all set.


fix: Change zone name in the mass update user zone script

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 86.616%

Totals

Change from base Build<PHONE_NUMBER>6:
0.05%

Covered Lines:
4282

Relevant Lines:
4785

💛  - Coveralls


Rails 7.1: mock_model is not always === the class it mocks

#54 reported, and #55 fixed an issue where Rails 7.1 has removed the ActiveRecord::Base#=== method. We found that the issue is only fixed in some cases, but not all. Specifically, when you have multiple mock_model’s in the same spec file, #=== only gets implemented on the first instance.
# frozen_string_literal: true

require "bundler/inline"

gemfile(true) do
  source "https://rubygems.org"

  gem "rails"
  gem "rspec-activemodel-mocks"
  gem "rspec"
  gem "sqlite3"
end

require "active_record"

ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
ActiveRecord::Schema.define { create_table(:posts) }

Post = Class.new(ActiveRecord::Base)

require "rspec/autorun"

RSpec.describe "#===" do
  before do
    @foo = mock_model(Post)
    @bar = mock_model(Post)
    @baz = mock_model(Post)
  end

  it "works for the first mock_model" do
    expect(Post === @foo).to be(true)
  end

  it "works for the second mock_model" do
    expect(Post === @bar).to be(true)
  end

  it "works for the third mock_model" do
    expect(Post === @baz).to be(true)
  end
end

The output from running this file is
Failures:

  1) #=== works for the second mock_model
     Failure/Error: expect(Post === @bar).to be(true)

       expected true
            got false
     # repro.rb:35:in `block (2 levels) in <main>'

  2) #=== works for the third mock_model
     Failure/Error: expect(Post === @baz).to be(true)

       expected true
            got false
     # repro.rb:39:in `block (2 levels) in <main>'

Finished in 0.0144 seconds (files took 0.22345 seconds to load)
3 examples, 2 failures

Failed examples:

rspec repro.rb:34 # #=== works for the second mock_model
rspec repro.rb:38 # #=== works for the third mock_model

If you'd like to tackle this please do so and I'd be happy to review, but I don't have enough free time right now to look into it myself (and I don't really use this library).


Doc feedback - Sample app mentioned isn't available in VS code with the latest version of Teams toolkit

Type of issue
Other (describe below)
Feedback
Hi,
We have received this Verbatim feedback. Requesting to look into this issue and help understand if any doc update would be required.

Doc feedback received: I'm using VS code with the latest version of Teams toolkit installed and the sample app mentioned in this guide doesn't exist.
Page URL
https://learn.microsoft.com/en-us/microsoftteams/platform/toolkit/build-a-rag-bot-in-teams?tabs=javascript%2Cjavascript1%2Cjavascript2%2Cjavascript4%2Cjavascript3
Content source URL
https://github.com/MicrosoftDocs/msteams-docs/blob/main/msteams-platform/toolkit/build-a-RAG-bot-in-teams.md
Author
@surbhigupta
Document Id
933feadd-483a-5562-81a3-38908b697a5a

@v-preethah - Thanks for reporting your issue. We will check and update you soon.

Hi @Nivedipa-MSFT
We have received the below inputs from the team, and we might not need a doc update.


The Spotify mute isn't working

Can you help?

And yes, I'm using the Mac Desktop app, not the web app. It was working last week and now it doesn't seem to be working.

Hey!, sure, can you share the ouput of the script after you execute it?, also, do you see any messages when you when you start playing a song?

this is a copy of the entirety--script included--and no, i don't see any mesages when i start playing a song:
Last login: Mon Jul  4 20:52:10 on ttys000
yo@MacBook-Pro ~ % mkdir -p ~/MuteSpotifyAds && curl https://raw.githubusercontent.com/gdi3d/mute-spotify-ads-mac-osx/master/NoAdsSpotify.sh > ~/MuteSpotifyAds/NoAdsSpotify.sh
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100  8710  100  8710    0     0  44226      0 --:--:-- --:--:-- --:--:-- 45602
yo@MacBook-Pro ~ % sh ~/MuteSpotifyAds/NoAdsSpotify.sh
Spotify Ads will be silenced while this program is running!. (This works ONLY with the Spotify App, not the web version)
This program was downloaded from https://gdi3d.github.io/mute-spotify-ads-mac-osx/ (check for documentation)
If the program is not working properly please open an issue at: https://github.com/gdi3d/mute-spotify-ads-mac-osx/issues/new
Press control+c to close this program or close the terminal window

🔈 Songs are playing 😀🕺💃. Audio back to normal

So weird, it's working now...

Oh WAIT!! I just remembered, it doesn't work if I use the Spotify app to play on the Alexa, is that correct??

As far as I know (I don't own an Alexa), it's not your computer playing the songs, it's Alexa. This means that all the downloading and playing is happing on the Alexa device.
You can test it by using Spotify with your headphones/speakers/bluetooth device and you'll see the script working properly.
 Audio back to normal
This means that the script has detected a song playing and that it's working properly.

You can close the ticket if the test I mentioned before works ok


On Country Map, Cross-Filtering is not operational as a source

Hello,
Country Map can be  filtered by the cross-filtering but it is not operational as a source because a click do zoom on map
Best regards

Yep. This is not a bug, really, but a lack of design/implementation for that plugin. Any interest in taking this on? We might close this as an Issue, but I'm happy to move it to an "Ideas" discussion if you want to figure out how to solve it or try to find help with it.

I agree that you move this issue to a discussion


remove h5py from spack and from tests

PR Summary
As requested by @mauneyc-LANL @dholladay00 and motivated by issues @annapiegra is having with oneapi, this PR does the following:

Stores an hdf5 "gold" file as a release
Downloads the file in cmake if needed
Removes py-h5py from spack and cmake dependencies.

PR Checklist

[x] Adds a test for any bugs fixed. Adds tests for new features.
[x] Format your changes by using the make format command after configuring with cmake.
[x] Document any new features, update documentation for changes made.
[x] Make sure the copyright notice on any files you modified is up to date.
[x] After creating a pull request, note it in the CHANGELOG.md file

Singularity-eos builds with a fresh spack checkout %oneApi w/ the following changes to package.py:
--- a/var/spack/repos/builtin/packages/singularity-eos/package.py
+++ b/var/spack/repos/builtin/packages/singularity-eos/package.py
@@ -19,6 +19,7 @@ class SingularityEos(CMakePackage, CudaPackage):
     maintainers = ["rbberger"]

     version("main", branch="main")
+    version("fixed", branch="jmm/goldfiles")
     version("1.6.2", sha256="9c85fca679139a40cc9c72fcaeeca78a407cc1ca184734785236042de364b942")
     version("1.6.1", sha256="c6d92dfecf9689ffe2df615791c039f7e527e9f47799a862e26fa4e3420fe5d7")

@@ -62,7 +63,6 @@ class SingularityEos(CMakePackage, CudaPackage):
<EMAIL_ADDRESS>when="+tests")
     depends_on("python@3:", when="+python")
     depends_on("py-pybind11@2.9.1:", when="+python")
-    #    depends_on("py-h5py", when="+tests build_extra=stellarcollapse")
     depends_on("py-sphinx", when="+doc")
<EMAIL_ADDRESS>when="+doc")
     depends_on("py-sphinx-multiversion", when="+doc")
@@ -116,9 +116,6 @@ class SingularityEos(CMakePackage, CudaPackage):
     # common MPI dependence
     for _flag in ("~mpi", "+mpi"):
         depends_on("hdf5~cxx+hl" + _flag, when=_flag)
-        depends_on("py-h5py" + _flag, when="+tests build_extra=stellarcollapse " + _flag)
-        #        depends_on("hdf5+hl" + _flag, when=_flag)
-        depends_on("py-h5py" + _flag, when=_flag)
         depends_on("kokkos-nvcc-wrapper" + _flag, when="+cuda+kokkos" + _flag)

Thanks, @annapiegra ! @dholladay00 @mauneyc-LANL does this satisfy?

Does not including h5py fix any of the issues we see with hdf5 and mpi @Yurlungur @mauneyc-LANL ? I'm curious if we could further simplify the spackage?

I don't think this simplifies hdf5 proper, which needs to be compiled as either with or without MPI.

@mauneyc-LANL I switched to FetchContent. Let me know what you think now.


A way to highlight linked stations

The idea is something like this:

Right click on a station
Click "show logistics links"
All linked stations are shown as mission guidances

Currently, I do not know how to create mission guidances like this. Suggestions are welcome.

Implemented. Special thanks to DeadAir for hinting at possible directions.


Can't right-click on objects or NPCs. Started in my POH now its the entire game.

Type
Incorrect behavior
Operating System
MacOS
OS Version
Ventura 13.5.1
Bug description
I can't right click on objects or NPCs. However, I can right click on players with no problem. I've ran runelite on safe mode and think it's definitely bugged. please let me know if you can help. Thanks!!
Screenshots or videos
No response
RuneLite version
<IP_ADDRESS>
2.6.8
Logs
No response

turn off "right click moves camera" or turn on "right click objects" in the camera plugin

That solved my problem, thank you XD


Refs #53 - User menu is bad positioned

Link to task: https://github.com/fixmydjango/fixmydjango/issues/53
Menu before login:

Menu after login:

@andersonresende Sure! gonna work on that now

Hey @andersonresende could we use the Email? Because the issue said "the name of the logged user as the button name is a bit confusing"

@andersonresende There are these two approaches:
Using name of the user:

Using the email:

What are your thoughts about them?

@alessandroHenrique just put the logout button directly in the bar

@filipeximenes @andersonresende

Hey @alessandroHenrique can you fix conflicts please?

@renatooliveira Done


StreamingClientDAO

A ClientDAO implementation that uses a BoxDAOListener to stream events on select(), as well as on listen(). Includes a few other minor fixes (see commit messages for details).

LGTM but try to get Adam's review.

TBR @adamvy


Adding a controller periodic task to clean up dead minion instances

Description
Pinot Minion is designed as a stateless service. In the Kubernetes deployment model, there is no guarantee for the minion instance id which means there could be many offline minion instances in the helix when we operate Pinot for a certain amount of time.
This PR introduces a new controller periodical task to cleanup dead pinot minion instances.
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes
If you have tagged this as either backward-incompat or release-notes,
you MUST add text here that you would like to see appear in release notes of the
next release.
If you have a series of commits adding or enabling a feature, then
add this section only in final commit that marks the feature completed.
Refer to earlier release notes to see examples of text
Documentation
If you have introduced a new feature or configuration, please add it to the documentation as well.
See https://docs.pinot.apache.org/developers/developers-and-contributors/update-document

Codecov Report

Merging #6543 (ead52b4) into master (1beaab5) will decrease coverage by 1.40%.
The diff coverage is 59.94%.

@@            Coverage Diff             @@
##           master    #6543      +/-   ##
==========================================
- Coverage   66.44%   65.04%   -1.41%
==========================================
  Files        1075     1336     +261
  Lines       54773    65843   +11070
  Branches     8168     9611    +1443
==========================================
+ Hits        36396    42825    +6429
- Misses      15700    19956    +4256
- Partials     2677     3062     +385

Flag
Coverage Δ

unittests
65.04% <59.94%> (?)

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

Impacted Files
Coverage Δ

...e/pinot/broker/api/resources/PinotBrokerDebug.java
0.00% <0.00%> (-79.32%)
:arrow_down:

...ot/broker/broker/AllowAllAccessControlFactory.java
71.42% <ø> (-28.58%)
:arrow_down:

.../helix/BrokerUserDefinedMessageHandlerFactory.java
33.96% <0.00%> (-32.71%)
:arrow_down:

...ker/routing/instanceselector/InstanceSelector.java
100.00% <ø> (ø)

...ava/org/apache/pinot/client/AbstractResultSet.java
66.66% <ø> (+9.52%)
:arrow_up:

...n/java/org/apache/pinot/client/BrokerResponse.java
100.00% <ø> (ø)

.../main/java/org/apache/pinot/client/Connection.java
35.55% <ø> (-13.29%)
:arrow_down:

...n/java/org/apache/pinot/client/ExecutionStats.java
15.55% <ø> (ø)

...inot/client/JsonAsyncHttpPinotClientTransport.java
10.90% <ø> (-51.10%)
:arrow_down:

...n/java/org/apache/pinot/client/ResultSetGroup.java
65.38% <ø> (+0.16%)
:arrow_up:

... and 1197 more

Continue to review full report at Codecov.

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


Fix: Fix all the terms without tag

Fix all the Web3 terms in the terms.json without a tag. e.g. we have some in this format is a .... or  This is a, change it to contain the terms like Bitcoin is a ...

Hey @Olanetsoft I would like to take on this task. Could you assign it to me. Thanks 😄

Done. Send Pr when it's ready... @ahmedsilat44

Hey, I have a question. Is this project also a part of hacktoberfest ? cause there is no hacktoberfest label on the issue. I am new to OSS and hacktoberfest so its a little confusing.

Hey, I have a question. Is this project also a part of hacktoberfest ? cause there is no hacktoberfest label on the issue. I am new to OSS and hacktoberfest so its a little confusing.

Yes, the project itself has the hacktoberfest label.