V3.0.0 C11<stdatomic.h> is not supported, is it OK to use it on a microcontroller?

hi：
I ported the latest v3.0.0 on top of the STM32F4. The development platform used keil , when compiling it shows cannot open source input file "stdatomic.h":, my compiler does not support C11 syntax.
Define LWRB_DISABLE_ATOMIC , compile can can pass, so I use j it in the microcontroller above to store the serial port to receive data is there any risk. In v2.0.2, the volatile keyword is used, but v3.0.0 does not support it.

Hi,
normally Atomic and volatile are not the there for the same purpose, basically:

Volatile will ensure that read and write operations are always from the memory (can be from cache tho), but every read operation will reload the CPU registers
Atomic ensures that read/write is atomic, especially between several cores.

One of the tricky points is when CPU cannot make a read/write in one bus read. If we focus on STM32F4, you may be fine with disabled ATOMICs because size_t is 32-bit on Cortex-M and 32-bit can be written in one cycle.
I am sure Keil supports C11, but it may need to be enabled in their settings.

Thanks for your reply, I understand the Atomic and volatile purpose now, and yes keil supports C11 and needs to be compiled with AC6.


Docker-compose trying to pull non-existant tag for jupyterhub

Command "PLATFORM_VERSION=development docker-compose pull"
Results in
"Pulling singleuser (rengahub/singleuser:development)...
ERROR: manifest for rengahub/singleuser:development not found"
After which pull does not continue.

So when is that :]

Two things here: 1) singleuser needs to be removed from the docker-compose.yml and 2) the singleuser image is built in the Makefile, e.g. you can do make tag to build the custom images we use.

@3C111 I'm not sure this is really a problem? If not, could you please close the issue?

@rokroskar why would this not qualify as an issue for you?

@3C111 because the image is built when you do make start


Ensure deprecated exercises are deprecated

This forum post says that Binary is deprecated for very long, but PHP still has it active.
As deprecating exercises does not do any harm to existing solutions and people currently attempting it, we should ensure, that all deprecated exercises from problem-specifications are marked as deprecated and so not shown to new users.
I'm sure, configlet has a command to list those exercises and they can easily be marked deprecated in the central config.json.

I don't find that command in the configlet --help info

So it is not an easy task to do. Feature request filed in forum.
Without a command, scanning problem-specifications and comparing to our central config.json is required...


WIP maven plugin with automatic trigger the NPEfix agent after detect any failures

Fix #374

Rebase against master such that the diff only contains the change in the readme and in the maven plugin.

No, we should never e.printStacktrace()
The good solution is to write e.toString() in the log.

No, we should never e.printStacktrace() The good solution is to write e.toString() in the log.

Commit a new fix.

Hello @monperrus, it's ready to merge. Please advise. FH

LGTM. Thanks a lot @cokefh and congrats for your first merged contribution to Repairnator!


[$250] IOU - Markdown isn't applied to the "Hold" message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: new feature
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: applause internal team
Slack conversation:
Action Performed:

Create an IOU in a chat
Put it on hold
Format the reason with markdowns

Expected Result:
Markdown should be applied.
Actual Result:
Markdown isn't applied to the "Hold" message.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011f83ef4d2ad62f49
Upwork Job ID:<PHONE_NUMBER>198360064
Last Price Increase: 2024-03-25

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU - Markdown isn't applied to the "Hold" message
What is the root cause of that problem?
We do not pars hold comment
What changes do you think we should make in order to solve the problem?
Update comment  with getParsedComment(comment)
What alternative solutions did you explore? (Optional)

@BartoszGrajdek @MonilBhavsar @robertjchen I see you in https://github.com/Expensify/App/pull/33897, can you please have a look here?

Proposal
Please re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
What changes do you think we should make in order to solve the problem?

Update the comment here and here with getParsedComment(comment)
There need updates in backbend too . The hold action will not contain html key in messages and originalMessage

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
On hold message not shown as HTML
What is the root cause of that problem?
The message from description for HOLD reason is saved as simple string
What changes do you think we should make in order to solve the problem?
There's 2 main areas that needs to be fixed.
1 - use ReportUtils.getParsedComment(comment)
We need to save the message as parsed before passed to Onyx here
https://github.com/Expensify/App/blob/20ca041595e76cdd289977e55b2af3fbb9618868/src/libs/ReportUtils.ts#L3530
2- Use <RenderHTML> not <Text>
After adding the ReportUtils.getParsedComment(comment)  the UI will display the markdown as it it is, something like <em> text </em>, so we need to use RenderHTML component here
https://github.com/Expensify/App/blob/20ca041595e76cdd289977e55b2af3fbb9618868/src/pages/home/report/ReportActionItemFragment.tsx#L135-L144

Proposal
Updated

On hold, pending the splitting of the message in  #36979

Looks like Robert has got this issue

Not overdue, on hold.  Robert is working on this https://github.com/Expensify/App/issues/36979

not overdue - Robert is still working on this: https://github.com/Expensify/App/issues/36979#issuecomment-1999128350

Off hold! 👍 Note that the hold message is now an independent message

cc: @hoangzinh for proposal review, thanks!! 🙇

@dragnoir there are significant differences in our proposals . And, as you can see, in my proposal there is no mention of the <RenderHTML> component. I suggested using RenderCommentHTML

Hm, the backend logs report that the comment field is missing when calling the HoldRequest API comand. Can you please try again on a fresh request with a comment when holding? 🤔

Good question, I'll update the backend to return HOLDCOMMENT as well for consistency 🤔

Actually, let's change the frontend to the ADDCOMMENT for now as there are many considerations on the backend 👍

Thanks for the review, the backend change should be going out later today so we can proceed here 👍

Fix: @dragnoir
PR Review: @hoangzinh
PR not merged yet

I'm suggesting we close this issue, to reflect where we've landed in the PR. https://github.com/Expensify/App/issues/40408#issuecomment-2123384330 - any thoughts, comment there. CC: @JmillsExpensify

Yes, agreed. Let's close.

Cool, thanks ya'll!


tests: dma_loopback: Intel ADSP ACE15 disable PM

Disable power management for this particular test case as it expects a particular pattern of pm get/puts that isn't matched by the driver and usage in SoF.
Tested on the device and verified things work as expected

@teburd  need to fix the build error


Ui router url changed but views not are loaded for sometime not always.

Hi,
I am using AngularJS v1.3.17 and the latest version of ui router.
I've searched for this but i could not find solution for that.
I've migrated from ng route to ui router.
After implementing named nested view, I got the problem like, url changed but views are not loaded.
I've implemented like the following. Before implement the sub view's sub view, I didn't face like this.,
I've tried to log to check whether there is any error, but found no exception
$rootScope.$on("$stateChangeError", console.log.bind(console));
But I didn't find anything.
If I refresh browser and work, there is no issues for sometimes.
$stateChangeStart event has been triggered.
When ui router works without error, i can catch the flow at interceptor.
when ui router is not loading, stateChangeStart event only has been triggered but i can't get the flow at the interceptor.
I can't figure out this problem.  :-1:
.state('home', {
url: '/home',
*--------------------------------------------
state('home.Page1', {
url: '^/page1',
*--------------------------------------------
.state('home.Page1.Subs', {
authenticate: true,
views: {
'Page1Master@': {
templateUrl: 'App/Page1/_Page1Master.html?v=' + version,
controller: 'Page1MasterController',
controllerAs: 'vm',
}
}
})

Removed ng href and window location hash and now using ui-sref and $state.go, transitionTo and reload based on need.
But having another issue.
posted here issue


sparse_softmax_cross_entropy_with_logits wrong answer under XLA

Related to https://github.com/tensorflow/probability/issues/975
System spec
TF version: 2.3.0-dev20200618 (tf-nightly)
OS: Linux Mint 19.3, CUDA-10.1.
Current behaviour
Under XLA compilation, tf.nn.sparse_softmax_cross_entropy_with_logits  returns nan if one or more logits are -inf.  In regular graph mode, -0.0 is returned.
Expected behaviour
The log_prob function above should return the cross entropy of obtaining labels k from logits logits, and be non-nan (-inf is okay) for valid values of the function parameters.
NB expected behaviour occurs under TF2.2.0 (stock Colab).  Only the current dev build is affected.
Minimal working example
colab here
probs = tf.constant([0, 1, 1], dtype=tf.int32)
logits = tf.math.log(probs)

@tf.function(autograph=False, experimental_compile=True)
def log_prob(k):
    lp = -tf.nn.sparse_softmax_cross_entropy_with_logits(labels=k, logits=logits)
    return lp
print(log_prob(1))  # Expected: -0.6931472, Actual: nan

@chrism0dwk
I have tried in colab with TF nightly version(2.3.0-dev20200619) and was able to reproduce the issue.But i am not seeing nay issue with TF 2.2 version.Please, find the gist here.Thanks!

Yup, confirmed that's it.  Thanks.

@srvasude could you take a look at this?

@srvasude could you take a look at this?

This is fixed by https://github.com/tensorflow/tensorflow/commit/e9b3d33ccfa4b670ca0a221db9bc7bc1281b0557

This is fixed by https://github.com/tensorflow/tensorflow/commit/e9b3d33ccfa4b670ca0a221db9bc7bc1281b0557


What I want to say is that there are a lot of blank lines in the text content of the console, and the problem that the mouse cursor cannot select the text content from right to left.

Issue Type: Bug
What I want to say is that there are a lot of blank lines in the text content of the console, and the problem that the mouse cursor cannot select the text content from right to left.
May I ask when the problem can be fixed?
VS Code version: Code 1.44.1 (a9f8623ec050e5f0b44cc8ce8204a1455884749f, 2020-04-11T01:48:12.622Z)
OS version: Windows_NT x64 10.0.19041

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-3210M CPU @ 2.50GHz (4 x 2494)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
9.59GB (4.87GB free)

Process Argv

Screen Reader
no

VM
67%

Extensions (27)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.3.0

vscode-eslint
dba
2.1.5

vscode-npm-script
eg2
0.3.11

prettier-vscode
esb
4.4.0

auto-close-tag
for
0.5.6

auto-complete-tag
for
0.1.0

auto-rename-tag
for
0.1.1

comment-translate
int
1.4.2

vuejs-extension-pack
mub
1.2.1

color-highlight
nau
2.3.0

vetur
oct
0.24.0

vscode-boot-dev-pack
Piv
0.0.8

vscode-concourse
Piv
1.16.0

vscode-manifest-yaml
Piv
1.16.0

vscode-spring-boot
Piv
1.16.0

java
red
0.60.0

vscode-xml
red
0.11.0

vscode-yaml
red
0.7.2

vue-vscode-snippets
sdr
1.9.0

vscodeintellicode
Vis
1.2.6

vscode-java-debug
vsc
0.25.1

vscode-java-dependency
vsc
0.9.0

vscode-java-pack
vsc
0.9.0

vscode-java-test
vsc
0.22.3

vscode-maven
vsc
0.21.2

vscode-spring-initializr
vsc
0.4.6

JavaScriptSnippets
xab
1.7.2

(1 theme extensions excluded)

![kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk](https://user-images.githubusercontent.com/27546390/79821700-0476a580-83c2-11ea-9921-6390533c86ad.png)
![kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk](https://user-images.githubusercontent.com/27546390/79821813-43a4f680-83c2-11ea-8071-e8576217a336.png)
![kkkkkkkkkkkkkkkkkkkkkkkkkkkkkkk](https://user-images.githubusercontent.com/27546390/79821885-78b14900-83c2-11ea-86b6-8d68a0d72578.png)
May I ask when the problem can be fixed?

For details, please refer to:
https://github.com/microsoft/vscode-java-pack/issues/359

Dup of https://github.com/microsoft/vscode/issues/2163


Smarter hard-coded credentials check

@gcmurphy @callidus
Check right-hand side expr for literals when looking for hard-coded credentials.
Example file:
package main

func main() {
  passwordBad := "password"
  passwordOk := getPassword()
}

With these changes, the first line in main will still warn, but the second line will now be fine. Previously both lines would produce a warning.
Anecdotally (based on running this on Square's internal Go code base) the hard-coded credentials check almost always yields false positives, since it's a common pattern to read a secret from e.g. a file and then load it into a variable like "password" or "token" or such.

nice 👍


In pluto_grid 4.0.1, data is getting hidden if entire table put into ScrollView, Appears and disappears when Column header is no longer visible.

@bosskmk I had updated this plugin yesturday only and I got to see weird things. Like mentioned In Question, I had put pluto_grid in a scrollable Screen. This was not the issue recently but after updating to latest version, when you drag your screen upward and your Column Headers no longer visible then the data from table dissapears and re appears when dragged down. I'm not getting any logs also. Everything is working fine only this shown like glitch to me.

@aadi-github
Version 4 has logic to render cells depending on whether the column is visible or not, which is the problem.
Changed the logic in the next version so the problem will no longer occur.
Please use version 3 until version 4.1 is released.
The next version work is about 80% complete.

@aadi-github Version 4 has logic to render cells depending on whether the column is visible or not, which is the problem. Changed the logic in the next version so the problem will no longer occur. Please use version 3 until version 4.1 is released.
The next version work is about 80% complete.

I'm just keeping it open in case of someone else encounters this issue. Thanks for your hard work, Sir.

PlutoGrid 5.0.0 has been deployed.
There are other changes from version 4, so please check the ChangeLog.
https://pub.dev/packages/pluto_grid/changelog#500---2022-7-11
The demo page has also been updated.
https://weblaze.dev/pluto_grid/build/web/#/

PlutoGrid 5.0.0 has been deployed. There are other changes from version 4, so please check the ChangeLog. https://pub.dev/packages/pluto_grid/changelog#500---2022-7-11
The demo page has also been updated. https://weblaze.dev/pluto_grid/build/web/#/

Thanks, this new update has resolved my issue. Now I'm closing this one now.


Prevent menu from popping on semi-colon or space

Is it possible to prevent tern from showing completions on entering a semi-colon or space? Its rather annoying because when I write a line and hit ';' to complete the line, the natural behavior is to hit enter to start the next line, but tern is showing completions and enter inserts a '$' (since jQuery is there, the $ shows first).
Good work though, thanks!
Edit:
After I submitted this, i noticed that it only happens after something like functionCall(); not on something like var x = 1;

+1 on this, this is very annoying.

v0.5.10 includes a temporary fix for that problem.
Not perfect, but it will work for now.
The total prefix logic needs a overhaul :)
But i'm to busy right now.

@tststs Thanks!

Thanks for the fix!


if/else statement seems to be useless

~@evaalonsoortiz unless I am missing something, I don't see the reasoning behind this if/else statement, which brings to the same syntax:~
https://github.com/spinal-cord-7t/coil-qc-code/blob/a6a8017de62ad80457bc7dbb7bd02033a05160ad/data_processing-human.ipynb#L202-L206
EDIT: My bad! I missed the subtle 'famp' vs. 'anat' difference 😅

Actually, digging more into this, I noticed that the registration solely relies on the segmentation (-param step=1,type=seg,algo=centermass), therefore the if/else statement is useless because the input image is not used for the registration. Unless I am missing something @evaalonsoortiz ?

Ah, more details: the if/else for MSSM is in fact required because the _acq-anat_TB1TFL.nii.gz file does not exist for this site (https://github.com/spinal-cord-7t/coil-qc-code/issues/43).
However, I think that the first if/else, referring to https://github.com/spinal-cord-7t/coil-qc-code/issues/48 which is now solved, is not required anymore.


Cleanup codes for old Rubies

This PR did a small refactoring that fixed the code for old Rubies.

Use symbolize_names option instead of deep_symbolize_keys
Remove needless branch for YAML.safe_load
Fix required Ruby version in README

coverage: 97.057% (+0.2%) from 96.845%
when pulling a5c216ef3b5e4f71c7f2e953b45cfefd8e86f368 on y-yagi:cleanup-codes-for-old-rubies
into 37811c52d27a1b3884af59ff6ad6bd17e626cbe5 on ridgepole:2.0.


Suggestion: Storage Scanner shows the name of the storage module.

I haven't gotten on Minecraft in awhile so don't hate me too much if it already does this, but I noticed recently from some YouTube videos you added the ability to name storage modules in an Anvil. This got me to thinking of a good idea, in that your Storage Scanner should show a list according to the name of the storage module, instead of just saying Modular Storage. That way you can create a file system. Can view the list of close Modular Storage blocks by Ores, Stone, Wood, RFTools Parts, or whatever you named the module. Would also be nice if you renamed chests or any other inventory tile entity, it would show the custom name. So if I named an Iron Chest "Misc Parts" it would show up as Misc Parts in the Scanner.

That would be interesting indeed! I would like to see this in a new version
of RFTools!
On Thu, Aug 4, 2016 at 2:32 AM, Rated01010010<EMAIL_ADDRESS>wrote:

I haven't gotten on Minecraft in awhile so don't hate me too much if it
already does this, but I noticed recently from some YouTube videos you
added the ability to name storage modules in an Anvil. This got me to
thinking of a good idea, in that your Storage Scanner should show a list
according to the name of the storage module, instead of just saying Modular
Storage. That way you can create a file system. Can view the list of close
Modular Storage blocks by Ores, Stone, Wood, RFTools Parts, or whatever you
named the module. Would also be nice if you renamed chests or any other
inventory tile entity, it would show the custom name. So if I named an Iron
Chest "Misc Parts" it would show up as Misc Parts in the Scanner.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/McJty/RFTools/issues/668, or mute the thread
https://github.com/notifications/unsubscribe-auth/AI9rvi7B1v96zoyto966dsj3bysaLTfXks5qcbGzgaJpZM4Jcg0m
.

Not sure I totally get this suggestion? The storage scanner already shows the name of those modules no?


Create Task Card Component

Description:
Develop a function to create a task card using the task details.
Acceptance Criteria:
A function createTaskCard(task) is implemented.
The function generates HTML for a task card with the task title, description, and delete button.
The generated task card is appended to the appropriate task lane.

Done and Dusted Out of this world


Typo in main.py

Line 22:       description = "This program simulates the influence of a raman active probe ..."
should be sample instead of probe.

Done.


WIP: Save waitTime value for sequence and conductor action (allow parent/child transaction ids)

Description
Currently, The openwhisk uses the waitTime annotation to store the internal wait time until the action code is provisioned. But this value is only present for non-sequence related activations.
Internally invoked actions can't know waitTime because this value is measured based on the start value of the first created TransactionId by topmost action.
So those actions invoked by topmost action must have a different TransactionId with a different start value.
To solve this problem I've modified the TransactionId case class to have a parent/child structure.
Define the starting point for internally invoked action
This is the invocation flow of the sequence and conductor action in the controller.
(Each action calls internal actions in a different way, so the starting point is different.)

Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x] I added tests to cover my changes.
[x] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Codecov Report

Merging #4819 into master will decrease coverage by 6.58%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4819      +/-   ##
==========================================
- Coverage   85.12%   78.53%   -6.59%
==========================================
  Files         197      197
  Lines        8879     8899      +20
  Branches      607      604       -3
==========================================
- Hits         7558     6989     -569
- Misses       1321     1910     +589

Impacted Files
Coverage Δ

.../openwhisk/core/containerpool/ContainerProxy.scala
91.89% <100%> (-0.09%)
:arrow_down:

...hisk/core/controller/actions/SequenceActions.scala
92.3% <100%> (+0.06%)
:arrow_up:

...la/org/apache/openwhisk/common/TransactionId.scala
94.73% <100%> (+1.18%)
:arrow_up:

...in/scala/org/apache/openwhisk/common/Logging.scala
74.3% <100%> (-9.32%)
:arrow_down:

...isk/core/controller/actions/PrimitiveActions.scala
87.82% <100%> (+0.57%)
:arrow_up:

...a/org/apache/openwhisk/http/BasicHttpService.scala
93.1% <100%> (-0.12%)
:arrow_down:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...enwhisk/connector/kafka/KamonMetricsReporter.scala
0% <0%> (-100%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0% <0%> (-100%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0% <0%> (-100%)
:arrow_down:

... and 15 more

Continue to review full report at Codecov.

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

Awesome! Thank you for adding this.

I've found a performance improvement point for this PR. As you know, using the string builder is much faster than the current version.
The blue line shows the performance of the transaction id generator currently used by openwhisk, and the orange line shows the performance of the implementation using the string builder.
I'll add a new commit.

(In the chart, the x-axis is the number of repetitions and the y-axis is the time taken.)
Source code for performance testing

object TransactionId {
  private val dict = ('A' to 'Z') ++ ('a' to 'z') ++ ('0' to '9')

  def generateTidByMap(): String = {
    (0 until 32).map(_ => dict(ThreadLocalRandom.current().nextInt(dict.size))).mkString("")
  }

  def generateTidByStringBuilder(): String = {
    val sb = new StringBuilder
    for (_ <- 1 to 32) {
      sb.append(dict(util.Random.nextInt(dict.length)))
    }
    sb.toString
  }
}

@upgle Sorry for letting this become stale.
Could you address the conflicts?

Codecov Report

Merging #4819 into master will decrease coverage by 6.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #4819      +/-   ##
==========================================
- Coverage   83.34%   77.33%   -6.01%
==========================================
  Files         200      201       +1
  Lines        9334     9469     +135
  Branches      408      394      -14
==========================================
- Hits         7779     7323     -456
- Misses       1555     2146     +591

Impacted Files
Coverage Δ

...in/scala/org/apache/openwhisk/common/Logging.scala
76.64% <100.00%> (-8.17%)
:arrow_down:

...la/org/apache/openwhisk/common/TransactionId.scala
95.12% <100.00%> (+1.47%)
:arrow_up:

...a/org/apache/openwhisk/http/BasicHttpService.scala
93.10% <100.00%> (-0.12%)
:arrow_down:

...isk/core/controller/actions/PrimitiveActions.scala
86.16% <100.00%> (+0.83%)
:arrow_up:

...hisk/core/controller/actions/SequenceActions.scala
80.00% <100.00%> (+0.33%)
:arrow_up:

.../openwhisk/core/containerpool/ContainerProxy.scala
93.80% <100.00%> (+0.23%)
:arrow_up:

...core/database/cosmosdb/RxObservableImplicits.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0.00% <0.00%> (-96.23%)
:arrow_down:

... and 29 more

Continue to review full report at Codecov.

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

@style95 @rabbah
I've updated this PR, and all test cases have been passed. Please review it again.


Verify ExecutionContext is needed in all Node.js extensions

I looked at Node.js extensions and did not see any checking about that ExecuationContext being in run mode. It seems we have an issue with that right now. If we use these extensions only in development, we should have to check that ExecuationContext is in run mode.
And I think it's better to change AddViteApp to WithRunViteApp.
That's a very good point, I'll add an issue tracking it.

Originally posted by @aaronpowell in #115

Do we have the same issue for CommunityToolkit.Aspire.Hosting.Deno?

Probably. Deno should deploy as a container IMO

Having a look into this issue and the way that we have it done at the moment is how it seem to work for the Node.js integration that is part of the Aspire core.
Taking https://learn.microsoft.com/dotnet/aspire/get-started/build-aspire-apps-with-nodejs as our guide (well, the samples repo) we can note that they always include the .PublishAsDockerFile method at the end of the chain.
This results in the following JSON in the manifest:
    "angular": {
      "type": "dockerfile.v0",
      "path": "AspireJavaScript.Angular/Dockerfile",
      "context": "AspireJavaScript.Angular",
      "env": {
        "NODE_ENV": "development",
        "services__weatherapi__http__0": "{weatherapi.bindings.http.url}",
        "services__weatherapi__https__0": "{weatherapi.bindings.https.url}",
        "PORT": "{angular.bindings.http.targetPort}"
      },
      "bindings": {
        "http": {
          "scheme": "http",
          "protocol": "tcp",
          "transport": "http",
          "targetPort": 8000,
          "external": true
        }
      }
    }

If we remove any publishing instructions we get this:
    "angular": {
      "type": "executable.v0",
      "workingDirectory": "AspireJavaScript.Angular",
      "command": "npm",
      "args": [
        "run",
        "start"
      ],
      "env": {
        "NODE_ENV": "development",
        "services__weatherapi__http__0": "{weatherapi.bindings.http.url}",
        "services__weatherapi__https__0": "{weatherapi.bindings.https.url}",
        "PORT": "{angular.bindings.http.targetPort}"
      },
      "bindings": {
        "http": {
          "scheme": "http",
          "protocol": "tcp",
          "transport": "http",
          "targetPort": 8000,
          "external": true
        }
      }
    }

And this is the same as we get out of our samples.
Because of this, I don't think there's really anything "special" we should be doing in our extensions (or integrations in the case of Deno) to handle the different execution modes. Probably the only real consideration would be the NpmPackageInstallerLifecycleHook (and similar hooks) which are doing a check for whether it's in publish mode or not and if it is, skipping the logic.
Is there anything specific you think we should add @Alirexaa for handling publish vs run mode in these?

@aaronpowell, Thanks for this research.
Skiping the logic in publish mode is good enough.


Update inifile_db_hdb.params.j2

fix issue 84. Set hdb.create.dbacockpit.user=false, seems this is not the default.

please pull, now it should work

please pull, it should work now


Apply FReLU to YOLOv5

Thank you for your great contributions! In YOLOv5 we skipped over using Swish or Mish due to their expensive nature, especially during training, and retained the same LeakyReLU(0.1) as YOLOv3. We have a PR for the addition of FReLU
https://github.com/ultralytics/yolov5/pull/556 as this. Is this a correct implementation? Should we use torch.max() or torch.maximum()?
# FReLU https://arxiv.org/abs/2007.11824 --------------------------------------
class FReLU(nn.Module):
    def __init__(self, c1, k=3):  # ch_in, kernel
        super().__init()__()
        self.conv = nn.Conv2d(c1, c1, k, 1, 1, groups=c1)
        self.bn = nn.BatchNorm2d(c1)

    @staticmethod
    def forward(self, x):
        return torch.maximum(x, self.bn(self.conv(x)))

Secondly, I saw you recommended using FReLU only in the backbone of a detection network. So would you recommend replacing all of our LeakyReLU(0.1) activations in the backbone with FReLU() to start, and then doing the same with the head activations if we see positive results? Our main model structure is here:
https://github.com/ultralytics/yolov5/blob/master/models/yolov5s.yaml
# YOLOv5 backbone
backbone:
  # [from, number, module, args]
  [[-1, 1, Focus, [64, 3]],  # 0-P1/2
   [-1, 1, Conv, [128, 3, 2]],  # 1-P2/4
   [-1, 3, BottleneckCSP, [128]],
   [-1, 1, Conv, [256, 3, 2]],  # 3-P3/8
   [-1, 9, BottleneckCSP, [256]],
   [-1, 1, Conv, [512, 3, 2]],  # 5-P4/16
   [-1, 9, BottleneckCSP, [512]],
   [-1, 1, Conv, [1024, 3, 2]],  # 7-P5/32
   [-1, 1, SPP, [1024, [5, 9, 13]]],
   [-1, 3, BottleneckCSP, [1024, False]],  # 9
  ]

# YOLOv5 head
head:
  [[-1, 1, Conv, [512, 1, 1]],
   [-1, 1, nn.Upsample, [None, 2, 'nearest']],
   [[-1, 6], 1, Concat, [1]],  # cat backbone P4
   [-1, 3, BottleneckCSP, [512, False]],  # 13

   [-1, 1, Conv, [256, 1, 1]],
   [-1, 1, nn.Upsample, [None, 2, 'nearest']],
   [[-1, 4], 1, Concat, [1]],  # cat backbone P3
   [-1, 3, BottleneckCSP, [256, False]],  # 17 (P3/8-small)

   [-1, 1, Conv, [256, 3, 2]],
   [[-1, 14], 1, Concat, [1]],  # cat head P4
   [-1, 3, BottleneckCSP, [512, False]],  # 20 (P4/16-medium)

   [-1, 1, Conv, [512, 3, 2]],
   [[-1, 10], 1, Concat, [1]],  # cat head P5
   [-1, 3, BottleneckCSP, [1024, False]],  # 23 (P5/32-large)

   [[17, 20, 23], 1, Detect, [nc, anchors]],  # Detect(P3, P4, P5)
  ]

Thank you in advance in advance for your time and any recommendations for our situation!

Hi @glenn-jocher ,
(1) you should use torch.max(), which is different form MegEngine API.
(2) yes you could try to use FReLU only in the backbone first, then try it in head. you could also replace all the LeakyReLU activations to FReLU because there is only one activation in each BottleneckCSP which are not originally many.

@nmaac great, thanks for the feedback! We'll try to experiment with this and see if it helps our results!

We'll try to experiment with this and see if it helps our results!

Hi glenn, is there any improment on COCO or VOC dataset about FReLU ?  Or how does it perform on the verification set? Thanks !

@clw5180 I've run some experiments with YOLOv5s, which seem to show some early improvement. I have not tried larger models, and improvements from one size to another sometimes does not extrapolate well, so I can't draw any conclusions across the entire v5 lineup, but so for small models it appears to help.
One warning is that FReLU increases training requirements greatly, similar to Swish perhaps, not quite as bad as Mish, so it's probably not a viable replacement for all activation functions in a model, but might be suitable for partial replacement.


Authentication

This is awesome. Thanks for your work here! Wondering how you/if you would go about setting up some sort of fake authentication system. Specifically, I want to use JWT. Do you think there's room for something like this in this project?

Hi Kent,
Thanks for the nice comment :)
I would try this:
https://github.com/typicode/json-server/issues/42
You should be able to add routes or use middlewares since the project is built on Express.
Tell me if that helps?

Wrong issue, this is the one I wanted to link :)
https://github.com/typicode/json-server/issues/30

I'll give it a look. Thanks!

Closing the issue.
Just for the record, now JSON Server can be used as a middleware so it should be easier to add authentication.

:+1:


UI changes from QA feedback

https://issues.redhat.com/browse/YARA-442
https://issues.redhat.com/browse/YARA-441
Changed systable column widths and increased system name font size

In the screenshot:

Removed unused checkbox
Made the system name font larger (to match Advisor's system table)
Removed OS sorting (because it doesn't work ATM) and fixed 'RHEL null' to 'Not available'


S3 Source connector based on iterators.

This implementation uses less memory that the original implementation, correctly handles the `poll' limit and solves several problems with excessive method arguments.

@Claudenw can we close this?

Closing this for now.


new release and documentation correction

Hello, thank you for the great package. I recently added befriended through composer and realized that following didn't work. I checked on the github page and saw that it was an issue that was corrected. I had to manually replace the file to make it work. Since it is a breaking error, maybe a release could help others not search for a long time. Also the documentation features errors in the block and like section. Instead of $user->liking() or $user->blocking() it is written $user->likings() and $user->blockings() which are misleading (I guess a typo mistake).
Again thank you very much for all

Hey @lukadriel7,
It's been pushed, it seems like the pull request messed up the following section of the package. I reverted that, also I fixed the documentation (i also counted consistency), so now the actual relationships remained exactly like that: blocking() and liking() (without plural).
The PR can be found here: https://github.com/rennokki/befriended/pull/28
Right after the PR passes tests, i will release as a minor since no breaking changes occured.


Loading timeseries data to Redshift using AWS Wrangler results in loss of timezone information

I am working on a use case which involves loading time series data with timezone information in Redshift. There are 50K datapoints.
There are two ways to do this:

Using  wr.redshift.to_sql() method
Using wr.redshift.copy() method

Using the wr.redshift.to_sql() method I am able to retain the timezone information in the Redshift. In the documentation for larger DataFrames (1K+ rows) they recommend to use wr.redshift.copy(). When I use the wr.redshift.copy() method I am not able to retain the timezone information. Please help me to resolve this. Below are the approaches I worked on and their results:
Environment:
OS: Linux
Python : 3.8.3
awswrangler : 2.16.1
redshift_connector : 2.0.913
pandas : 1.4.1
Code to reproduce:
Creating a time series Dataframe
from datetime import timedelta
import awswrangler.redshift as rs
import redshift_connector
import pandas as pd
import os

a = pd.Timestamp('2020-03-14T15:32:52.192548651', tz="UTC")
time_index = []
timestamp_tz = []
for counter in range(1,10):
    time_index.append('time_'+str(counter))
    timestamp_tz.append(a+timedelta(days=counter))
dataframe = pd.DataFrame(data = {'time_index': time_index, 'timestamp_info':timestamp_tz})
dataframe.info()

Creating the Redshift Connection
rs_conn = redshift_connector.connect(host=host, database=database, user=user, password=password)
rs_conn.autocommit = True

Loading the DataFrame to Redshift using  wr.redshift.copy() method
Approach 1:
rs.copy(
  df=dataframe,
  path='S3_path',
  con=rs_conn,
  table='timestamp_test',
  schema='schema_test',
  )

Result : Stores data in the Redshift without timezone. The datatype is timestamp instead of timestamptz.
Approach 2:
rs.copy(
  df=dataframe,
  path='S3_path',
  con=rs_conn,
  table='timestamp_test',
  schema='schema_test',
  dtype = {'timestamp_info':'TIMESTAMPTZ'}
  )

UnsupportedType: Unsupported Athena type: timestamptz
Approach 3: Replaced 'TIMESTAMPTZ' with 'timestamp with time zone' but it also gave output as UnsupportedType.
Approach 4:
rs.copy(
  df=dataframe,
  path='S3_path',
  con=rs_conn,
  table='timestamp_test',
  schema='schema_test',
  sql_copy_extra_params=["TIMEFORMAT AS 'auto'"]
  )

ProgrammingError: {'S': 'ERROR', 'C': '0A000', 'M': 'TIMEFORMAT argument is not supported for PARQUET based COPY', 'F': '../src/pg/src/backend/commands/commands_copy.c', 'L': '2933', 'R': 'report_keyword_error'}
Any help in this regard is appreciated.

Hi @Mahima-ai, could you double check that your table actually has the timezone_info set to the TIMESTAMPTZ data type?
The code below returns a data frame with the timezone, but note how the column has the right type at table creation:
from datetime import timedelta
import awswrangler as wr
import pandas as pd

schema = "public"
table = "test_timezone_2413"
iam_role = "arn:aws:iam::..."
path = f"s3://bucket/{table}/"

con = wr.redshift.connect(connection="aws-sdk-pandas-redshift")
con.autocommit = True

with con.cursor() as cursor:
    cursor.execute(
        f"""
        CREATE TABLE {schema}.{table} (
            time_index VARCHAR(100),
            timestamp_info TIMESTAMPTZ
        );
        """
    )

df = pd.DataFrame({
    "time_index": [f"time_{i}" for i in range(1, 10)],
    "timestamp_info": [pd.Timestamp('2020-03-14T15:32:52.192548651', tz="UTC") + timedelta(days=i) for i in range(1, 10)],
})
print(df)
print(df.dtypes)

wr.redshift.copy(
    df=df,
    path=path,
    con=con,
    table=table,
    schema=schema,
)

df_redshift = wr.redshift.read_sql_table(con=con, table=table, schema=schema)

print(df_redshift)
print(df_redshift.dtypes)

Output:
  time_index                      timestamp_info
0     time_1 2020-03-15 15:32:52.192548651+00:00
1     time_2 2020-03-16 15:32:52.192548651+00:00
2     time_3 2020-03-17 15:32:52.192548651+00:00
3     time_4 2020-03-18 15:32:52.192548651+00:00
4     time_5 2020-03-19 15:32:52.192548651+00:00
5     time_6 2020-03-20 15:32:52.192548651+00:00
6     time_7 2020-03-21 15:32:52.192548651+00:00
7     time_8 2020-03-22 15:32:52.192548651+00:00
8     time_9 2020-03-23 15:32:52.192548651+00:00
time_index                     object
timestamp_info    datetime64[ns, UTC]

  time_index                   timestamp_info
0     time_1 2020-03-15 15:32:52.192548+00:00
1     time_3 2020-03-17 15:32:52.192548+00:00
2     time_5 2020-03-19 15:32:52.192548+00:00
3     time_7 2020-03-21 15:32:52.192548+00:00
4     time_9 2020-03-23 15:32:52.192548+00:00
5     time_2 2020-03-16 15:32:52.192548+00:00
6     time_4 2020-03-18 15:32:52.192548+00:00
7     time_6 2020-03-20 15:32:52.192548+00:00
8     time_8 2020-03-22 15:32:52.192548+00:00
time_index             string[python]
timestamp_info    datetime64[ns, UTC]

Hi @Mahima-ai, could you double check that your table actually has the timezone_info set to the TIMESTAMPTZ data type?
The code below returns a data frame with the timezone, but note how the column has the right type at table creation:
from datetime import timedelta
import awswrangler as wr
import pandas as pd

schema = "public"
table = "test_timezone_2413"
iam_role = "arn:aws:iam::..."
path = f"s3://bucket/{table}/"

con = wr.redshift.connect(connection="aws-sdk-pandas-redshift")
con.autocommit = True

with con.cursor() as cursor:
    cursor.execute(
        f"""
        CREATE TABLE {schema}.{table} (
            time_index VARCHAR(100),
            timestamp_info TIMESTAMPTZ
        );
        """
    )

df = pd.DataFrame({
    "time_index": [f"time_{i}" for i in range(1, 10)],
    "timestamp_info": [pd.Timestamp('2020-03-14T15:32:52.192548651', tz="UTC") + timedelta(days=i) for i in range(1, 10)],
})
print(df)
print(df.dtypes)

wr.redshift.copy(
    df=df,
    path=path,
    con=con,
    table=table,
    schema=schema,
)

df_redshift = wr.redshift.read_sql_table(con=con, table=table, schema=schema)

print(df_redshift)
print(df_redshift.dtypes)

Output:
  time_index                      timestamp_info
0     time_1 2020-03-15 15:32:52.192548651+00:00
1     time_2 2020-03-16 15:32:52.192548651+00:00
2     time_3 2020-03-17 15:32:52.192548651+00:00
3     time_4 2020-03-18 15:32:52.192548651+00:00
4     time_5 2020-03-19 15:32:52.192548651+00:00
5     time_6 2020-03-20 15:32:52.192548651+00:00
6     time_7 2020-03-21 15:32:52.192548651+00:00
7     time_8 2020-03-22 15:32:52.192548651+00:00
8     time_9 2020-03-23 15:32:52.192548651+00:00
time_index                     object
timestamp_info    datetime64[ns, UTC]

  time_index                   timestamp_info
0     time_1 2020-03-15 15:32:52.192548+00:00
1     time_3 2020-03-17 15:32:52.192548+00:00
2     time_5 2020-03-19 15:32:52.192548+00:00
3     time_7 2020-03-21 15:32:52.192548+00:00
4     time_9 2020-03-23 15:32:52.192548+00:00
5     time_2 2020-03-16 15:32:52.192548+00:00
6     time_4 2020-03-18 15:32:52.192548+00:00
7     time_6 2020-03-20 15:32:52.192548+00:00
8     time_8 2020-03-22 15:32:52.192548+00:00
time_index             string[python]
timestamp_info    datetime64[ns, UTC]

Hi @jaidisido,
Thank you for your response. The example given by you is working but it fails in the following two scenarios:

When we use the parameter mode='overwrite' in the wr.redshift.copy(). My use case uses the overwrite mode.
We query the table using wr.redshift.unload() instead of wr.redshift.read_sql_table(). wr.redshift.unload() is the recommended method for 1K+ rows.

Thanks!


UnicodeDecodeError when importing photos

Hi there!
Thank you for this amazing software :-) It was just what I was looking for.
I've started to import a set of photos that I have with the suggested:
./elodie.py import --debug --destination=~/Pictures/organized ~/Pictures/conv
But after processing almost all, I got this error:
/home/familia/Pictures/conv/IMG_0039.JPG -> /home/familia/Pictures/organized/2015/Unknown Location/2015-07-18_16-31-41-img_0039.jpg
****** ERROR DETAILS ******
Traceback (most recent call last):
  File "./elodie.py", line 345, in <module>
    main()
  File "/home/familia/.local/lib/python2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/home/familia/.local/lib/python2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/home/familia/.local/lib/python2.7/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/familia/.local/lib/python2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/familia/.local/lib/python2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "./elodie.py", line 119, in _import
    result.write()
  File "/home/familia/installs/elodie/elodie/result.py", line 29, in write
    print(tabulate(error_result, headers=error_headers))
  File "/home/familia/.local/lib/python2.7/site-packages/tabulate.py", line 1119, in tabulate
    ['\t'.join(map(_text_type, row)) for row in list_of_lists])
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 47: ordinal not in range(128)

I thought it was one off so I deleted the organized folder and the ~/.elodie/hash.json and started again.
Same error. I then tried to just import that file but the error didn't appear so I'm guessing it's not that file but maybe the next in list ?
What do you suggest ?
Thank you.

Do you mind adding print(_file) in filesystem.py on line 324.?
Then run the command again and let me know which file it fails at. Ideally if you could send it to me then I could reproduce it. jaisen (at) githubusername (dot) com.
    def process_file(self, _file, destination, media, **kwargs):
        print(_file)
        move = False
        if('move' in kwargs):
            move = kwargs['move']

Unfortunately I can no longer reproduce this :(
I'm not aware of anything that has changed on the ~/Pictures/conv folder that was used for importing.
But I do notice that is a discrepancy on how much items were processed.
So I was expecting some similar like:
$ find ~/Pictures/conv -type f \( -iname "*.jpg" -o -iname "*.jpeg" -o -iname "*.png" -o -iname "*.mpg" -o -iname "*.mov" -o -iname "*.mp4" \) | wc -l
6275
And elodie output is:
****** SUMMARY ******
Metric      Count
--------  -------
Success      6129
Error          80

So in total elodie processed 6209 vs 6275 of what I was expecting ? Should I raise a separate issue for this ?

I am able to reproduce this problem:
Traceback (most recent call last):   File "./elodie.py", line 345, in <module>     main()   File "/home/agrieco/.local/lib/python2.7/site-packages/click/core.py", line 716, in __call__     return self.main(*args, **kwargs)   File "/home/agrieco/.local/lib/python2.7/site-packages/click/core.py", line 696, in main     rv = self.invoke(ctx)   File "/home/agrieco/.local/lib/python2.7/site-packages/click/core.py", line 1060, in invoke     return _process_result(sub_ctx.command.invoke(sub_ctx))   File "/home/agrieco/.local/lib/python2.7/site-packages/click/core.py", line 889, in invoke     return ctx.invoke(self.callback, **ctx.params)   File "/home/agrieco/.local/lib/python2.7/site-packages/click/core.py", line 534, in invoke     return callback(*args, **kwargs)   File "./elodie.py", line 115, in _import     trash, allow_duplicates)   File "./elodie.py", line 38, in import_file     _file = _decode(_file)   File "/home/agrieco/elodie/elodie/compatability.py", line 23, in _decode     return string.decode(encoding) UnicodeDecodeError: 'ascii' codec can't decode byte 0xef in position 49: ordinal not in range(128)
Have the photo handy if useful

$ sudo python -c "import sys; print(sys.stdout.encoding)"
Gives me UTF-8
I added the print(_file) statement. Will report back.

It didnt print out the file... What to do?

Can you include the stack trace?

****** ERROR DETAILS ******
Traceback (most recent call last):
  File "./elodie.py", line 371, in <module>
    main()
  File "/home/jogai/.local/lib/python2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/home/jogai/.local/lib/python2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/home/jogai/.local/lib/python2.7/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/jogai/.local/lib/python2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/jogai/.local/lib/python2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "./elodie.py", line 144, in _import
    result.write()
  File "/home/jogai/src/elodie/elodie/result.py", line 29, in write
    print(tabulate(error_result, headers=error_headers))
  File "/home/jogai/.local/lib/python2.7/site-packages/tabulate.py", line 1119, in tabulate
    ['\t'.join(map(_text_type, row)) for row in list_of_lists])
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 47: ordinal not in range(128)

I did the whole thing in batches and then I didn't encounter the error.


Should we add option to set seed?

It seems like there is no option to set seed?

Not supported as a cli arg but you can prob set it yourself doing something like this
torch.manual_seed(42)
torch.cuda.manual_seed_all(42)

as done in test_benchmark.py

Oh, I see. Thx!


jsii should enforce consistent accessors between properties and their overrides

In .NET, an override of a get-only property may not contain a setter. So this is invalid:
class Base {
    public abstract string Foo { get; }
}

class Child : Base {
    public override string Foo {
        get => /* ... */ ;
        set => /* ... */ ;
    }
}

When jsii detects inconsistent accessors, it should report an error.

This was fixed by #115.


sfx does not reload

in-game sfx does not reload with textureldr. look into GameSoundManager / FMODAudioEngine.

fixed, will push commit after adding some more options to options menu

fixed in today's commit (v2.0a).


Move 1103

Issue Addressed
This PR closes MOVE-1103 - multiple location selection responsive fixes.
Description
some CSS tweaks for the multi-select sidebar.
Bringing visual parity between multi-select sidebar and the single-select sidebar.
Prevents stretching of width for long location names, and removes some hardcoded widths/heights.
A 'Clear all' button is removed, as it was already hidden behind the sidebar during many interactions.
Other:

use .min file of material Design icons (320kb vs 380kb)
shows 'include intersections ...' toggle only if multi-locations selected

Feedback and questions welcome!


Post Request to my api and conv.ask is not even working.

const {SignIn, Suggestions} = require('actions-on-google');
var request = require('request');
var querystring = require('querystring');
const functions = require('firebase-functions');
const config = require('../config/config.json');
module.exports = {
    'ask_for_sign_in' : (conv) => {
        conv.ask(new SignIn());
    },
    'search_data' : (conv, params) => {
        const payload = conv.user.profile.payload;
        if(payload.name)
        {
            if(params.skill)
            {
                var searchParam = geocity = geostate = rate = searchText = "";
                searchParam = params.skill;
                // for (var i = 0 ; i < searchParam.length; i++) {
                //     searchText += searchParam[i] +" ";
                // }
                searchText = searchParam;
                if(params["geo-city"])
                {
                    geocity = params["geo-city"];
                }
                if(params["geo-state-us"])
                {
                    geostate = params["geo-state-us"];
                }
                var post_data =  querystring.stringify({
                    'searchParam' : searchText,
                    'geocity' : geocity,
                    'geostateus': geostate
                });

                var options = {
                    url: "https://"+config.host+"/api/search",
                    method: "POST",
                    body: post_data,
                    json: true,
                    headers: {
                      'Content-Type': 'application/x-www-form-urlencoded',
                      'xxxapitoken' : 'xxxxxtokenxxxxx'
                    }
                };
                var data = "";
                request(options, function (err, res, body) {
                    conv.ask("Inside Response API"); // it is not working even just simple text response.
                    data = body+res+err;
                });
                conv.ask("Response : "+data); //it show response on google assistant but data is blank
                  //conv.ask(`Request : ${searchText} Geocity ${geocity} STATE ${geostate}`);
            }else{
                conv.ask(`Great! ${payload.name}. Welcome back. What do you want to do next?`);
            }
        }else{
            conv.ask(`Please sign in to use the app. Can we sign you in by using your Google Account?`)
            conv.ask(new Suggestions(['Ok Sign Me In', 'No']));
        }
    },
}

My Api is working fine using Rest Client
Here is the response
{
"keyword": {
"params": "Java",
"lat": 34.0522342,
"lng": -118.2436849
},
"result": {
"rateAvg": [
  "101-150"
],
"rateLow": [
  "101-150"
],
"rateHigh": [
  "300+"
],
"resumeCount": 5
}
}

Thank you

@chantrea999 You need to use the app.intent('YOUR_INTENT_NAME', conv => { ... }) function to register your code to handle for each intent. That is, the current code
'ask_for_sign_in' : (conv) => {
    conv.ask(new SignIn());
},

should be changed to like the following code:
const {SignIn, Suggestions, dialogflow } = require('actions-on-google');
...
const app = dialogflow({ debug: true });
app.intent("ask_for_sign_in", conv => {
    conv.ask(new SignIn());
});
...
exports.foobar = functions.https.onRequest(app);

@yoichiro Yes thank for your reply. Yes I did use the app.intent but it is inside the modules
please refer to this link and look at the functions folder
https://github.com/actions-on-google/dialogflow-helper-intents-nodejs

@yoichiro
Here is index.js
'use strict';
const functions = require('firebase-functions');
const app = require('./app');
module.exports.helperIntentExampleAction = functions.https.onRequest(app);

app.js
'use strict';
const askForConfirmation = require('./helper-intents/ask-for-confirmation');
const askForDateTime = require('./helper-intents/ask-for-datetime');
const askForPermission = require('./helper-intents/ask-for-permission');
const askForPlace = require('./helper-intents/ask-for-place');
const askForSignIn = require('./helper-intents/ask-for-sign-in');
const askToDeepLink = require('./helper-intents/ask-to-deep-link');
const contactManager = require('./helper-intents/contact_manager');
const search_data = require('./helper-intents/search_data');
const app = require('actions-on-google').dialogflow();

/** Adds Intent-name & callback key value pairs to app */
function addIntents(...args) {
  for (let i = 0; i < args.length; i++) {
    for (const key in args[i]) {
      if (args[i].hasOwnProperty(key)) app.intent(key, args[i][key]);
    }
  }
}

addIntents(
  askForConfirmation,
  askForDateTime,
  askForPermission,
  askForPlace,
  askForSignIn,
  askToDeepLink,
  contactManager,
  search_data
);

app.intent('Default Welcome Intent', (conv) => {
  conv.ask('Welcome to the Helper Intents Tutorial.');
});

module.exports = app;

search_data.js
it was the first comment code.

@chantrea999 Here is a place to report issues about the SDK itself, is not a place to ask how to use this sdk, I think. You should do the codelab first to learn the usage of this sdk. After that, if you have a question, I recommend that you ask your question at Stackoverflow, AoG support form and others.

@yoichiro You have no ideas that is the issue. Did you read the title conv.ask it is not event working under any callback function.
request(options, function (err, res, body) {
conv.ask("Inside Response API"); // it is not working even just simple text response.
data = body+res+err;
});
I am not come here to ask you how to use it I am asking you to fix the bugs if you are not the right guy that is fine. I will figure out my self.

You have no ideas that is the issue. Did you read the title conv.ask it is not event working under any callback function.

@chantrea999 You need to use Promise to handle the async process. This is a specification of this SDK v2, is not a bug. You should read the document: Intent handler helpers and arguments. This is the issue how to use the sdk.


digeq-146 rich text always bold, CSS using wrong font

What
Rich text is always bold in survey runner
How to test

Create a questionnaire with rich text in, but without making it bold
Open survey runner and make sure the rich text isn't bold

Who can review
Anyone but LJBabbage

:+1:


Upgrade to Vitess v17.0.1

This PR updates the project to Vitess v17.0.1 and adds some automation for proto file syncing.

Codecov Report

Merging #5 (49967c6) into main (7db1314) will decrease coverage by 0.1%.
The diff coverage is 0.0%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files

Files Changed
Coverage Δ

vitess-grpc/src/generated/binlogdata.rs
3.1% <0.0%> (-0.6%)
:arrow_down:

vitess-grpc/src/generated/mysqlctl.rs
0.0% <0.0%> (ø)

vitess-grpc/src/generated/query.rs
0.7% <0.0%> (-0.1%)
:arrow_down:

vitess-grpc/src/generated/replicationdata.rs
0.0% <ø> (ø)

vitess-grpc/src/generated/tabletmanagerdata.rs
0.0% <0.0%> (ø)

vitess-grpc/src/generated/tabletmanagerservice.rs
0.0% <0.0%> (ø)

vitess-grpc/src/generated/topodata.rs
0.0% <ø> (ø)

vitess-grpc/src/generated/vtadmin.rs
0.0% <0.0%> (ø)

vitess-grpc/src/generated/vtctldata.rs
0.0% <0.0%> (ø)

vitess-grpc/src/generated/vtctlservice.rs
0.0% <0.0%> (ø)

... and 1 more

... and 2 files with indirect coverage changes


iOS system exception "You should reference the NuGet package"

I installed the NuGet in PCL and iOS and also configurated the iCloud Kit on developer.apple, but I got this error:
System.Exception: This functionality is not implemented in the portable version of this assembly. You should reference the NuGet package from your main application project in order to reference the platform-specific implementation.

You need to link the iOS dll files from your iOS project. To find the required dll, go to the NuGet package -> lib -> MonoTouch10 -> select the 2 dll files there & add it to your iOS project reference.

I shouldn't reference it like that, you should install it as a NuGet also on your iOS project.

But the problem is the NuGet is outdated and it doesn't work on iOS at all.

@riandesign I will look into it, please try not to go through every issue and report the error you have. Once is enough. I will try to find some time tomorrow to have a look.

I am having the exact same issue with the nuget.  Did this ever get fixed and if so how?

I'm having the same issue with an iOS project. Don't understand why this issue is closed if it's still a problem.

I'm having the same issue!  Any news about it?

Same issue

I am facing the same issue. Any clean solution?

Me too have the same issue? An update on it?

same problem

tested in a blank project

@CarLoOSX this is fixed on the fork of this in my repo and accompanied NuGet.
In an effort to further develop this plugin I have moved it into my own account: https://github.com/jfversluis/FilePicker-Plugin-for-Xamarin-and-Windows.
Please look for the pre-release package.


Added: Login and Signup pages

Login and SignUp components

You have included css within js, this is considered as a bad practice.
Read more

Please move all the css to index.css

When will this be complete? @FL45Hi

Make a Dev branch and push there


App cannot start

Hi!
Please can you check this one?
It's not Blazebit bug, but Hibernate is ignoring this issue.
https://hibernate.atlassian.net/browse/HHH-15004
Thanks!

Thanks for the ping. I wasn't aware that you were using Blaze-Persistence 😄
Like I wrote on the Hibernate issue, you seeing this error because of an unfortunate SPI change, which requires that you switch to the new integration module as pointed out in https://persistence.blazebit.com/news/2022/blaze-persistence-1.6.5-release.html


Fixing unescaped hash signs in a Markdown tab.

The text within these tabs are parsed as Markdown, so the # hash signs were breaking the markdown and rendering the page differently than intended.  I've escaped the hash signs to fix this.

/hold

@JayBeale As mentioned by @sftim, would you investigate using a code block?
After changing, feel free to remove the hold.

I apologize for leaving this pull request open - I had missed coming back to it.  I'll close it.


Batch request for Vision API "web"?

Hey, is there a way to send batch requests when using the Vision API (.image and .web methods specifically)

There is a way to submit a single API request for multiple images. You can do this using the Project#annotate method:
require "google/cloud/vision"

vision = Google::Cloud::Vision.new

face_img = vision.image "path/to/face.jpg"
landmark_img = vision.image "path/to/landmark.jpg"
text_image = vision.image "path/to/text.png"

annotations = vision.annotate do |annotate|
   annotate.annotate face_img, faces: true, labels: true
   annotate.annotate landmark_img, landmarks: true
   annotate.annotate text_image, text: true
end

Image#web will only annotate the single image. To process multiple images you must use Project#annotate.

@9mm Does this work for you? If so, can you close the issue? Thanks!

Thanks! I'll give it a shot, I didn't get a chance yet.


[refactor] #4161: rewrite config, minimally

Description
This PR is a complete overhaul of Iroha & Client CLI configuration. It is based on the Configuration Overhaul RFC.
Although this PR has many changes already, there are still big parts of the RFC to be completed separately.
Configuration changes

Change format: TOML, lower-case fields
Restructure and rename parameters for users' convenience
Localise chain-wide parameters in the [chain_wide] section to ease further transition: https://github.com/hyperledger/iroha/issues/4028
Limit set of ENV aliases for the parameters
Support composition via extends mechanism
Split user config view from strongly-typed internal config representation
Emit as many errors at once as possible: collecting all missing fields, all ENV parsing failures, and semantic errors.

Subsequent changes

Revise sample configurations:

remove alt branch configs (configs/peer/lts|stable, configs/client/lts|stable)
put client.template.toml, peer.template.toml, and prometheus.template.yaml in the root of configs. These configs are not for direct use, but rather a part of documentation.
move docker compose files into configs/swarm
move docker compose related configurations into configs/swarm as well (client.toml, genesis.json, and executor.wasm)
some tests & deployment docs relied to /configs directory to use client/peer configs. It was implied that the configs are actually workable and filled with everything needed. It was ambiguous. Now those places rely on /configs/swarm content.

Update test environment:

test_env.py:

doesn't use ENVs to set configuration
generates a shared config, and individual configs (extending the shared one) for each peer

Pytests:

Split CLIENT_CLI_DIR (implying it has iroha_client_cli and config.json content) setting into CLIENT_CLI_BINARY and CLIENT_CLI_CONFIG

Iroha Swarm:

Doesn't rely on IROHA_CONFIG variable anymore, and doesn't expect Iroha configuration to be present in the configs directory; relies only on genesis.json, and all necessary configs are set in ENV.
TODO: allow users to customise configuration? For all peers or for individual ones somehow?

maybe via --config-shared=shared.toml --config-peer0=peer0.toml CLI

Iroha Torii:

split constants (namely - URIs) into a iroha_torii_const crate for easier re-use in iroha_client. Previously those were defined in iroha_config, which is not an appropriate place for it.

Kagami:

remove kagami config command, which used to produce "sample" peer/client configurations

Intentionally left issues (temporarily)

Using Report as a universal error to not bother with errors (yet)
Default is used for *Partials to create empty layers, while Default for actual config structures fills them with default values.

Not implemented

Tracing of configuration: basic debug logging to track how config is read, where and what is overwritten, how final value looks like etc
Enhanced errors:

emitting errors with fancy diagnostics by ariadne, spans to config files and/or ENVs, hints etc
"lazy" parsing of TOML with multiple-errors emitting

Human-readable durations and byte sizes; although I put HumanDuration and HumanBytes as stubs for future.

Post-merge
Upon merging of this PR, create these as separate issues:

Create a separate general purpose configuration crate (project soukousei), move some of iroha_config_base's stuff there, define a procmacro to generate boilerplate
Enhance errors (might be part of the previous one):

use fancy diagnostics and hints
use spans to source files and envs
don't use eyre::Report as a universal error

Remove chain-wide configuration from here: https://github.com/hyperledger/iroha/issues/4028
Cover client.template.toml and peer.template.toml with some tests
Implement config tracing as per RFC
Ensure how Snapshot uses its path from configuration: it should automatically create the top-level directory for it, but not upper. It allows specifying ./storage/snapshot as a default value, which is convenient.
Human-readable durations and bytes
Update reference: https://github.com/hyperledger/iroha-2-docs/issues/392
Change PrivateKey deserialisation format: allow algorithm and payload fields, deprecate digest_function

Linked issues
Closes #4161, #3499, #3500, #3501, #3504, #3358

@AlexStroke btw this PR changes the way Pytests are configured. Please take into account.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-493 of 2063 (76.1%)  changed or added relevant lines in 44 files are covered.
1265 unchanged lines in 30 files lost coverage.
Overall coverage decreased (-0.3%) to 56.498%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cli/src/main.rs
4
5
80.0%

core/src/query/store.rs
4
5
80.0%

core/src/smartcontracts/wasm.rs
7
8
87.5%

tools/swarm/src/compose.rs
101
102
99.02%

config/src/client_api.rs
1
3
33.33%

core/src/kiso.rs
14
16
87.5%

core/src/queue.rs
51
53
96.23%

core/src/wsv.rs
14
16
87.5%

telemetry/src/dev.rs
0
2
0.0%

config/src/kura.rs
11
14
78.57%

Files with Coverage Reduction
New Missed Lines
%

config/base/src/lib.rs
1
86.08%

crypto/src/signature/secp256k1.rs
1
91.94%

tools/kagami/src/main.rs
1
0.0%

tools/swarm/src/compose.rs
1
91.9%

client_cli/src/main.rs
2
9.45%

cli/src/samples.rs
2
0.0%

core/src/sumeragi/mod.rs
2
0.0%

primitives/src/unique_vec.rs
2
91.24%

data_model/src/metadata.rs
3
78.52%

core/test_network/src/lib.rs
7
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
22437

Relevant Lines:
39713

💛  - Coveralls


DocumentFragment should have append method.

According to spec DocumentFragment implements ParentNode interface, yet polyfill doesn't account for that.
> 'append' in document.createDocumentFragment();
< false

Should I write additional test cases?
Also, why you catching exception in case if mutating document.createDocumentFragment().prototype fails? Where this could happen?

Thanks for the report. I'll answer to everything at once in here:

please do not modify manually anything inside the build folder. Just modify the file into the src one. If you want to PR properly run make dependencies and after that just run make
in the following list of devices there were edge cases covered by the try catch. This should not bother you at all so I don't see the problem in having it there. The idea is to limit compatibility issues when this file is in. Removing the try catch won't give you any advantage so please leave it as it is now ;-)
your check is obtrusive. You are testing that the property is either append or prepend but you are nto checking if the engine already implements native those two methods. You should temporarily grab the prototype and verify it doesn't have already those two methods and only if that's true you can add them

I might just quickly patch this so don't worry about this if you don't have time and thanks again for bringing this up.

Thanks for extensive reply. I'm wondering about try/catch because this polyfill deployed on our mobile site (which is having millions of unique users). So compatibility is the issue though we don't pursue support of outdated user agents. So not really important, just got my attention.

You are testing that the property is either append or prepend but you are nto checking if the engine already implements native those two methods.

Yup, that's true! I'll fix it on the weekend.

no need to fix anything, the version 1.7.0 has everything you need plus tests ;-)

FYI the commit with all needed changes is here: https://github.com/WebReflection/dom4/commit/11c0275a70691ee323839346c003e79711d4c002
I've done few little extra adjustments too plus added tests

Great, thanks! :-)


The requested package marcelog/pami No version set

I did a update for composer and run install. But it says "The requested package marcelog/pami No version set (parsed as 1.0.0) is satisfiable by marcelog/pami[No version set (parsed as 1.0.0)] but these conflict with your requirements or minimum-stability". Please let me know wha can be done on this issue?

PAMI does not offer a 1.0.0 version, there is no such thing. To see the list of available versions for use with composer, see https://packagist.org/packages/marcelog/pami


[openvr] Added Linux support and updated to v1.10.30

ValveSoftware/openvr contains linux64 and linux32 but wasn't being used in the Vcpkg port file so I added them.
osx32 (no 64 bit and latest version Catalina is 64 bit only) exists too but hasn't been updated in 4 years. I think VR support on Mac is basically non existent but I know Valve is focusing on Linux a lot.
I updated the version as well.

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

Hi @makitsune
Thanks for this PR.
I noticed this port has passed on Linux platform. Could you please remove openvr:x64-linux=fail from ci.baseline.txt?

Thank you so much @NancyLi1013 for replying so quickly.
I made the changes you suggested.
I didn't replace the file(COPY ... calls for /lib and /bin with file(INSTALL ... because the file permissions get reset. Specifically they're not executable (+x) anymore.
Also, VCPKG_CMAKE_SYSTEM_NAME STREQUAL "" for detecting Windows doesn't work anymore. Though it's stated in the documentation here:
https://github.com/microsoft/vcpkg/blob/master/docs/users/triplets.md#vcpkg_cmake_system_name
Probably best to add that it's deprecated. I could open a pr?

/azp run

@makitsune
thanks for your update and reminder.
Currently, we suggest to use VCPKG_TARGET_IS_WINDOWS, VCPKG_TARGET_IS_LINUX , VCPKG_TARGET_IS_OSX,  VCPKG_TARGET_IS_UWP  etc to judge the platforms instead of VCPKG_CMAKE_SYSTEM_NAME ,
As for the usage of `VCPKG_CMAKE_SYSTEM_NAME  for detecting Windows, it is the correct way to use like this:
NOT VCPKG_CMAKE_SYSTEM_NAME
Of course, we need to update the deprecated functions to document.
You don't need to open a new PR to update this.

Thanks for the PR!


Device list error: TypeError: Cannot set properties of undefined (setting '2-1')

Dreame L40 Ultra
following warnings/errors in current Github version:

Besides this, the already repoted problem about data type mismatch still exists:
has to be type "string" but received type "number"
Regardless, the adapter is working fine and I appreciate the work by you guys, thanks!

fixed


How can I migrate to v3 from v2 not using lazy or other plugin

Problem
rror detected while processing /home/jinlei/.config/nvim/init.vim[3]../home/jinlei/.vimrc:
line  685:
You are trying to call the setup function of indent-blankline version 2, but you have version 3 installed.
Take a look at the GitHub wiki for instructions on how to migrate, or revert back to version 2.
Error detected while processing VimEnter Autocommands for "*":

Expected behavior
can migrate

Which plugin manager do you use?

plugged

I add this codes in vimrc, but cannot work, seems cannot find `ibl`

You need to run PlugUpdate to update the plugin
https://github.com/junegunn/vim-plug#commands

I have updated the plugin, but not work

❯ cat init.vim
set runtimepath^=~/.vim runtimepath+=~/.vim/after
let &packpath = &runtimepath
source ~/.vimrc

❯ git show --summary

commit 29be0919b91fb59eca9e90690d76014233392bef (HEAD -> master, tag: v3.3.7, origin/master, origin/HEAD)
Author: Lukas Reineke<EMAIL_ADDRESS>Date:   Mon Oct 30 20:56:43 2023 +0900

    fix: config overwrite

    The documentation was not consistent with the code and comments.
    I think the version that is described in the documentation is more
    useful, so this should be what the code does.

    I think it is fair to classify this as a bug, so this is not a breaking
    change. But it does behave differently now. If you want the old
    behavior, call `ibl.setup` once before.

    ~/.local/share/nvim/plugged/indent-blankline.nvim  on   master ?1                                                                                  at eks-us-west-2/wztest-live ⎈  with jinlei@debian  4.07   at 02:39:18 PM 
❯ pwd
/home/jinlei/.local/share/nvim/plugged/indent-blankline.nvim

You have the correct version checked out. Maybe Plug uses a cache somewhere? Sorry I don't know much about Plug, I can't really help you. Try opening an issue in the Plug repo.


How to swap buffer1 and buffer2

How do I move the location of the buffer

Don't have yet, this will be added in buffet.nvim, not in this plugin.


bug(dialog): mat-dialog doesn't close when a navigation happens

Expected Behavior
A dialog should close when an angular navigation happens
Actual Behavior
A dialog closes only when you hit the "back" button, but doesn't close when you navigate via routerLink or Router.navigate. It does make sense, since OverlayRef subscribes to Location's events, not Router's. But what's the point of closeOnNavigation option if it doesn't close on angular navigation? I've googled and haven't found an answer to my question, so I'm asking here. Why do you guys use Location's events? It seems like this code should look like
    if (this._config.disposeOnNavigation) {
      this._locationChanges = this._router.events
        .pipe(filter(event => event instanceof NavigationEnd))
        .subscribe(() => this.dispose());
    }

where this._router is a Router's instance. But surely you know more than I do, so I just wanna know why it is designed like so.
Environment

Angular: 11.0.0
CDK/Material: 11.0.0
Browser(s): Google Chrome Version 87.0.4280.88 (Official Build) (x86_64)
Operating System (e.g. Windows, macOS, Ubuntu): MacOS 10.15.7

The reason it's set up this way is that we don't want to add a dependency between Material and @angular/router since all apps might not necessarily be using Angular's router. If you want a generic way to handle this in your app, you can add something like the following in your main module:
constructor(router: Router, dialog: MatDialog) {
  router.events.subscribe(event => {
    if (event instanceof NavigationEnd) {
      dialog.closeAll();
    }
  });
}

@crisbeto Thanks


Upgrades the version of black used in CI

@draganagrbicllnl has been running into issues with black while trying to get formatting working in her PR #126. After looking at our CI, I realized we were using a 2.5 year old version of black with quite a few known bugs. The version of black we were using was particularly problematic since it was a beta version.
This PR upgrades black in CI to the newest version that supports our target Python version (3.9).

@pearce8 I've updated the title and description.
I was originally going to upgrade flake8 as well, but every newer version I tried had some type of internal error. This is likely to do a version mismatch between flake8 and one of the other tools it uses behind the scenes. flake8 is really just a thin wrapper around 3 or 4 other formatting tools by the same authors. The biggest problem with flake8 is that its dependencies on those other tools are hyper-specific. Sometimes when those dependencies are off even by a patch version, flake8 breaks when you try to run it.
Regarding the part of your comment asking for a "reason for the version choice", the last sentence (shown below) covers that:

This PR upgrades black in CI to the newest version that supports our target Python version (3.9).

There wasn't any other complex reason for the choice. It was just the newest version supporting the Python version that we use when running black and flake8 in CI. It is also coincidentally the newest version of black in general.

FWIW, the issues I ran into with flake8 are also a bit reason why a lot of developers are looking different tools for formatting and linting in Python (e.g., ruff).


kongingress CRD fails in openAPIV3Schema validation error

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
deploy kongingresses.configuration.konghq.com into cluster failure with error messages as following
The CustomResourceDefinition "kongingresses.configuration.konghq.com" is invalid:

spec.validation.openAPIV3Schema.properties[route].type: Required value: must not be empty for specified object fields
spec.validation.openAPIV3Schema.type: Required value: must not be empty at the root

Expected Behavior
deploy kongingresses.configuration.konghq.com successfully
Steps To Reproduce
kubectl apply -f kongingress.yaml

Kong Ingress Controller version
2.0.0-beta.2

Kubernetes version
No response
Anything else?
No response

This was fixed .


feat(workflows): build and release on tag

Insane, this will save a lot of time. Ty for contributing!


fix: (prereq-permanents): resolves issue #458

Please review as this would be just a patch fix until a "refactor the whole requirements generator" is doable.

That issue is not that important to deserve a patch. Let's ignore the problem. Thanks anyways!


Web3 user authentification

Changes
Support of web3 signature registration. Works with web3login to verify signature and save user web3 address in database.
How to test these changes?
Module tests
Related issues

@bugout-dev check

[ ] Run migration

@bugout-dev locust

@bugout-dev check

[ ] Add env var BUGOUT_WEB3_SIGNATURE_APPLICATION_HEADER


chore: release main

:robot: I have created a release beep boop
accesskit_c: 0.7.4
Dependencies

The following workspace dependencies were updated

dependencies

accesskit_windows bumped from 0.16.1 to 0.16.2

accesskit_python: 0.1.5
Dependencies

The following workspace dependencies were updated

dependencies

accesskit_windows bumped from 0.16.1 to 0.16.2

accesskit_windows: 0.16.2
0.16.2 (2024-02-25)
Bug Fixes

Bump windows-rs dependency to 0.52 (#359) (69d74f6)

accesskit_winit: 0.18.4
Dependencies

The following workspace dependencies were updated

dependencies

accesskit_windows bumped from 0.16.1 to 0.16.2

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/AccessKit/accesskit/releases/tag/accesskit_windows-v0.16.2 :sunflower:

:robot: Release is at https://github.com/AccessKit/accesskit/releases/tag/accesskit_winit-v0.18.4 :sunflower:

:robot: Release is at https://github.com/AccessKit/accesskit/releases/tag/accesskit_c-v0.7.4 :sunflower:

:robot: Release is at https://github.com/AccessKit/accesskit/releases/tag/accesskit_python-v0.1.5 :sunflower:


editing cells in a new sheet is not working as the corresponding row …

…is not cretaed.

We will merge this one as well.


Update to FontAwesome 4.5

Currently only FontAwesome 4.2 is included in the SDK.
This pull request contains the updated font itself and the updated AwesomeUtils.as.
Link to FontAwesome:
https://fortawesome.github.io/Font-Awesome/icons/
If somebody could merge it, I'll test it by downloading the latest dev build.

I've just seen that my pull request contains 12 commits instead of 2.
Seems that other commits from other users are included in this pull request cause I target the 4.15.0 branch for merging?
Should I target the develop branch instead of the 4.15.0 branch?
These are my commits:
72f4d3936d8513b5953d0db1f14edab65e866839
38c1a7565a95baebcf68f664902424d1e4d40e11


Update docker install instructions

The existing README.md uses an incorrect docker command for running the freshly built image. This just updates it to use the correct command.

Thank you for spotting this.


Sync async 2

Opening this PR just to discuss the problems I've seen so far.
1: if an error is thrown in a custom function, it's run again here: https://github.com/webdriverio/wdio-sync/compare/sync-async-2?expand=1#diff-168726dbe96b3ce427e7fedce31bb0bcR146, and throws the same error again at the natural fn failed line. This is weird from the developer's POV.
2: Before this change (https://github.com/webdriverio/wdio-sync/compare/sync-async-2?expand=1#diff-168726dbe96b3ce427e7fedce31bb0bcR220) I saw that each time a custom command was run, it was wrapped in another layer of wdioSync, so every call after the first one would return undefined.
3: It's actually not waiting as I'd expect now. Can you suggest how to fix this:
		browser.addCommand('wait1', function() {
			this.pause(1000);
			console.log('wait1 complete', Date.now() - start);
		});

		browser.addCommand('wait2', function async() {
			return new Promise((resolve) => {
				setTimeout(() => {
					console.log('wait2 complete', Date.now() - start);
					resolve();
				}, 1000);
			});
		});

		browser.addCommand('wait3', function async() {
			return new Promise((resolve) => {
				console.log('wait3 starts', Date.now() - start);
				return this.wait1()
				.then(() => {
					console.log('wait3 middle', Date.now() - start);
				})
				.then(this.wait2)
				.then(() => {
					console.log('wait3 complete', Date.now() - start);
					resolve();
				});
			});
		});

		var start = Date.now();
		browser.wait3();
		console.log('wait3 after', (Date.now() - start));

This gives:
wait3 starts 1
wait1 complete 2
wait3 middle 2
wait3 complete 3
wait3 after 5

OK, now at least the wait1 call does wait:
		browser.addCommand('wait1', function async() {
			return this.pause(1000)
			.then(() => {
				console.log('wait1 complete', Date.now() - start);
			})
		});

		browser.addCommand('wait2', function async() {
			console.log('wait2 starts', Date.now() - start);
			return new Promise((resolve) => {
				console.log('wait2 middle', Date.now() - start);
				setTimeout(() => {
					console.log('wait2 complete', Date.now() - start);
					resolve();
				}, 1000);
			});
		});

		browser.addCommand('wait3', function async() {
			return new Promise((resolve) => {
				console.log('wait3 starts', Date.now() - start);
				return this.wait1()
				.then(() => {
					console.log('wait3 middle', Date.now() - start);
				})
				.then(this.wait2)
				.then(() => {
					console.log('wait3 complete', Date.now() - start);
					resolve();
				})
			});
		});

Gives:
wait3 starts 1
wait1 complete 1003
wait3 middle 1003
wait2 starts 1003
wait2 middle 1003
wait3 complete 1004
wait3 after 1005

Before this change (https://github.com/webdriverio/wdio-sync/compare/sync-async-2?expand=1#diff-168726dbe96b3ce427e7fedce31bb0bcR220) I saw that each time a custom command was run, it was wrapped in another layer of wdioSync, so every call after the first one would return undefined.

This is basically the same implementation that is right now on master

Ok seems that it works for wdio promises but not for 3rd party/native ones (tried q as well). Will look into this.

Instead of calling the function async to force promise usage we could introduce addAsyncCommand instead. What do you think? I like the fact that you can change the behaviour by the function name and it doesn't fills up the scope with another method.

Closing this PR - not useful


[3118] Add provider filter for admins

Context
Adds a provider filter to the trainee index page for system admins only.
Changes proposed in this pull request

New provider filter populated with all providers ordered by name.
Rendered only for system admins.
New AdminFeature component which wraps the provided content in admin feature styles. Can be optionally passed extra classes/html attributes to apply to outermost div. We'll be adding more admin features soon.

Guidance to review

Sign in as a system admin, check that you can filter by provider.
Sign in as a normal user, check that the filter is not visible/you cannot filter by provider.

 [M] Add filter for providers


Removed space from file name

This caused broken link in API reference.

Hi @svick, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Rebased and merged. Thanks


Restore Python 3.2 compatibility

Current coverage is 96.38%

Merging #498 into dev will increase coverage by +0.01% as of 944eee8

@@              dev    #498   diff @@
=====================================
  Files          25      25
  Stmts        2289    2295     +6
  Branches      205     205
  Methods         0       0
=====================================
+ Hit          2206    2212     +6
  Partial        34      34
  Missed         49      49

Review entire Coverage Diff as of 944eee8

Uncovered Suggestions

+0.31% via ...y_forms/bootstrap.py#77...83
+0.18% via ...spy_forms_filters.py#102...105
+0.13% via ...crispy_forms_tags.py#66...68
See 7 more...

Powered by Codecov. Updated on successful CI builds.


[Feature] add "format" attribute to Schema objects

add support to the "format" property for Schema Object as described here:
format - See Data Type Formats for further details. While relying on JSON Schema's defined formats, the OAS offers a few additional predefined formats.

Hey @galshachaf, which additional formats? I've just checked the links you sent and found nothing about more than already supported formats. Current format property placed here - enumeration.py

the Schema class should have a "format" property, used to describe, for example, that a certain string parameter is in a "date-time" format, but it's missing:
@dataclass
class Schema:
    type: DataType
    title: Optional[str] = None
    enum: Optional[list[Any]] = None
    example: Optional[Any] = None
    description: Optional[str] = None
    default: Optional[Any] = None
    nullable: Optional[bool] = None
    read_only: Optional[bool] = None
    write_only: Optional[bool] = None
    deprecated: Optional[bool] = None
    extensions: Optional[dict] = field(default_factory=dict)

    # all_of: Any  # TODO
    # one_of: Any  # TODO
    # any_of: Any  # TODO
    # not: Any  # TODO

Yes, Schema itself doesn't contain format property since Schema is not a type, it cannot be used separately. Moreover, each DataType (which inherits the Schema class) has its own format - specification.py. So, after parsing the document you will have many schemas, but the format will be available only for supported data types (string, int, number, etc), but not for others like an array, object, boolean.


Add buildCallWithPermit variations

Close #323
Main changes:

swap with permit2 signatures
add liquidity with permit2 signatures
remove liquidity with permit signatures

Also:

approveTokens and setTokenBalances were exposed from test helpers, so we have more flexibility when setting up a test
add/remove/swap test helpers now have an extra usePermitSignatures flag
add/remove/swap tests were split into permit direct approval and permit signatures which basically replicate tests for those 2 conditions, so we make sure we’re not missing any scenario that needs to be tested
new input validator for buildCallWithPermit was added

Notes:

addLiquidity and swap when wethIsEth = true is currently not supported by SC (may be fixed in the future, but I suggest we leave it out of scope for this PR)

I agree that it would be nice to document Permit/Permit2 signatures nuances. I created #369 so we don't forget to do it. Just to be clear, the idea is to add content to readme (including eventual limitations, such as does not support wethIsEth = true) and also some examples on how to use it, right?
Ps: I'll wait on a final decision from the SC team to add details about what's supported and what's not - if I understood correctly, everything will be supported in the next testnet deployment.

I agree that it would be nice to document Permit/Permit2 signatures nuances. I created #369 so we don't forget to do it. Just to be clear, the idea is to add content to readme (including eventual limitations, such as does not support wethIsEth = true) and also some examples on how to use it, right? Ps: I'll wait on a final decision from the SC team to add details about what's supported and what's not - if I understood correctly, everything will be supported in the next testnet deployment.

Replying on the issue directly.


Problem in sftp.json missing name

Do you read the FAQ?

[x] Yes.
[ ] I am going to read now.

Describe the bug
Vscode finds one problem in the sftp.json file. Missing "name".
{
"resource": "/c:/Users/opicr/Desktop/tseportal/.vscode/sftp.json",
"owner": "generated_diagnostic_collection_name#0",
"severity": 4,
"message": "Missing property "name".",
"startLineNumber": 1,
"startColumn": 1,
"endLineNumber": 1,
"endColumn": 2
}
To Reproduce
Use the configuration from the following Gist https://gist.github.com/opicron/013e66e9f3f577266d06f06dc39bad2b.
Expected behavior
The configuration should be correct. No problems should be mentioned I think.
Desktop (please complete the following information):

OS: Windows 10 64bit
VSCode Version 1.3
Extension Version 1.8.5

Extension Logs from Startup - required
https://gist.github.com/opicron/bfab461531ec1fde60358d082897408a

This is expected. The name is required.

Sorry maybe I am stupid, but there are two names defined. If I add another name there is another error.
Do i miss something here?

In the profiles?

I have the same issue


add projects with mmdet

Thanks for the contribution. We have added your mentioned projects manually to resolve the conflict


Make it possible to unittest applications depending on dart:html on the Dart VM.

This issue was originally filed by @rbeeger

I have unittests that mock away the view of the app I'm developing. The unittests test the logic by simulating the triggering of events and by checking what values are put into ui components.
They run without a problem in the browser, but the browser isn't really necessary since no DOM operations occur. I'd like to run them on Dart VM.
Unfortunately dart:html is not available in Dart VM.
It would be nice to have some kind of empty dart:html classes on the VM that only make the app compilable on the VM and allow the creation of mocks for them.

This issue has been moved to dart-lang/test#165.


.eslintcache file should be in the node modules folder and should be ignored

.eslintcache file keeps getting generated even after trying the various available method to .gitignore this file, the files keeps generating.
As of now the only viable solutions seems to be deleting the .eslintcache file and then push the changes upstream.
Here is a reference of this issue - https://github.com/facebook/create-react-app/pull/9977
can you guys please look into this issue and confirm if this happens on your devices as well?

Adding it to the .gitignore should be sufficient since it remains untracked

Yeah I think .gitignore should handle it if the gitignore file and the cache file are in the same directory. @suraj-singh127

@yashika51 this can be closed


Print initial validation loss + final validation loss

EDIT: After Adrian already added the final validation, I greatly simplified this PR. So this only adds the initial validation. Since there was not that much enthusiasm from others, I made it optional and disabled it by default. I think it's a super useful feature to be able to know what the initial val_loss of the model is so that one can compare it to the final one and be able to tell if/how much things improved after training.

To address the "it's slow" concern, I introduced a new --skip_validation option if a user is interested in not doing any evaluation, e.g., for benchmarking training performance, or for hparam tuning and deferring the validation to a later point in time when all the checkpoints were generated.
But I'd say in 95% of the cases you want to know the training loss and validation loss during and at the end of training to gauge overfitting and to know how well the model is doing in general.
What do you think?

I greatly simplified this @carmocca . Let me know if that's ok now. It should introduce any changes to the defaults.

Is it really needed to add an argument for this? Why not just use the initial sanity check as the initial validation loss?

I originally had that but others found it was misleading because it's calculated only on 2 batches and is not reliable. I then replaced it with the full loss calculation but then this was too slow. So the new argument is a compromise so that I can enable it for me but it doesn't slow it down for everyone.


Looking for maintainers

Hi everyone,
Thanks for your continued support to my repository for the last 4 years. During this time, I have been able to expand my repository to include a lot of awesome tutorials from all over the internet thanks to your contributions. Sadly, this past year I have had a lot of personal issues that prevent me from paying attention to this project. As a result, I have basically neglected it and caused frustration for everyone. I'm slowly getting back to this and looking at the growing backlog of PRs, but by myself it would be difficult. Therefore, I am requesting your help in keeping this alive.
Please comment if you're interested in becoming one of the maintainers and I'll work out the logistics.
Thanks all <3
Tu

Oh, it's so unfortunate to hear that (including divided resource/effort by fork, possibly).
I was interested in content getting updated as time elapses, so I've watched this repo for a pretty long time from my inbox.
Therefore I understand how many requests it gets.
(Like, for example, dozens of notifications are there for just  a few past hours )

I am not an expert on every area this repo includes.
I can say I know javascript(main), dart(flutter), python, java related topics.
However, if I can and you can trust me, I'd like to help!
I can merge/argue very deliberately, hearing as much feedback/opinion from other people as possible.
Anyway, I hope your, and our(!), balance between personal schedule and opensource contribution would be getting better and better.
Thanks.

Hey @tuvtran ! I tried to contant you via email weeks ago. and got now answer, so I figured that you're not able to maintain this repo anymore. Thanks for showing interest again, and I'm sorry about your issues.
So how you do think this would work? Because the correct way doing this would include setting up an organization, and inviting people as collaborators.

I hope you don't disappear again for months. It's been a week you posted this. What's up?

Here, if you want someone to maintain I would love be of some help

Hi @enkeyz , I have transferred this repo to https://github.com/practical-tutorials/project-based-learning/. Will work on adding a core team next

Nice!
Also, you could set-up a comm channel here: https://gitter.im/ But only you can do it, as the project's owner.

I'm gonna look into it. In the meantime I have added you to the organization

As the member of the orgi, I could set up a gitter channel: https://gitter.im/practical-tutorials

@jjangga0214 @booleanhunter happy to get you guys onboarded

@tuvtran still looking for maintainers?

Hi @tuvtran
Just in case you are still looking for maintainers. I'm available to help. Sorry about your issues. I hope you are in a better place now?

@enkeyz @tuvtran Would love to help maintain this repo. I could begin cleaning through dead links and projects behind a paywall immediately! Just let me know

@tuvtran I think I am very late to the discussion. However, if you are still looking for maintainers do let me know I would love to help maintain this repo.

@tuvtran Hi , i am interested in working on this project .

I would be happy to maintain it in my spare time as well.

I am Up for the task

I would love to be part of... Let me know what is required

I would like to give some help

i would love to help on maintaining

interested in becoming one of the maintainers 😊

I am always available for help and becoming one of the maintainers

Hi. Im Hussein
Im new to the tech field. It'll be great to have a master.
في السبت، ٢ ديسمبر ٢٠٢٣, ١٠:٢٥ ص Himanshu Tekade @.***>
كتب:

I have experience of working in Java, Python, and Javascript. I would like
to help you maintaining this repo.
—
Reply to this email directly, view it on GitHub
https://github.com/practical-tutorials/project-based-learning/issues/310#issuecomment-1837085686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEIJCUH7TPDJAQMAJ4KU2GLYHLQW3AVCNFSM5CHF47D2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBTG4YDQNJWHA3A
.
You are receiving this because you are subscribed to this thread.Message
ID: <practical-tutorials/project-based-learning/issues/310/1837085686@
github.com>

I've worked as a Backend Developer and want to work as a contributor /maintainer in this project.
Thanks

Hi, i see there are many volunteers for maintaining this repo. I myself following this for a while and i must say this work is future-promising. I am not an expert on anything but interested working with .NET framework using c# mainly, also have front-end experience with JS, CSS, React, Vite, Next, also some other stuff that i do not take too much text-space with those.
I can happily want to contribute to all these effort if it's acceptable.
Thanks.

Hello i want to help you maintain to this repo if it is still available.

Hello i want to help you maintain to this repo if it is still available.

Hi @enkeyz @tuvtran I'd love to help maintain this project in my spare time. I've already contributed with PR #626 and would love to contribute more. Let me know!

Hello! If you require maintainers I have some experience. I have a lot of python, assembly, and Julia experience.

Gatsby co-founder here. Would love to be a maintainer!