sap.ui.webc.main.Table vs. sap.m.Table vs. sap.ui.table.*Table: Which One Should I Choose?

In the documentation there is a great article «Tables: Which One Should I Choose?» with a comparison table and main use cases when each type of table is an option choice. However, this table doesn't cover relatively recently added sap.ui.webc.main.Table.
The questions:

Is sap.ui.webc.main.Table intended to replace the previously added table classes?
What are the benefits of using sap.ui.webc.main.Table over the rest UI5 table classes?
Is there any preference to use one table class over another for the future development? In other words, are there any plans to replace sap.ui.table.Table with sap.ui.webc.main.Table or something similar?

Thanks.
P.S. This issue can be perceived as suggestion to elaborate the «Tables: Which One Should I Choose?» article and to address sap.ui.webc.main.Table as well.

Hi @pubmikeb ,
the sap.ui.webc.main.Table control is part of the UI5 Web Components enablement of OpenUI5. This means the UI elements developed as UI5 Web Components will be wrapped as UI5 controls and can be used natively in OpenUI5. More details about this can be found here: https://blogs.sap.com/2022/03/10/ui5-web-components-enablement-for-openui5-sapui5/
The UI5 Web Components will be the future direction of writing UI elements in a framework agnostic way and currently the enablement is experimental. The APIs are not yet perfect for the usage in OpenUI5 and requires some more massage. As of today, most of the UI5 Web Components control wrappers are just 1:1 wrappers and there is work in progress to ensure a proper usage within OpenUI5 with databinding support. The UI5 Web Components APIs are more classic web development (HTML) centric.
But my blog post linked above will hopefully shed more light...
Independent from that, you are right that the documentation «Tables: Which One Should I Choose?» should be updated once the sap.ui.webc.main.Table is not experimental anymore.

@petermuessig , thanks for the link to the blog post with detailed info regarding UI5 WebComponents.
A further question regarding UI5 WebC, in the script-section of the HTML bootstrapper it's written:
data-sap-ui-libs = "sap.ui.webc.main"

In classical UI5 HTML bootstrapper I didn't use data-sap-ui-libs. Is it mandatory to use data-sap-ui-libs = "sap.ui.webc.main" in the HTML bootstrapper additionally to the dependencies block in the manifest.json?
"dependencies": {
	"libs": {
		"sap.ui.webc.main": {}
	}
},

Isn't the same configuration? Once made in declarative way via manifest.json and once in index.html?
And one more question regarding UI5 WebC, do I understand it correctly, that for new apps development based on OpenUI5 without SAP Fiori Launchpad it's better to use sap.ui.webc.main instead of sap.m?

Hi @pubmikeb,

Isn't the same configuration? Once made in declarative way via manifest.json and once in index.html?

This answer might help: https://stackoverflow.com/a/56136611/5846045

For new apps [...] it's better to use sap.ui.webc.main instead of sap.m?

See Peter's comment at https://blogs.sap.com/2022/03/10/ui5-web-components-enablement-for-openui5-sapui5/#comment-620432
While sap.ui.webc controls are still experimental, you're highly welcome to try them and to give feedback!

Hello,
As Peter said, the webc library is currently under an experimental flag, we are looking to gradually improve its documentation and its should be more complete when the it is no longer experimental.
Kind regards,
Nikolay


Fix GTK drag & drop and xprop

Closes #52.
Turns out that the section I found in i3's source code when first diagnosing the issue was exactly what I needed.
Also, it turns out xprop was also broken by WM_STATE not being set! Go figure.
It's not exactly as clean as the rest of your XChangeProperty calls since WM_STATE was neither in netAtoms or ipcAtoms, but I wasn't sure where I should put a new atom so I just interned it inline.

That is so cool, thank you for figuring this out so much! Tried it and it seems to fix the issue with firefox and a couple gtk apps.

Wait, revisiting this:
For some reason this seems to work perfectly for me in Xephyr, but break when I run it in as my actual WM. The same is true even post-#59.
 If you don't have a similar experience it's entirely possible it's a peculiarity of my system somehow that I'll have to hunt down since NixOS can be weird like that sometimes.

I was trying it on Xephyr when I tested, however right now at least between firefox and thunar it seems to be working fine. Some random old image to test:

I'm truly lost now -- it seems to be working again. I am testing a minor change currently, let me revert it and get back to you.

Nope, still working. Maybe I was still running a pre-fix version by accident somehow.

Oh well, at least it's working now :shrug:


Bugfix : try_push blocks

Hello,
Using the following queue:
  using Queue = atomic_queue::AtomicQueueB2<
  /* T = */ float,
  /* A = */ std::allocator<float>,
  /* MAXIMIZE_THROUGHPUT */ true,
  /* TOTAL_ORDER = */ true,
  /* SPSC = */ true
  >;

when the queue is full, try_push blocks.
A fix for this behaviour is here : https://github.com/OlivierSohn/cpp.algorithms/commit/37ccb8dbc95c0a1541073741166224b889a29e62
I verified that it solves my use case.
Feel free to use it!

when the queue is full, try_push blocks.

Thanks for a tip, I'll see if I can reproduce it in a unit test.

I added a unit test for try_push and it passes.
It is likely that the error was in your code, not the queue.

@max0x7ba Thanks for looking at it, I think the bug existed a year ago or so, and has been fixed since by a commit on master (or maybe I messed up something in my local version).
So I'll resync with master and everything will be fine ;)


Master

2014302580168-艾麦尔·图尔荪-Assignment1.zip

2014302580168-艾麦尔·图尔荪-Assignment1.zip

2014302580168-艾麦尔·图尔荪-Assignment1.zip

2014302580168

This looks like an accidental pull request and/or spam, closing this out.


Create a score mechanism

Create a mechanism to calculate, display and store the score of a player. Additionally maintain a history of players (usernames) and their respective high scores. Possibly display a leaderboard through a menu option at the beginning or after GAME OVER display.

Score mechanism added.

score mechanism added


Link to list of SIGs from calendar page

Also wouldn't hurt to link to SIG calendar pages so they know where to update their calendar entries
/assign

/remove-lifecycle stale

/remove-lifecycle rotten

/close


converstion to REST initial checkin

Changing from custom protocol to HTTP/HTTPS.
Removing dependency on LIBUV and adding dependency for Simple-Web-Server built on ASIO.
Cleaning up RPC model. Moving towards a source/sink model for re-eventing and data ingestion.

GCC is much fussier than VC, which is good. GCC picked up some goofy return types and non-standard templating. Standalone ASIO appears to require pthreads, which are likely linked by automatically by any C++11 (or greater) linux compiler, but it would appear you must explicitly ask for them to make it link.


Check for missing open source template files

Use node to check for missing open source template files and add them to your project as needed. Also activates .env file if you have one, adds it if you don't
Additions

check-it.js checks for missing open source files including TERMS, CHANGELOG, CONTRIBUTING
also looks for .env file in prep for pulling in the license-exceptions module in future PR
package.json adds dependencies for the above to function
options/flags for running on command line: -fix to automagically add missing files; -scrub to run the scrub function (tba, just checks for .env file right now)
thank you vax for basically doing exactly what i needed this to do already 👏

Testing

npm install
run npm link so the command will work
do one of these:

check-it -fix -scrub .
or try it without the optional flags
check-it .
or try it on another directory
check-it -fix -scrub ../path/to/another/repo/
Review

@contolini

Preview this PR without the whitespace changes
Todos

update readme so someone can actually use this

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

👍 Very cool tool!


Create/Edit page - Unnecessary "Could not save!" message

Edit an existing approved listing
Scroll down to the icons section and delete one of the icons by pressing the trashcan
Add an icon to replace the one you just deleted by pressing "Choose File" and choosing an image file
Press the Save button
You get a "Could not save!" pop-up.

The listing changes are correctly saved, but the pop-up is confusing and wrong.

@wski  This is still an issue in production can you take a look

@clarkjefcoat - tested this and could not recreate. Theory - its been resolved.
@tonyalletag

Agreed looks like this has been fixed.

@tonyalletag
@akonsowski
@wski
@clarkjefcoat - The issue does not occur based on the use-case above, however when doing the following the issue returned:

Edit an existing approved listing
Scroll down to the icons section and delete one of the icons by pressing the trashcan
Don't replace the icon you just delete
Press the Save button
You get a "Could not save!" pop-up (expected)
Replace the deleted icon with a new icon
Press the Save button
You get a "Could not save!" pop-up (not expected)
The listing is updated and saved (expected)

“The listing changes are correctly saved, but the pop-up is confusing and wrong.”

Fixed. Please use the use-case in the above thread to test.
@wski
@akonsowski
@clarkjefcoat

@clarkjefcoat - I thought I had created another bug by fixing this one, but I didn't . If you have multiple screenshots and you update one, the one you update will drop to the bottom of the list. Please confirm that this is how the page should work. - thanks!

@rkenyon1969  It sounds like you have already confirmed that the reordered screenshot issue existed prior to your work on this bug.  I think you should file a new bug report about it so that we have a record of it.  I don't anticipate a fix for that issue will be at the top of anyone's list.

@wski
@akonsowski
@clarkjefcoat
Please review:

Edit an existing approved listing
Scroll down to the icons section and delete one of the icons by pressing the trashcan
Don't replace the icon you just delete
Press the Save button
You get a "Could not save!" pop-up (expected)
Replace the deleted icon with a new icon
Press the Save button
You get a "Could not save!" pop-up (not expected)
The listing is updated and saved (expected)

Passed on latest

Passed on Latest.

Passed on Staging.


Update MSRV to 1.71.0

With the 1.73.0 release today update our MSRV to a one-higher version. This will cause 1.73.0 to be tested in CI.

The most recent failure was a MIRI failure which I've now addressed through refactoring how Table with static memory is represented.


How to integrate CppMicroservices with Qt (LIKE framework) to have something like TornadoFX or DromblerFX (OSGi) like

Greetings all!
I would like to know if someone already achieved integrating CppMicroservices with a UI framework like Qt?
I've come across those really advanced UI frameworks on Java OSGi side:

https://github.com/edvin/tornadofx
https://www.drombler.org/drombler-fx

And I've looked how easy they are to use and it seems that it is really well designed...
https://www.youtube.com/watch?v=liOFCH5MMKk
So basically you can just hot reaload/install any UI components without ending the OSGi runtime.
I was wondering if this was achieve with CppMicroservice and if you guys are already using something like this.
Best Regards.

The framework is not tied to a particular use case, but I am not aware of anything similar to what you linked.
No one else commented so far, so I am closing this for now. Feel free to open a new issue if you have a more concrete question or problem.

@saschazelzer can you reopen this thread it seems that I found out what I was relating to for C++.
https://www.mitk.org/wiki/BlueBerry
BlueBerry is a UI Framework that is part of MITK project. But since MITK is a massive project I started an initiative and tried to move it one level lower within CTK Framework.
The repo is here : https://github.com/al-sabr/CTK/tree/ctk-blueberry-framework
Do you think you can help me to figure out the missing parts to make it work?
Best regards

I am afraid this discussion doesn't belong here.
I don't have the capacity to maintain or work on BlueBerry and it is maintained by MITK developers currently. I doubt that CTK is a good place either, since none of the CTK-devs (AFAIK) know the BlueBerry codebase. There just doesn't seem to exist enough interest in the existing communities for this, so maintaining your own independent fork of it looks like the best way to go.


chapter 4

in chapter 4, equation 4.226 there should be a minus (-) sign in the upper bound, but it is written as a plus (+) sign.

Actually i think the sign error was in 4.225...


Support a local ARM image build

The current Dockerfile builds from eclipse-temurin:17-jdk-alpine which is not available for ARM. Update to use an ARM image.
The eclipse-temurin:17-jdk-jammy image works for ARM and I've tested it locally.

This may also be a good time to update to use a JRE image. The current image is JDK-based, but Data Prepper shouldn't need the entire JDK since has already been built, just the JRE to run.


Switch to udx-native

Supersedes #82. Depends on mafintosh/dht-rpc#40.

Landed in master, with the additional handshake tweak from our other PR \o/ - leaving this branch around as we have some tests using it


Peer to peer code review

Very good work done. we have just one suggestion. Please make sure that your live demo link is not broken.

Very compact and great code structure! Well done Rachel 💯


Objective unicity

Fix #221

Thanks @Kelvin-M. LGTM

@jmorel  I'm closing it, feel free to take the code when you want to do this update :)


[Bug][dolphinscheduler-server] For a process that is executing a failed retry task, stopping it will change the status to success

Describe the bug
A task in process B depends on process A, and the dependent task supports failed retry
When process A fails to execute, process B is executing the failed retry task
At this time, if process B is ready to pause, the process status will be changed to failed
While process B is ready to stop, the process status is changed to success
Expected behavior
I think that when process B is stopped, the process status should also be changed to failed.
Additional context
In the method runProcess(), the failed retry task will not be put into the completeTaskList. If the process is ready to stop, the failed retry task will be skipped, and update the process status to success.
However, if the process is ready to pause, the method hasRetryTaskInStandBy() is used to judge whether the failed retry task is included, and if it exists, it will be regarded as a failure.
private void runProcess() {
            ......
            // node fails, retry first, and then execute the failure process
            if (task.getState().typeIsFailure()) {
                if (task.getState() == ExecutionStatus.NEED_FAULT_TOLERANCE) {
                    this.recoverToleranceFaultTaskList.add(task);
                }
                if (task.taskCanRetry()) {
                    addTaskToStandByList(task);
                } else {
                    completeTaskList.put(task.getName(), task);
                    if (task.isConditionsTask()
                            || DagHelper.haveConditionsAfterNode(task.getName(), dag)) {
                        submitPostNode(task.getName());
                    } else {
                        errorTaskList.put(task.getName(), task);
                        if (processInstance.getFailureStrategy() == FailureStrategy.END) {
                            killTheOtherTasks();
                        }
                    }
                }
                continue;
            }

private void updateProcessInstanceState() {
        ExecutionStatus state = getProcessInstanceState();
        if (processInstance.getState() != state) {

private ExecutionStatus getProcessInstanceState() {
    ......

    // pause
    if (state == ExecutionStatus.READY_PAUSE) {
        return processReadyPause();
    }

    // stop
    if (state == ExecutionStatus.READY_STOP) {
        List<TaskInstance> stopList = getCompleteTaskByState(ExecutionStatus.STOP);
        List<TaskInstance> killList = getCompleteTaskByState(ExecutionStatus.KILL);
        if (CollectionUtils.isNotEmpty(stopList)
                || CollectionUtils.isNotEmpty(killList)
                || !isComplementEnd()) {
            return ExecutionStatus.STOP;
        } else {
            return ExecutionStatus.SUCCESS;
        }
    }

private ExecutionStatus processReadyPause() {
    if (hasRetryTaskInStandBy()) {
        return ExecutionStatus.FAILURE;
    }

    List<TaskInstance> pauseList = getCompleteTaskByState(ExecutionStatus.PAUSE);
    if (CollectionUtils.isNotEmpty(pauseList)
            || !isComplementEnd()
            || readyToSubmitTaskList.size() > 0) {
        return ExecutionStatus.PAUSE;
    } else {
        return ExecutionStatus.SUCCESS;
    }
}

Which version of Dolphin Scheduler:
-[1.2.0]

Already fixed in 2.x.


feat(oxc): add oxc_napi crate

#7612  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


[5.4] Output migration name before starting a migration or rollback

Laravel shows the migration name as output when an UP or DOWN has finished, but does not show any output before starting either.
It would be useful to see the migration name before it starts,  in the runUp and runDown methods.
I spent some time today trying to figure out why my simple migration was just hanging -- rebooting my VM and such, when it was actually a 10+ minute long migration a teammate had slipped in that was running before mine. Since there is no output until finished, I thought something was hanging as the cursor just sat below the command prompt with no indication of anything happening.
It would have been useful to know that it was busy running THAT migration.
For language, I’m propose this in this pull request:
$this->note("<info>Rolling back:</info> {$file}");
$this->note("<info>Migrating:</info> {$file}");

(to matched the ‘Rolled Back: ‘ and ‘Migrated: ' messages that are show on completion.)

There is a formatting difference where yours has the whole file path and the completed notification just has the basename style output.

Thanks - I fixed that. I had tested on 5.3 where $file is what was displayed, and pulled the code from there.
I changed this to $name to match.


Johnbillion contextual parameters

Additional tests for #1077

Coverage decreased (-1.3%) to 87.198% when pulling 78f4f1adc4d0d6d6be90bbad8a9ae23782a3468b on johnbillion-contextual-parameters into e8b344b25a89e48304adb92065b9e8a4dad57a72 on master.

Looks good!


Makefile?!

Are you open to something like:
sudo make install

As this script do not require building any code, or having a developer environment installed, I am not a fan of the idea of using make to install as this would add extra requirements. I know I do not have make installed on many of my systems. However, make is the standard way to install source software, so if somebody whip something up I can take a look.


Polylines appear "broken"

As seen in the attached image polylines are not really straight. I've just got black lines with a stroke of 0.3 (have also tried stroke width of 1 and 3). In all cases lines appear broken. It's worth mentioning I have several hundred lines shown, but it's still unclear to my why they're not just straight.
Haven't done anything special, no custom tile server. Map is saved as a png with default options, but even setting the compression level to 0 yields a similar result
Running on macOS catalina
node v14.4.0
staticmaps ^1.4.1

Hi @Tyler-Keith-Thompson
Thanks for the hint.
The effect you're describing is called Anti-Aliasing. I need to investigate if it could be easily implemented into the used components.

https://github.com/lovell/sharp/issues/1401


ceph_deploy/mon:ceph-deploy should setup the keys.

In my opinion making authorization keys is a
configuration management task not an init script task.
The systemd init scripts in ceph do not make keys
for mon daemon as they do with system V init scripts.
Signed-off-by: Owen Synge<EMAIL_ADDRESS>

Seems this has been fixed by you guys already

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-deploy-pull-requests/1580/
Tests failed for this pull request.


Email Stuck

Marketing team tried to send out an email blast,
Server ran out of disk space & RAM.
We rebooted the server, upgraded the RAM and moved the postal database over to a NAS to resume operations.
Now all email sent to postal is in state pending, but nothing is delivered.
Is there a way to force all queued messages to attempt delivery?

Do you mean new messages go to pending or the messages that were trying to be sent before the crash?
If it was before the crash, you would have to go in the database and check the queued_messages and clear the locked_by, locked_at, and retry_after fields or something along those lines.

clearing the locked messages by following the steps described in -
Issue when mail stuck in queue possibly solved #1021


Fauna Entomologica Scandinavica

Series of Scandinavian publications. https://brill.com/view/serial/FES

ID
Title
Volume

B15
The Sepsidae (Diptera) of Europe
37

B20
The Sphecidae (Hymenoptera) of Fennoscandia and Denmark
4

B458
The Dryinidae and Embolemidae (Hymenoptera: Chrysidoidea) of Fennoscandia and Denmark
30

B481
The Formicidae (Hymenoptera) of Fennoscandia and Denmark
8

B566
The Sphecidae (Hymenoptera) of Fennoscandia and Denmark 1
4

B567
The Sphecidae (Hymenoptera) of Fennoscandia and Denmark 2
4


Fix "time ago" string generation for Featured Components

The "time ago" value used to always show a relative timestamp i.e. 1m ago 3w ago but now it displays a relative timestamp only for short diffs and a date for longer diffs. We are always appending the string ago to the time value, which leads malformed output when we get a date
This screenshot shows both types in effect

Solution options:

Revert to always showing relative time and continue appending ago to every entry
Determine the breaking point where the time display switches to date and omit ago in those cases

Both makes sense, it's a matter of UX preference
Relevant Code
near/widget/ComponentCard
<CardTag>
  <i className="bi bi-clock"></i>{" "}
  <Widget
    src="mob.near/widget/TimeAgo"
    props={{
      blockHeight: props.blockHeight,
      keyPath: `${accountId}/widget/${widgetName}`,
    }}
  />{" "}
  ago
</CardTag>

Eugene published updates to the TimeAgo component which caused this bug. Not his fault, our implementation makes assumptions that his component would always return in a certain format. We can execute a quick fix by version locking our embed of his component to a previous block height— reverting the changes which caused this bug

@k3y0k3y my PR reverts to the original behavior which is an improvement, but I see why Eugene changed it. Older values are actually represented in days e.g. 96d ago
I will put in a low prio ticket for adopting the new behavior as time allows


[Request] Option to create a minimum number of samples

Using some samples instead of the whole file saves a ton of time, but if there are too few samples, results could get skewed more often.
In my experience using ab-av1, that is one of the biggest sources of inaccuracy and it affects everything, the vmaf score, recommended crf, estimated encoded percent and so on. So the results for longer files are usually more accurate than for shorter files.
I'm trying to batch process multiple files of very different duration. Right now we have --sample-every, that works better as a general rule for longer videos, and --samples that is useful if we want a fixed number, but as one option overrides the other, there is no way to have a solution that works well for both short and long videos at the same time.
I'd like to set a minimum number of samples (--min-samples) that does not override the default --sample-every argument to increase the accuracy of results for short files, without negatively affecting long files, that should still have more samples based on the default 12 minutes. So, for instance, if --min-samples is set to 4 for a 20 minutes file, it would create 4 samples instead of the current 2, and for a 120 minutes file it would still create 10 samples.
Would you please consider adding an option that works something like that? I think it could be quite helpful.

Related to that, but as a separate idea, there could be a way to improve the default behavior of creating samples to increase or decrease the frequency based on the duration of the video instead of the current fixed frequency or a fixed sample number. That way, it would automatically create some more samples for shorter files and not so many for very long files, so it would adjust better to a greater variety of files durations.
I'm suggesting this because of some of the results I've been getting lately with shorter videos. Right now, due to how video encoding works, in files shorter than 12 mins with only 1 sample it seems almost like playing the lottery, more a guess than an estimation.
Example
Lets take a not so short video as an example:
Video duration of 14m --> 2 samples by default --> VMAF target 93
crf 26.5 VMAF 93.41 predicted video stream size 175.29 MiB (14%) taking 14 minutes
Runtime: 00:04:09

But then the real encoded file was VMAF 94.39 and 202MB.
I know it does not seem that much, but this was a small file, it is just an example, the vmaf was quite higher than target and the file size is 15% higher than expected. With bigger files or higher target vmafs the differences would have been greater. That is the problem with having so few samples that the results get skewed.
By running sample-encode at suggested crf 26.5, but this time with 3 samples, this is the resulting prediction:
VMAF 93.83 predicted video stream size 206.25 MiB (17%) taking 12 minutes
Runtime: 00:01:20

That is almost spot on (and with the new alternative method proposed in the other thread to estimate size it would have been even more accurate, I checked). The problem now is that I was targeting VMAF 93, and the encoded result was quite higher, so I could have probably used a lower crf, and that margin is not caused by a loose crf-search, but the low sample count.
So if I run crf-search again but with 3 samples, this is the result I get:
crf 27 VMAF 93.36 predicted video stream size 192.34 MiB (16%) taking 12 minutes
Runtime: 00:05:26

The runtime gets slightly increased, but not very significantly (its a reasonable time), and now the prediction almost matches the real encoded file (vmaf is still underestimated, so probably a crf27,5 would have meet the vmaf target, but well, it is better than the initial prediction), which makes this tool more reliable. I think that is quite important, to know beforehand that the predictions are expected to be close enough to the real encoded files, so we don't have to double check if something went wrong, or wonder if I could have encoded differently and still achieve the minimum targets.
I understand this tool offers an approximation and it will never be perfect, but I think it could be improved a bit.
Possible automatic solution for any given file duration
A linearly scaling sample frequency could be a possible solution to this situation. One way to do it would be with a function using a base number that stabilizes the frequency + a percentage of the video duration so the frequency scales up with it, but not too little or too much. A scaling rate of 5% is pretty smooth. I also tried a 10% rate, but it was way too fast scaling in my opinion. Maybe a middle ground could work very well, like 6%, as shown in the example below.
sample_frequency = 4m + 0.06 * video_duration

sample_frequency = 4m + 0.06 * 10m = 4,6m (it results in 3 samples instead of the current 1, which would greatly improve the accuracy)
sample_frequency = 4m + 0.06 * 20m = 5,2m (it results in 4 samples instead of the current 2)
sample_frequency = 4m + 0.06 * 30m = 5,8m (6 samples instead of current 3)
sample_frequency = 4m + 0.06 * 60m = 7,6m (8 samples instead of current 5)
sample_frequency = 4m + 0.06 * 120m = 11,2m (11 samples instead of current 10)
sample_frequency = 4m + 0.06 * 180m = 14,8m (13 samples instead of current 15)

I'm showing this scaling rate because it benefits both the accuracy of shorter files while it also offers time savings in longer files, so everybody could take advantage of it.
It could be tweaked to scale more or less aggressively, maybe even by the user, or have a default mode and then an option to enable a more thorough mode, like the mode in crf-search. Nevertheless, it should always respect the --min-samples argument.
Of course, this would increase the runtime for most files, proportionally more for shorter videos, but the number of samples is still not that big, so it should not extend too much, specially in sample-encode as it does not need to do multiple runs with the same samples, and we gain way better predictions, which helps the reliability of ab-av1. Besides, any time penalization is compensated by the new cached results if the file is retested for a different vmaf or something like that, which would offer almost instant results while maintaining the higher accuracy.
I think this improvement to the sample splitting logic could also be a nice addition.

Thank you, that mostly covers it.


Not being able to log in to the demo app

using:
username: demo
password: demodemodemo
Did the user and password changed for the demo app?

@viniciusgama ,
it was deleted. Try again.


Add case insensitive string using i suffix

Allow writing "str"i, "STR"i or 'str'i and it will match all case combinations
of str: str, Str, STr, STR, sTr, and so on.
This makes it unnecessary to write a rule like:
str <- [Ss] [Tt] [Rr] ~;
This simplifies writing parsers for languages where the keywords are
case insensitive.

At Alert Logic we depend on Neotoma and this PR is important to us.
Until this PR has been merged we will be maintaining our own fork at https://github.com/alertlogic/neotoma


[VQC-CHALLENGE] [Feynman-Prodigies] [October 27, 2022]

version 1 of the model

Team members
Rohit Saini
Abdullah Kazi
Pranav Maheshwari
Rishabh Singh
Vansh Jhunjhunwala
Soubhadra Maiti
Submission summary
Depth - 6
Shots - 200
Iterations - 50
Batch Size - 10
Step Size - 0.1
Test Accuracy: 100%
Additional comments
Checked the model on 2 different datasets and test accuracy is consistent.

Congrats! Your model achieved 100% accuracy on our judging data set and joins the top of the leaderboard. As a next step, your team is encouraged to take on the quantum kernel estimator challenge, which will act as a tie-breaker to determine an overall winner. Happy hacking!


Find out where circular structure error originates

Periodically circular references show up in my code, but I never have any indication of what socketcluster is trying to serialize. Is there any way I can get a more descriptive stack trace of what it's trying to do, ie, where the error originates?

@seiyria Which version are you using on the server?

@jondubois - I'm running 4.3.2 right now. The problem was actually a problem in my code, for sure, where I had a recursive structure I was trying to serialize (it was an ES6 promise - I forgot to use await and instead just called a function that returned a promise). To be clear, my problem was that I had no idea where that stack trace was originating (it only showed internal errors in socketcluster).

Do you think if we switched the serializer to https://www.npmjs.com/package/json-stringify-safe it would solve the problem?

I know that hprose's serializer can do references just fine:
> var hprose = require("hprose")
undefined
> var a = { foo : 42 }
undefined
> a.ref = a;
{ foo: 42, ref: [Circular] }
> var out = hprose.serialize(a)
undefined
> out.toString()
'm2{s3"foo"i42;s3"ref"r0;}'

https://github.com/hprose/hprose-nodejs
You should be able to use the serializer and de-serializer independently of the rest.

I think I definitely want to be told that my object is circular - I definitely didn't intend for it (maybe having this as an option would be good!). I was just having problems with figuring out where the circular object came from.

@seiyria Did the error look like this?
child_process.js:479
    var string = JSON.stringify(message) + '\n';
                      ^
TypeError: Converting circular structure to JSON
    at Object.stringify (native)
    at process.target._send (child_process.js:479:23)
    at process.target.send (child_process.js:416:12)
    at handleError (/projects/sc/node_modules/socketcluster/lib/workercluster.js:133:13)
    at EventEmitter.emit (events.js:107:17)
    at EventEmitter.SCWorker.errorHandler (/projects/sc/node_modules/socketcluster/lib/scworker.js:348:8)
    at Domain.<anonymous> (/projects/sc/node_modules/socketcluster/lib/scworker.js:33:23)
    at Domain.emit (events.js:107:17)
    at emitError (domain.js:83:24)
    at Domain.errorHandler [as _errorHandler] (domain.js:122:16)

No, it looked like this:
1456704624387 - Origin: Worker (PID 2757)
   [Warning] Error: Cannot traverse circular structure
    at convertBuffersToBase64 (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/sc-formatter/index.js:52:11)
    at convertBuffersToBase64 (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/sc-formatter/index.js:75:21)
    at convertBuffersToBase64 (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/sc-formatter/index.js:75:21)
    at convertBuffersToBase64 (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/sc-formatter/index.js:75:21)
    at Object.module.exports.stringify (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/sc-formatter/index.js:83:22)
    at Emitter.SCSocket.stringify (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/socketcluster-server/scsocket.js:208:20)
    at Emitter.SCSocket.sendObject (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/socketcluster-server/scsocket.js:214:16)
    at /Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/socketcluster-server/scsocket.js:249:14
    at EventEmitter.SCServer.verifyOutboundEvent (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/socketcluster-server/scserver.js:684:5)
    at Emitter.SCSocket.emit (/Users/seiyria/GitHub/backend/node_modules/socketcluster/node_modules/socketcluster-server/scsocket.js:227:17)
Cannot traverse circular structure

@seiyria I think the problem in your case might be that your stack trace isn't big enough to show the source of the error inside your code. Try running with node --stack-trace-limit=1000 server.js

Even with that, the stack trace is the same.

What node version are you using?

Can you provide a simple/short code snippet I could copy-paste to reproduce the error on my machine?

You could also try https://www.npmjs.com/package/longjohn

It's tough to nail down something small, but I'm thinking async/await is factoring into this mess somehow. I'm on node 4.2.2. I haven't tried longjohn, but I'll give it a try tomorrow.

Thanks for reporting this. Let me know how things go.
The --stack-trace-limit flag definitely works for me in Node v4.2.0.
It's weird that the stack trace doesn't go further than Emitter.SCSocket.emit - Something definitely must have called it but it's not showing. Maybe it is related to async await.
I did fix another potential issue related to circular structure in Error objects a few minutes ago but this is not related to your problem.

Strangely, longjohn doesn't work either. I'm not sure what's up, but, I'm not sure if this is anything you can adjust at this point. While I can't provide you a working demo, here's the gist of it:

export default async ({ lat, lon }, playerLevel) => {

    return new Promise((resolve) => {
        resolve([]);
    });

};

        const monsters = nearbyMonsters(player.homepoint, player.currentLevel);
        // normally you'd do await nearbyMonsters instead of just nearbyMonsters
        socket.emit('update:monsters', monsters);

That said, if this is out of your control entirely, feel free to close it.

We have now moved away from the Node.js domain module which was responsible for cyclic errors so hopefully this is no longer a problem.


new XML files to add Azure Mobile packages

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

:x:Error

Details

[Error] System.ArgumentException: An item with the same key has already been added.
at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add) at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable1 source, Func2 keySelector, Func2 elementSelector, IEqualityComparer`1 comparer)
at ECMA2Yaml.Models.ECMAStore.Build()
at ECMA2Yaml.Program.LoadAndConvert(CommandLineOptions opt)
at ECMA2Yaml.Program.Main(String[] args)
Line 210: [Error] W:\vzgq-s.optemp\packages\opbuild.scripts.1.35.5\tools\opbuild\plugin.executor.ps1 failed to execute customized script in the plugin step docset_postbuild in the docset azuredotnet
Line 80: [Warning] Failed to run script W:\vzgq-s_dependentPackages\ECMA2Yaml\tools\Run.ps1 with exit code -1
Line 80: [Warning] Failed to run script W:\vzgq-s_dependentPackages\splitToc\tools\run.ps1 with exit code -532462766
Line 80: [Warning] Failed to run script W:\vzgq-s_dependentPackages\ECMA2Yaml\tools\PostBuild.ps1 with exit code 16

[Error] System.ArgumentException: An item with the same key has already been added.
at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add) at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable1 source, Func2 keySelector, Func2 elementSelector, IEqualityComparer`1 comparer)
at ECMA2Yaml.Models.ECMAStore.Build()
at ECMA2Yaml.Program.LoadAndConvert(CommandLineOptions opt)
at ECMA2Yaml.Program.Main(String[] args)
Line 210: [Error] W:\vzgq-s.optemp\packages\opbuild.scripts.1.35.5\tools\opbuild\plugin.executor.ps1 failed to execute customized script in the plugin step docset_postbuild in the docset azuredotnet
Line 80: [Warning] Failed to run script W:\vzgq-s_dependentPackages\ECMA2Yaml\tools\Run.ps1 with exit code -1
Line 80: [Warning] Failed to run script W:\vzgq-s_dependentPackages\splitToc\tools\run.ps1 with exit code -532462766
Line 80: [Warning] Failed to run script W:\vzgq-s_dependentPackages\ECMA2Yaml\tools\PostBuild.ps1 with exit code 16

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@meganbradley Are you ready for this to be merged? The system tags things "do-not-merge" by default, so I didn't want to assume. :)

Crap, I closed the PR so I could re-open and force another staging build/verification, but the branch from your PR doesn't exist anymore.  :(  Please advise.


Scheduler Jobs in Spoon Web

Expected Behavior
Hi, I need to raise a Spoon Web  in SAP CLOUD but I need to schedule the JOBS and transformation with the SAP Scheduler, is this possible? Is it possible to schedulate the JOBS using Spoon Web in the Cloud?
Actual Behavior
Screenshots and error messages are helpful.
Steps to Reproduce the Problem

Specifications
Server

OS (e.g., Mac OS X 10.12.6):
Java (e.g., Oracle JDK 8u151):
Servlet container (e.g., Apache Tomcat 8.5.20):
webSpoon (e.g., version: <IP_ADDRESS> or commitId: 0211d84b):

Client

OS (e.g., Mac OS X 10.12.6):
Browser (e.g., Google Chrome 64.0.3282.140):

I can see two questions:

I need to schedule the JOBS and transformation with the SAP Scheduler, is this possible?
Is it possible to schedulate the JOBS using Spoon Web in the Cloud?

For the 1st question, I'm not familiar with the SAP Scheduler so I'm not so sure but I'd doubt it. Pentaho Community (now known as Vantara Community) or SAP community, I'm not sure if it exists, would be the best place to ask this question.
For the 2nd question, no and yes.
No, because webSpoon is based on CE version of PDI/Spoon and CE does not include the job scheduler. This is only available in EE (see here for more details about the job scheduler).
Yes, you could "technically" upgrade webSpoon to EE by copying things like Jar files from EE to webSpoon, but I do not support this operation (i.e., do it at your own cost and risk).

Hi Hiromu, thank you so much
Regards
Teresa Duartez Uribe
2018-05-23 14:28 GMT-03:00 Hiromu Hota<EMAIL_ADDRESS>
I can see two questions:

I need to schedule the JOBS and transformation with the SAP
Scheduler, is this possible?
Is it possible to schedulate the JOBS using Spoon Web in the Cloud?

For the 1st question, I'm not familiar with the SAP Scheduler so I'm not
so sure but I'd doubt it. Pentaho Community (now known as Vantara
Community) or SAP community, I'm not sure if it exists, would be the best
place to ask this question.
For the 2nd question, no and yes.
No, because webSpoon is based on CE version of PDI/Spoon and CE does not
include the job scheduler. This is only available in EE (see here
https://help.pentaho.com/Documentation/5.4/0L0/0Y0/060 for more details
about the job scheduler).
Yes, you could "technically" upgrade webSpoon to EE by copying things like
Jar files from EE to webSpoon, but I do not support this operation (i.e.,
do it at your own cost and risk).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/HiromuHota/pentaho-kettle/issues/105#issuecomment-391432419,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARMgwbKCqHU-9_rtIn-LpJ5Lk2VvngHMks5t1ZxPgaJpZM4UIz12
.


Added TypeScript definition

I added the TypeScript definition for getOwnPropetyDescriptors. It doesn't include definition for the polyfill. It might be better to just add a comment to readme on how to modify the Object definition for getOwnPropertyDescriptors.

This (like all typescript definitions) should live in the DefinitelyTyped project; I don't use TypeScript and would have no way of ensuring the types remain accurate.


Features, improvements wanted?

I like hpack and I have also been reading through the sources for tools that work with cabal files, so I would like to contribute to the portion of this library that parses cabal files. Any Todo's already on your mind?

Hey there :)
This library does no cabal file parsing, that's in the Cabal package. If you're interested in it, there's a nice to have which would be not to lose any information when parsing/converting (currently it'll lose it on some places; I can't tell of the top of my head, but looking at the AST it'd be +/- clear what it was).
Other than that... I'm not sure if there're things to catch-up on the hpack side; that'd be nice, to par it up with hpack (rebase/merge it, fix conflicts, make sure it works).
And there's one thing I'd like to have which is a YAML/JSON pretty printer that uses a template or some structured way to define field ordering, so the output is nice and the way to output it isn't a hack. This would be useful for several pretty printers; there's a JavaScript library NPM uses that could be used for inspiration.
But all-in-all, I've no requests. This package solves the problem really well for my personal uses. Maybe @sol has ideas.

Okay, that should be plenty to start. We'll see how much time I get in on these, then go from there.

Other than that... I'm not sure if there're things to catch-up on the hpack side; that'd be nice, to par it up with hpack (rebase/merge it, fix conflicts, make sure it works).

Since this is mostly additive, it might be nice to switch to depending on hpack as a library eventually. If it turns out to be a pretty quick effort, I will create a PR for that later.

The initial project was a PR to hpack, but it is/was a lot of code. The actual diff between our code and the upstream shouldn't be really large, unless I messed-up.
https://github.com/sol/hpack/pull/114

I think #5 is highest priority (see also https://github.com/sol/hpack/issues/170).
After that, I think rebasing on the latest version of hpack and catching up with any changes would be nice  (#5 may actually be solved by that, not sure).
Medium-term, maybe give "using hpack as a library" an other shot.

Sorry about that.
Fixed #5. Going to wait for the travis build to run then publish v0.14.7.
As always, it should be buildable from source given the LTS, available as a web service and with binary distributions for OSX and Linux, which is why this didn't seem like a big deal to me.

would be not to lose any information when parsing/converting (currently it'll lose it on some places; I can't tell of the top of my head, but looking at the AST it'd be +/- clear what it was).

Besides multiple source repos, I think you are handling everything currently? I couldn't find anything else that was obviously missing or truncated for Cabal 1.22.

Pkgconfig, data-dir have open hpack issues. Could parse them for now.

Also, there might be some clever usage of a built in condtree traversal operator to reduce code here.

You can use that snippet if you have time to refactor later.
That's probably all for now... Long term, maybe Cabal the library can use the skeleton of the logic hpack-convert and hpack use to model the logic for converting existing cabal files into their new front end syntax automatically, and for expanding the front end syntax to the internal syntax.


Configure email services

When the domain name is registered email services need to be set up.

Updated NS records in the DNS settings for the domain.
Awaiting propagation of changes, which can take up to 8 hours.

DNS records have been updated successfully.
Added support and admin accounts at examplium.com.


General cleanup of refactored tests in testing-jvm

Some cleanup of the recently refactored JUnit tests.  This is almost exclusively moving some things around and/or renaming things for clarity:

Reorganize JUnit-specific tests into packages underneath the junit package
Convert the failfast integration test to a multiversion test since it has almost identical structure
Merge the categories tests into one test
Eliminate the "jupiter:"/"vintage:" prefixes from versions
Break up JUnitIntegrationTest into function-specific tests
Break up TestingIntegrationTest into function-specific tests and remove it
Rename JUnitMultiVersionIntegrationTest to TestingMultiVersionIntegrationTest since it can be used with TestNG tests as well as JUnit

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

I've triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

I've triggered a build for you.


lr_find() got an unexpected keyword argument 'train_dataloaders'

Bug description
Bug description
lr_find() got an unexpected keyword argument 'train_dataloaders'
model = BasicLightningTrain() # First, make model from the class

trainer = L.Trainer(max_epochs=20)

lr_find_results = trainer.tuner.lr_find(model,
                                        train_dataloaders=dataloader, # the training data
                                        min_lr=0.001, # minimum learning rate
                                        max_lr=1.0,   # maximum learning rate
                                        early_stop_threshold=None) # setting this to "None" tests all 100 candidate rates
new_lr = lr_find_results.suggestion() ## suggestion() returns the best guess for the optimal learning rate

print(f"lr_find() suggests {new_lr:.5f} for the learning rate.")

model.learning_rate = new_lr

How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

lr_find() got an unexpected keyword argument 'train_dataloaders'
Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 1.10):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response
How to reproduce the bug
run notebook:
https://drive.google.com/file/d/1-UXgqb9H-i9YVmHn2HpihYNGCdTnYiac/view?usp=sharing
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 1.10):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response

@skr3178 You didn't provide the information that was required in the bug report template. The version of Lightning and the fully error trace would be useful.

@awaelchli i added a link to notebook.
It is run with colab GPU enabled.

@skr3178 Which version of Lightning do you have installed?


Reading connection strength

Not really an issue but rather a question, so if this is not the right place for that, I'd be happy to post where it should be.
I've been using esp32-nimble crate a for a small DIY project (basically doing https://www.thingiverse.com/thing:2146425 but with esp32-c3, Rust, and over Bluetooth). One thing I would like to add to my remote controller is connection strength information.
There's a get_power on BLEDevice https://taks.github.io/esp32-nimble/esp32_nimble/struct.BLEDevice.html#method.get_power but I suspect it just returns the configured power transmit, and not the actual signal level ?
There is also rssi exposed on BLEAdvertisedDevice https://taks.github.io/esp32-nimble/esp32_nimble/struct.BLEAdvertisedDevice.html#method.rssi ; does it means one need to keep track of the BLEAdvertisedDevice after connection to keep the signal information, or this is only valid after sanning for bluetooth devices ? (I suspect it is the latter).
Searching for RSSI on Espressif's docs yields only https://docs.espressif.com/projects/esp-idf/en/v5.1.1/esp32c3/api-reference/bluetooth/esp_gap_ble.html?highlight=read_rssi#_CPPv421esp_ble_gap_read_rssi13esp_bd_addr_t which I suspect is what is exposed by BLEAdverisedDevice::rssi() ?
So, question is: is there a way to get signal strength on a live connection ? Or one of the above is the right solution ?

Question is welcome.
There is a function to obtain the RSSI of the last communication.
https://taks.github.io/esp32-nimble/esp_idf_sys/fn.ble_gap_conn_rssi.html
Currently, BLEClient does not have an API that uses this function.
I will add an API to my program that uses this function.

That was quick, thanks !

Tested and my two esp32-c3 boards being ~30cm apart, it reports -38~-41 of RSSI (I suspect it is reported in dBm)

Received signal strength indication in dBm is mentioned in https://github.com/espressif/esp-nimble/blob/bdf2c7588a5c7d0c16e98a168b0cf41578b119ef/nimble/host/include/host/ble_gap.h#L431


⚠ Secret Website 3 has degraded performance

In 7183866, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2891 ms

Resolved: Secret Website 3 performance has improved in ef49f80.


Set name for restore point/snaphot

Before doing potential dangerous things or having successfully set up something important on an Azure VM - I would like to name the specific snapshot/restore point I manually trigger - so that I can easier understand what the state of the vm where in that snapshot/restore point in the future.
I could not found any azure cli command for how to manually create a restore point - but in the portal in VM it is created like so:

Describe the solution you'd like
After clicking Backup now - entering the following page:

... There could be and entry as well  where one could optionally give a name/description to the restore point.
And/or: Create cli command like az backup recoverypoint create. Add a -name flag with the command.
And a column for name/description in the list:

Describe alternatives you've considered
Manually keeping a note where I describe the state of the vm - with the time stamp of the restore point. Or create a azure table doing the same thing.

Could not make this in the next semester planning but acknowledging the feedback. Added this to our backlog request. Closing this issue as a feature-request in backlog.


Debian package fails

Debian package fails:
ubuntu@ubuntu1604:~$ sudo dpkg -i bitcoinz-wallet_1.1.0.deb
Selecting previously unselected package bitcoinz-wallet.
(Reading database ... 270283 files and directories currently installed.)
Preparing to unpack bitcoinz-wallet_1.1.0.deb ...
Unpacking bitcoinz-wallet (1.1.0) ...
dpkg: dependency problems prevent configuration of bitcoinz-wallet:
bitcoinz-wallet depends on bitcoinz; however:
Package bitcoinz is not installed.
dpkg: error processing package bitcoinz-wallet (--install):
dependency problems - leaving unconfigured
Errors were encountered while processing:
bitcoinz-wallet

Just what it says - your are missing DEB dependencies:
bitcoinz-wallet depends on bitcoinz; however: Package bitcoinz is not installed.
So you probably need this:
https://github.com/bitcoinz-pod/bitcoinz/releases (bitcoin deb or compile it)
DEB bitcoinz wallet info:
dpkg-deb -I bitcoinz-wallet_1.1.0.deb
new debian package, version 2.0.
size 335522 bytes: control archive=426 bytes.
66 bytes,     2 lines      bitcoinz-wallet      #!/bin/bash
196 bytes,     9 lines      control
139 bytes,     2 lines      md5sums
Package: bitcoinz-wallet
Version: 1.1.0
Section: misc
Priority: low
Architecture: all
Depends: default-jdk, bitcoinz
Installed-Size: 348
Maintainer<EMAIL_ADDRESS>Description: BitcoinZ Wallet

You just need to install bitcoinz package dependency from releases page.


 You only added check that run is exits within 10 seconds.

I was going to ask the same question as @LK4D4.  The removal has to be done by the time docker run exits, so unless there is some other inconsistency in daemon state, I'm unclear what is "racing" and whether this just hides it?

@estesp I can't reproduce that race on my machine, nor with master, neither with this PR.
But it failing pretty often on jenkins, with leaving containers after docker run --rm

@LK4D4 @estesp I was using TESTFLAGS='-race -v -run TestRunContainerWithRmFlagExitCodeNotEqualToZero' make test-integration-cli as @jfrazelle suggested to reproduce and indeed I can always reproduce it inside dev container
however, seems that when running a container with a nonexistent command or when a command gives error the cli wait a bit before exiting, so I added that. It was failing even with waitInspect from utils

@runcom I don't understand motivation of done channel still. It is like voodoo or orcs magic from warhammer.

@LK4D4 it's just to sync. To me, it seems that run doesn't fully block and with that -race above it not wait to finish and move over. Putting that in a go routine and wait if finished seemed the way to go to properly wait to finish. I don't know of course if there is some inconsistency with deamon state

@runcom No, if we don't take GC/scheduler stuff in account, then your code totally equivalent to previous except for timeout check.
We can merge it as is, but I want to be sure, that all understand that code is same.

@LK4D4 no, I'm more into understanding why these tests fails randomly or are racy than merging this

I don't know what's up here but I can reproduce this anymore with
TESTFLAGS='-race -v -run TestRunContainerWithRmFlagExitCodeNotEqualToZero' make test-integration-cli
:confused:

@runcom Yup, I'm too.

hmmm ya the overlay tests I am running to fix the issue are still apparent with this patch :(

I'm going to close this sorry I think I have a different fix :(

btw it was not these tests, it was specific to running overlay in overlay and containers failing to be removed from build tests

good to know!


Volume affinity conflict with node

I am facing following issue with volume affinity
Volumes:
isamconfig:
Type:       PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
ClaimName:  isamconfig
ReadOnly:   false
isamconfig-logs:
Type:       PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
ClaimName:  isamconfig-logs
ReadOnly:   false
default-token-v674z:
Type:        Secret (a volume populated by a Secret)
SecretName:  default-token-v674z
Optional:    false
QoS Class:       BestEffort
Node-Selectors:
Tolerations:     node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:
Type     Reason            Age                 From               Message

Warning  FailedScheduling  33s (x19 over 24m)  default-scheduler  0/2 nodes are available: 1 node(s) didn't match node selector,1 node(s) had volume node affinity conflict.

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

ID: d1ac572f-c613-71fc-580a-2805b740b793
Version Independent ID: b4aeef15-7b0f-abc7-872e-28051e10d6ca
Content: Dynamically create Azure disks volume - Azure Kubernetes Service
Content Source: articles/aks/azure-disks-dynamic-pv.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@VIPULKAM Thank you for reaching out, our team will review and get back to you at the earliest.

@VIPULKAM Apologies in delayed response. As per the error shared, it seems this could be because of taint/tolerance values.
Can you run kubectl describe node  and see if any taints are been setup and is that matching with tolerance in the pod?

Nope I have not set taint and tolerance on nodes. I have set label and
using selector in deployment defination
On Tue, Dec 1, 2020, 4:11 AM VikasPullagura-MSFT<EMAIL_ADDRESS>wrote:

@VIPULKAM https://github.com/VIPULKAM Apologies in delayed response. As
per the error shared, it seems this could be because of taint/tolerance
values.
Can you run kubectl describe node and see if any taints are been setup
and is that matching with tolerance in the pod?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/66843#issuecomment-736375070,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIE3EX6JQOSHMOJ3BEZFQGLSSS6LZANCNFSM4UGVG6JQ
.

@VIPULKAM I would recommend to open a support case as this need deeper investigation and troubleshooting.
Once the issue is resolved, request you to post the solution/resolution here for the benefit of community.
Incase you don't have a support plan, let me know so that I can help you by enabling a onetime free support on your subscription.

Thanks Vikas , I do not have a support plan with my subscription.
Best Regards,
Vipul Kadam
On Tue, Dec 1, 2020 at 8:52 AM VikasPullagura-MSFT<EMAIL_ADDRESS>wrote:

@VIPULKAM https://github.com/VIPULKAM I would recommend to open a
support case as this need deeper investigation and troubleshooting.
Once the issue is resolved, request you to post the solution/resolution
here for the benefit of community.
Incase you don't have a support plan, let me know so that I can help you
by enabling a onetime free support on your subscription.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/66843#issuecomment-736602814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIE3EX2RMJL4BRF5ZVPFAULSST7MXANCNFSM4UGVG6JQ
.

@VIPULKAM You can email me with Subject as "ATTN: Vikas" at<EMAIL_ADDRESS>and provide me with your SubscriptionID and link to this thread. I can then enable your subscription for one time free support.
Once the issue is resolved, request you to post the solution/resolution here for the benefit of community.
Looking forward for your email.


Clean-up unused functors

ShuffleAndReverse, InflatePadAndShuffle and TransformDepth were included in the very first public commit of tensorflow.
They are used at the beginning, but as far as I can see they are not used anywhere in the codebase for a long time.
Is there any reason keeping them around?

@smit-hinsu can i merge this PR internally ?

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!

I'm interested in pursuing this but the mentioned build failure with clang cannot be reproduced by me.

Rebased.

Can one of the admins verify this patch?

@smit-hinsu I did a rebase. Will you have a chance to review it again?


Problem with main repo app: book purchase

After deploying the app both in local env and production env (zeit), I found: a regular user could not open the book and read the chapter even though:

it shows the book on the list of 'Your books' after going through the purchasing procedure.

After trying open it, it show the TOC, and big blue 'Buy button'

but it worked when I follow the instructions and code of chapter-8 book/8-end

I am using macPro, chrome.

thanks

Hi @klyburke
The issue is related to the main repository, not about the tutorial code (book chapter-8 code).
the only changes in my clone are

the ROOT_URL
all kinds of env variables

-John

Hi @klyburke
I tried a clean deployment locally with my own secrets, and completed the purchase process with the demo-book (forked from your demo-book repository), I checked the database:

No Purchase document inside the purchase collect

there is parameter  purchasedBookIds in the User document as below:

{
    "_id": {
        "$oid": "5b7ac4c6672fe01f04a62e89"
    },
    "isAdmin": false,
    "purchasedBookIds": [
        "5b7ac4ad672fe01f04a62e88"
    ],
    "freeBookIds": [],
    "isGithubConnected": false,
    "createdAt": {
        "$date": "2018-08-20T13:40:22.868Z"
    },
     .....

    "__v": 0
}

let me know if you need more information to resolve it.
thanks
-John

@johnking Thanks for your response about DB.
The fact that your User doc has purchasedBookIds (line 237 does run) but Purchase doc is not created (line 265 does not run) - means that either AWS SES or Mailchimp returned caught error. And code that created Purchase doc did not run.
@johnking please take a look at below snippet:
https://github.com/builderbook/builderbook/blob/146ade6be3f39261d174360a08cd55921667bd03/server/models/Book.js#L237-L272
Clear DB and try purchasing book again.
Please monitor logs in your terminal, you should see either server error:
logger.error('Email sending error:', error);
or
logger.error('Mailchimp error:', error);
@klyburke Thanks for narrowing down on potential problem.

Hi @tima101 @klyburke ,
I repeated the steps based on your instructions, but I did not find any log errors related to AWS SES or Mailchimp, and I did not receive the purchase notification email either. I could receive the notification email based on chapter 8-end code.

both the console logs and in-application message give me a not found error message.
The DB has the same status, there is no Purchase document created inside the purchase collection.

@johnking Alright, this helps to narrow down problem even further.
You did not get purchase email via AWS SES.
Was your email address added to Mailchimp list?
Please comment out these 2 blocks of code and test purchase again (remember to clear relevant data in DB):
    const template = await getEmailTemplate('purchased', {
      userName: user.displayName,
      bookTitle: book.name,
      bookUrl: `https://builderbook.org/books/${book.slug}/introduction`,
    });

    try {
      await sendEmail({
        from: `Kelly from builderbook.org <${process.env.EMAIL_SUPPORT_FROM_ADDRESS}>`,
        to: [user.email],
        subject: template.subject,
        body: template.message,
      });
    } catch (error) {
      logger.error('Email sending error:', error);
    }

    try {
      await subscribe({
        email: user.email,
        listName: 'purchased',
      });
    } catch (error) {
      logger.error('Mailchimp error:', error);

The goal is to create Purchase doc successfully.
After that we will investigate why purchase email was not sent by AWS SES.
Thanks.

Hi @tima101 ,
The purchase flow is successful this time by commenting out the lines as specified above!
I double checked that my second gmail address IS in the mailchimp MAILCHIMP_PURCHASED_LIST_ID.
let me know what I should test next.
thanks

@johnking Great. This means that Mailchimp works as expected but not AWS SES.
Please comment out only this block of code to confirm our conclusion:
    const template = await getEmailTemplate('purchased', {
      userName: user.displayName,
      bookTitle: book.name,
      bookUrl: `https://builderbook.org/books/${book.slug}/introduction`,
    });

    try {
      await sendEmail({
        from: `Kelly from builderbook.org <${process.env.EMAIL_SUPPORT_FROM_ADDRESS}>`,
        to: [user.email],
        subject: template.subject,
        body: template.message,
      });
    } catch (error) {
      logger.error('Email sending error:', error);
    }

If purchase works then problem is within above code block.
I may have found the problem, try fixing, replace purchased with purchase inside:
const template = await getEmailTemplate('purchased', {
I think your DB has docs in EmailTemplates collection, one has name welcome and second has name purchase. Thus error.
This means that third-party APIs work fine and you created/added proper env variables. Problem is in wrong name for email template. template variable is likely undefined in:
    const template = await getEmailTemplate('purchased', {
      userName: user.displayName,
      bookTitle: book.name,
      bookUrl: `https://builderbook.org/books/${book.slug}/introduction`,
    });

If you confirm all above, please create pull request for fix.
Thanks for running code and debugging, I could not run app since my env variable are on different machine.

@johnking Have you seen our SaaS boilerplate: https://github.com/async-labs/saas ?
If you happen to run/test/use our SaaS app, you are welcome to create pull requests in that repo as well.
Thanks.

hi @klyburke ,
thanks for your invitation, I will try your SaaS boilerplate later.


Allow screen reader to announce dynamic changes when filtered

Description
Style changes in response to the PQ Tracker accessibility audit Dec 2022. Sections 1.11, 1.4, 1.5.
1.4 - added aria-live="polite" to #count element as a means to inform the screen reader that the number of filtered results has changed. Otherwise too much content will be read out. Example, the screen reader will read out "0 parliamentary questions out of 1." when filter results are updated.
1.5 - Added contrast to focus indicator on filter inputs (E.g. datepicker and keywords). Other dropdown contrasts were added in PR https://github.com/ministryofjustice/parliamentary-questions/pull/955
Self-review checklist

[ ] (1) Quick stakeholder demo done OR
[ ] (2) ...bug with before and after screenshots
[ ] (3) Tests passing
[ ] (4) Branch ready to be merged (not work in progress)
[ ] (5) No superfluous changes in diff
[ ] (6) No TODO's without new ticket numbers
[ ] (7) PR Prefixed with ticket number e.g. CT-7654 ...

Screenshots

Related JIRA tickets
https://dsdmoj.atlassian.net/jira/software/c/projects/CDPT/boards/1152?modal=detail&selectedIssue=CDPT-519
Deployment

Manual testing instructions

@lucas-shaw 1.4 looks good. For 1.5, reading https://design-system.service.gov.uk/get-started/focus-states/ it says to use a yellow border to show focus. Is that possible, or are you following other guidance?

Thanks Andrew. I aimed to keep the outline colour the same as the main navigation bar focus outline, which uses the -webkit-focus-ring-color.
The recommendations for other styling changes (1.14) were due to the yellow borders not having enough contrast against a white and grey background.  I could instead use the black (#0b0c0c) focus however its inconsistent with the main nav.
Happy to change in needed.


Get the Default Value of a Method Parameter

As far as I can see from the code this is not possible at the moment.
The default value ist contained in the full method name but it might be convenient to make this value available more easily.
Thanks for the great work on Sourcery!

We already received similar request in #70 but could not come up with any use case. Do you have anything in mind how that can be used in templates?

Did I put my comments to the wrong issue now? I added them to #70...
Sorry if I created a confusion!

Can you move these comments here, please? Your use case make sense, will be useful for other tasks like generated decorators and mocks. I will try to look into that,


Insufficient plural matching

This is a cross-posted issue from https://github.com/tailoredmedia/gradle-localize-plugin/issues/1 (by @stefangeyer), which is also valid for this fastlane plugin.
Detailed description in the original issue, TL;DR:
When defining a plural without the one option, it is not interpreted as a plural.
Example (which wrongly will not be detected as a plural):
zero|No items available
other|Available items: %d

As mentioned in the original issue, the following line seems to be the problem:
https://github.com/tailoredmedia/fastlane-plugin-localize/blob/12d15603bcc42f87ff7e1fd74bb662591b9530f5/lib/fastlane/plugin/google_sheet_localize/actions/google_sheet_localize_action.rb#L414
That check would need to be expanded to match all possible keywords.

PR sind gerne gesehen (:


Updated The Footer of all Pages with New Links ✅

Hey @Suchitra-Sahoo,
issue closes #1077
Here's what i did

I Updated the footer for all the Pages with new Pages Links
Fixed navigation issue of "Home" in Navbar in termsandconditions and privacypolicy pages.
Fixed The Spacing between Card and footer in termsandconditions and privacypolicy pages.

Please take a look.
Type of PR

[x] Feature enhancement

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Recently some changes in footer is made so now you just pull all the changes from origin and do two things correct the spacing of card and make sure that the circular cursor is visble on both the pages.

Ok @Suchitra-Sahoo I'll again fix the spacing of cards and will make sure the circular cursor will be present. For those two pages!

@Suchitra-Sahoo Please check the new PR (#1124)
I resolved everything and did the tasks you said.


MODCXEKB-80 Get packages

Purpose
Impement getting collection of packages
Approach
Add implementation similar to getting instances,
Use folio-holdingsiq-client for building urls and validation of parameters
TODOS and Open Questions

[x] Add unit tests for CodexPackagesImpl
[x] This PR depends on following PR https://github.com/folio-org/folio-holdingsiq-client/pull/13

Thanks for review, @carolegodfrey,

Before, mod-codex-ekb was building list of queries, and pagination was handled in AbsctractQueryBuilder, but now that we use a library for HoldingsIQ we don't build queries directly, we just pass pagination parameters to library service. So I changed AbsctractQueryBuilder into PaginationCalculator and made it return list of pages instead of list of queries.
ValidationException is the exception thrown by folio-holdingsiq-client, I changed mod-codex-ekb so that QueryValidationException is thrown only in cases where CQL query is invalid, and in cases where CQL is valid but some parameter combination doesn't make sense for particular HoldingsIQ endpoint it throws ValidationException


Modifying parts of css

Maybe related to my lack of experience with engines...
I would like to override parts of the css for upmin. Particulary i need to have a top margin to all body on pages in upmin. What is the best way to override this without duplication the entire set of css files (the main part have incudes as well?

(Not a collaborator on upmin but I ran into this too and thought what I learned might be useful).
I also needed to do this, to allow for a taller than standard boostrap nav.  I did it by adding a custom class to the body, and then calling it in the regular app's css.
I made a new file:  app/views/layouts/upmin/application.html.haml
In application.html.haml:
!!!
%html
%head
%title
Upmin Dashboard
= stylesheet_link_tag("upmin/application", media: "all")
= javascript_include_tag("upmin/application")
= csrf_meta_tags
%link{href: "//cdn.jsdelivr.net/bootstrap/3.2.0/css/bootstrap.min.css", rel: "stylesheet", type: "text/css"}
%link{href: "//maxcdn.bootstrapcdn.com/font-awesome/4.1.0/css/font-awesome.min.css", rel: "stylesheet"}
%script{src: "//cdn.jsdelivr.net/bootstrap/3.2.0/js/bootstrap.min.js"}
%script{src: "//cdnjs.cloudflare.com/ajax/libs/moment.js/2.8.3/moment.min.js"}

- if content_for?(:head)
  = yield(:head)

%body{ id: "upmin-body", class: body_classes, data: body_data.merge(root_path: root_path), :style => "padding-top: 65px;" }
.page-wrap
= render('layouts/upmin/navbar')
= render('layouts/upmin/flash')
= yield
-# = render('layouts/footer')

- if content_for?(:javascript)
  = yield(:javascript)

Note the body id "upmin-body"
And then I did standard css stuff in the application.css.scss file with #upmin-body

With all Rails engine (at least to my knowledge) you can create local files in your app (eg app/views/layouts/upmin/application.html.haml) and the engine will prefer your local file over its own. This basically means that to override css, the easiest way is often just to clone the application.html.haml file, add it to your app, then add in the changes you need (eg load another css file).
Aside from that I'm not 100% sure of the best way because I haven't spent much time on this. If this solution ends up being an annoyance for some reason let me know, but I don't see the application.html.haml file changing frequently so this shouldn't be too hard to maintain.

Tanks for quick reply from you both! I was a bit reluctant to make the application file local, so this is right on!


Issue with markdown

Discussed in https://github.com/open-webui/pipelines/discussions/233

Originally posted by shadowfaxproject August 18, 2024
I have noticed certain markdown syntax does not render correctly when using pipelines in open_webui.
Specifically noticed this when using pipelines with open_webui 0.3.12/13.
e.g. following markdown code [![Test](/favicon.png)](https://google.com)
works with open_webui 0.3.10 displays the image and the link to google.com works as expected

With open_webui 0.3.13, here is the output rendered, does not display the image. Link to google.com works as expected.

Has anyone experienced this? I am unable to higher open_webui version because of this.

Make sure you're testing with latest versions of things. WebUI is at 0.3.16 now.

Tested it again with 0.3.16 webui and still the same issue.
I will open the issue with open-webui since looks like it's on that side
rather than core pipelines issue.
On Thu, Aug 29, 2024 at 5:16 AM Justin Hayes @.***>
wrote:

Make sure you're testing with latest versions of things. WebUI is at
0.3.16 now.
—
Reply to this email directly, view it on GitHub
https://github.com/open-webui/pipelines/issues/241#issuecomment-2317485877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCY5QU3CMS5D6T7JJ2RVBKTZT4GKVAVCNFSM6AAAAABNHCN3D2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJXGQ4DKOBXG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Moving this to open-webui https://github.com/open-webui/open-webui/discussions/5057


feat(note): ✨ add showNatural option to EnglishNoteNotation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1041

Relevant Lines:
1041

💛  - Coveralls


fix: feature flag "download all" button

The "download all" button is blocking our latest release. Rather than try to fix it further on main, let's disable it and fix it out-of-band
The feature hook wasn't being used, so it seems like a good opportunity to simplify it. Our analytics setup has changed and I don't think that we need to identify here any more
The features has been created in posthog staging and production
After

:tada: This PR is included in version 1.9.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


List users not accesible if we use custom login

Bug description
We have deployed our apache superset on AWS fargate, we have our internal dashboards from there we want our users to login hence we have implemented custom login setup on superset side, i was able to do it, now everything works fine, but if i navigate to list/users its redirecting me to login screen and logs in back but getting access denied in right bottom popup, If i revert my custom login code it works properly am using the same user credentials, not sure why it behaves differently, attaching my custom login code
 class RemoteUserMiddleware(object):
  def __init__(self, app):
    self.app = app
  def __call__(self, environ, start_response):
    user = environ.pop('HTTP_USER', None)
     environ['REMOTE_USER'] = user
  return self.app(environ, start_response)

 ADDITIONAL_MIDDLEWARE = [RemoteUserMiddleware]

class CustomRemoteUserView(AuthRemoteUserView):
 login_template = ""

@expose("/login/")
def login(self):
    logger.info("Using custom security manager")
    username = ""
    url = ""

    if g.user is not None and g.user.is_authenticated:
        return redirect(self.appbuilder.get_url_for_index)

    def getUserName(user_roles):
       return "dummy"

    try:
        token = "Bearer " + token
        response = requests.get(url, headers={"Tenant_identifier":tenantIdentifier, "Authorization": token})

        if response.status_code == 200:
            user_data = response.json()
            user_roles = user_data['data']['userRoles']

            security_manager = self.appbuilder.sm
            username=getUserName(user_roles)

            logger.info(username)

            user = security_manager.find_user(username=username)

            logger.info("details")
            logger.info(user)
            if user is not None:
                logger.info("Inside if condition")
                login_user(user)
                return redirect(self.appbuilder.get_url_for_index)
        else:
            print('Error:', response.status_code)
            logger.warning("User not found")
            return redirect('/login/')
    except requests.exceptions.RequestException as e:
        logger.error('Error:')
        return redirect('/login/')

class CustomSecurityManager(SupersetSecurityManager):
authremoteuserview = CustomRemoteUserView
CUSTOM_SECURITY_MANAGER = CustomSecurityManager
AUTH_TYPE = AUTH_REMOTE_USER`
How to reproduce the bug

Implement custom login on forte-fit/superset/docker/pythonpath_dev/superset_config.py
Login using with any admin role credentials
Navigate to users/list

Screenshots/recordings

Superset version
3.1.3
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

Check this link may help.

@dosu am not getting any roles array am just getting firstname and lastname alone from the userdb query

@Habeeb556  thanks for the link, but my case is different, i already have admin role only for the user, still its not working, am using superset 3 is that causing a problem here?

@kannant-sf Hmm, it's not causing a problem, but Superset 3.x is not yet supported. It is preferable to upgrade to 4.x.
Also, make sure to update the list of roles and permissions with the following commands:
export FLASK_APP=superset
superset db upgrade
superset init

@Habeeb556
tried updating to superset 4 as well, but still getting access denied error

@dosu please reply, solve this problem

@dosu i did exactly what you said but am still getting Access is Denied if i try to navigate, help me out

@dosu
these commands from @Habeeb556  made it work
superset db upgrade
superset init
thank you @Habeeb556

@dosu
Now how can i combine the existing username and password flow along with my custom login as well

Am I correct in understanding that the problem is solved, and we can close this? Either way, thanks @Habeeb556!

@rusackas  yes go ahead


pagination doesn't work, config is just as yours.

just as title mentioned , and other function is ok.

Can you be bit more specific. How does it not work, what version, component or directive and also still seeing the full config you are using would help.

I consider that I'm'not expressing it clearly,I don't know why I can't see the pagination when setup it .I start a new project, and I still can't see the pagination,the following is the config and the html.

thank you for you response.

I have the same problem. When using the "component" approach the pagination shows up as expected. However, when using the directive, even with correct configuration it won't show up. Not sure why that's happening. On StackBlitz it works fine though if I change the type to 'directive' and enable pagination configuration. Could be related to angular version (using angular 7).

 Component includes it automatically, but with directive you need to import the stylesheet in your app code.

 Component includes it automatically, but with directive you need to import the stylesheet in your app code.

Indeed I have included the stylesheet in the scss file of my component, same result unfortunately.

If you include it in you components file its bound by the angular view encapsulation. So you need to either turn off encapsulation or load the styles in global styles.


[release-2.3] build refresh

Addresses:

https://github.com/stolostron/backlog/issues/22155

/lgtm


Gwen template matching

Template Matching
This PR introduces pattern matching capabilities to gwen. This will allow users to match any source content with a target template that can either be specified as a text literal or text stored in a file.  Supported also is the ability to extract, ignore, and inject attributes.
Examples
The following examples assume that the following JSON value is bound to an attribute named my value in the currently available scope:

{"id":42,"category":{"name":"pet"},"name":"tiger","status":"available"}

Templates can contain JSON, XML, HTML, or any type of text and can also have multiple lines. The examples below use single-line JSON only.
Match
The following will successfully match my value:
Then my value should match template "{"id":42,"category":{"name":"pet"},"name":"tiger","status":"available"}"

Same as the above, but using a DocString:
Then my value should match template
     """
     {"id":42,"category":{"name":"pet"},"name":"tiger","status":"available"}
     """

Same as the above, but using a multi-line DocString:
Same as above, but assumes template is in a file relative to location where Gwen is invoked:
Then my value should match template file "path/to/Template.json"

Match and Ignore
The following example will successfully match my value and:

Ignore the id value where !{} appears

Then my value should match template
     """
     {"id":!{},"category":{"name":"pet"},"name":"tiger","status":"available"}
     """

Match, Ignore, and Extract
The following example will successfully match my value and:

Ignore the id value where !{} appears
Extract the tiger value where @{pet name} appears and bind it to the attribute named pet name in the feature scope

Then my value should match template
     """
     {"id":!{},"category":{"name":"pet"},"name":"@{pet name}","status":"available"}
     """

Subsequent steps can access the extracted pet name attribute like any other Gwen binding. For example, the following checks it's value:
And pet name should be "tiger"

Match, Ignore, Extract, and Inject
The following example will successfully match my value and:

Ignore the id value where !{} appears
Extract the tiger value where @{pet name} appears and bind it to the attribute named pet name in the feature scope
Inject the value bound to the my status attribute into the template where ${my status} appears.

Given my status is "available"
 Then my value should match template
      """
      {"id":!{},"category":{"name":"pet"},"name":"@{pet name}","status":"${my status}"}
      """
  And pet name should be "tiger"

The @{} ingore syntax just doesn't seem quite right. My other concern is that users might start requesting for loops, if statements, regex matchers, date format checkers, etc that could complicate matters and compromise the simplicity of Gwen.
I'm a little hesitant to merge this one atm. Thoughts anyone?

I think that @{} is a bit confusing indeed, but probably !{} or @{!} would be more suggestive for ignoring something.

Changed ignore syntax from @{} to !{} and updated examples in PR description above.


[FEAT]WorkPro: 企微扫码登录后，是否有获取所有包含当前账号的群的方法

Please make sure you have read the Wechaty Puppet Service: WorkPro from Wechaty Official Website before you continue writing this issue.
IMPORTANT
DO NOT post your token in the issue. If we need to investigate the behavior of certain token, we will provide a private contact route.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

const group = await wechat.Room.findAll()
console.log("group>>>>>", group);


Make cmake builds work on macOS again

Would normally put link libraries with their respective targets, but I’m ok with this.

Well, it looked uglier mixing the calls to this function prototype with the link targets mixed in :)

[approve ci autest]

[approve ci autest]


vz: fix mu

Follow-up to #1494
Fix #1381

@AkihiroSuda
Changes looks fine for me.
But am trying to understand why mu would be needed here.
We are creating wrapper once and updating the stopped in one place only.
Even if both tick and stateChange happens at same time the tick would fail and stopped would be set to true and on next tick it would technically be successful
Am i missing something here 🤔


Can I use different MotionTransitionType in one UIViewcontroller

Say I have a UIViewController UV1 which is embedded in an AppNavigationController and Another two UIViewController UV2 and UV3. I want to push UV2 or UV3 from UV1.
class AppNavigationController: NavigationController {
  open override func prepare() {
    super.prepare()
    isMotionEnabled = true
    motionNavigationTransitionType = .autoReverse(presenting: .zoom)
  }
}

The problem is both transitions for UV2 and UV3 is "zoom".
If I want the transition for UV2 is zoom, but the one for UV3 is "slide", what should I do to achieve it?
Thanks!

You can adjust the different motion transition in each view controller. So in VC2 you can add in the viewDidLoad method:
navigationController?.motionNavigationTransitionType = .autoReverse(presenting: .pageIn(direction: .down))

and so forth in the different view controllers. That is one way and a straight forward way to do so. If you need please reopen the issue for any further help on this topic. Thank you!

It works. Thank you very much!

Any time :)


TaxonName - synonym endpoint

  when I search for a taxon name, or provide an ID, then I want to get back a list of synonyms, or the valid id

At this point Keep the JSON response very simple, likely just an id to id map.

Valid ids are presented in every individual response, including filter. Closing this until we hear from users that they want something different.


Documentation should be good, accessible and user centric

There needs to be a website that is fast, accessible and user centric to be as easy to consume as possible.

Documentation will be available on Github (Basic) and https://litr.dev (Extended).

To ensure not duplicating work on the documentation there should always only be one source, therefore here a clarification what comes into what place:
GitHub Wiki, Development documentation such as:

Script parsing
Command line parsing
Used software
etc.

Website, usage documentation such as:

Quick setup (installation, setup)
Detailed configuration
FAQ and common usage cases
etc.


Fix emoji font metrics

What does the pull request do?

What is the current behavior?

The font extents provided by HarfBuzz are sometimes not correct
What is the updated/expected behavior with this PR?

Use the correct ascender, descender, etc.
How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Avaloniaui.net with user documentation

Breaking changes

Fixed issues

Fixes: #4860

working great, well done


Livewire no sync

Hello,
First, thanks for this plugin.
I run in troubles with Livewire where the hidden input don't get the value from the user input. So I used your trick about refresh and it works.
But, and maybe because, the hidden input where the wire:model is can't sync with Livewire when it's updated by JS.
So I had to add theses line to get it works in my x-init :
initPhone() {                 let vm = this;                 document.querySelector("#phone").addEventListener('telchange', function(e) {                     if(e.detail.valid) {                         vm.phone = e.detail.validNumber;                     }                 });             }
And phone: @entangle('phone').
Is there a better way ?
Have a nice day.
Arnaud

You can just use wire:model.lazy (lazy loading) and it'll work as expected.
And the hidden input will only update if the format is correct/valid.


Git

URL to the linked Repository or Organization
https://github.com/
Steps to reproduce the problem

What is the expected result?
What happens instead?
Any other information? (attach screenshot if possible)
Used browser and browser version?
Any other tested browsers/devices(OK/FAIL):

URL to the linked Repository or Organization
https://github.com/
Steps to reproduce the problem
What is the expected result?
What happens instead?
Any other information? (attach screenshot if possible)
Used browser and browser version?
Any other tested browsers/devices(OK/FAIL):


Add url diff to PullRequest

Closes #211
I added diff url to PullRequest.
Implementation:

[x] BasePullRequest
[x] Github
[x] Gitlab
[x] Pagure

But I found diff url in api for github, so I "guess" url.
But for github there are two option "/files"  (consistent with other services) or ".diff"  (consistent with Github api).
For example:
https://github.com/packit-service/ogr/pull/1/files
and
https://patch-diff.githubusercontent.com/raw/packit-service/ogr/pull/1.diff
I think ".files" looks better and that option was in issue as example, that why I chose this one. But if you want I can change to github api option ".diff"
For others services I couldn't find in api, so I "guess" url.

About tests fails,
Issue: https://github.com/packit-service/ogr/issues/281
PR: https://github.com/packit-service/ogr/pull/282

Sorry for the late response. I missed the notifications.
We made some big changes in the code structure last days -- Can you rebase, please? (Creating a new PR may be quicker.)
Thanks!

I'm out of range my laptop, I will update this PR on the weekend

I'm out of range my laptop, I will update this PR on the weekend

No worries, thanks!

@pawelkopka do you still want to work on this? (I can finish it otherwise...)

I'd say the answer is no and you can carry it Franto :)

Closing in favour of #335


Add block-no-unnecessary-nesting rule

Fixes #679.

Thanks! Looks good, just a couple of small things that I commented on :)

One probablem that I noticed when reviewing this rule is that the rule does not seem to take into account that Sass supports nesting CSS properties.
This accepted test should pass:
    {
      code: `
        .foo {
          font: {
            size: 16px;
          }
        }
      `,
      description: "Sass' nested properties"
    }

...as it compiles to:
.foo {
  font-size: 16px;
}


Enhancement

various icons and logo image updated #7

Please share screenshot of the change you have made.

Please share screenshot of the change you have made.
This file has the basic changes
ScreenshotBOSSY.docx

Looks great. Can you change the design at the top/

That's the place for the logo so should I add some random image or should I design a logo ?

Closing this as redundant.


Show a consistent navbar across docs.rs and rustdoc pages

Moves the docs.rs internal navbar links to the right-hand side, and keeps the crate-related navbar links when viewing the crate details/builds/source pages (while removing the dropdown that duplicates the crate details).
There is one remaining issue that I don't want to fix here; the "go to latest version" link doesn't appear on the non-rustdoc pages. The metadata used for rendering builds and source files doesn't currently pull enough details from the database for this, I think it might make the most sense to make both of them get the full CrateDetails and move the is_latest_version etc. checking into Tera helpers rather than being extra context variables, but that seems better done as a separate refactoring.
related to #1077
Screenshots:

I think I like this better. @jsha , @GuillaumeGomez , @Kixiron , what do you think?

This is great! I like the consistency across all pages. I have a small branch locally to give the About / Releases / Rust menus the same treatment we recently gave the crate / Source / Platform menus: Click to open and click to close rather than hover. I'll rebase on this branch once it lands.

I prefer this new one as well!

@Nemo157 thanks for working on this!


[DEVOPS-1020] add logs in dev mode and nix improvements

These are some basic changes to help QA with running daedalus with a demo cluster.
Overrides runTimePath to "./" when ran with yarn dev so logs are written to Logs/pub/Daedalus.log in current directory.
Improves nix shell code to give instructions on how to use.
Minor refactor to support multiple derivations in shell.nix using passthru.
Todo:

[ ] Todo
[ ] Todo
[ ] Todo

Screenshots:

Screenshot

Review Checklist:
Basics

[ ] PR is updated to the most recent version of target branch (and there are no conflicts)
[ ] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance tests are passing (yarn run test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn run dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn run package / CI builds)
[ ] There are no flow errors or warnings (yarn run flow:test)
[ ] There are no lint errors or warnings (yarn run lint)
[ ] Text changes are proofread and approved (Jane Wild)
[ ] There are no missing translations (running yarn run manage:translations produces no changes)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn run storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly documented and commented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-rendering
[ ] Any code that only works in Electron is neatly separated from components

Testing

[ ] New feature / change is covered by acceptance tests
[ ] All existing acceptance tests are still up-to-date
[ ] New feature / change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

@disassembler if I got this correctly once this PR is merged even us (Daedalus developers) will have Daedalus logs saved in the project directory when running Daedalus in dev mode?

@DominikGuzei @cleverca22 please review this PR too! Thanks!