Unable to Obtain Thread IDs for all Zephyr k_thread_create() APIs in application

For the Native-POSIX board target, I need to obtain the thread ID for all threads in the application, both those that result from kernel configuration and ones that I create in the application using the K_THREAD_DEFINE macro.  The intention here is to be able to increase the thread priorities on a commodity Linux workstation so that the application proper has a more pronounced Linux scheduler priority.
For the main() thread, I use the pthread_self() API to obtain the handle that can successfully increase its thread priority via the standard POSIX pthread_setschedparam() API.  Using this API, this thread when viewed using the top or htop application shows a higher priority than the Zephyr default of level 20 for all threads.
This ID would presumably be the return value from k_thread_create(), which I am unable to access.
I intend to use thread callback mechanism via the k_thread_foreach() API to iterate through all application threads.
While this callback mechanism is called for each application thread, I am unaware of how to obtain the thread ID
for the Zephyr created threads that is compatible with the pthread_setschedparam() API similar to the way the pthread_self()
API works for the main application thread.
Obtaining this handle via the callback mechanism is not apparent through an API from the k_thread pointer.
My code looks something like the following, what is missing the Thread ID obtained in the callback.
I need an ID of the data type pthread_t to use with the standard Linux API pthread_setschedparam().
The use of k_thread pointer provided as an argument to the callback does not work as the thread does not have an enhanced Linux host thread priority.
k_thread_foreach(App::threadCallback, (void *)0);

void App::threadCallback(const struct k_thread *cthread, void *user_data)
{
    char state_str[32];

    struct k_thread *thread = (struct k_thread *)cthread;

    const char *tname = k_thread_name_get(thread);

    printk("%s%p %-10s\n",
           (thread == k_current_get()) ? "*" : " ",
           thread,
           tname ? tname : "NA");

    printk("\tID: %p, options: 0x%x, priority: %d timeout: %d\n",
           thread,
           thread->base.user_options,
           thread->base.prio,
           (int64_t)thread->base.timeout.dticks);

    printk("\tstate: %s, entry: %p\n",
           k_thread_state_str(thread, state_str, sizeof(state_str)),
           thread->entry.pEntry);

#ifdef CONFIG_ARCH_POSIX
    if (isRootUser())
    {
        int policy = SCHED_FIFO;
        uint32_t sched_priority = zephyr_to_posix_priority(thread->base.prio, &policy);

        pthread_t threadid = *(pthread_t *)thread;

        printk("Changing thread '%s' ID %ld Zephyr priority %d to Linux priority %d, policy %d\n", tname, threadid, thread->base.prio, sched_priority, policy);

        SysUtils::setThreadScheduleAndPriority(threadid, sched_priority, policy);
    }
#endif
}

Not a bug, but a question.
In very short: the Zephyr API does not know and cannot know anything about the fact that each Zephyr thread is mapped to an underlying Linux pthread. So there is no way to get the pthread number from a kthread structure.
In any case, what you are doing is not advisable (trying to raise priorities of a Zephyr thread in Linux), but you can do that easily when creating the process (calling the zephyr.exe executable), or later by using renice.
This seems to be a continuation of https://github.com/zephyrproject-rtos/zephyr/issues/58031#issuecomment-1570726586

Thanks for the quick.  I am using the pthread_self() API to obtain the pthread info for the main application thread as I stated, but this does not do my any good when trying to promote the dozen or so threads created within Zephyr.  We will disagree on the danger of promotion of threads into real-time behavior, this is done frequently for developments for embedded Linux and I have done so personally successfully on several project, as well as OS tuning.  I need near real-time behavior from a Zephyr port to a th Native-POSIX (Linux) board target, and am finding it does not behave real-time and appears to have OS suspensions entire which causes problems for me.  I will not near term have access to any underlying hardware, thus the necessity to get near real-time behavior when operating under standard Linux.  The reason for the "bug" category, is nobody was offering up any comments on a post that I have in June, so the "bug" category served the purpose of getting some eyes on the issue.  If what I am requiring simply can not be done, that is fine, I just wanted to pursue all possible options.  I had a suspicion from my logging that the k_thread_create ID might not correlate to an ID that the Linux operating system could use, but I needed to confirm that.


fix: removed obs flattening from executor's action selection

Closes #494
Removed implicit obs flattening from executor's action selection. It should rather be done by the networks
What?
Changed action_selection.py to not flatten observations before passing them to the neural network and instead unsqueeze them.
Why?
Because the current behavior is undesirable for grid world style environments where agents can benefit from spatial knowledge
How?
The change simply adds a batch dimension without flattening the observations

Thanks @sash-a


Arni/config/remove redundent mempool node config tests

This change is

Codecov Report
Attention: Patch coverage is 65.21739% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 60.11%. Comparing base (9ad5ef6) to head (d2893db).

Files
Patch %
Lines

crates/mempool_node/src/bin/dump_config.rs
0.00%
5 Missing :warning:

crates/mempool_node/src/config/mod.rs
0.00%
0 Missing and 2 partials :warning:

crates/gateway/src/config.rs
92.30%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #158      +/-   ##
==========================================
+ Coverage   50.86%   60.11%   +9.24%
==========================================
  Files          19       21       +2
  Lines         694      712      +18
  Branches      694      712      +18
==========================================
+ Hits          353      428      +75
+ Misses        322      260      -62
- Partials       19       24       +5

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


Error - TAGUI is reading xpath incorrectly

Hi,
I have used tagui in past and its great, but this is my first time dealing with xpath. I am using below flow but tagui is reading count of rows as zero while in the webpage there are 51 instances of the xpath (I checked xpath using chrome console result was 51)
results from tagui:

START - automation started - Tue Aug 03 2021 10:08:37 GMT+0530 (India Standard Time)
keyboard [win]r
keyboard chrome[enter]
wait 2
keyboard [ctrl][v][enter]
wait 2
dump date, community, propertyname, type, bedroom, price, sqft, pricesqft to drivenproperties.csv
'row counted' 0

flow used:

keyboard [win]r
keyboard chrome[enter]
clipboard('https://www.drivenproperties.com/dubai-real-estate-market-guide/rental-transactions')
wait 2
keyboard [ctrl][v][enter]
wait 2
dump date drivenproperties.csv
rows = count('//[@id="dpx-sold-grid-tbody"]/tr')
echo 'row counted' rows
for row from 1 to rows
echo 'for started ' row
read //[@id="dpx-sold-grid-tbody"]/tr[row]/td[2] to date

Hi @munishmarya
Instead of using keyboard and clipboard to access the webpage you can directly access it with the url as a step
https://www.drivenproperties.com/dubai-real-estate-market-guide/rental-transactions
rows = count('//*[@id="dpx-sold-grid-tbody"]/tr')
echo `rows`

doing this in live mode gives the result of 51 for me:

thanks for reply Ruth. I tried again following your approach but same results for me 0 rows, i am saving the flow and using cmd to run. I have used the latest package from https://tagui.readthedocs.io/en/latest/setup.html
flow results

https://www.drivenproperties.com/dubai-real-estate-market-guide/rental-transactions
rows = count('//*[@id="dpx-sold-grid-tbody"]/tr')
echo rows
flow used:
_________________ https://www.drivenproperties.com/dubai-real-estate-market-guide/rental-transactions rows = count('//*[@id="dpx-sold-grid-tbody"]/tr') echo rows

Hi @munishmarya, seems like the webpage takes a while to load. You may add a wait step in between like this for a 3 second delay:
https://www.drivenproperties.com/dubai-real-estate-market-guide/rental-transactions
wait 3
rows = count('//*[@id="dpx-sold-grid-tbody"]/tr')
echo `rows`

see diff after adding the wait step:

@ruthtxh thanks, you are a star. It worked after wait time, have patience let it load :)

@munishmarya no problem, feel free to ask anything!

Adding on, if you prefer a dynamic wait time instead of a fixed time, you can use hover step on some element that only shows up when the page is fully loaded. That way, once the page is done loading, automation proceeds.


Add range comment to pool decls in dhcpd_compare2 output

Resolves issue #988.
This one's pretty trivial. The diff used to look like this:
 subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
      pool {
          +option routers <IP_ADDRESS>;
      }
 }
 subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
     -pool {
     -    range <IP_ADDRESS> <IP_ADDRESS>;
     -}
 }

Now it looks like this:
 subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
      pool { # <IP_ADDRESS> - <IP_ADDRESS>
          +option routers <IP_ADDRESS>;
      }
 }
 subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
     -pool { # <IP_ADDRESS> - <IP_ADDRESS>
     -    range <IP_ADDRESS> <IP_ADDRESS>;
     -}
 }

It's redundant for pools that are added or removed, because you can see the range parameter itself, but it still might help if a pool has more than a few lines.

I don't use dhcpd_compare so I don't know the basis for wanting this (the attached issue doesn't say anything about it either) but this is such a minor change that I don't intend to hold it up.


Output indices_distances of _py4dgeo.region_growing not equal to distances of neighborhood_similarity

Issue:
In segmentation.py the dtw distances within the initial neighborhood (similarities) computed in the seed_sorting_scorefunction should be equal to the objdata.indices_distances of the initial neighborhood points computed in _py4dgeo.region_growing.
This is not the case. The objdata.indices_distances for the neighborhood points are all zero instead and the distances outside the neighborhood seem too small.
Solution:
The problem could be that in the c++ function ( _py4dgeo.region_growing) the dtw distance computation is done using the full time series instead of only the temporal segment between end_epoch and start_epoch of seed. This could explain why the DTW distances in similarities and objdata.indices_distances are not equal, but this does not necessarily explain why the  objdata.indices_distances in the neighborhood are zero.

I read a bit through the code and indeed it seems like something seems wrong. I am unsure whether it is a regression that I introduced at some point or whether it always has been wrong (which would be slightly embarrassing). On the bright side: I am fairly confident that the problem is on the Python side. The restriction to (start_epoch, end_epoch) is not done on the C++ side, instead the required restriction is done on the calling Python side by slicing numpy arrays:

Correct slicing:

https://github.com/3dgeo-heidelberg/py4dgeo/blob/main/src/py4dgeo/segmentation.py#L943
https://github.com/3dgeo-heidelberg/py4dgeo/blob/main/src/py4dgeo/segmentation.py#L1047
https://github.com/3dgeo-heidelberg/py4dgeo/blob/main/src/py4dgeo/segmentation.py#L1058

Notable absence of such slicing:

https://github.com/3dgeo-heidelberg/py4dgeo/blob/main/src/py4dgeo/segmentation.py#L744
There might be more issues, but this immediately seemed dubious to me.

I tested the additional slicing in https://github.com/3dgeo-heidelberg/py4dgeo/blob/main/src/py4dgeo/segmentation.py#L744. When added, the program crashed at this function, though.
Without the slicing, we would expect the distances between compared time series to be larger (than in the limited timespan). So I assume the return of 0.0 is a separate issue.


Adding on Upward arrow for scrolling to the top Feature

This feature is a must as after scrolling to the bottom we usually find it lethargic to scroll to the top  so i can work on that for the same case
So basically there will be an arrow and on clicking it we will just reach at the top of the webpage

@Aditya-edith work on it

@Aditya-edith are you still working on this issue? if not, I'm interested in working on this feature.

@sameer-soni this issue is already solved


Fix example

Current coverage is 68.32% (diff: 100%)

Merging #20 into master will not change coverage

@@             master        #20   diff @@
==========================================
  Files             8          8
  Lines           281        281
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            192        192
  Misses           89         89
  Partials          0          0

Powered by Codecov. Last update 80bfc24...5091537

Thanks :)


Failed to list the current directory

I recently setup my upspin server on Google compute engine, following the upspin docs.
And I thought I should try the GUI to upspin. But when I input my email (upspin username), it says
Failed to list the current directory

What OS? Is this on the same machine containing your upspin config and keys?
On Sat, Sep 9, 2017 at 12:37 PM Khushmeet Singh<EMAIL_ADDRESS>wrote:

I recently setup my upspin server on Google compute engine, following the
upspin docs.
And I thought I should try the GUI to upspin. But when I input my email
(upspin username), it says
Failed to list the current directory
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jnglco/browser/issues/16, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAx1lML03qWutmcyjzcyw1KNPnMKEdqLks5sgniVgaJpZM4PR-EH
.

I am running macOS sierra(latest). My config and keys are stored in it. My store and dir server are running on Google compute engine.

Okay. I’m unfortunately traveling where I don’t have much access to the
internet until later in the month. I’ll have to debug when I return. Thanks
for opening the issue.
On Sun, Sep 10, 2017 at 4:40 PM Khushmeet Singh<EMAIL_ADDRESS>wrote:

I am running macOS sierra(latest). My config and keys are stored in it. My
store and dir server are running on Google compute engine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jnglco/browser/issues/16#issuecomment-328350810, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAx1lAa3HnWoZ3F9CkNXsYLv4gBshotjks5shAL4gaJpZM4PR-EH
.


[pre] Update to 4.0.0-rc.1 causes failed plugin loads

When updating to the new version, the following tracebacks appeared on the console:
reloading plugin SublimeLinter.commands
Traceback (most recent call last):
  File "/opt/sublime_text/sublime_plugin.py", line 109, in reload_plugin
    m = importlib.import_module(modulename)
  File "./python3.3/importlib/__init__.py", line 90, in import_module
  File "<frozen importlib._bootstrap>", line 1584, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1565, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1529, in _find_and_load_unlocked
ImportError: No module named 'SublimeLinter.commands'
reloading plugin SublimeLinter.sublime_linter
Traceback (most recent call last):
  File "/opt/sublime_text/sublime_plugin.py", line 109, in reload_plugin
    m = importlib.import_module(modulename)
  File "./python3.3/importlib/__init__.py", line 90, in import_module
  File "<frozen importlib._bootstrap>", line 1584, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1565, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1529, in _find_and_load_unlocked
ImportError: No module named 'SublimeLinter.sublime_linter'

As a result, the plugin seemingly wasn't initialized correctly and also failed to backup the User settings file.
I'm not sure what this means exactly, but you might have to do something similar to what I do with PackageDev here: https://github.com/SublimeText/PackageDev/blob/master/main.py. Not sure if that actually works, though, since I never tested the update process.
Note that the plugin loads after an ST restart.
I'm on Arch and 3156.

Not sure what all this means or what can be done about it. I made sure you don't get swamped in popups, and the upgrade message now tells you to restart (which isn't unreasonable for an upgrade). I don't know if it's worth the effort to try and make the upgrade cleaner than that given how the runtime is really just one big ball of python.


Save auto-detected markup type in Page.Markup

Always make markup type available to the theme.
Solution to issue #1950.
Also renames guessMarkupType() to determineMarkupType() as discussed in #1950.

 You should sign our Contributor License Agreement in order to get your pull request merged.

Can you squash the commits into one -- and add a commit message in line with the guidelines in the README.

There is a breaking shortcode test. I don't understand why it's broken ...

Sorry that code was plain wrong. Thank you @moorereason for your analysis!
New solution builds on the fact that helpers.GuessType(p.Markup) returns 'unknown' as a default (i.e. also on empty input string).

Merged in 9f6b5ad. Thanks.


build: add root to the sourcemap so it won't use absolute pathing

Fixes #541, I think.
Need to chat to @jelbourn to try to see if that fixes the release itself, not jsut local.

This requires the latest release of the CLI to get https://github.com/angular/angular-cli/pull/1028.

Also fix prod builds, which actually don't make sense for material, but :shrug:

I'm going to merge this since it's better, but I think the sourceMaps still aren't exactly right.

Thanks guys. We hit this yesterday.


swift 3.1

Autoupdated syntax and manually changed something with dispatch queue

Safely dissmiss notification even if there's none displayed

Anyone here


Add dependency to package.json

Also—this project isn't on npm? I had to install it by linking here...

You're right, it's not on npm, I hadn't quite finished with it, then forgot about it. Thanks for the pull, I will publish a version to npm.

Ok, available on npm.

Awesome, thank you!


Accuracy formula in the doc doesn't match the one coded

In the Accuracy documentation, the formula presented is not the one coded in the class.
https://github.com/RubixML/ML/blob/master/src/CrossValidation/Metrics/Accuracy.php

Good observation. The Accuracy Metric scores the predictions differently that the accuracy value computed from the confusion matrix (in Multiclass Breakdown). Specifically the Metric only considers true positives while the Report does. I tried to show the differences by providing the formulas in the docs but they could be wrong - I don't consider myself a mathematician. Perhaps someone with more experience can give it a look.


You can unanchor a syndicate bomb by removing the floor tile under it

Description
Using a RCD or fireaxe to remove the floor tile under a syndicate bomb allows you to drag it away without having to cut any wires.
Intended?
Reproduction

arm a syndicate bomb.
use RCD or fireaxe to remove the floor tile under it
drag the bomb away.

Resolved as of #22799


Use File.open instead File.write because on Windows it don't create file

https://ci.appveyor.com/project/ribose/relaton/builds/23278193/job/cctd7mk4vhbj28wu
https://github.com/metanorma/relaton/issues/27

ping @opoudjis for this fix.

Ticket title is misleading: code fix is to use not Unix glob in removing directories (FileUtils.rm_rf dir + '/.'), but generic glob (FileUtils.rm_rf(Dir.glob(dir + '/*'), secure: true)).
That's @andrew2net 's code; for my part, I always do just FileUtils.rm_rf dir, and Appveyor seems not to have had a problem with that.


Fix warning when running tests

Bug/issue #103533986, if applicable:
Summary
Fix warning: [Vue warn]: Invalid prop: type check failed for prop "type". Expected String, got Undefined
when running tests
Dependencies
NA
Testing
Steps:

Run npm run test
Assert no warnings are displayed

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

@swift-ci test


Light_yellow not a color?

Describe the bug
CVE_Prioritizer/scripts/helpers.py has an error:
KeyError: 'light_yellow'
To Reproduce
As root and regular user run python script.
CVE_Prioritizer# python3 cve_prioritizer.py -l CVE-2023-2825
Expected behavior
some kind of rating with light_yellow
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

/home/ubunuser/CVE_Prioritizer# cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04.5 LTS (Focal Fossa)"

Screenshots

Extra context:
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File "/usr/lib/python3.8/threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "/home/ubunuser/CVE_Prioritizer/scripts/helpers.py", line 149, in worker
print_and_write(working_file, cve_id, 'Priority 2', epss_result.get('epss'),
File "/home/ubunuser/CVE_Prioritizer/scripts/helpers.py", line 119, in print_and_write
color_priority = colored_print(priority)
File "/home/ubunuser/CVE_Prioritizer/scripts/helpers.py", line 109, in colored_print
return colored(priority, 'light_yellow')
File "/usr/local/lib/python3.8/dist-packages/termcolor/termcolor.py", line 147, in colored
text = fmt_str % (COLORS[color], text)
KeyError: 'light_yellow'

I changed it to magenta in the helpers.py
def colored_print(priority):
if priority == 'Priority 1+':
return colored(priority, 'red')
elif priority == 'Priority 1':
return colored(priority, 'light_red')
elif priority == 'Priority 2':
return colored(priority, 'magenta)
AND that worked


slack import: Use a slightly different version of check_subdomain_available

When doing an import on the command line with manage.py import, we probably don't want to enforce reserved subdomains (etc.).  I think this is probably could be best implemented as an option to check_subdomain_available with a name like from_management_command (since that would clearly encode where the extra permission is coming from); that should help protect against the option being accidentally migrated to when we allow import on the web.

@zulipbot  claim
I can take a look at this. Are the extremal dash and bad character checks in that function just stylistic? Or should we enforce those here as well?

I think I disagree with this.  We may want a command-line flag to override, but the default should not be that a human can't be convinced to create a subdomain with a reserved name by social engineering.
This also removed the check that errors nicely if the domain already exists, by short-circuiting returning before doing that check.

Hmm, yeah, I think in retrospect the right algorithm is probably that we should offer a flag --allow-reserved-subdomain

Opened https://github.com/zulip/zulip/issues/16924 to track an improved version..


tests(openshift/nightly): Change demo and cleanup

Signed-off-by: nshankar13<EMAIL_ADDRESS>
Description: Edit demo for openshift nightly job and add cleanup step.

Affected area:

Functional Area

New Functionality
[ ]

Documentation
[ ]

Install
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Ingress
[ ]

Egress
[ ]

Networking
[ ]

Observability
[ ]

SMI Policy
[ ]

Sidecar Injection
[ ]

Security
[ ]

Upgrade
[ ]

Tests
[X]

CI System
[X]

Demo
[X]

Performance
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? No

Did you notify the maintainers and provide attribution?

Is this a breaking change? No

Codecov Report

Merging #3872 (e0c8306) into main (748ff4b) will increase coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3872      +/-   ##
==========================================
+ Coverage   67.72%   67.83%   +0.10%
==========================================
  Files         199      199
  Lines       11076    11076
==========================================
+ Hits         7501     7513      +12
+ Misses       3525     3514      -11
+ Partials       50       49       -1

Flag
Coverage Δ

unittests
67.83% <ø> (+0.10%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/providers/kube/client.go
65.00% <0.00%> (-0.63%)
:arrow_down:

pkg/crdconversion/crdconversion.go
75.00% <0.00%> (+14.13%)
:arrow_up:

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 748ff4b...e0c8306. Read the comment docs.

You should also modify the create-container-registry-creds.sh script to use the USE_PRIVATE_REGISTRY variable to determine whether or not to create the secret.
The script is currently doing that in a roundabout way by checking to see if the password is empty: https://github.com/openservicemesh/osm/blob/main/scripts/create-container-registry-creds.sh#L16

You should also modify the create-container-registry-creds.sh script to use the USE_PRIVATE_REGISTRY variable to determine whether or not to create the secret.

@ksubrmnn I think it would make sense to do this in a different PR because multiple files use the create-container-registry-creds.sh file, and we would need to create the USE_PRIVATE_REGISTRY variables in these scripts as well.

You should also modify the create-container-registry-creds.sh script to use the USE_PRIVATE_REGISTRY variable to determine whether or not to create the secret.

@ksubrmnn I think it would make sense to do this in a different PR because multiple files use the create-container-registry-creds.sh file, and we would need to create the USE_PRIVATE_REGISTRY variables in these scripts as well.

Sure whatever is easiest for you.


[5.7] Fix wrong class being used when eager loading nullable MorphTo with withDefault()

Fixes #27369
Currently, MorphTo inherits the newRelatedInstanceFor() method from BelongsTo, which ignores the passed $parent argument and instantiates a related model based on the generalized relationship instance (without constraints) produced by the query builder . However, in the case of MorphTo relations, the no-constraint relationship instance does not know anything about related classes, and wrongly defaults to the parent model class. This commit simply makes use of the $parent argument to produce a related instance that is appropriate for the parent model instance.

I am closing this pull request because it lacks sufficient tests. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests.
Thanks!


Move gemini rating logic to server

Adds a Cloud function in /functions dir that serves Gemini powered ratings.
Removes gemini logic from the frontend

@devnook this PR has conflicts and can't be merge. Is it still going to be updated or should it be closed?


Enforce non-empty values for required keys

Asset validation for new_assets and modify_assets operations now fail,
when a required key has an empty value.
Closes #481

Note, that coverage is to be addressed by #501


Add first version of gardener integration tests

What this PR does / why we need it:
Added a basic gardener integration test that tests the following flow:

get a gke cluster
setup gardener with the latest release version e.g 0.26.2 with its corresponding garden-setup version e.g. 1.6.0
create 3 shoots (aws, gcp and azure)
update gardener to the newest master version with the master garden-setup version
wait until all shoots of the gardener are successfully reconciled
6a. test the previously created shoot with all gardener integration tests
6b. delete the previously created shoots
7a. create 3 new shoots (aws, gcp and azure)
7b. test the new created shoots with all gardener tests
7c. delete the newly created clusters.
delete the gardener
release the GKE cluster

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
e2e tests are currently skipped as of their time consuming execution time
Release note:

Added gardener integration tests to test a full gardener lifecycle with a specific commit.

@schrodit Is this still WIP?

It's not WIP anymore but I want to wait for all reviews

can be merged


[SPARK-13603] [SQL] support SQL generation for subquery

What changes were proposed in this pull request?
This is support SQL generation for subquery expressions, which will be replaced to a SubqueryHolder inside SQLBuilder recursively.
How was this patch tested?
Added unit tests.

Test build #52264 has started for PR 11453 at commit 9876ffc.

Test build #52264 has finished for PR 11453 at commit 9876ffc.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class SubqueryHolder(query: String) extends LeafExpression with Unevaluable

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52264/
Test FAILed.

Test build #52267 has started for PR 11453 at commit efa545f.

Test build #52267 has finished for PR 11453 at commit efa545f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52267/
Test PASSed.

cc @liancheng

Test build #52443 has started for PR 11453 at commit 5fbc714.

Test build #52443 has finished for PR 11453 at commit 5fbc714.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52443/
Test PASSed.

Merged build finished. Test PASSed.

LGTM, merged to master. Thanks!


discourage excessive changesets

I'm seeing users Save after every single edit. For example, http://www.openstreetmap.org/user/Rps333/edits?page=1
Two ideas

Keep changesets open for a time, even after uploading changes. I think potlatch2 used to do this.
A help dialog to suggest to users that they can save less often.

Bump bump bump. I have people in Poland that click 'save' every single building. They clearly don't understand how it works.
It's nuts. A user with 1200 edits not even a week old. Good luck at reverting this pile of crap:
https://www.openstreetmap.org/changeset/32310453#map=18/51.20153/19.49770

#2251 is the issue for keeping a changeset open if the user is making a bunch of similar changes over a short period of time, and we agree it's worth implementing.

It's nuts. A user with 1200 edits not even a week old.  Or using OSMHV to check someone's edits thoroughly, as it works on per-changeset basis.

Why would anyone need to revert all those edits?  Couldn't a power user with JOSM just select them all and remove the unnecessary name tags in a few seconds?  It's not vandalism.  Just send the user a nice message telling them thank you and they don't need to write "single family home" in the name field for each building.


feature: provide access to different APIs of the same compute cluster

Feature Description
I would like to have two personas that need to sync resources to a compute cluster part of our service offering:

administrators that deploy infrastructure components: they will use resources as deployments.apps,services,ingresses.networking.k8s.io
end users that mainly need to create Tekton resources. They should not consume compute resources in other ways, i.e. deployments.

Both administrators and end users have their own workspaces, where they are allowed to bind the apiexport of the compute cluster.
Preventing end users to create deployments in their workspace could be addressed by quotas. This is however not ideal to expose an API in their workspace that they cannot leverage, to block them to use this API through a binding of an apiexport for their own compute.
Proposed Solution
Currently I am considering creating two SyncTargets in two separate workspaces (as the APIExport is always called kubernetes) and allow:

adminstrator to bind the one with deployments.apps,services,ingresses.networking.k8s.io
end users to bind the one with Tekton resources

Challenge:
kcp sync command generate manifests where the following resources: deployments.apps", "secrets", "configmaps", "serviceaccounts are always synced. These manifests can be amended before they get applied to the compute cluster but:

is it future proof? The way resources to sync are specified may change
secrets and serviceaccounts are used to make possible for workload to talk to the kcp API (this is currently not a concern for us). In the first phase the Tekton controllers talk to the compute cluster API.

Alternative Solutions

Use quotas
Make possible to limit (through RBAC?) the resources that are available for binding to a SyncTarget: all resources deployment + Tekton get synced by the same syncer but administrators can "bind" all of them whereas end users only bind a subset.

Want to contribute?

[ ] I would like to work on this issue.

Additional Context
Discussion in this slack thread: https://kubernetes.slack.com/archives/C021U8WSAFK/p1663666920366639

kcp sync command generate manifests where the following resources: deployments.apps", "secrets", "configmaps", "serviceaccounts are always synced.

This definitely has to change. @qiujian16 has a PR up or even merged that makes the syncer automatically get the resources from the supported APIExport. I would guess with that merged, you should have a solution, shouldn't you?
Plus the topic about selective configmap/secret/serviceaccount syncing is necessary. With those two you should have Tekton-only syncing.

Plus the topic about selective configmap/secret/serviceaccount syncing is necessary. With those two you should have Tekton-only syncing.

There's an issue already about this topic: https://github.com/kcp-dev/kcp/issues/727

Is this feature still relevant, or is it made obsolete by #1895?

I think it relates to PR https://github.com/kcp-dev/kcp/pull/2164
After it is merged. It could work like this:
You can have two workspaces
location workspace: where synctarget resides, and administrator can access
user workspace: where user can access

admin run the sync command kubectl kcp sync --apiexports root:compute|kubernetes, tekton-workspace|tekton-apiexport to start the syncer.
admin can create deployment in location workspace.
user creates tekton APIBinding in user workspace.
user creates a placement point to the location workspace.
user can create tekton resource in user workspace.

Does the above flow makes sense?

/transfer-issue contrib-tmc


Closing old issues

To me current issues are unpleasant because:

old not closed issues messes the true issues of the project
many opened issues informs that user input is not really welcome (at first glance)
issues are not properly tagged - this would inform contributors what they can improve

Regards

True, that's a good point. I'll cruise through and close some of these!
(and as a meta thing, will close this one once kicking off the audit)

True, that's a good point. I'll cruise through and close some of these!
(and as a meta thing, will close this one once kicking off the audit)

Now just 2 issues are open!

Now just 2 issues are open!


Create a CAP Business Service with Node.js Using Visual Studio Code

Tutorials: https://developers.sap.com/tutorials/cp-apm-nodejs-create-service.html
Write here how you think we can improve the tutorial ...
Please update the respective libs:
(base) d055092@C02XT13BJGH6 ~ % npm i -g @sap/cds-dk
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to @mapbox/node-pre-gyp: the non-scoped node-pre-gyp package is deprecated and only the @mapbox scoped package will recieve updates in the future
npm WARN deprecated<EMAIL_ADDRESS>This version of tar is no longer supported, and will not receive security updates. Please upgrade asap.

Hi @VolkerSaggau,
Thank you for this feedback. The dev colleagues are already working on it. It should be fixed asap.
Best regards,
Iwona


Maven artifacts with dummy source - may compromise debugging

As recently noted in https://github.com/odpi/egeria/issues/1989, for egeria a change was made (by me!) before we published to maven central so that source files were also included.
For regular jars this is fine, but there were some modules where there was no source explicitly included since the modules just offered an aggregate jar composed from other modules. This is the case in #1989
When we look at a jar such as in the atlas connector above (accepting this isn't base egeria), there is code included in the jar. So a developer of a third party app may depend on this jar.
When debugging their app, the debugger (like that within intellij) will go and try and retrieve the source to aid in debugging. Retriving a empty jar file - whilst it gets past maven central checks - isn't entirely useful.
This isn't a problem with a 'package' module is simply a packaging jar with dependencies, but it IS an issue where there is a jar provided within the package that one might expect to be able to debug.
It's probably worth disabling the auto-zero-length docs creating and re-evaluating which modules this applies to to either figure out

how to add the correct source
redefine the module as a simple packaging pom rather than a jar module

Also to test with the debugger & an external app
cc: @cmgrote

will investigate for 1.3

Am going to close this. We are in the process of stripping down our jars to only unique dependencies - or rather without including all the platform jars. Instead it will be assembled via packaging. This should remove most manifestations of this issue.

Am going to close this. We are in the process of stripping down our jars to only unique dependencies - or rather without including all the platform jars. Instead it will be assembled via packaging. This should remove most manifestations of this issue.


Unicode support?

I'm trying to send messages with Chinese characters but can't seem to get it to work. Is it not supported at the moment?

Hello,
are you using python 2 ?
telepot has no problem with utf-8 characters , see issue #149
you most likely have problems with utf support in python 2, try putting a u before your strings, so instead of '何' , write u'何', or even better , upgrade to python 3 which has native unicode support and all strings are utf-8 by default.

Yes I'm using Python 2 as I'm trying to modify some script which was written on that version. I tried the u method but was still getting errors.

can you please post the error messages and the line numbers that caused the error?
without more information, we can't really help you.

This is what I got when I tried sendMessage with just one Chinese character:
Traceback (most recent call last):
File "C:\Python27\lib\threading.py", line 801, in _bootstrap_inner
self.run()
File "main0.py", line 1397, in run
telebot.sendMessage(chat_id=telegram, text= u'\u5751', parse_mode= 'HTML',disable_web_page_preview='False',disable_notification='False')
File "C:\Python27\lib\site-packages\telepot_init.py", line 409, in sendMessage
return self._api_request('sendMessage', rectify(p))
File "C:\Python27\lib\site-packages\telepot_init.py", line 398, in _api_request
return api.request((self._token, method, params, files), **kwargs)
File "C:\Python27\lib\site-packages\telepot\api.py", line 130, in request
r = fn(*args, **kwargs)  # fn must be thread-safe
File "C:\Python27\lib\site-packages\urllib3\request.py", line 148, in request_encode_body
return self.urlopen(method, url, **extra_kw)
File "C:\Python27\lib\site-packages\urllib3\poolmanager.py", line 244, in urlopen
response = conn.urlopen(method, u.request_uri, **kw)
File "C:\Python27\lib\site-packages\urllib3\connectionpool.py", line 594, in urlopen
chunked=chunked)
File "C:\Python27\lib\site-packages\urllib3\connectionpool.py", line 361, in _make_request
conn.request(method, url, **httplib_request_kw)
File "C:\Python27\lib\httplib.py", line 1057, in request
self._send_request(method, url, body, headers)
File "C:\Python27\lib\httplib.py", line 1097, in _send_request
self.endheaders(body)
File "C:\Python27\lib\httplib.py", line 1053, in endheaders
self._send_output(message_body)
File "C:\Python27\lib\httplib.py", line 895, in _send_output
msg += message_body
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe5 in position 83: ordinal not in range(128)

Can you just try it like this (after substituting the token and user id):
import telepot
bot = telepot.Bot(TOKEN)
bot.sendMessage(999999999, u'我愛保齡球')

And see what happens?

Hey that actually worked! Now I need to find out why it didn't work in the other program...

FYI: with Python2, somehow the api_key shouldn't be unicode:
bot = telepot.Bot(bot_model.api_key) # bot_model.api_key gives unicode
bot.sendMessage(r.recipient_id, u"Авторизация успешна")

>> 'ascii' codec can't decode byte 0xd0 in position 83: ordinal not in range(128)

Yet this works:
bot = telepot.Bot(bot_model.api_key.encode('utf-8'))
bot.sendMessage(r.recipient_id, u"Авторизация успешна")


CPDF constructors and Conv

Currently the constructors of the CPDFs only work with mappings that get vectors or matrices as input (mapping(x::AbstractMatrix)::AbstractMatrix same for vector)
This is due to the detect_mapping_variant function that tries to infer which variance type to use. It would be quite tedious to make it work for higher dimension input (as is needed for convs). Should we get rid of this automatic detection?
The specific constructors like CGaussian{Float32,UnitVar} of course till work, as the are not calling detect_mapping_variant.

duplicate of #39


Document installation via MR Feature Tool.

Wrote up one main page on MR Feature Tool and WLT,
then added a lot of links to that, everywhere it says how to install WLT without MR Feature Tool.

/azp run wlt_docs


[Feature Request] ownCloud/nextCloud Mail App add multiple mail accounts in cloud user settings

Hello,
after my first tries with the app, I really like it. Great job, I think about enabling it for all cloud users, because it brings a real benefit.
One question/feature request: Is it possible to enter multiple mail accounts as autologin in the cloud personal settings page?
Best regards,
Plastikschnitzer

Not needed ;-)
Found out its already existing, but not the way I expected it:

Add one mail account to your cloud settings page.
Then go to the rainloop mail app and add all other mail accounts there. The app will save the login credentials so you will have all mail accounts available, they do not need to be added to the cloud settings page.


⚠ ClamAV has degraded performance

In 883ea07, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 5404988.


Update README.md

I propose the change from wss://wss.hyperspace.node.glif.io/apigw/lotus/rpc/v1 to wss://wss.hyperspace.node.glif.io/apigw/lotus.
When running a lotus lite node it only works with the latter  FULLNODE_API_INFO=wss://wss.hyperspace.node.glif.io/apigw/lotus lotus daemon --lite
In this issue (https://github.com/filecoin-project/lotus/issues/10099) you can see that this is the intended wss that should be used when running a lotus lite node.
Please correct me if I should have changed it elsewhere. I can also revert my changes and add a new bullet point to indicate that when using a GLIF node to run lotus lite one should use the wss without the rpc/v1 part:
- **RPC - Public Endpoints**:
  - These endpoints are limited to all read-only Filecoin JSON RPC API calls + `MPoolPush` (for sending already signed messages)
  - **Glif Nodes RPC:**
    - https://api.hyperspace.node.glif.io/rpc/v1
    - web socket endpoint: wss://wss.hyperspace.node.glif.io/apigw/lotus/rpc/v1
    - *NOTE:* When using the web socket endpoint to run a lite node please use `wss://wss.hyperspace.node.glif.io/apigw/lotus`

More details on the Filecoin's slack  - https://filecoinproject.slack.com/archives/C04JEJB82RY/p1674398487957749

Thanks for reviewing @Reiers! I’m afraid I don’t have write permissions, could you merge please?

cc: @eshon - I don't have write permission here !

Others might use the WSS for eth filters and subscriptions though, which requires the extra v1 endpoint path.
So let me revise this to add a note about Lotus lite underneath.
Thanks for the flag! And sorry review took so long.


Push notifications

Add support for push notifications to spread updates to users about our other apps or in case of a bug send a warning to users

^Is this a "please update to version X.X.X for best performance" message? I think telling a user to download our other apps is against an apple policy by we could have a link to our apps in the about section.

content of push is independent of this feature, we need support for pushing any arbitrary message.


Not working after upgrading to 1.0.1.

I upgraded Octopress from 3.0.0 to 3.0.6, ink from 1.0.0.rc to 1.0, and littlefoot from 1.0.0 to 1.0.1.  And now littlefoot doesn't work.  The footnote still appears at the bottom of the post, and the reference is a link to the footnote (as if littlefoot isn't installed at all).  Using my meager JavaScript skills, I was able to confirm that the event listener is added to the footnote reference.
I wiped out the various caches and the generated site itself.  I confirmed that I'm loading the JavaScript and CSS in the head.
I'm happy to help debug this, but I don't know what to try next.

Sorry for your troubles! I'll take a look at this tonight.

Thanks so much for posting this! I found a fix. In an update to Octopress Ink I changed the default behavior for Javascript to be loaded asynchronously, this means that the DOMContentLoaded event may be fired before the javascript is fully downloaded. So I added a check to see if the document.readyState was "interactive", in which case I setup LittleFoot immediately.

I've just released 1.0.2 which should fix this for you. Thanks again!

That worked.  Thanks!


Tailoring: Error when filtering to previous expansion patterns

Message: Interface/AddOns/CraftSim/Data/Classes/NodeData.lua:31: attempt to index local 'nodeInfo' (a nil value)
Time: Thu Mar  9 12:52:31 2023
Count: 1
Stack: Interface/AddOns/CraftSim/Data/Classes/NodeData.lua:31: attempt to index local 'nodeInfo' (a nil value)
[string "=[C]"]: ?
[string "@Interface/AddOns/CraftSim/Data/Classes/NodeData.lua"]:31: in function `new'
[string "@Interface/AddOns/CraftSim/Libs/classic.lua"]:66: in function `NodeData'
[string "@Interface/AddOns/CraftSim/Data/Classes/SpecializationData.lua"]:49: in function `parseNode'
[string "@Interface/AddOns/CraftSim/Data/Classes/SpecializationData.lua"]:75: in function `new'
[string "@Interface/AddOns/CraftSim/Libs/classic.lua"]:66: in function `SpecializationData'
[string "@Interface/AddOns/CraftSim/Data/Classes/RecipeData.lua"]:71: in function `new'
[string "@Interface/AddOns/CraftSim/Libs/classic.lua"]:66: in function `RecipeData'
[string "@Interface/AddOns/CraftSim/Main/CraftSim.lua"]:529: in function `TriggerModulesByRecipeType'
[string "@Interface/AddOns/CraftSim/Main/CraftSim.lua"]:189: in function `TriggerModulesErrorSafe'
[string "@Interface/AddOns/CraftSim/Main/CraftSim.lua"]:234: in function <Interface/AddOns/CraftSim/Main/CraftSim.lua:207>
[string "=[C]"]: ?
...
[string "@Interface/AddOns/Blizzard_Professions/Blizzard_ProfessionsFrame.lua"]:155: in function `SetProfessionInfo'
[string "@Interface/AddOns/Blizzard_Professions/Blizzard_ProfessionsFrame.lua"]:55: in function <...s/Blizzard_Professions/Blizzard_ProfessionsFrame.lua:53>
[string "=[C]"]: ?
[string "@Interface/SharedXML/CallbackRegistry.lua"]:143: in function <Interface/SharedXML/CallbackRegistry.lua:142>
[string "=[C]"]: ?
[string "@Interface/SharedXML/CallbackRegistry.lua"]:146: in function `TriggerEvent'
[string "@Interface/AddOns/Blizzard_ProfessionsTemplates/Blizzard_Professions.lua"]:1139: in function `set'
[string "@Interface/FrameXML/FilterDropDown.lua"]:103: in function `func'
[string "@Interface/SharedXML/UIDropDownMenu.lua"]:1007: in function `UIDropDownMenuButton_OnClick'
[string "*UIDropDownMenuTemplates.xml:136_OnClick"]:1: in function <[string "*UIDropDownMenuTemplates.xml:136_OnClick"]:1>

Locals:

Im sorry but I cannot reproduce it, when I am on my tailor and filter to show shadowlands or classic recipes it just works ^^
are you on CraftSim 8.0+?
You can try to delete the savedVariables wtf file (addon settings, also doable with curseforge manager) while logged out and then log in again

Updated from 8.1 to 8.2 and removed settings, same issue still.

I just tested it out of interest and I'm getting the same error when filtering to a previous expansion. Did a bit of testing and I think I understand what's causing it and how to reproduce it:

Open any profession, does not have to be Tailoring specifically
Set one or multiple dragonflight recipes as favorite
Switch to a different expansion in the filter options
Error pops up

Also interesting:

select a non-dragonflight recipe -> no error
select one of the favorited dragonflight recipes -> error pops up again
if a non-dragonflight recipes is at the top of your favorites list and you filter to a different expansion you don't get the error

Conclusion is that selecting a dragonflight recipe while not having the filter set to dragonflight is what causes the error, maybe because some things get unloaded when switching to older expansions?
21x CraftSim/Data/Classes/NodeData.lua:31: attempt to index local 'nodeInfo' (a nil value)
[string "@CraftSim/Data/Classes/NodeData.lua"]:31: in function `new'
[string "@CraftSim/Libs/classic.lua"]:66: in function `NodeData'
[string "@CraftSim/Data/Classes/SpecializationData.lua"]:49: in function `parseNode'
[string "@CraftSim/Data/Classes/SpecializationData.lua"]:75: in function `new'
[string "@CraftSim/Libs/classic.lua"]:66: in function `SpecializationData'
[string "@CraftSim/Data/Classes/RecipeData.lua"]:71: in function `new'
[string "@CraftSim/Libs/classic.lua"]:66: in function `RecipeData'
[string "@CraftSim/Main/CraftSim.lua"]:523: in function `TriggerModulesByRecipeType'
[string "@CraftSim/Main/CraftSim.lua"]:189: in function `TriggerModulesErrorSafe'
[string "@CraftSim/Main/CraftSim.lua"]:234: in function <CraftSim/Main/CraftSim.lua:207>
[string "=[C]"]: ?
[string "=[C]"]: in function `Init'
...
[string "@Blizzard_Professions/Blizzard_ProfessionsFrame.lua"]:155: in function `SetProfessionInfo'
[string "@Blizzard_Professions/Blizzard_ProfessionsFrame.lua"]:55: in function <...s/Blizzard_Professions/Blizzard_ProfessionsFrame.lua:53>
[string "=[C]"]: ?
[string "@SharedXML/CallbackRegistry.lua"]:143: in function <SharedXML/CallbackRegistry.lua:142>
[string "=[C]"]: ?
[string "@SharedXML/CallbackRegistry.lua"]:146: in function `TriggerEvent'
[string "@Blizzard_ProfessionsTemplates/Blizzard_Professions.lua"]:1139: in function `set'
[string "@FrameXML/FilterDropDown.lua"]:103: in function `func'
[string "@SharedXML/UIDropDownMenu.lua"]:1007: in function `UIDropDownMenuButton_OnClick'
[string "*UIDropDownMenuTemplates.xml:136_OnClick"]:1: in function <[string "*UIDropDownMenuTemplates.xml:136_OnClick"]:1>

Locals:
self = <table> {
 affectsRecipe = false
 recipeData = <table> {
 }
 nodeRules = <table> {
 }
 nodeID = 40008
 idMapping = <table> {
 }
 professionStats = <table> {
 }
 childNodes = <table> {
 }
}
recipeData = <table> {
 isEnchantingRecipe = false
 isCooking = false
 recipeID = 376556
 supportsCraftingStats = true
 categoryID = 1669
 learned = true
 isRecraft = false
 supportsCraftingspeed = true
 isSalvageRecipe = false
 recipeIcon = 4549303
 maxQuality = 3
 recipeName = "Azureweave Bolt"
 supportsInspiration = false
 isGear = false
 supportsMulticraft = false
 supportsQualities = true
 isSimulationModeData = false
 isOldWorldRecipe = false
 supportsResourcefulness = false
 professionData = <table> {
 }
 subtypeID = 5
 numSkillUps = 1
}
nodeRulesData = <table> {
 1 = <table> {
 }
}
parentNode = nil
configID = 0
nodeInfo = nil
(*temporary) = <function> defined =[C]:-1
(*temporary) = 0
(*temporary) = 40008
(*temporary) = "attempt to index local 'nodeInfo' (a nil value)"

Just read #94 and it's the same issue, can probably be merged

Same issue with Inscription when filtering on Classic. Maybe it's because I have some DF, WoD and Pandaria patterns in my Favourites list?
1x CraftSim/Data/Classes/NodeData.lua:31: attempt to index local 'nodeInfo' (a nil value)
[string "@CraftSim/Data/Classes/NodeData.lua"]:31: in function new' [string "@CraftSim/Libs/classic.lua"]:66: in function NodeData'
[string "@CraftSim/Data/Classes/SpecializationData.lua"]:49: in function parseNode' [string "@CraftSim/Data/Classes/SpecializationData.lua"]:84: in function new'
[string "@CraftSim/Libs/classic.lua"]:66: in function SpecializationData' [string "@CraftSim/Data/Classes/RecipeData.lua"]:71: in function new'
[string "@CraftSim/Libs/classic.lua"]:66: in function RecipeData' [string "@CraftSim/Main/CraftSim.lua"]:523: in function TriggerModulesByRecipeType'
[string "@CraftSim/Main/CraftSim.lua"]:189: in function TriggerModulesErrorSafe' [string "@CraftSim/Main/CraftSim.lua"]:234: in function <CraftSim/Main/CraftSim.lua:207> [string "=[C]"]: in function Init'
[string "@Blizzard_Professions/Blizzard_ProfessionsCrafting.lua"]:286: in function SelectRecipe' ... [string "@Blizzard_Professions/Blizzard_ProfessionsFrame.lua"]:155: in function SetProfessionInfo'
[string "@Blizzard_Professions/Blizzard_ProfessionsFrame.lua"]:55: in function <...s/Blizzard_Professions/Blizzard_ProfessionsFrame.lua:53>
[string "=[C]"]: ?
[string "@SharedXML/CallbackRegistry.lua"]:143: in function <SharedXML/CallbackRegistry.lua:142>
[string "=[C]"]: ?
[string "@SharedXML/CallbackRegistry.lua"]:146: in function TriggerEvent' [string "@Blizzard_ProfessionsTemplates/Blizzard_Professions.lua"]:1172: in function set'
[string "@FrameXML/FilterDropDown.lua"]:103: in function func' [string "@SharedXML/UIDropDownMenu.lua"]:1007: in function UIDropDownMenuButton_OnClick'
[string "*UIDropDownMenuTemplates.xml:136_OnClick"]:1: in function <[string "*UIDropDownMenuTemplates.xml:136_OnClick"]:1>
Locals:
self =  {
affectsRecipe = false
recipeData =  {
}
nodeRules =  {
}
nodeID = 34835
idMapping =  {
}
professionStats =  {
}
childNodes =  {
}
}
recipeData =  {
isEnchantingRecipe = false
isCooking = false
recipeID = 383325
supportsCraftingStats = true
categoryID = 1748
learned = true
isRecraft = false
supportsCraftingspeed = true
isSalvageRecipe = false
recipeIcon = 4554993
maxQuality = 5
recipeName = "Darkmoon Deck Box: Inferno"
supportsInspiration = false
isGear = true
supportsMulticraft = false
supportsQualities = true
isSimulationModeData = false
isOldWorldRecipe = false
supportsResourcefulness = false
professionData =  {
}
subtypeID = 0
numSkillUps = 3
}
nodeRulesData =  {
1 =  {
}
2 =  {
}
3 =  {
}
4 =  {
}
5 =  {
}
6 =  {
}
7 =  {
}
}
parentNode = nil
configID = 0
nodeInfo = nil
(*temporary) =  defined =[C]:-1
(*temporary) = 0
(*temporary) = 34835
(*temporary) = "attempt to index local 'nodeInfo' (a nil value)"

I think it's something related to old professions not reporting information about specialization nodes without ranks.
I thought to solve this issue with this piece of code in NodeData.lua (row 31):
    local nodeInfo = C_Traits.GetNodeInfo(configID, self.nodeID)

    if (nodeInfo) then
        self.active = nodeInfo.activeRank > 0
        self.rank = nodeInfo.activeRank - 1
        self.maxRank = nodeInfo.maxRanks - 1
    else
        self.active = false
        self.rank = 0
        self.maxRank = 0
    end

As this can be related to having favorites from dragonflight and then switching to other expansions filters, perhaps my code is only a workaround to not have the lua error, but a more deep analysis on why it happens can be better.


Improve the error message and logic when Dropbox's Permissions are incorrect

@arjunprakash027
Looks like dropbox client / code is broken
WARNING: The folder Marker Data already exists.
ERROR: Error at division
Traceback (most recent call last):
  File "urllib3/util/ssl_.py", line 447, in ssl_wrap_socket
FileNotFoundError: [Errno 2] No such file or directory

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "urllib3/connectionpool.py", line 789, in urlopen
  File "urllib3/connectionpool.py", line 490, in _make_request
  File "urllib3/connectionpool.py", line 466, in _make_request
  File "urllib3/connectionpool.py", line 1095, in _validate_conn
  File "urllib3/connection.py", line 730, in connect
  File "urllib3/connection.py", line 909, in _ssl_wrap_socket_and_match_hostname
  File "urllib3/util/ssl_.py", line 449, in ssl_wrap_socket
urllib3.exceptions.SSLError: [Errno 2] No such file or directory

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "requests/adapters.py", line 667, in send
  File "urllib3/connectionpool.py", line 843, in urlopen
  File "urllib3/util/retry.py", line 519, in increment
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='api.dropboxapi.com', port=443): Max retries exceeded with url: /oauth2/token (Caused by SSLError(FileNotFoundError(2, 'No such file or directory')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "airlift/dropbox_client.py", line 41, in __init__
  File "dropbox/base.py", line 1207, in files_create_folder
  File "dropbox/dropbox_client.py", line 304, in request
  File "dropbox/dropbox_client.py", line 372, in check_and_refresh_access_token
  File "dropbox/dropbox_client.py", line 405, in refresh_access_token
  File "requests/sessions.py", line 637, in post
  File "requests/sessions.py", line 589, in request
  File "requests/sessions.py", line 703, in send
  File "requests/adapters.py", line 698, in send
requests.exceptions.SSLError: HTTPSConnectionPool(host='api.dropboxapi.com', port=443): Max retries exceeded with url: /oauth2/token (Caused by SSLError(FileNotFoundError(2, 'No such file or directory')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "airlift/cli.py", line 38, in cli
  File "airlift/dropbox_client.py", line 45, in __init__
airlift.utils_exceptions.CriticalError: ('Error during Dropbox client creation', SSLError(MaxRetryError("HTTPSConnectionPool(host='api.dropboxapi.com', port=443): Max retries exceeded with url: /oauth2/token (Caused by SSLError(FileNotFoundError(2, 'No such file or directory')))")))

@arjunprakash027
We can add something like Dropbox API Permissions Error. Check your scopes in the Permissions tab under Dropbox's App Console.

I will be close this issue as we have identified the issue.


Update README.md

Adding "Community How-Tos, Videos & Articles" section. Feel free to change it, move it, remove it or approve it.
Either way, keep up the good work!
Thanks,
Kiril

Thanks - Good idea to add this in. Will add it in the next update shortly.
Mark

On 15 Oct 2020, at 16:00, peyanski<EMAIL_ADDRESS>wrote:

Adding "Community How-Tos, Videos & Articles" section. Feel free to change it, move it, remove it or approve it.
Either way, keep up the good work!
Thanks,
Kiril
You can view, comment on, or merge this pull request online at:
https://github.com/mlfunston/node-red-contrib-broadlink-control/pull/56
Commit Summary
Update README.md
File Changes
M README.md (8)
Patch Links:
https://github.com/mlfunston/node-red-contrib-broadlink-control/pull/56.patch
https://github.com/mlfunston/node-red-contrib-broadlink-control/pull/56.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


[WIP] C3 charts yaxis cu

Purpose or Intent

Fix overlapping Y axis labels in small C&U charts.

Screenshots
Before:
After:

@miq-bot add_label wip

@miq-bot remove_label wip

@martinpovolny please review


Drop some logs from info to debug

Signed-off-by: Flynn<EMAIL_ADDRESS>

Gonna merge this without waiting to restart infrastructure things.


Non zero exit failure

Not sure whats happening here, but my build is failing with this:
    vmware-iso: SDelete is set for 1 pass.
    vmware-iso: Free space cleaned on C:\
    vmware-iso: 1 drives zapped
    vmware-iso:
==> vmware-iso: Provisioning with shell script: /var/folders/b4/grsjj0d52t33m9jvc6g8_5440000gn/T/packer-shell510891355
    vmware-iso: rm: cannot remove '/tmp/vmware-vmsvc.log': Device or resource busy
    vmware-iso: rm: cannot remove '/tmp/vmware-vmusr.log': Device or resource busy
==> vmware-iso: Stopping virtual machine...
==> vmware-iso: Deleting output directory...
Build 'vmware-iso' errored: Script exited with non-zero exit status: 1

==> Some builds didn't complete successfully and had errors:
--> vmware-iso: Script exited with non-zero exit status: 1

What version of vmware fusion are you using?

Professional Version 8.0.0 (2985594)

hrm, okay. I'm still running my builds on v7 and haven't yet upgraded to 8. You may want to try and post this issue on https://github.com/mitchellh/packer/issues since it may not be related to what we're doing with the box.
Which image were you trying to build? Did you make any modifications to it when you ran it?

I'm using the windows_2012_r2 profile. Only modification I made was that I added my own powershell provisioner. If I remove:
{
      "type": "shell",
      "inline": [
        "rm -rf /tmp/*"
      ]
    }

It works OK.

I can reproduce this issue also.

Fixed by 975eac3ae2871bf2632bdc5eded09a958a948b48


Ace editor & Model Job UI Tweaks

Implements #738.

[x] Added only the minimal amount of minified Ace libraries
[x] Ace editor syncs with the custom_network flask box upon interaction
[x] Implement for classification job
[x] Implement for generic job
[x] Change the model creation UI to [3 rows; 1 row; 1 row]

Note that for purposes of backwards compatibility and maintaining flask/python integrity, the custom_network container is still in place, and only syncs with the ace editor when it has to. Although the file diffs seem to indicate i did a lot, the ace editor is only four .js files and a few LOC, while the majority of the code change is just moving around the <div>'s to tweak the model creation UI.

Very cool, great addition! I've tried it and it works great so far, let me start reviewing the code...

I need a signed CLA from you before I can accept this: https://github.com/NVIDIA/DIGITS/blob/v3.3.0/.github/CONTRIBUTING.md#pull-requests

After addressing any code nitpicks, would you mind implementing this pull request in two separate commits: (1) rearranging the new.html templates and (2) adding Ace? That will make git blame more useful later, as well as making the commit stream easy to read.

Sure Luke. Note the CLA link is dead in https://github.com/NVIDIA/DIGITS/blob/master/.github/CONTRIBUTING.md#L33

Note the CLA link is dead in https://github.com/NVIDIA/DIGITS/blob/master/.github/CONTRIBUTING.md#L33

Whoops, good call. https://github.com/NVIDIA/DIGITS/pull/753.

For reference, here are some alternate themes available for Ace:
https://ace.c9.io/build/kitchen-sink.html
I don't have a strong opinion on the theme - "Monokai" works for me. @jmancewicz: do you have an opinion?

Based on their license, can you add these lines to the top of those minimized JS files?
// Copyright (c) 2010, Ajax.org B.V.
// All rights reserved.

[x] Cursor Position To Start
[x] Theme to Chrome .js, remove monokai js
[x] Append Ajax BV licence header to minified ace files
[x] div id removal
[x] squash commits and rebase to two for (1) template rearranging and (2) Ace-related.

All set and ready for review.

div id removal

Technically you didn't do this one, but who cares.
I still need your CLA before I can merge, but otherwise this looks great! I like the Chrome theme, it seems to fit in pretty well.

CLA is on the other computer, keep forgetting to send it. Actually i did do the id removal but only for the generic and not the classification. In fact i encounter those minor 'mistakes' more often wrt the classification/generic files.

In fact i encounter those minor 'mistakes' more often wrt the classification/generic files.

I know, it's a pain. Fortunately @gheinrich is helping us forge ahead towards a nice, general solution that will let us de-duplicate a lot of that code eventually.

div id removal

Technically you didn't do this one, but who cares.

Fixed.

CLA received. Looks great. Thanks for the contribution!


Adds upstream auto sync script

This is a slightly modified version of the same script used in the capk fork. We should be able to use this to sync upstream to downstream.

/lgtm


Authenticating GCS from within a kubernetes cluster

Hi. I'm currently trying to get the GCS sample working in our kubernetes cluster hosted on google cloud. Kubernetes offers (as far as I'm aware) two approaches to authenticate with the google cloud services. One is the workload identity which provides temporary tokens and the other are mounted credentials using the "automountServiceAccountToken" option.
The workload identity approach provides a json file but once provided to this library it gives an error as there is no type field in the json. I've tried various type fields like "service_account", "authorized_user" and "anonymous". All seem to expect different json files.
The mounted credential approach also doesn't seem to work as that does not provide any json files. It mounts a token, namespace and ca.cert file. The token file looks like a bearer token, but I honestly don't know.
The third approach would be to generate the credential file outside the cluster and mount it when launching the cluster. But I'd rather not go with that approach as that doesn't seem very secure and CI/CD friendly.
Does anyone have any experience in authenticating from within a kubernetes cluster?

Are you running on GKE or your own cluster?  On GKE you should be able to use the metadata server to get credentials for the cluster's service account.  That might not be secure enough if you want to run workloads with and without access to GCS in the same cluster.
You can also use secrets and load a service account, as we do here:
t

Hi. I'm currently trying to get the GCS sample working in our kubernetes cluster hosted on google cloud. Kubernetes offers (as far as I'm aware) two approaches to authenticate with the google cloud services. One is the workload identity which provides temporary tokens

I think that is the one you want, but using this:
https://cloud.google.com/kubernetes-engine/docs/how-to/workload-identity#using_from_your_code
Starting with v1.13.0 (you did not mention what version of the library you were using, can you share?) the client library should be able to recognize that it is inside a GKE cluster (I think you are using GKE?  You mentioned k8s, but not GKE explicitly, maybe me assuming GKE is unwarranted) and use the cluster's metadata server to get access tokens.  You do not need to "mount" any files or set the GOOGLE_APPLICATION_CREDENTIALS environment variable, the library contacts the metadata service directly.
Mind you, we have not tested that configuration explicitly, so it is possible we missed something.

and the other are mounted credentials using the "automountServiceAccountToken" option.
The workload identity approach provides a json file but once provided to this library it gives an error as there is no type field in the json. I've tried various type fields like "service_account", "authorized_user" and "anonymous". All seem to expect different json files.
The mounted credential approach also doesn't seem to work as that does not provide any json files. It mounts a token, namespace and ca.cert file. The token file looks like a bearer token, but I honestly don't know.

I suspect all those are not in the formats expected by the auth classes in the library.

The third approach would be to generate the credential file outside the cluster and mount it when launching the cluster. But I'd rather not go with that approach as that doesn't seem very secure and CI/CD friendly.
Does anyone have any experience in authenticating from within a kubernetes cluster?

I have used both these approaches in the past successfully:
https://cloud.google.com/kubernetes-engine/docs/how-to/workload-identity#alternatives
They are adequate for experiments and tests, but maybe not for deployments facing the Internet.

Can we leave this open until the GKE metadata server is fixed?

Sure.

This is ready to close. The fix is in gke-metadata-server > 0.3.6. I see the latest GKE versions in prod on 0.3.11 and 0.3.19. Thanks for the report!

Thanks for keeping us updated.  Closing.


Token Expiration

There are no ways where a generated Token for asp.net web api can be forced to expire other than the default time allocated to the token.
It may be good to have a mechanism where the token can be expired when the user logout or some other scenarios where the token needs to be deactivated.
Thanks

How and where are you generating token?

@satyajit-behera I achieved this with JwtBearerAuthentication(VNext), the tokens are self contained, so you can't force it, however, you can persistently store the token and validate it exists when making requests or delete it on logout so subsequent requests will be denied. You could also achieve sliding token lifetimes in the same way. You could do the same with the older version OAuthBearerTokens.

Thanks @stephenstroud Can you give some more details or reference examples to elaborate this.

thanks @rustd . I am using the default asp.net identity provider while using the web api template in visual studio.

@satyajit-behera  which template version are you using? Are you using VS 2013?

@stephenstroud  VS 2013


Needs a note to distinguish GCC from GCCH as this is not supported in the latter.

Hi Team - this is not currently a supported feature in GCCH, the article is a bit ambiguous and it would be helpful to call out that this is for GCCModerate, and not GCCH/DOD.  Thanks!

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 6be7caf9-523d-2e84-fb17-14b1606aa97a
Version Independent ID: 34a7cb37-5f79-c1d3-f40f-8afb91f48862
Content: Connect to Microsoft 365 Government Community Cloud (GCC) data with Usage Analytics - Microsoft 365 admin
Content Source: microsoft-365/admin/usage-analytics/connect-to-gcc-data-with-usage-analytics.md
Service: microsoft-365-business
GitHub Login: @efrene
Microsoft Alias: efrene

@drdenson Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/12458
Once the author approves, the changes will be live.
Thanks
Sri


[FEATURE] Settings

Feature Description
Add the website settings page.
What Will Happen?
Change the website settings

Add to (#6) Documentation


Wallets are now linked to coins via id.

Updated linking code so that wallets are tied to a coin by its id instead of its name.  Added additional check to differentiate between coins that aren't tied to any wallet (None) and coins that are tied to a wallet that can't be found ().
Mitigation Strategy
When opening up the coins screen for the first time any configured coin will report its wallet as <unknown>.  To fix the user will need to edit the coin and select the wallet from the drop-down.  This only needs to be done once per-coin to relink them to the correct wallet.  If the user attempts to mine the coin without doing this first they'll receive a (somewhat misleading) error saying that the wallet could not be found.
@evan-cohen What are your thoughts on this?  I think it's a reasonable compromise since this is a pre-release and the alternatives are very hackish.

@evan-cohen What are your thoughts on this? I think it's a reasonable compromise since this is a pre-release and the alternatives are very hackish.

Agreed - not ideal, but we're still sub-1.0 to your point. Let's make sure to still call it out in the release notes so that this caveat is noted in a user-accessible way.


cant init user  table in mysql

       public static readonly string TableStruct = "(" + ColumnName_Id + " INT NOT NULL AUTO_INCREMENT, " + ColumnName_Username + " VARCHAR(255), " + ColumnName_Password + " VARCHAR(255), " + ColumnName_UserProfile + " INT, " + ColumnName_Email + " VARCHAR(255), " + ColumnName_CreatedOn + " **TIMESTAMP**, " + ColumnName_ModifiedOn + " **TIMESTAMP**, CONSTRAINT " + DocumentName + "_pk PRIMARY KEY (" + ColumnName_Id + "))";
MySQL not allow exist two timestamp columns  in one table ,so we need to  change column type to datetime or any other way.

When I have done MySQL testing, two timestamp columns worked for me. Which version of MySQL are you using?

When I have done MySQL testing, two timestamp columns worked for me. Which version of MySQL are you using?

mysql version	5.7.22
version_comment	MySQL Community Server (GPL)

MySQL versions before 5.6.1 would not let two TIMESTAMP columns in the same table, unless as you rightly noted without defaults and allowing null.
MySQL 5.6.+ allows two or more TIMESTAMP columns in a table.


Add env directive to docker

FEATURE
@tlunter @adamjt @bfulton
Add a directive to the docker DSL called env. It allows users to specify environment variables to be set for the build, but these will not be set in the generated image. Usage:
docker :sweet_docker do
  env 'SOME_VARIABLE' => 'value', 'OTHER_VARIABLE' => 12
end

Note that all keys and values will be shell escaped.

I think that by using the same keyword that the Dockerfile uses for persistent environment variables there is a high chance of confusion here. Would like to see it called something with clearer intent, like build_envor whatnot.

@adamjt Good call, env => build_env.

:+1:

@adamjt @tlunter Got dockly's specs in a working order, exercised the build_env logic in the specs here.

:+1:

sweet! +1


[question] is there a way to customize the loading "circle" gif while a page is loading?

Not an issue but a question: Is there an elegant way to customize the loading "circle" gif while a page of an epub is loading? I was thinking about adding a loading bar at the top of the content.

There is no "elegant" way that I know of.  You would need to fork the code and do some programming in JavaScript

Ric is (W)right, you would need to replace the Spinner AMD module with the one of your choice. The API is really simple (start/stop functions, IIRC)

Can this issue be closed? The solution (albeit indirect) has been posted above.


Add --iterations to specify a budget for the experiment

Similar to https://github.com/BuoyantIO/slow_cooker/pull/68
Note that ~10% of the time, I see the following problem with this PR :
./strest-grpc client --interval=10s --iterations=3
2018-10-22T11:07:59+02:00   0    0.0B  94256/0 10s L:   0 [  0   0 ]    7 J:   0   0
2018-10-22T11:08:09+02:00   1    0.0B  91003/0 10s L:   0 [  0   0 ]    6 J:   0   0
2018-10-22T11:08:19+02:00   2    0.0B  84834/0 10s L:   0 [  0   0 ]    0 J:   0   0
2018-10-22T11:08:29+02:00   3    0.0B      1/0 10s L:   0 [  0   0 ]    0 J:   0   0
{
  "good": 270094,
  "bad": 0,
  "bytes": 0,
  "latency": {
    "p50": 0,
    "p75": 0,
    "p90": 0,
    "p95": 0,
    "p99": 0,
    "p999": 7
  },
  "jitter": {
    "p50": 0,
    "p75": 0,
    "p90": 0,
    "p95": 0,
    "p99": 0,
    "p999": 0
  }
}

There is an extra line printed with very few (I have only seen "1") requests in an extra and undesired iteration. Any idea how we can reliably avoid this ?

Hi @farquet, similar to https://github.com/BuoyantIO/slow_cooker/pull/68#issuecomment-432334438, curious about whether totalRequests already solves this use case?

In a mode where we don't set --totalTargetRps, strest-grpc will show a throughput per iteration. This is not fixed, since this is a performance metric of your run.
In that case, there is currently no way to say, I want 60 iterations of 10 seconds (hence running for a total of 10 minutes). With this PR, one can set --iterations=60 --interval=10s for such use case.

@farquet Ah, this all makes sense, I didn't realize that difference between slow_cooker and strest-grpc.
Thinking about this a bit more, I'm wondering if we should just make --totalTargetRps required, and default to 1, to match slow_cooker. We would be removing the ability to run with unlimited --totalTargetRps, but as a workaround a user could just set the value very high.
Would like to get your thoughts on this. In our own testing, we've found that running a load tester with unlimited qps to be not very useful. At max qps, a test subject typically exhibits latency, memory, and cpu characteristics that are not useful for production. What we find more useful is answering questions like "what is the p99 latency at 10,000qps?"

I agree with you on the fact that under high pressure, a system can exhibit higher latency, memory consumption, etc. But isn't that interesting to analyze a bit more ?
Since slow_cooker already works in a way where a constant load is sent. I find interesting that strest-grpc has the ability to do a stress test too and see how different systems perform in that scenario.
For instance, let's compare this quick run of strest-grpc on my laptop on Oracle JDK versus Graal EE.
Running it with strest-grpc_darwin_amd64 client --address=localhost:4143 --latencyUnit=us --interval=10s, going through Linkerd, before reaching the grpc server (strest-grpc_darwin_amd64 server).
Oracle JDK :
2018-10-29T20:08:53+01:00   0    0.0B  17523/0 10s L: 324 [1134 2065 ] 393930 J:   0   0
2018-10-29T20:09:03+01:00   1    0.0B  26281/0 10s L: 314 [428 511 ] 5075 J:   0   0
2018-10-29T20:09:13+01:00   2    0.0B  24975/0 10s L: 333 [447 522 ] 3055 J:   0   0
2018-10-29T20:09:23+01:00   3    0.0B  24084/0 10s L: 351 [458 510 ] 2589 J:   0   0
2018-10-29T20:09:33+01:00   4    0.0B  23327/0 10s L: 362 [474 532 ] 4663 J:   0   0
2018-10-29T20:09:43+01:00   5    0.0B  22999/0 10s L: 358 [490 537 ] 2337 J:   0   0
2018-10-29T20:09:53+01:00   6    0.0B  23175/0 10s L: 358 [481 549 ] 2015 J:   0   0
2018-10-29T20:10:03+01:00   7    0.0B  24119/0 10s L: 341 [451 519 ] 1924 J:   0   0
2018-10-29T20:10:13+01:00   8    0.0B  24683/0 10s L: 344 [433 505 ] 2133 J:   0   0
2018-10-29T20:10:23+01:00   9    0.0B  24796/0 10s L: 338 [445 501 ] 1923 J:   0   0

Graal Enterprise :
2018-10-29T20:18:43+01:00   0    0.0B   9800/0 10s L: 314 [2107 3739 ] 382722 J:   0   0
2018-10-29T20:18:53+01:00   1    0.0B  24995/0 10s L: 273 [600 691 ] 7465 J:   0   0
2018-10-29T20:19:03+01:00   2    0.0B  29382/0 10s L: 275 [396 472 ] 2686 J:   0   0
2018-10-29T20:19:13+01:00   3    0.0B  26188/0 10s L: 287 [537 712 ] 4345 J:   0   0
2018-10-29T20:19:23+01:00   4    0.0B  26393/0 10s L: 306 [442 558 ] 34966 J:   0   0
2018-10-29T20:19:33+01:00   5    0.0B  27670/0 10s L: 303 [405 480 ] 3163 J:   0   0
2018-10-29T20:19:43+01:00   6    0.0B  26997/0 10s L: 304 [427 523 ] 3636 J:   0   0
2018-10-29T20:19:53+01:00   7    0.0B  28371/0 10s L: 293 [394 450 ] 3611 J:   0   0
2018-10-29T20:20:03+01:00   8    0.0B  29162/0 10s L: 288 [372 433 ] 2356 J:   0   0
2018-10-29T20:20:13+01:00   9    0.0B  29268/0 10s L: 281 [381 442 ] 2516 J:   0   0

From those numbers, one could say that Graal can handle higher throughput while still giving you a significantly smaller p99 latency. Having a way to show that you can win on both metrics at the same time seems relevant to me.
And isn't this scenario a realistic setup of Linkerd getting a sudden huge amount of requests ?

Sorry for the late reply.
I'm not sure how I could get a static number of iterations without this extra --iterations flag.
If I try to adjust --totalTargetRps and --totalRequests, I can never ensure a strict number of iterations.
If --totalTargetRps is not set, the duration will depend on how much time it requires to handle --totalRequests.
If --totalTargetRps is set to something achievable, it's likely that the first iteration(s) might not reach it due to warmup effects. That will then make the number of iterations unpredictable.
If --totalTargetRps is set to something low enough to have the first iteration reach it, we get a low fixed number of iterations, but that benchmark is not very useful anymore.
That's why I think it's important to have --iterations. In the JVM benchmarking world, this is usually how you specify a budget. And the score of each iteration is relevant to analyze the warmup time for instance. The famous JMH framework also works like that.
However, if that's not relevant to your approach, I would totally understand. In that case, I'll continue to use my fork of your projects for my use case and close the PR.

Great! Thanks for the feedback. I’ll integrate that change asap.
However, do you have any hint regarding the problem I raised in the initial comment ? How to make sure that an extra request won’t trigger an undesired logging due to parallelism ?

@farquet I believe that last log line is due to logIntervalReport getting called on shutdown:
https://github.com/BuoyantIO/strest-grpc/blob/c3cb195e500f7b8fa6d88c49617acf9cf12140b7/client/client.go#L624-L628
Thinking about this a bit, I think it's fine to remove that entire code block, since an incomplete interval report during shutdown isn't that interesting any way. Once you remove that, you'll also be able to remove the count and previousinterval vars, which is a nice cleanup!


different pa2 pipelines

what is the difference between the PA2 walkthrough pipeline and mine?
why am i getting a " Pair<S,S> cannot be converted to Pair<T,S>" error? don't both methods eventually create a Pair<S,S>, which is permissible when passed into the pipeline? thanks.

I think you need to put 'return new StateM<S, S> (s -> ...)', because if you don't put the <S, S> in the return statement, java will think that you are outputing StateM<T, S> as defined in 'class StateM<T, S>'

yea think u must clearly initialise a Pair<S,S>, since the one in the parent constructor is Pair<T,S>

Since StateM has 2 parameter types, T and S, when calling its constructor, you HAVE to specify what these 2 parameters are. e.g. new StateM<x, y> , even though this does not take in a type until later on

I had a similar problem as you during the lab itself but i tweaked it slightly.
Below is my implementation of get()
    public static <S> StateM<S,S> get() {
        return new StateM<>(s -> new Pair<>(s, s));
    }

Not sure if you have fixed it but this helped me definitely. What worked is that type inference can often "magically" determine the correct types based on the context in which a method or constructor is called. This can make the code shorter and cleaner, as it avoids the redundancy of specifying type parameters that the compiler can infer for itself.
Hope this helps!

finals is over you can close this issue


Replace rbind with union_all

Doing #20 would mean I think that rbind will not work so you'll need to use union_all (ideally batching for multiple cohorts)

hey @edward-burn I have done the union_all part and could you please do the batching part thanks!

closing this as the results should not be that long


RNN cell with list of symbol as input

@piiswrong
when using list of symbol as input, the layer is assumed as TNC order, While the default layout input is NTC. It is not properly parsed for list input case.
assert len(inputs) == length
inputs = [symbol.expand_dims(i, axis=0) for i in inputs]
inputs = symbol.Concat(inputs, dim=0)
if begin_state is None:
begin_state = self.begin_state()

Also I think the Concat is not right here. which should be:
inputs = symbol.Concat(*inputs, dim=0)

problem fixed with #5166


Feat:ADDING SLIDESHOW IN BACKGROUND FOR FAQ AND BUY PAGE

Describe the feature

Add ScreenShots
.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Content should be clearly visible.


Create a finite state machine analysis pass to propagate Pauli basis states, and use it to disentangle CNOTs

Context:
In circuits that mostly involve the six Pauli eigenstates, the effect of gates can be described by a simple six-state finite state machine. This allows us to deduce their midcircuit state at compile time.
The knowledge of the snapshot midcircuit states can be useful when:

Deducing CNOT/Toffoli/SWAP/... effect and decouple them. We decouple CNOT in this PR.
Replace complicated circuits whose results are |01+-LR> with standard preparation routines
Potentially more

Description of the Change:
Added an analysis pass --propagate-simple-states (and tests).
Also added CNOT decomposition (and tests).
Benefits:
More compile time circuit information for potential compile time transformation/optimization.
Possible Drawbacks:
This algorithm is from an external paper. We need to properly cite it.
https://arxiv.org/abs/2012.07711 (fig.5)
Related GitHub Issues:
sc epic: https://app.shortcut.com/xanaduai/epic/74389
[sc-74649]
[sc-74650]

Note that this is a hackweek project and is not on any roadmaps, so it's not urgent

Adding do-not-merge label to freeze this branch for assessment


Make routes optional

Honestly I don't think it makes sense on a package like that to define routes. Maybe I'm just misunderstanding what this project is supposed to be.
Relates to #16.

@emiliopedrollo Hi and thank you for your contribution. The package can be used as an api or consumed via the world facade. This is the reason why the routes are defined. I will pull this route and test your proposed modification.
Thank you

@emiliopedrollo
in world.php config files you can find the seeders key:
*/
	'seeders' => [
		'states' => true,
		'cities' => true,
		'timezones' => true,
		'currencies' => true,
		'languages' => true,
	],

What do you think if routes and seeder keys are combined into :
'modules' => true,
		'cities' => true,
		'timezones' => true,
		'currencies' => true,
		'languages' => true,

@emiliopedrollo Thank you for this. I would opt to keep things simple in a config file if you agree with this approach.

@emiliopedrollo thank you, i am merging the PR.

@emiliopedrollo Thanks dude, that's awesome. I was hoping to implement this feature when I got time. Thanks  @nnjeim for this package.

@kodjosama Thank you. All the credit goes to @emiliopedrollo


Inclusion in distro package managers

Is your feature request related to a problem? Please describe.
I would like to see Zplugin become the de-facto standard ZSH config manager.  To do that (IMHO) it needs to be available by default in most distro package managers.
Describe the solution you'd like
Get ZSH Zplugin working on the major distros supported by OBS and then upstreaming everything from there.
Describe alternatives you've considered
Using a ZSH config manager that is included in Ubuntu by default.
Additional context
I don't like having to write custom scripts to install my ZSH config manager, and I doubt most system admins do either 😝.

By "Get ZSH working on the major distros supported by OBS and then upstreaming everything from there." you've meant: "Get Zplugin working on the major distros supported by OBS and then upstreaming everything from there." ?
I will think on the request. It requires much work.

I will think on the request. It requires much work.

I would think most of the work comes from CI to push new updates out.  Does Zplugin rely on software/locations/settings that are distro specific?

No, it doesn't depend on such things.

I've took a look at the OBS. It makes the process of creating Linux packages fairly easy. I'm looking into it right now.

I’d like to package the version 3.5 for MacPorts. Is it appropriate to use the AUR package (with the patch from #243) as a starting point?

Yes. You can also use the fresh release v3.7.


Bicaridine metabolism bandaid

About the PR

Bicaridine, Dylovene and Crytobiolin now metabolize at the same rate for all species.
Fixes #21397
Why / Balance

Previous behavior was causing more or less healing than expected, as well as unexpected ODs.
Technical details

Since these three reagents have both Medicine and Alcohol metabolisms, there were some interactions there that messed up the math. I simply moved the Drunk effect to the Medicine metabolism.
I marked this is a bandaid, as there are design questions around how to handle different rates of metabolism on the same reagent (diona and slime stomach handles both Medicine and Alcohol), or if there are two organs metabolizing at once (human liver for Alcohol and human heart for Medicine).
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

fix: Bicaridine, Dylovene and Crytobiolin now metabolize at the same rate for all species.

Nice! I was about to try and fix it myself, but this implementation seems better and more logical.
I was going to make Bicaridine, Dylovene and Crytobiolin generate Ethanol when metabolized just like drinks do at the moment since in my search for a way to fix this I came across this PR:

#7398 , where alcoholic drinks generate ethanol to avoid putting the alcohol effect on all of them, since that can generate unwanted behavior. Somewhat related issue to what we have here.

If we do it this way when some species can only metabolize 2 chemicals at a time means that it will be an indirect downgrade on these chems. Since Ethanol will take up a metabolic slot and thus reduce the metabolizing rate of other chems, thats how I understand this.
Anyway just wanted to mention that PR since I am unsure if it should apply to medicine in order to avoid the issues mentioned in that PR.


Update README

Fix docker-compose service name and add an example how to run Uvicorn based service.

🙏


qmenumodel: init at 0.9.2

Description of changes
Working towards #99090.
QMenuModel, a Qt5 renderer for Ayatana Indicators. Used by Lomiri itself, presumably to render the active indicators in the top bar?
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Bumped to 0.9.2, resolved merge conflict.


Use binaries from pre-release/testing apt repository

Otherwise binary jobs can fail if breaking-ish changes are introduced, since we still build part of the code from source.
Signed-off-by: Christophe Bedard<EMAIL_ADDRESS>

This doesn't actually fix the current CI failures, but the change definitely makes sense, so we'll go ahead.


Alert: SiteExtrenallyDown in monitoring

Alert SiteExtrenallyDown firing in monitoring namespace
This is an automated issue created by the monitoring system. Please do not edit this message.
Alertmanager URL: https://alertmanager.ankhmorpork.thaum.xyz
Issue was last updated at 2023-03-07 15:53:54.403378307 +0000 UTC m=+6386.239340128.
Common Labels

alertname
SiteExtrenallyDown

cluster
ankhmorpork

job
probe/monitoring/uptimerobot

namespace
monitoring

prometheus
monitoring/k8s

severity
warning

Common Annotations

runbook_url
https://runbooks.thaum.xyz/runbooks/thaum-xyz/SiteExternallyDown

summary
Site is not available from external network

Alerts

instance
monitor
url
description
StartsAt
Links

https://home.ankhmorpork.thaum.xyz/
homeassistant - bln
https://home.ankhmorpork.thaum.xyz

2023-03-07 15:48:23.974 +0000 UTC
GeneratorURL

https://recipes.krupa.net.pl/
cookbook
https://recipes.krupa.net.pl/

2023-03-07 15:25:53.974 +0000 UTC
GeneratorURL

(DO NOT MODIFY: e7e9f8aad3259f1032f061bc7ec9197a9785c7d6d00b133a5297fd5d3d70be3f )

Same as #201


BUG: ClassList Upload w/ CWL Updates Break Deliverable Repo Provisioning

If a pre-existing user is updated with new CWL information by uploading an updated Class List, only after a Deliverable is created, the Repo Provisioning for that Deliverable will break.
Step 1) Upload class list and create a Deliverable.
Expectation: You see the ability to provision repos for the Deliverable:

You can see that the Team objects for this deliverable are already created in the database:

To Break Deliverable:
Step 2) Upload a Class List with modified CWL users:

Step 3) Attempt to view the Admin's Provision Repo view:

Although the Repos have not been provisioned, no repos appear in the list to provision.
As the Team objects are created as the Provision Repos page is loaded, and it occurred before the new class list was uploaded, the cross-comparison of the new class list names and old team names did not match up, which resulted in a null value downstream:

Wow, definitely something to fix. Oy...

Starting work on  fix/classlist-cwl-update-bug.

After investigating the code base, I have tentative fix proposals:
Each time an Admin --> Manage Repositories view is loaded, the planProvision() function is triggered on the back-end before the front-end view receives the list of Unprovisioned Repositories to display. The planProvision() function generates Team objects for each single student team, or custom student team with multiple participants if the Team object does not already exist for the Deliverable. If new students are added to the list, then new Team objects are generated when the Admin --> Manage Repositories view is accessed on the front-end again.
The break occurs in the planProvision() logic, which attempts to query every computed team name in a the class list to retrieve a repo.

Source: AdminController.ts
Fix 1: Use CSID for Repo Names. As CSIDs are unchanging, the repository name never has to change. The computed name for the query will always work.
--- However, what other logic may be affected? Is the repo name used to collect any grade data based on CWL?
Fix 2: Each time the Class List is uploaded, if a user already exists based on CSID, run update logic to sequentially change areas of the application that may need to be updated based on CWL.
--- I currently do not know what other logic heavily relies on the CWL logic, but it is possible that more than the Unprovisioned Repos list is breaking. This could tentatively affect other areas like grades and AutoTest timer settings. This needs either more investigation, or we can start implementing it to see if it comes in handy.
Fix 3: Each time the  Admin --> Manage Repositories view is loaded, the planProvision() logic runs. If a team === null, we can attempt to run a query for a team that matches all person Ids for a team.

The person Ids for the team are available just above the team === null logic.
--- This could fix the Unprovisioned Repo list view right away, but the Release Repos logic might also have to be updated. Fix 3 needs to investigate if Release Repos logic is also broken for similar reasons as Unprovisioned Repo logic.

CWLs are no longer used in the Deliverable logic that broke. Uploading a ClassList should no longer break this issue.
Fix in: https://github.com/ubccpsctech/classy/pull/24.