Add v0.20 to nightly builds

Just enable builds for branch-0.20.

cc @quasiben @pentschev (for vis)


Update OpenSSL 1.0.2q to 1.0.2s

I need to keep OpenSSL always up to date in my project (client requirement) so if someone would update the existing windows and linux ports to the newest OpenSSL 1.0.2s packages I would be grateful :)

I believe there is talk of upgrading OpenSSL to 1.1.1 to the latest which supports TLS 1.3 (and not to upgrade to 1.0.2s). Please see Issue #4267

@ZacharLDZ Updating the port file for OpenSSL windows is fairly trivial.
If your "Client" requires you to do something as a condition that you are paid, would you kindly ask yourself who's job that really is?
Thanks.

@heydojo people want everything for free these days.

See https://github.com/microsoft/vcpkg/pull/6854

#6854 is merged. You should be able to use 1.0.2s now, @ZacharLDZ!


Pico8 file support

Not really an issue, but nice to have. Would you mind adding "p8" files to the list of supported files in the grammars/lua.cson file? There's a big community of Pico-8 (http://www.lexaloffle.com/pico-8.php) developers who I'm sure would be very grateful ;)

https://github.com/keiyakins/language-pico8


Verify leaking memory for GLib arrays

GLib has several different array implementations:

GLib.Array
GLib.PtrArray
GLib.ByteArray

Those array types are already used inside the generated code but are treated like regular arrays. As those arrays can increase in size they must be memory managed / freed.
Verify if memory is leaked or the C# side of things needs to free the array. If so handle the memory of those kinds of array, probably through some dedicated class per array type.
The api of this arrays is already generated automatically. The code must use the generated types.

One problem with GPtrArray is that the ownership semantics of the array contents seems to depend on whether an element_free_func has been set, even if we get the array from a function with (transfer full). Unfortunately we have no way of knowing if an element_free_func has been set (as far as I can see). Here is a related issue on the gir repo.


dtool item properties raises KeyError if invalid identifier is supplied

We should catch the KeyError and print out an error message that can be understood by humans.

Fixed in 775db66


fix an issue with saving collapsable states

Coverage remained the same at 94.444% when pulling a39d52a4d8da378169f29dd9a74d5a8e0624b070 on FinalAngel:develop into 6341fb84138e1c9fdbfb310811dec69c884fce0c on divio:develop.

Coverage remained the same at 94.444% when pulling a39d52a4d8da378169f29dd9a74d5a8e0624b070 on FinalAngel:develop into 6341fb84138e1c9fdbfb310811dec69c884fce0c on divio:develop.

Coverage remained the same at 94.444% when pulling a39d52a4d8da378169f29dd9a74d5a8e0624b070 on FinalAngel:develop into 6341fb84138e1c9fdbfb310811dec69c884fce0c on divio:develop.


NIFI-6998 PutAzureEventHub to utilize Batch, Properties & Partitioning key

Description of PR
Batch & Partitioning key

Asynchronous sending in limited batches
Property to determine attribute name carrying partitioning key
Maximum batch size property
Carrier object

Attributes to User Defined properties
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on both JDK 8 and JDK 11?
[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[x] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[x] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

I have corrected the checkstyle violations. I have had difficulties to run complete build of NiFi. It is crashing on nifi-properties-loader. So I have been just building the azure processors submodules.

Outside of the failing PutAzureEventHubTest, the code looks correct. I also verified that functionally this work and messages are being sent to the Partition being specified in the custom attribute.

You are right. The test was working until I applied organize imports. Unfortunately I assumed that reorganization of imports can't break anything so I have just commited aftewards. I fix it immediately.

@cima When do you think you might have that?  I'm keeping an eye on this from a release manager point of view for NiFi 1.11.  No rush just saying if it is resolved I'd be happy to grab it.

All right, after build without tests downloaded the entire internet and run out of diskspace (40 GB approx.)I was able to at least run the azure processors project to verify it is compiling and passing test. However I am not going to test the whole NiFi project because there are tests NiFiPropertiesLoaderGroovyTest.groovy that crashes on windows just because they assume some capability of filesystem. But that is different story. I have pusshed hopefully the last change that should make CI/CD to pass.


Archive class channels in some manner and allow users to access them at the end of a semester

It would be a nice QoL feature to archive the class channels at the end of the semester. Some of the content in the class channels can be very useful, or just really cool. At the end of the semester, all of the content posted is lost.
It would be nice to have a way to view the channel at the end of the semester in some fashion. While warnings are given that the channel will be deleted, not all users have the ability to go and save the stuff they want. It is also possible for a user to forget to grab something or think it isn't worth it and regret not having it after the channels are deleted.
Possible Implementation steps (modified from @Armintron):

Log all messages and/or pins in a given class channel to a database
At archival time, take a snapshot of the users who are allowed to access the channel, this will be the list of allowed users
Create some method of retrieving the stored messages and/or pins for a given channel, and check that the accessor is on the list of allowed users

https://github.com/Tyrrrz/DiscordChatExporter this repo may be of use for this

Can you add suggested implementation steps to summarise discussion in discord

Before channels are deleted, the current users in a class channel are added to a list of people that can access the archives for that channel.
Using Lion, the channels and their pins are stored in the database
An endpoint is created that can (given credentials that are allowed to access the request archieve) fetch an archive

Can you add suggested implementation steps to summarise discussion in discord

Closing this because I personally dont want to dedicate time to do this and I dont think there will be much traction on this going forward.


Make it work with relative URLs too

When testing locally, the link URL always points to the absolute URL. It would be nice if it could work too using a relative URL.

Hi @jorgesumle , The current version is to add the relative, such as  this demo.

Sorry, I don't understand, what do you mean? The link is broken (Not found).

@jorgesumle  sorry.
http://www.fangpeishi.com/

Exactly, that's what is missing, so when you are developing locally, you can execute make html and you get relative URLs and when you want to upload the website you execute make publish and get absolute URLs.


session menu doesnt have ctrl-enter shortcut

Ctrl+Enter should be added.
It’s in the docs, but not in the product (Ctrl+Enter):

Won't fix. This is not possible given the way VS works today.


Unreasonable display of imagePullSecrets

Setup

Rancher version: v2.6.7
Browser type & version: Chrome os Mac 104.0.5112.101 x86_64

Describe the bug

Kubernetes supports specifying container image registry keys on a Pod. imagePullSecrets must all be in the same namespace as the Pod. The referenced Secrets must be of type kubernetes.io/dockercfg or kubernetes.io/dockerconfigjson.
refer to: https://kubernetes.io/docs/concepts/containers/images/#specifying-imagepullsecrets-on-a-pod
However, when creating a workload, all the secrets are displayed in the dashboard.
To Reproduce

When creating a workload, switch to the kube-system namespace, and it's easy to see the problem. Because this namespace will contain a lot of secrets.
In addition, when trying to filter secrets, this metadata.name will also cause troubles and trigger js errors.

Result
All secrets show in the form.
Expected Result

Only shows kubernetes.io/dockercfg or kubernetes.io/dockerconfigjson secrets.
Screenshots

Additional context

retested on rancher - 2.6 head 68215bc
retest : failed
see video recording:
when creating a deployment i cannot get the 'pull secrets' section dropdown to display any data even when the selected namespace has secrets and I am logged in as admin.

see video for more examples.
https://zube.io/files/rancher/e7c34377c0348bc53f0f91bba1cfc35b-retest-6725.mp4

@Shavindra thank you for the correction!
retest- passed
note: I was unable to prove dockercfg files display. based off of this documentation I believe users can no longer create dockercfg type secrets, when I create one via kube command the system recognizes it as a dockerconfigjson. logging bug to attach to this work item shortly.
testing video
https://user-images.githubusercontent.com/89030218/190681062-4ac4f534-da4c-4a47-a2a7-e74bbea843ff.mp4


[BUG] if DnD column, then edit label, column is moved to the end of table

even if the state of react-table columnOrder is correct, when you edit a label if you previously DnD the column, is displaced to the end

I found the problem. It needs to update colOrder id


Force User to change password on first login

ESA support this? how i do it?

No, this is out of scope for ESA.


deploy: false config not respected

Clone SafeMarket and check out 7a7012827dc4bb686ba75284a48fe4358c8b7901
https://github.com/SafeMarket/dapp/tree/7a7012827dc4bb686ba75284a48fe4358c8b7901
then run embark blockchain and embark run. Multiple contracts that shouldn't deploy will deploy.

something is wrong with the tabs in the config file, try replacing it with 2 spaces intead

That fixed it. Thanks!


PIL Packaging Bug

With a vanilla Anaconda python 3.6 distribution on Windows 7, the following install command clobbers something in the PIL module.
conda install -c conda-forge cartopy

When trying to import cartopy.io.img_tiles, an import error then occurs.
import cartopy.io.img_tiles
Traceback (most recent call last):

  File "<ipython-input-106-feb3ede8296d>", line 1, in <module>
    import cartopy.io.img_tiles

  File "C:\Anaconda3\lib\site-packages\cartopy\io\img_tiles.py", line 32, in <module>
    from PIL import Image

  File "C:\Anaconda3\lib\site-packages\PIL\Image.py", line 56, in <module>
    from . import _imaging as core

ImportError: DLL load failed: The specified module could not be found.

This can be fixed with conda upgrade -c conda-forge pillow or conda config --add channels conda-forge, depending if you want to add a channel or not.
I suspect cartopy is expecting pillow 4.3.0 (conda-forge), not 4.2.1 (standard conda)

Indeed cartopy is installing pillow. I don't think you can use it with PIL. If you need PIL for other projects you'll need to create separate envs.


Designer doesn't work for nested dynamic UI

Describe the Bug with repro steps
We are trying to generate a UI like:

We have a SAP manifest which looks like below:

Where we have a dynamic reference on the parameter bodyType, which can be [XML, JSON, Dynamic]
Based on particular selection (i.e bodyType as Dynamic) a call is being made to:
POST https://management.azure.com/subscriptions/f34b22a3-2202-4fb1-b040-1332bd928c84/resourceGroups/siddharth-imp/providers/Microsoft.Web/sites/sid-sap-testbox/hostruntime/runtime/webhooks/workflow/api/management/operationGroups/sap/operations/getCallRfcInputVarientSchemaSwagger/dynamicInvoke?api-version=2018-11-01

...<With bodyType as Dynamic in payload>...

Which responds with another dynamic parameter bodyType & rfcName:

Now when we enter a valid entry for rfcName a call to dynamicInvoke is not being made to fetch further set of Dynamic UI parameters.
What type of Logic App Is this happening in?
Standard (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Edge
Additional context
No response
AB#25863805

Something is not correct in your screenshot, firstly when you put dynamic properties in a schema that object schema becomes a dynamic property so anything defined inside properties of that schema will not be shown in the UI.
Please state your ask, what do you actually want to do in the operation?

So we have a UI like below:

And it makes two dynamic UI calls,

First is when we select bodyType as Dynamic. We see new parameters appear, ie. rfcName and groupRfcFilter.  Below is the response form this dynamic call:

When a rfcName is selected, it should ideally call another dynamicInvoke to retrieve new parameters depending on rfcName.

Is this a scenario that is not currently supported?

discussion can continue in #3709 and in person


Remove lockfree-stack implementation that is no longer used

Dead code as of #10897

No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Known failures: #11383 #11223


Issue on page /notebooks/13-t_F_tests.html

"Let’s A more-than-two-factors example would be if you were, say, comparing the means of more than two types of insect groups – below, we compared means of two groups (dragonflies and damselflies); if we added stoneflies to this comparison, the t-test would not be the way to go as we would be making three pairwise comparisons (n = 3, p = 3).
OK, assuming you interested in making just one paiwise comparison, let’s learn about t
tests."
Should it be "let's say" in the first sentence, as you don't refer to any "A" further?
Also, should be "OK, assuming you're interested" in the last sentence

fixed, thanks!


rethink specifying initial values as DeepPartial

this causes some problem in edge causes which can lead to returned initial values being initialised to illegal type (related to objects inside arrays)

Hi @dgreene1! I am happy that you find this project interesting :)
Sorry if the issue is not described properly - We didn't expect other people getting involved at this stage :D
this sandbox illustrates the issue:
https://codesandbox.io/s/issue-25-3jf0i?file=/src/example.tsx
relevant piece of code:
https://github.com/VirtusLab/formts/blob/master/src/core/helpers/create-initial-values.ts
relevant spec: (it probably should have a skipped failing test with a TODO to indicate the issue exists rather than suggesting all is good 😅)
https://github.com/VirtusLab/formts/blob/master/src/core/helpers/create-initial-values.spec.ts#L98
context:
We can generate initial values automatically based on defaults for each field decoder, so rather than requiring consumer to pass all initial values, we require DeepPartial and do deepMerge with defaults.
It falls apart when we merge empty array (default initial value for array fields) with array containing partial objects provided by the user - state contains undefined where it expects e.g. string
We need to either improve algorithm for creating initial values or use better types that reflect the implementation.
Easiest fix would be to just use Partial instead of DeepPartial as nested objects in form values are rare anyway.
There's probably a better way of doing this though - feel free to submit a PR if you have an idea :)

fixed by adjusting DeepPartial type to ignore arrays


Feature : Dark mode

Issue Template
can add new feature switch to dark mode

@student-aditipaliwal
You can start working on this issue.

ok sir  i will done it by tommorow

@AnupK1234  can u assign this issue to me... I am interested in contributing to this project under swoc

@gayathrik23
This issue is currently assigned. Have a look on other issue. If this issue gets unassigned then I will assign it to you.

@student-aditipaliwal
Are you done with this issue ?

I am sorry ,but I am not able to complete this task  , thanku for assigning
me this issue , please assign this issue to any other .
On Tue, 30 Jan, 2024, 11:53 PM Anup Khismatrao, @.***>
wrote:

@student-aditipaliwal https://github.com/student-aditipaliwal
Are you done with this issue ?
—
Reply to this email directly, view it on GitHub
https://github.com/AnupK1234/MealMatters/issues/62#issuecomment-1917638237,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDDHYL4EAOCVP63GITVGYALYRE3CRAVCNFSM6AAAAABCB53TOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJXGYZTQMRTG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@gayathrik23
Would you like to work on this issue ?

@gayathrik23 Would you like to work on this issue ?

yes

@gayathrik23 Would you like to work on this issue ?

yes

Ok. You can start working on this.

@gayathrik23
Are you done working on this?

@gayathrik23
Any updates ?


Deprecate this role in favor of the new geerlingguy.mac collection

I have consolidated my macOS-related roles into the new geerlingguy.mac collection.
This collection contains the exact same copy of this role at version 3.2.0 in its 1.1.1 release.
All future work will go into the collection role.
Migration steps:

Make sure you're running at least Ansible 2.9 (Ansible Core 2.10 or Ansible 3.0.0 or later recommended).
Install the collection: ansible-galaxy collection install geerlingguy.mac (you can also specify collections: inside your requirements.yml file separately from roles:).
Change all references to the geerlingguy.homebrew role to geerlingguy.mac.homebrew.

If you have any migration issues, please consider opening an issue on the geerlingguy.mac collection repository, a this repository will be archived, and the role on Galaxy will be marked as 'deprecated'.
TODO:

[x] Add a deprecation note to the README of this role and release a new tag so it appears on Galaxy.
[x] Add a deprecation note to the role's repository description.
[ ] Archive the repository.

Archiving next!


Extracting multiple items

Feature Request
Is your feature request related to a problem? Please describe.
I couldn't find a way to extract multiple items by LLMParser. My usecase is that I want to extract ingredients list from HTML of recipe site.
I defined the fields as follows but only one ingredient name could be obtained.
const parser = new LLMParser({
  model: 'gpt-3.5-turbo',
  fields: [
    {
      name: 'name',
      description: 'The name of an ingredient.',
      type: 'string',
    },
    {
      name: 'quantity',
      description: 'The quantity of the ingredient.',
      type: 'string',
    },
  ],
});

Describe the solution you'd like

Describe alternatives you've considered

Are you willing to resolve this issue by submitting a Pull Request?

[ ] Yes, I have the time, and I know how to start.
[ ] Yes, I have the time, but I don't know how to start. I would need guidance.
[ ] No, I don't have the time, although I believe I could do it if I had the time...
[x] No, I don't have the time and I wouldn't even know how to start.

Hi! There is no official way yet. I've added it to the roadmap.
The workaround today is to put in the description "a list of all ingredients separated by a comma" and then just split the string by comma.
const ingredients = "chicken, salt, water"; const list_of_ingredients = ingredients.split(",");


Custom serializer doesn't work with MobileOptionsBuilder

3.5.1
java.lang.NullPointerException: Cannot invoke "it.auties.whatsapp.controller.ControllerSerializer.linkMetadata(it.auties.whatsapp.controller.Controller)" because "this.serializer" is null
	at it.auties.whatsapp.controller.Controller.phoneNumber(Controller.java:105)
	at it.auties.whatsapp.api.MobileRegistrationBuilder$Unregistered.register(MobileRegistrationBuilder.java:125)

It works properly with web version. Also mobile version works if serializer is not set.

Fixed on the master branch


Protocol handlers for common app views

landing page (initial view)
cluster view
cluster settings
extensions
add cluster

landing page (initial view)
33.000___****___]()

cluster view

cluster settings

extensions

add cluster

landing page (initial view)
23.000[]()

cluster view

cluster settings

extensions

add cluster


Allow any prefix as trigger

use Trigger to set the prefix for triggering derek, currently set
to "Derek ", but "//" is useful as well.
Use Const instead of Contant for constants and clean up import lines
Signed-off-by: Miek Gieben<EMAIL_ADDRESS>

I think the issue is the following "Merge branch 'master' into my-derek"
How about stashing your changes and applying them to a fresh master branch?
Alternatively you can reset backwards 2 commits and then apply your changes again.

GitHub: This branch cannot be rebased due to conflicts

Alex

Yeah, it's the merge conflict commit.

Sorry, this has already taken too much of my time and yours. Let me close this.


Unexpected global_steps/ accelerator behaviour

🐛 Bug
I observed some, imo,  unexpected behaviour when experimenting with different combinations of gpus/ num_processes and accelerator trainer args.
To Reproduce
I use the following bug_report_model.py. I adapted the one from pl to take accelerator, gpus, and num_processes args, only use fake train_data, and train for ten epochs.

When running it with the default args (accelerator=None, gpus=0, num_processes=1), trainer.global_step=640 as expected:

.venv ❯ python -W ignore bug_report_model.py
GPU available: True, used: False
TPU available: None, using: 0 TPU cores
Epoch 9: 100%|████████████████████████████████████████████| 64/64 [00:00<00:00, 969.92it/s, loss=718]
trainer.accelerator_backend=<pytorch_lightning.accelerators.cpu_accelerator.CPUAccelerator object at 0x7f8e39ea7dc0>
trainer.gpus=0
trainer.num_processes=1
trainer.global_step=640

When running it with  --num_processes 2 --accelerator ddp_cpu, trainer.global_step=0 which is not expected:

.venv ❯ python -W ignore bug_report_model.py --num_processes 2 --accelerator ddp_cpu
GPU available: True, used: False
TPU available: None, using: 0 TPU cores
initializing ddp: GLOBAL_RANK: 1, MEMBER: 2/2
initializing ddp: GLOBAL_RANK: 0, MEMBER: 1/2
Epoch 9: 100%|███████████████████████████████████████████| 32/32 [00:00<00:00, 305.89it/s, loss=45.2]
trainer.accelerator_backend=<pytorch_lightning.accelerators.ddp_cpu_spawn_accelerator.DDPCPUSpawnAccelerator object at 0x7f325f6e6df0>
trainer.gpus=0
trainer.num_processes=2
trainer.global_step=0

When running it with --gpus 2, trainer.accelerator_backend is DDPSpawnAccelerator and trainer.global_step=0 which is not expected:

.venv ❯ python -W ignore bug_report_model.py --gpus 2
GPU available: True, used: True
TPU available: None, using: 0 TPU cores
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1,2,3]
initializing ddp: GLOBAL_RANK: 0, MEMBER: 1/2
initializing ddp: GLOBAL_RANK: 1, MEMBER: 2/2
Epoch 9: 100%|██████████████████████████████████████████| 32/32 [00:00<00:00, 284.00it/s, loss=0.159]
trainer.accelerator_backend=<pytorch_lightning.accelerators.ddp_spawn_accelerator.DDPSpawnAccelerator object at 0x7fa00deafe50>
trainer.gpus=2
trainer.num_processes=2
trainer.global_step=0

When running it with  --gpus 2 --accelerator ddp, trainer.global_step=320 which makes sense due to ddp, but changes logging behaviour and max_steps calculations:

.venv ❯ python -W ignore bug_report_model.py --gpus 2 --accelerator ddp
GPU available: True, used: True
TPU available: None, using: 0 TPU cores
LOCAL_RANK: 0 - CUDA_VISIBLE_DEVICES: [0,1,2,3]
LOCAL_RANK: 1 - CUDA_VISIBLE_DEVICES: [0,1,2,3]
initializing ddp: GLOBAL_RANK: 1, MEMBER: 2/2
initializing ddp: GLOBAL_RANK: 0, MEMBER: 1/2
Epoch 9: 100%|███████████████████████████████████████████| 32/32 [00:00<00:00, 233.42it/s, loss=0.19]
trainer.accelerator_backend=<pytorch_lightning.accelerators.ddp_accelerator.DDPAccelerator object at 0x7f1b88743370>
trainer.gpus='0,1'
trainer.num_processes=2
trainer.global_step=320
trainer.accelerator_backend=<pytorch_lightning.accelerators.ddp_accelerator.DDPAccelerator object at 0x7fb45ae2ee80>
trainer.gpus=2
trainer.num_processes=2
trainer.global_step=320

Expected behavior
Regarding 2.:  trainer.global_step should be 320 or 640?
Regarding 3.:  trainer.accelerator_backend should be DDPAccelerator   as this is the recommended one for multi GPU training?
Regarding 4.: Would it be possible and/ or make sense to have total_steps that multplies global_steps and gpus that is used for max_steps and logging?
Environment
* CUDA:
        - GPU:
                - GeForce RTX 2080 Ti
                - GeForce RTX 2080 Ti
                - GeForce RTX 2080 Ti
                - GeForce RTX 2080 Ti
        - available:         True
        - version:           10.2
* Packages:
        - numpy:             1.19.4
        - pyTorch_debug:     False
        - pyTorch_version:   1.7.1
        - pytorch-lightning: 1.1.2
        - tqdm:              4.55.0
* System:
        - OS:                Linux
        - architecture:
                - 64bit
                - ELF
        - processor:         x86_64
        - python:            3.8.6
        - version:           #51~18.04.1-Ubuntu SMP Sat Sep 5 14:35:50 UTC 2020

Hi, these are all expected behaviour. In all modes that spawn children processes (ddp_spawn, ddp_cpu (spawn)), the main process does not do any training, therefore trainer.global_step = 0. If you want to access global_step, you need to do it in the code that runs in training, i.e. in training_step or similar, then you will see it has a meaningful value.
And in ddp (no spawn) max_steps, batch_size etc. are settings per gpu.

Hi, these are all expected behaviour. In all modes that spawn children processes (ddp_spawn, ddp_cpu (spawn)), the main process does not do any training, therefore trainer.global_step = 0. If you want to access global_step, you need to do it in the code that runs in training, i.e. in training_step or similar, then you will see it has a meaningful value.
And in ddp (no spawn) max_steps, batch_size etc. are settings per gpu.

Hey @chris-clem,
As this behaviour is expected and @awaelchli  provided a great answer, I will be closing this issue.
Best regards,
T.C

Hey @chris-clem,
As this behaviour is expected and @awaelchli  provided a great answer, I will be closing this issue.
Best regards,
T.C


/validate is not working

Template,

What kind of issue is it? bug
In which Minecraft Server Software did it occur, Spigot
In which version did it occur? 1.8.8 Minecraft Version. Nameless Plugin for v2 beta-29
How would you describe the issue?
When a player register in the website, it doesn't ask them to do /validate (numbers) in the server.
Do you have (relevant) resources about this issue (Logs, pictures, etc.)?
no

PLEASE UPLOAD IMAGES TO IMGUR OR ANY OTHER IMAGE UPLOADER WEBSITE!
PLEASE UPLOAD LOGS TO HASTEBIN.COM!
Example
What kind of issue is it? (question, bug, suggestion, etc.)
Bug
In which Minecraft Server Software did this occur?
Spigot
In which version did it occur?
Nameless Plugin for v2 beta-29
How would you describe the issue?
Command /validate not working
Do you have (relevant) resources about this issue (Logs, pictures, etc.)?
No

More information please. Does the command throw an error when it is run?

So, when a player register in the website, it never asks to do /validate (numbers) in game. It just says the account has been validated.

That's an issue on the website end. Either it's a bug or the option to skip verification is still enabled. The best way to get help with this is through discord or the discussion section on spigot

Tried discord, all helpers ignored me, welp.....k then

Been 3 days asking for help in discord smh

https://github.com/NamelessMC/Nameless/issues/1577


Fine-Grained Control Over TOCO Quantization

Our (Syntiant Corp’s) neural network inference chips use quantized weights and biases in order to minimize storage and energy consumption. The new Tensorflow experimental quantization feature tf.contrib.quantize.experimental_create_training_graph supports quantizing weights to between 2 and n bits, but the tf.contrib.lite.toco_convert tool currently only supports 8 bit quantization. As a result, we have to internally fork the TFLite pipeline before generating the Flatbuffer.
Feature request: Update TOCO to support arbitrary (i.e., 2 to n bit) signed fixed point quantization of weights and biases for both symmetric and asymmetric quantization. Our desired solution would process the quantization specified at the op or Keras layer level and not involve quantization specification within the TOCO tool API.

We are thinking about how to handle things things like this, but a solution is not currently available. It's likely this will be quite difficult. @raghuraman-k , @suharshs , could you please take a look.

This will likely involve needing to introduce new inference types to TOCO/TFLite. It will be a non-trivial change, as we will likely need to also support mixed precision model to make this not be very detrimental to accuracy. This is something we will try to determine a priority for. Thanks!

Nagging Assignees @suharshs, @raghuraman-k: It has been 17 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @suharshs, @raghuraman-k: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @suharshs, @raghuraman-k: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @suharshs, @raghuraman-k: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @suharshs, @raghuraman-k: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @suharshs: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @suharshs: It has been 89 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @suharshs: It has been 104 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

We are working to make things more configurable in new converter iterations, will close this issue since its a bit stale.


New Workspace Preview does not give access to Dataset when App is published

When app is published from new workspace (preview) then access to dataset that are related with published reports is not granted automatically. I had to add access to dataset manually.

@lukaszjakubczyk Thanks for your comment. We’ve forwarded your feedback to the writer, who should respond to this thread.

Hi, @lukaszjakubczyk -- again thanks for your comment! Looks like this is product feedback, rather than documentation feedback – you can submit product feedback or feature ideas by going to https://ideas.powerbi.com. You can also engage with the Power BI engineering team by being active in the Power BI Community, found at https://community.powerbi.com.


NIFI-4426 - Replaced Java7 jBCrypt implementation which was made for …

…Java7 backwards compatibility. It now uses a normal maven import to provide jBCrypt.
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Ran contrib-check and all tests pass. +1, merging.


Rename get_igv_omit_override(), refactor use

Some housekeeping I have to do as a part of #1923.
The naming and usage of this helper makes no sense. We use the AppSettingAPI to get the setting and its default value is defined in the Django settings. We will return a value regardless, so there is no need to query for an "override" separately. This should be simplified.

Done.


Mobile home page

-Resize/change margins for "Welcome to the Bitcoin Poker Tour" so it doesn't stretch so far down the screen.

Fixed in PR https://github.com/bitcoinbrisbane/bitcoinpokertour/pull/37

@TexasHODL  Something wrong?

The welcome message needs to be stretched out a bit. Its too slim, there is so much empty space on the sides of the screen

Please leave here an image explaining better what do you want

With the telegram link as well. Looks like the telegram link has dissapeared from the home page to

I think the telegram link under coming soon was better

ok leave it at that for now then


Todo: Remove freestanding workarounds to use C++23 standard headers for freestanding

Since C++23, more headers and features (like std::addressof) are available now. Therefore, we need to remove all the workaround.
GCC provides extensions like making <array> and <span> be also in the freestanding C++ toolchain. However, we will still avoid them unless the C++ standard says something.
This will trim the library and make it more "standardized."

#273 changes around freestanding part will be made on branch 'experimental', and will not be merged before it's clear that no bugs are triggered by these changes.

all benchmarks, tests and examples have passed. 'experimental' branch merged

Done


Additional conditions to setCenter & setZoom.

In the case when you need to change mapState to use bounds instead of center / zoom, we should not call setCenter / setZoom, because they will have an undefined argument (this causes an error). In the reverse order everything works fine.

Nice catch! Thanks for the contribution


Refactor tests & fix some declarations

This should significantly improve the UX when designing tests, especially since we will add more configuration options for the upcoming byzantine tests.

Thank you for the review. Updated.

This was approved, merging.


Q: In a multi process deployment, can I adjust the number of replias?

I'm using it according to the official documentation. I have a question because I couldn't find a part where I could set up a replica.
P.S. It would be nice to have a Discussion for Q&A...

Hi @myyrakle. There is no easy way to adjust the number of replicas. Here's the code that defines the number of replicas:
https://github.com/ServiceWeaver/weaver/blob/2c48ca8f415cb9dba82ce28b854957512789c0e2/internal/tool/multi/deployer.go#L48
If you want, you can edit the source code and recompile the weaver tool.
We could allow users to configure the number of replicas. If you don't mind sharing, what were you trying to accomplish by changing the number of replicas?

P.S. It would be nice to have a Discussion for Q&A...

I've never used GitHub Discussions before. We have a Google Group for Q&A right now, but maybe Discussions are better? What has your experience been with Discussions vs Google Groups?

I didn't need that feature right away.
However, if the function to adjust the number of processes is provided, I think that users will be able to make the most of their resources within the limit.
What do you think of handling these values in weaver.toml?

Personally, the experience of github discussion was simple and good.
And it's also very visible.
In addition, it was nice to be able to write an article right away without additional login.

What do you think of handling these values in weaver.toml?

What do you think of handling these values in weaver.toml?

We've been trying to keep config files as minimal as possible, but if we find there's a lot of desire to specify the number of replicas for a multiprocess deployment, we can definitely add it!

Personally, the experience of github discussion was simple and good.

Awesome, thanks so much for sharing. I'll chat with the team about enabling Discussions then. It sounds nice.


Implement the ceph osd destroy feature in ceph-deploy.

Feature:
    1. handle osd already out
    2. handle osd did not blong any hostname
    3. handle osd is not in acting set

Usage: ceph-deploy osd destroy <hostname> --osd-id <osd_id>

I have tested in CentOS6/7, Ubuntu12.04/14.04 and it work normally.
This feature can help to remove osd quickly. It maybe is helpful for
newbie who uses ceph.

If you have any questions, feel free to let me know.
Thanks!

Signed-off-by: Vicente Cheng<EMAIL_ADDRESS>

@Vicente-Cheng I wanted to point you to http://tracker.ceph.com/issues/7454, which has some details about how we ultimately want to roll this capability into Ceph.
A lot of the logic contained in this PR can be in ceph-disk instead.  ceph-deploy uses ceph-disk where it can, so it doesn't have to concern itself with too many low-level disk details.  We would like to remove OSDs via ceph-disk as well, then ultimately have ceph-deploy use the relevant ceph-disk command instead.
This also makes a lot of the logic required to remove an OSD re-usable by other tools besides ceph-deploy.

@trhoden
OK, I will study this issue and its discussion (w/ related issues) and think how we implement this feature via ceph-disk.

@trhoden
I remove the manual "ceph osd tree" called component.
To complete my goal (get the osd tree) by osd_tree() function.
I plan to give the "ceph-deploy osd" two subcommand like our discussion "deactivate" and "destroy".
The "destroy" will have dependency on "deactivate". how did you think?
As you work on "deactivate", I am going to work on ceph-disk "destroy".
When these two subcommand complete, I will modify the ceph-deploy to complete these two features.

@Vicente-Cheng closing because I'm assuming your latest contribution to ceph-disk address the same problem. Feel free to re-open if I'm mistaken.

@dachary Sure, I will complete the ceph-disk feature first.
Then ceph-deploy could use the ceph-disk related feature to complete it.


[Rebranch] Resolve Immediate Conflicts

Resolves a main -> rebranch merge conflict in Immediate.

@swift-ci please test

/home/build-user/swift/lib/Immediate/SwiftMaterializationUnit.cpp:85:11: error: no viable conversion from 'llvm::JITTargetAddress' (aka 'unsigned long') to 'llvm::orc::ExecutorAddr'
      ES, llvm::pointerToJITTargetAddress(&handleLazyCompilationFailure));
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

/home/build-user/swift/lib/Immediate/SwiftMaterializationUnit.cpp:85:11: error: no viable conversion from 'llvm::JITTargetAddress' (aka 'unsigned long') to 'llvm::orc::ExecutorAddr'
ES, llvm::pointerToJITTargetAddress(&handleLazyCompilationFailure));
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Ah yup let me update that as well

@swift-ci please test

Thanks @zachary0kent!


[Front-end Project] Salon Site

Salon Site Project

Close #3496

Checklist:

[✅] I have mentioned the issue number in my Pull Request.
[✅] I have commented my code, particularly in hard-to-understand areas

@MohitGupta121  @TusharKesarwani  @pranjay-poddar  Kindly, review this PR and merge the file please

@MohitGupta121 @TusharKesarwani kindly, review this please


[SPARK-33306][SQL][FOLLOWUP] Group DateType and TimestampType together in needsTimeZone()

What changes were proposed in this pull request?
In the PR, I propose to group DateType and TimestampType together in checking time zone needs in the Cast.needsTimeZone() method.
Why are the changes needed?
To improve code maintainability.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By the existing test "SPARK-33306: Timezone is needed when cast Date to String".

Test build #130522 has started for PR 30223 at commit f02b64d.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35122/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35122/

Merged build finished. Test PASSed.

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

Test build #130522 has finished for PR 30223 at commit f02b64d.

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

Merged build finished. Test PASSed.

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

thanks, merging to master!

Hi, @cloud-fan and @MaxGekk .
This is not merged to master.

@dongjoon-hyun Thanks! My merge script didn't finish successfully and I missed that :(


[BUG] scap-driver: fails to compile with kernel 6.3

Trying to rebuild scap-driver against 6.3:
CC [M]  /tmp/portage/dev-util/scap-driver-0.29.3-r1/work/libs-e5c53d648f3c4694385bbe488e7d47eaa36c229a_build/driver/src/ppm_fillers.o
In file included from ./include/linux/sched/signal.h:10,
                 from ./include/linux/rcuwait.h:6,
                 from ./include/linux/percpu-rwsem.h:7,
                 from ./include/linux/fs.h:33,
                 from ./include/linux/compat.h:17,
                 from /tmp/portage/dev-util/scap-driver-0.29.3-r1/work/libs-e5c53d648f3c4694385bbe488e7d47eaa36c229a_build/driver/src/ppm_fillers.c:12:
/tmp/portage/dev-util/scap-driver-0.29.3-r1/work/libs-e5c53d648f3c4694385bbe488e7d47eaa36c229a_build/driver/src/ppm_fillers.c: In function 'f_proc_startupdate':
./include/linux/cred.h:378:23: error: passing argument 1 of 'inode_permission' from incompatible pointer type [-Werror=incompatible-pointer-types]
  378 |         current_cred()->xxx;                    \
      |                       ^
      |                       |
      |                       struct user_namespace *
./include/linux/cred.h:395:34: note: in expansion of macro 'current_cred_xxx'
  395 | #define current_user_ns()       (current_cred_xxx(user_ns))
      |                                  ^~~~~~~~~~~~~~~~
/tmp/portage/dev-util/scap-driver-0.29.3-r1/work/libs-e5c53d648f3c4694385bbe488e7d47eaa36c229a_build/driver/src/ppm_fillers.c:1250:67: note: in expansion of macro 'current_user_ns'
 1250 |                                 exe_writable |= (inode_permission(current_user_ns(), file_inode(exe_file), MAY_WRITE) == 0);
      |                                                                   ^~~~~~~~~~~~~~~
./include/linux/fs.h:2476:22: note: expected 'struct mnt_idmap *' but argument is of type 'struct user_namespace *'
 2476 | int inode_permission(struct mnt_idmap *, struct inode *, int);
      |                      ^~~~~~~~~~~~~~~~~~
./include/linux/cred.h:378:23: error: passing argument 1 of 'inode_owner_or_capable' from incompatible pointer type [-Werror=incompatible-pointer-types]
  378 |         current_cred()->xxx;                    \
      |                       ^
      |                       |
      |                       struct user_namespace *
./include/linux/cred.h:395:34: note: in expansion of macro 'current_cred_xxx'
  395 | #define current_user_ns()       (current_cred_xxx(user_ns))
      |                                  ^~~~~~~~~~~~~~~~
/tmp/portage/dev-util/scap-driver-0.29.3-r1/work/libs-e5c53d648f3c4694385bbe488e7d47eaa36c229a_build/driver/src/ppm_fillers.c:1251:72: note: in expansion of macro 'current_user_ns'
 1251 |                                 exe_writable |= inode_owner_or_capable(current_user_ns(), file_inode(exe_file));
      |                                                                        ^~~~~~~~~~~~~~~
./include/linux/fs.h:1607:47: note: expected 'struct mnt_idmap *' but argument is of type 'struct user_namespace *'
 1607 | bool inode_owner_or_capable(struct mnt_idmap *idmap,
      |                             ~~~~~~~~~~~~~~~~~~^~~~~
cc1: some warnings being treated as errors

I have not investigated further yet.

Hi! Thanks for opening this issue!
Our ci actually is lagging behind because it ensures that we are able to build latest archlinux kernel, but arch is still at 6.2.12 (at least this morning).
We are gonna look at this and fix it asap!

It looks like this was caused by abf08576 or related followups.

/milestone 5.0.0+driver

Ok arch upgraded the linux kernel package ;) And our daily CI job started failing indeed: https://github.com/falcosecurity/libs/actions/workflows/latest-kernel.yml
I am going to work on this asap!

A PR with the fixes has been opened :)

A PR with the fixes has been opened, see #1071 :)

All that macro mess and bitshifted typecasts are insane. Thanks SO much for this!

I managed to backport the relevant bit to the older driver we have in Gentoo and it now builds & works correctly on 6.3.1. :raised_hands:


Make middle-click do copy-and-paste in integrated terminal

Maybe this is a dup of #26902.
The traditional mouse button for copy-and-then-paste on Linux is the middle button, while it is the right button on Windows. However, the notable MSYS2 terminal emulator, mintty, which is also utilized by the Git Bash, defaults to the linux configuration (but it's configurable). As a dual-platform user, having to be aware of this divergence is really a pain. Thus, I suggest we make this configurable, as I am more accustomed to the middle button.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Tested on MATE terminal, Linux Mint 21.2 'Victoria' x86_64:

Uncheck Edit -> Profile Preferences -> Copy selected text into clipboard.
Select random text "text one" with the mouse, then press Ctrl-Insert to copy it. Selected text is now copied into clipboard.
Select another block of text "text two" with the mouse.
Press Shift-Insert. "text one" is inserted, so pressing Shift-Insert pastes the clipboard.
Press the mouse middle button. "text two" is inserted, so pressing middle button pasts the selection.
Press Shift-Insert again. "text one" is still inserted, so pressing middle button just duplicates the selection, having no effect on the clipboard.

This is acceptable. The MSYS2 mintty, by configuration, assigns middle button the Paste command. So a middle click pastes the system clipboard; it's no difference from Ctrl-Shift-V or Shift-Insert.

I think I have to disable this on Linux (and consequently, on Windows too) for now, as the touchpad is very easily mishandled, causing vscode to paste text not only in the terminal, but in the editor area too.

Any update on this? I work with vscode and "native" Linux shells side-by-side on a Windows workstation. This different behavior on what's essentially two windows for me is driving me crazy.

Any update on this? I work with vscode and "native" Linux shells side-by-side on a Windows workstation. This different behavior on what's essentially two windows for me is driving me crazy.

Me, too. And the solution is to disable auto copy-on-select, and always copy explicitly with Ctrl-Insert and paste with Shift-Insert. It's cumbersome, but at least is consistent.

I am kinda confused by this GitHub Issue, I thought that "middle-click to Paste" was already an available feature as per
https://stackoverflow.com/questions/36723333/how-to-enable-middle-mouse-button-click-paste-in-vs-code
https://github.com/microsoft/vscode/issues/14610

Please set "editor.selectionClipboard": false in your user settings. This will make it so we don't paste from the selection clipboard when you press the middle button and we don't write to the selection clipboard when a selection is made in the editor.

However, searching for editor.selectionClipboard in the VS Code settings on version code-server: v4.14.1 / Code: 1.79.2  does not yield any results.
Also kinda confused because it seems like the desired behavior would be "copy on select" followed by "paste on middle-click".
also kinda confused by this

The traditional mouse button for copy-and-then-paste on Linux

been using Linux daily for many many years, I have heard of a "traditional mouse button behavior" in Linux. Aren't these behaviors controlled by the third-party software such as your terminal emulator and desktop environment? If you are seeing some default behavior in your Linux installation, I am not sure that its coming from Linux itself, but from the third-party packages you are running which are not standardized.
regardless of what is perceived as "traditional mouse button behavior" it seems like the "copy" and "paste" functionalities related to mouse behavior in the integrated terminal should be configured separately. So you can have "highlight on select" and "paste on middle-click", or whatever combination of actions, that you prefer

@stevekm On Linux there are two clipboards, one is provided by X which can be printed with xclip -o, and the other is provided by my desktop environment which is controlled by Ctrl-C, Ctrl-V, etc.
I am on Cinnamon so I take it as an example:

After some text is selected (no whether it is in a terminal or not), the X clipboard is updated and the output of xclip -o changes, but the Cinnamon clipboard doesn't change.
If I enable System Settings -> Mouse and Touchpad -> Paste the current selection when middle-click is pressed, pressing middle moused button pastes the X clipboard. This has almost the effect of combination of copy-on-select and then paste-on-middle-click, except that the Cinnamon clipboard (Ctrl-C, Shift-Insert) is unaffected.
This explains why I am not willing to enable copy-on-select: It may unintentionally overwrite the clipboard just because something is selected. Copy-and-paste on click is less likely to be mishandled.

Thanks @lhmouse
X is not a part of Linux. It is included in some desktop Linux distros but its not a core part of the Operating System. Linux desktop environments and desktop distro's are not standardized and are heavily influenced by whatever arbitrary combination of third-party packages their designers decide to include. It seems very strange and not quite appropriate to change the behavior of a cross-platform app like VS Code in response to a non-standard configuration of third party packages on a subset of distributions of an OS.

https://unix.stackexchange.com/questions/149066/what-is-x-window-system#149075

https://en.wikipedia.org/wiki/X_Window_System

This is why I suggest that the correct implementation of this is to allow for de-coupled behaviors for both copying and pasting. So that if one or the other or both are causing you problems on your specific combination of non-standard third party desktop environment packages, you can more easily choose the exact combination of configurations to suit your needs.
Also I am refering to the "Copy On Select" setting in VS Code, not in your OS or desktop environment.
terminal.integrated.copyOnSelection

This is a VS Code Terminal setting. It does not cause text selected in e.g. the VS Code Editor to automatically copy.

This explains why I am not willing to enable copy-on-select: It may unintentionally overwrite the clipboard just because something is selected.

If you are using copy-on-select, and you dont want some text to be copied, then just dont select it. If you accidentally selected the wrong text for copying, then just re-select the correct text. If you are worried about losing your clipboard contents, then make sure to Paste it before you make another selection.


[FLINK-14781] [docs-zh] clarify that a RocksDB dependency in pom.xml may not be needed

What is the purpose of the change
clarify that a RocksDB dependency in pom.xml may not be needed
Brief change log
clarify that a RocksDB dependency in pom.xml may not be needed
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Thanks @catkint for the PR. Can you actually rebase this onto the latest master and without those additional (merge) commits? Depending on your setup, something like this and then only keep your new commit:
git rebase -i upstream master

Maybe there is something wrong with my operation. I will correct it asap

I have modified ，please review it

Hi @catkint could you please rebase the new master branch to fix the conflict first?

Hi, It seems that friend finished this issue before me. And without using jira.  I modify some minor points, such as the reference to Chinese  document. Please review it. thx.

Hi,@wuchong ,excuse me, this PR takes a long time, please review it, maybe I should close it ?

Looks good to me % a minor comment

right ，I have corrected it ，thx


Global 'View' and Workspace 'Add': Cycle Time Summary Data

As an ECMPS user, I want to view Cycle Time Summary Data in the Global-View and add new Cycle Time Summary Data in the Workspace so that I can look at real-time data or add test information as needed
Acceptance Criteria: (Global-view)
Given a user has selected Cycle Time Summary from the Test Type Group dropdown and a data record is available
When the data is displayed
Then the user can see data displayed as a separate table
Columns:
Total Time
Given a user is viewing Cycle Time Summary Data
When the data is displayed in a table
Then the default sort order is alphanumeric descending by the contents in the 1st column
And the user will have the option to sort the columns in ascending or descending alphanumeric order

[x] The fields and their order specified

Pagination?

[ ] Yes
[x] No

Filter by keyword?

[ ] Yes
[x] No

Columns are sortable?

[x] Yes
[ ] No

Nested rows, expandable?

[x] Yes
[ ] No

Acceptance Criteria: (Workspace - Add)
Given the user has selected Cycle Time Summary from the Test Type Group dropdown
When the user clicks on 'Add' for Cycle Time Summary data
Then the system displays a modal
and the user can add new data for:
Total Time (Text)
Given the user has added Cycle Time Summary
When the user chooses to save and close
Then the data is saved
And the modal closes
And the user is returned to the Landing screen
And the Cycle Time Summary record is reflected on the UI
Given the user has added a new Cycle Time Summary
When the user chooses to Cancel
Then the user is returned to the Landing screen for Cycle Time Summary
And no updates are saved
Dev Note: This data section is at the second level of nesting (Test Data -> Cycle Time Summary)
Reference Screenshot:

Testing
Test Case: TC1107
Tester @mosesdeeCVP

Cycle Time Summary Data - Table View Official

Cycle Time Summary Data - Modal View Official

Cycle Time Summary Data - Table View Workspace

Cycle Time Summary Data - Modal View Workspace

Test Passed


Install locations from feed not always as stated

In the section
.NET SDK and CLI environment variables
DOTNET_ROOT, DOTNET_ROOT(x86)
you write:

Specifies the location of the .NET runtimes, if they are not installed in the default location. The default location on Windows is C:\Program Files\dotnet. The default location on macOS is /usr/local/share/dotnet. The default location on Linux varies depending on distro and installment method. The default location on Ubuntu 22.04 is /usr/share/dotnet (when installed from packages.microsoft.com) or /usr/lib/dotnet (when installed from Jammy feed).

This is not necessarily the case - on running sudo apt-get install dotnet-sdk-7.0 the runtimes ended up in /usr/lib/dotnet but the sdks ended up in /usr/share/dotnet - this has been very confusing to debug (dotnet finds the runtime but not the sdk) and I have no good idea how it happened (possibly because I added a microsoft feed to get powershell? Which also supplies the sdk, but not the runtime, so I get a jammy runtime but a microsoft sdk?)

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

ID: c1fab316-eb04-50f5-03c8-bf3004981d3e
Version Independent ID: 0c8e3542-470a-fa11-0d27-061384c4c3ba
Content: .NET environment variables - .NET CLI
Content Source: docs/core/tools/dotnet-environment-variables.md
Product: dotnet-fundamentals
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

I have run a few tests backward and forwards.
I can reliably recreate the issue when I add MS repos to get powershell.
My current workaround has been to get powershell from Snap and not add the MS repos to APT.
Still, that seems silly - what am I supposed to advice my colleagues here? "Avoid adding MS repos if you have to do dotnet development" seems like very weird advice.

Hi @SRNissen This sounds like a package mixup problem. You'll need to configure your package manager to AVOID getting the .NET packages from the MS Feed while allowing all other packages (such as powershell) to be sourced from the MS Feed.
The Install .NET on Ubuntu Overview page includes a lot of information about resolving these problems. Read the intro and the warning message there, then check the Decide how to install .NET section. I added a decision tree there to help these types of scenarios. Hopefully this gets you working. Please provide feedback on that page if there's a problem or improvement to that page.
I'm going to close this issue here because I think that link should provide all the information you need. If you feel that you want to continue the conversion here, please feel free to reopen the issue, otherwise, use the feedback link on the overview page and I'll get alerted to the new issue. Thanks!

Hi
Thank you, it is a thorough and useful guide (better than the one for powershell) but I don't need it today - I found a solution on the same day I opened the issue.
I returned to the issue yesterday because a junior ran into the exact same problem and as I was explaining the solution, I wanted to update this issue to make it known to the dotnet team that Microsoft guidance on how to install Powershell will break casually break dotnet installs.
As a purely administrative matter you can leave the issue closed or you can re-open it - inside our org it is solved, our internal handbook now has guidance on how to avoid this, or how to get back out if you accidentally got into this situation.
But as a fact about documentation, the issue has not been solved until Microsoft guides don't give mutually exclusive directions.
For reference, the powershell guide that doesn't tell you it will break dotnet installs is this one:
https://learn.microsoft.com/en-us/powershell/scripting/install/install-ubuntu?view=powershell-7.3

Thanks for the link @SRNissen I'll file a bug on that documentation telling them to add matching (or point to the ubuntu article) documentation.


(WIP) Add option to package individually without moving module content to root folder (i.e. like base serverless)

What it does
Adds option IndividuallyMoveUpModules to allow customization of packaging, when individually: true.

Let's see an example that tries to show, why this might be important...
We have a root-directory containing one lambda (as a module) located at ./fn, some tests (./tests) and our serverless.yml, giving this structure:
$ ls -lR
.:
total 80
drwxrwxr-x 2 bweigel bweigel  4096 Nov 12 18:14 fn
-rw-rw-r-- 1 bweigel bweigel   431 Nov 12 18:12 serverless.yml
drwxrwxr-x 2 bweigel bweigel  4096 Nov 12 18:15 tests

./fn:
total 52
-rw-rw-r-- 1 bweigel bweigel   0 Nov 12 14:08 __init__.py
-rw-rw-r-- 1 bweigel bweigel 156 Nov 12 16:49 main.py
-rw-rw-r-- 1 bweigel bweigel  61 Nov 12 16:49 other_module.py
-rw-rw-r-- 1 bweigel bweigel   0 Nov 12 14:15 requirements.txt
-rw-rw-r-- 1 bweigel bweigel  24 Nov 12 16:46 yam.py

./tests:
total 20
-rw-rw-r-- 1 bweigel bweigel  0 Nov 12 16:47 __init__.py
-rw-rw-r-- 1 bweigel bweigel 80 Nov 12 16:48 test_other_module.py

This is the content of the serverless.yml, where we define our function with its entry-point fn.main.lambda_handler located at the fn/main.py. We also define that we want individual packaging of functions using individually: true:
service: packaging_test

provider:
  name: aws
  runtime: python3.6

package:
  individually: true
  exclude:
    - '**/*'

functions:
  function-one:
    handler: fn.main.lambda_handler
    package:
      include:
        - 'fn/**'

Using base serverless functionality lets us package our (single function) service and we see, that the structure that is present in our root-folder has not changed. The module fn is still there:
$ sls package
Serverless: Packaging service...
Serverless: Excluding development dependencies...

$ zipinfo .serverless/function-one.zip
Archive:  .serverless/function-one.zip
Zip file size: 834 bytes, number of entries: 5
-rw-rw-r--  4.5 unx        0 bl stor 80-Jan-01 00:00 fn/__init__.py
-rw-rw-r--  4.5 unx      156 bl defN 80-Jan-01 00:00 fn/main.py
-rw-rw-r--  4.5 unx       61 bl defN 80-Jan-01 00:00 fn/other_module.py
-rw-rw-r--  4.5 unx        0 bl stor 80-Jan-01 00:00 fn/requirements.txt
-rw-rw-r--  4.5 unx       24 bl defN 80-Jan-01 00:00 fn/yam.py
5 files, 241 bytes uncompressed, 212 bytes compressed:  12.0%

However, if we were to use the serverless-python-packaging-plugin in our serverless.yml this would change:
# serverless.yml
...
plugins:
  - serverless-python-requirements
...

$ sls package
Serverless: Generated requirements from /home/bweigel/Projects/OSS/sls_package/fn/requirements.txt in /home/bweigel/Projects/OSS/sls_package/.serverless/requirements.txt...
Serverless: Copying from /home/bweigel/Projects/OSS/sls_package/.serverless/requirements.txt into /home/bweigel/Projects/OSS/sls_package/.serverless/fn/requirements.txt ...
Serverless: Installing requirements from /home/bweigel/Projects/OSS/sls_package/.serverless/requirements/requirements.txt ...
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Serverless: Injecting required Python packages to package...

$ zipinfo .serverless/fn-packaging_test-dev-function-one.zip
Archive:  .serverless/fn-packaging_test-dev-function-one.zip
Zip file size: 727 bytes, number of entries: 5
-rw-rw-r--  3.0 unx        0 b- stor 98-Jan-01 00:00 __init__.py
-rw-rw-r--  3.0 unx        1 b- defN 98-Jan-01 00:00 requirements.txt
-rw-rw-r--  3.0 unx      156 b- defN 98-Jan-01 00:00 main.py
-rw-rw-r--  3.0 unx       61 b- defN 98-Jan-01 00:00 other_module.py
-rw-rw-r--  3.0 unx       24 b- defN 98-Jan-01 00:00 yam.py
5 files, 242 bytes uncompressed, 215 bytes compressed:  11.2%

Now the content of fn has moved to the root-level of the deployment artifact.
This is bad for a couple of reasons:
It breaks imports: fn/other_module.py imports a function from fn/yam.py using from fn.yam import .... However, this import statement will not work, when the content of fn is moved to the root of the zip-artifact, because now module fn will not be on our PYTHONPATH. It will cease to exists in our lambda runtime environment. I guess one could argue to just import like fn is the base folder and do from yam import ..., however this is where you break something else...
You break testing: tests/test_other_module.py contains unittests for some functions in fn/other_module.py. Again, inside  test_other_module.py we use absolute imports from fn.other_module import .... This will work fine in the IDE (given it is configured correctly) until the Python interpreter parses the first line in other_module.py, which reads from yam import ... and the module yam cannot be found, because it is not on the path (only fn is...).
I guess you see the point that I am trying to argue.  Something I would like to try and avoid and hence came up with this PR.

Implementation details

Don't break existing code bases. Keep default behavior as is.

if IndividuallyMoveUpModules: true (default behavior) moves module contents to root of deployment artifact (./fn/handler.py→./handler.py)
if IndividuallyMoveUpModules: false leaves module contents as is (default serverless behavior; ./fn/handler.py→./fn/handler.py)

During the writing of the new (individual) zip-file the option is checked in the function moveModuleUp
this will be configurable once accepted:

Variable Name
Value
Description

IndividuallyMoveUpModules
false/true
Default: true. Disable or enable moving module contents to the top of the zip-file artifact

TODO

[x] implement feature
[x] add simple test
[x] fix tests
[x] add more tests?
[ ] update documentation
[ ] iterate through feedback
[ ] await merge

related Issues

#194
#208
#209
#223

Update 2018-11-15
While working on this (i.e. on fixing failed tests) I also ran into some bugs, that have been lurking behind #209 and #194, which I will fix in this PR.

neat. I think this matches how I'd envisioned this working, but I wasn't using the feature, nor had time to develop my vision of how it should work.

Note to self: check funky behavior of requirement injection. Failed tests turn green, when hello4 is moved to first place in functions in serverlss.yml ...

Thank you so much for this!

Would love to see this merged. :) Thanks for the feature!

Hello, I can't wait for this PR to be merged too since I also have trouble invoke my functions when packaged individualy.

Anyone is invited to pick up the work. I am swamped right now and not in the mood to address broken windows tests....well this and merge hell :)

Could an admin look into re-invoking the tests? It looks like it's failing while setting up the environment... Could've been a fluke?


Roll Engine from b22809b08434 to 40c226e25310 (5 revisions)

https://github.com/flutter/engine/compare/b22809b08434...40c226e25310
2020-10-15<EMAIL_ADDRESS>Revert "Add flag to not publish the observatory port over mDNS (#21632)" (flutter/engine#21882)
2020-10-15<EMAIL_ADDRESS>Fix destruction order in C++ plugin registrar (flutter/engine#21840)
2020-10-15<EMAIL_ADDRESS>Add flag to not publish the observatory port over mDNS (flutter/engine#21632)
2020-10-15<EMAIL_ADDRESS>Add a style note about Linux embedding style (flutter/engine#21819)
2020-10-15<EMAIL_ADDRESS>Migrate TextInputPlugin API to TextRange (flutter/engine#21854)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


How to set custom point Sydney for example as default center of globe

Hi Chris,
Thank you very much for sharing this excellent globe.  I want to ask you how can I set custom point, for example, Syndey's coordinates, as default center of the globe?

Hi @xwenc, try using the lookAt props (see code).
Let me know if that is what you're looking for!

Yes, That's what i want, thanks！  O(∩_∩)O


docs: How to bind to nullable objects with fluent-select

🙋 Documentation Request
I want to use FluentSelect component to allow my users to select items from a predefined list.
In my case I have a list of countries which I want to to assign to MyModel. A new instance does not have a country selected at first that's why it holds the CountryId as nullable.
For example:
public class Country
{
    public int CountryId { get; set; }
    public string Name { get; set; }
}

public class MyModel
{
    public int? CountryId { get; set; } // For new MyModels, no country is preselected
    /* Other properties */
}

My countries are all held in memory like this:
private List<Country> _countries = new
{
    new Country { 1, "Germany" },
    new Country { 2, "Netherlands" }
}

In default blazor I can achieve this with this code:
<InputSelect @bind-Value="Input.CountryId">
    <option value="">--- None ---</option>
    @foreach (var item in _countries)
    {
        <option value="<EMAIL_ADDRESS>    }
</InputSelect>

But how can I achieve this using the FluentSelect component?
I tried this so far but I cannot use bind-value with anything else besides a string. Also I don't know how I get the null state in here as well.
<FluentSelect TOption="Country"
              Label="Country"
              Items="@_countries"
              OptionValue="@(p => p.CountryId.ToString())"
              OptionText="@(p => p.Name)" />

Could you provide some details about this use case?

You can do either this :
<FluentSelect TOption="Country"
              Label="Country"
              Items="@_countries"
              OptionValue="@(p => p.CountryId.ToString())"
              OptionText="@(p => p.Name)" >
</FluentSelect>

@code {
    public class Country
    {
        public int? CountryId { get; set; }
        public string? Name { get; set; }
    }

    public class MyModel
    {
        public int? CountryId { get; set; } // For new MyModels, no country is preselected
        /* Other properties */
    }

    private List<Country> _countries = new List<Country>
    {
        new Country { CountryId = null, Name="Select a country"},
        new Country { CountryId = 1, Name = "USA" },
        new Country { CountryId = 2, Name = "Canada" },
        new Country { CountryId = 3, Name = "Mexico" }
    };

}

Or
<FluentSelect Label="Country">
              <FluentOption Value="null">Select a country</FluentOption>
              <FluentOption Value="1">USA</FluentOption>
              <FluentOption Value="2">Canada</FluentOption>
              <FluentOption Value="3">Mexico</FluentOption>
</FluentSelect>

@vnbaaij thanks for clarification. But your examples are missing the bind mechanismn. Could you specify how to bind it to MyModel.CountryId correctly?

Ah, that was because there was no bind in your code as well :)
You can just use  @bind-Value="@SelectedValue" or @bind-SelectedOption="@SelectedPerson" as shown in the example on the demo site:
@inject DataSource Data
<FluentSelect TOption="Person"
              Label="Select a person"
              Items="@Data.People.WithVeryLongName()"
              Id="people-listbox"
              Width="200px"
              Height="250px"
              OptionValue="@(p => p.PersonId.ToString())"
              OptionText="@(p => p.LastName + ", " + p.FirstName)"
              @bind-Value="@SelectedValue"
              @bind-SelectedOption="@SelectedPerson" />

    Selected value: @SelectedValue

    Selected person (strongly typed): @SelectedPerson?.ToString()

@code
{
    Person? SelectedPerson;
    string? SelectedValue = "4";
}

@vnbaaij my problem here is that I need to bind to the CountryId of MyModel I need the Id in integer not as string. Is there any easy way to bind to int directly?

You either need to parse it yourself (@bind-Value), or get it from the TOption (@bind-SelectedOption). Like with SelectedPerson.PersonId in the example above

I think I got it. I can use the SelectedOptionChanged EventCallback
<FluentSelect TOption="Country"
              Label="Country"
              Width="100%;"
              Items="@_countries"
              OptionValue="@(p => p.CountryId.ToString())"
              OptionText="@(p => p.Name)"
              OptionSelected="(option) => Input.CountryId == option.CountryId"
              SelectedOptionChanged="(option) => Input.CountryId = option.CountryId > 0 ? option.CountryId : null" />

@code {
    public class Country
    {
        public int CountryId { get; set; }
        public string? Name { get; set; }
    }

    public class MyModel
    {
        public int? CountryId { get; set; } // For new MyModels, no country is preselected
        /* Other properties */
    }

    private List<Country> _countries = new List<Country>
    {
        new Country { CountryId = -1, Name="Select a country"},
        new Country { CountryId = 1, Name = "USA" },
        new Country { CountryId = 2, Name = "Canada" },
        new Country { CountryId = 3, Name = "Mexico" }
    };
}

Another problem is that the ID itself of the country cannot be null, only from the MyModel itself.

This implementation is even better and more easily manipulatable.
<FluentSelect TOption="Country"
              Label="Country"
              Width="100%;">
    <FluentOption Value="null" Selected="@(Input.CountryId is null)" OnSelect="(val) => Input.CountryId = null">--- Select ---</FluentOption>
    @foreach (var item in _countries)
    {
        <FluentOption TOption="Country" Value="@item.CountryId.ToString()" Selected="@(Input.CountryId == item.CountryId)" OnSelect="(val) => Input.CountryId =<EMAIL_ADDRESS>    }
</FluentSelect>


Transitions aren't working

I'm not able to see the slide transitions as shown in the demo videos, even on the demo site. I didn't dig into your code and/or css yet, but thought I'd report what I'm seeing in case you know off-hand what might be causing it.
I've created a sandbox here to show how I've attempted to adjust the props to enable transition effects:
https://codesandbox.io/s/recursing-agnesi-7s6n1z?file=/src/App.js
Tested on Chrome, FF, Edge
Kind Regards,
Jonathan

Hi @JSteeleAZ  ! Thank you for raising this issue! 😀
Did you set "Reduce motion: on" or something similar in settings?

Can you give me more information?


Option to also download Profile Picture when using :feed

Provide us a use case of the feature
How could the user invoke the new function? Which problem would it solve? If new information is obtained, how would it be further processed?
Add an option so also store the profile picture of the user in the :feed function. I currently use this statements:
instaloader/instaloader.py -l $USERNAME -p $PASSWORD :feed --fast-update --no-captions --dirname-pattern= --filename-pattern={owner_id}/{date_utc:%Y}/{date_utc:%m}/{date_utc:%d}/{date_utc:%Y-%m-%d_%H-%M-%S}_{profile}
I know username and pw in Console is unsafe blabla... (It's not a problem if account gets hacked.)
Describe the solution you'd like
A clear and concise description of what you want to happen.
Maybe store the picture like this too, because currently it only stores it like this when I use instaloader.py @user --no-posts it stores it like this:
/{username}/
	id
	..._profile_picture.jpg

the thing is, I have an structure, to parse it into a database for a little webapp for me at home here, because my instagram app doesn't work correctly sometimes, that sorts the posts like this for parsing:
/:feed/
	{owner_id}/
		year/
			month/
				day/
					2010-04-20_04-20-00_username.jpg
and what i want:			2010-04-20_04-20-00_profile_pic.jpg

maybe we can add the functionality, that it can store it like this too... so that the profilepicture can be stored like in the example above. I know, there is json.xz and so on also, but I did not include it here in this examples because of simplicity.
so in short: its a foldername problem.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
If this is not possible, maybe make it possible, that instaloader.py @user will be possible to use same filestructure like the instaloader.py :feed ... it does this at the moment /username/{content}
If the feature request is accepted, would you be willing to submit a PR?
Yes / No (Help can be provided if you need assistance submitting a PR)
Ehhmm... I don't know what PR is currently... but if you tell me maybe.
Additional context
Add any other context about the feature request here.
I need this "Permanent" Structure of filenames, because the database that runns in combination with instaloader keeps only static paths to the files on the webserver (running local here).
(please also see https://instaloader.github.io/contributing.html for how to suggest a feature)

This would be a typical case for using Instaloader as a module rather than its CLI, since it is a very specific use case.


Use new JavaScript standards (ECMA-Script 6=<)

I want to increase the performance in terms of memory a bit more by using let and const instead of var

Decided to stay with the old stuff, just because of backward compatibility~


Fix: DebugLog.on_result_judge() KeyError: 'judge'

例外が上がってたので1行パッチです。
DebugLog.on_result_judge() raised a exception >>>>
Traceback (most recent call last):
  File "/Users/kshimo69/repos/IkaLog/ikalog/engine.py", line 50, in call_plugins
    getattr(op, event_name)(self.context)
  File "/Users/kshimo69/repos/IkaLog/ikalog/outputs/debug.py", line 90, in on_result_judge
    context['game']['judge'], context['game'].get('knockout', None))
KeyError: 'judge'

<<<<<

Thanks!


core(packaging): use upstream buildpack

There were the following issues with your Pull Request

Commit: 2a65606770b9230b2913bfcc42fe9e84d77044e6
Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore, revert

Guidelines are available at https://github.com/benlangfeld/QueueFlex/blob/master/CONTRIBUTING.md#pull-requests

This message was auto-generated by https://gitcop.com

Coverage remained the same at 47.368% when pulling 301bd7d42b9a3a1c777ee40ec09c53dad4f87811 on feature/upstream into 9492acc4b6ac97e4162b475d3cb463f4c2d69ead on master.

Coverage remained the same at 47.368% when pulling 301bd7d42b9a3a1c777ee40ec09c53dad4f87811 on feature/upstream into 9492acc4b6ac97e4162b475d3cb463f4c2d69ead on master.

LGTM


Added support to pack components of ReplicatedTransform, off by default

Usecase for this would be interesting to hear.

Usecase for this would be interesting to hear.

From what I know it's a common practice when you need to sync many objects - to pack transforms.
I'm just not sure if it make sense to have individual flags for position/rotation. Probably it does, I just don't have any hands-on experience for such case.

What if we had scene-level constants for "max values" and flag set in component?
Just to save on attributes, and simplify value editing too.
I am also curious how much it saves in practice.


I donot understand what is the meaning of ‘’maximize D(x)-1-D(G(z))‘’ ?

It's my first time to touch with GAN, and I try to search the original paper. I have understood the loss function of Generator net writed in your code(maybe it refers to ESRGAN‘s and adds TV loss to denoise), but I found that the loss function of Discriminator net is different from the paper of SRGAN  and ESRGAN(or maybe I make mistake understanding it).....could you help me explain the loss function of Discriminator net writed in your code or give me some advise to solve it? thanks very much!!! （if I change the code "d_loss = 1 - real_out + fake_out" to code “d_loss = - log(real_out ) - log(1 - fake_out)”, will it work? I found your loss function will get Score_D and Score_G close to zero after training of some epoch)

I found "https://github.com/amanchadha/iSeeBetter" have same code of your project...I want to run it finally, but I want to understand SRGAN and RBPN respectively


Unable to load course content. Try again later.

NSURLSession/NSURLConnection HTTP load failed (kCFStreamErrorDomainSSL, -9813)
[ERROR|ANALYTICS] @ Optional("CourseOutlineViewController.swift"):126 - Unable to load block: Error Domain=org.edx.error Code=-100 "Unable to load course content.
Try again later." UserInfo={NSLocalizedDescription=Unable to load course content.
Try again later.}
We are getting the above error while accessing the courseware in the mobile application v2.1. Please let us know if there is any way to fix? issue.

Hey @krishnakanthjb can you tell me the content of the request the server is sending back? That's just the generic error we show when we don't have something more specific to show

We found the response is  "{
"detail": "Authentication credentials were not provided."
}"
Can you please let us know, is there any specific configuration change needs to be done?
If yes, please let us know, if that is from server side or in the Xcode application.

@krishnakanthjb I would guess it's a server side error then, but it's worth verifying that the app is sending the expected oauth Authentication http header.
Are you using basic-auth by any chance on your server? We've tried to make it work in conjunction with oauth, but there are some weird interactions.
Was this working elsewhere or previously? Perhaps you're missing a config flag on your server. Did you follow the instructions here: http://edx.readthedocs.org/projects/edx-installing-configuring-and-running/en/latest/mobile.html

Yes.. It's working well with v1.0 with the same configurations, also we were able to play the course videos on v1.0. Not sure what differed on v2.1.
Any help appreciated.

2.1 depends on new API that did not exist for 1.0 @krishnakanthjb if you haven't upgraded your server you will need to do that

What exactly did you mean by up gradation..do we need to create the new client Id and secret..or any thing else..
Please clarify as I'm new to server terminology

@krishnakanthjb over the last several months, we have made changes to the edx-platform server that the new code app code depends on. If you haven't pulled and deployed the latest version of edx-platform in a while, you will need to do that

We are using cypress as platform version. Please let us know if any thing specific. Is The new API containing the 2.1 instead of v 1.0??

@krishnakanthjb you will need to upgrade from cypress to dogwood to support the v2 app code.

Hi ALeffert,
Thanks for the reply.
We were able to view the list of courses after logged into the application(with cypress server), also we have observed some of the data in the Handouts, Updates.
We facing problem only for the course content.
Will it help in accessing the course content by upgrading the server to dogwood?
Thanks & Regards,
Krishna Kanth J B
-----Akiva Leffert<EMAIL_ADDRESS>wrote: -----
To: edx/edx-app-ios<EMAIL_ADDRESS>From: Akiva Leffert<EMAIL_ADDRESS>Date: 01/29/2016 08:12PM
Cc: krishnakanthjb<EMAIL_ADDRESS>Subject: Re: [edx-app-ios] Unable to load course content. Try again later. (#652)
@krishnakanthjb you will need to upgrade from cypress to dogwood to support the v2 app code.
—
Reply to this email directly or view it on GitHub.
=====-----=====-----=====
Notice: The information contained in this e-mail
message and/or attachments to it may contain
confidential or privileged information. If you are
not the intended recipient, any dissemination, use,
review, distribution, printing or copying of the
information contained in this e-mail message
and/or attachments to it are strictly prohibited. If
you have received this communication in error,
please notify us by reply e-mail or telephone and
immediately and permanently delete the message
and any attachments. Thank you

@krishnakanthjb yes. You will need to upgrade to dogwood.

I also get this issue, ok good info I'll upgrade from Cypress to Dogwood


Cleanup post shared components are moved out

Currently, there are multiple share components within Pooler that need to be moved into their separate repositories, as outlined in issue #14 (https://github.com/PowerLoom/product/issues/14). Once this separation is complete, it will be necessary to conduct a thorough cleanup of the Pooler libraries and dependencies to ensure optimal performance and clean codebase.
Related to #8

Completed as part of https://github.com/PowerLoom/product/issues/14


Updates 04.12

(@Apollon77) Shows a progress indicator when deleting controller nodes
(@Apollon77) Cuts names and labels to 32 or 64 characters as needed by Matter
(@Apollon77) Improves error handling on devices and bridges
(@Apollon77) Clear storage when removing a bridged device
(@Apollon77) Processes changed objects with a 5s delay to prevent too many changes at once
(@Apollon77) Fixes version determination

Re Queue: The minimum needed is a delay in processing the changed objects if someone in the UI klicks very fast because else re-initializations could overlap and lead to strange behavior. SO I thought it is a good idea to also queue that all if someone changes multiple devices, so that logs are easier to read and we do not overload "smaller systems"


Internal link without using href

Very nice demo! Hi do you know how to solve the internal link routing? Example passing [routerLink] looks like it didn't compile.

@hijoncon I'm not sure what you're trying to do. By internal, you mean link inside a page #my-heading-somewhere ? If so, I don't think you can with the Angular router.

@ludohenin thanks for your reply.  Example in your post usually we use about me but this will reload the page.  I tried to use [routerLink] directive, but it didn't compile...

Can you provide the error logs ?


How Imagecredentials can be access if its values are defined in secret file instead of values file

I am using below configuration for Imagepullsecrets.
My query- Here within defin Imagepullsecrets , we are pulling imagecredentials.registry, imageCredentials.username imageCredentials.password from values.yaml file. If I don't want to expose these username, registry and password in values file then what is the other way to achieve this.
For example - If I want to pass these username, registry and password in secrets file then how we can access it in helm chart i.e. .Secrets.imagecredentials.registry instead of .Values. imagecredentials.registry ?
{{- define "imagePullSecret" }}
{{- printf "{"auths": {"%s": {"auth": "%s"}}}" .Values.imageCredentials.registry (printf "%s:%s" .Values.imageCredentials.username .Values.imageCredentials.password | b64enc) | b64enc }}
{{- end }}
apiVersion: v1
kind: Secret
metadata:
name: regcred
namespace: {{ .Values.namespace }}
type: kubernetes.io/dockerconfigjson
data:
.dockerconfigjson: {{ template "imagePullSecret" . }}

Duplicate of #11270.

@bacongobbler can you please help me with answer. I am seeing you have closed this issue with a say that it is duplicate. However I am not seeing any answer for this query.


Combined client chart v2

Adds a chart below each router summary.
Modifies UpdateableChart to support polling via the metricsCollector.

:star: :turtle: Looks good to me, modulo @rmars's comment.

:+1: :octopus: :dog: yay graphs!


STENCIL-3126: Add pull request template for contribution guidelines

See title. CONTRIBUTING.md has a link to this which currently 404s.

@bigcommerce/stencil-team Any input welcome

I would rather add a github template for new pull request based on https://help.github.com/articles/creating-an-issue-template-for-your-repository & just point the url to "New Pull Request" link  (https://github.com/bigcommerce/stencil/pull/new/master)

Might consider moving files to .github directory so we dont clutter the root directory of the project.

@pedelman done
@junedkazi so someone more slimmed down and change the PR link in CONTRIBUTING.md?

I did not mean moving CONTRIBUTING.md under .github. I meant more like updating the broken link to  https://github.com/bigcommerce/stencil/pull/new/master

@junedkazi Okay. I can do that. I was also making the change that Patrick requested.

yup I had requested the same change with a link on how to do it :)

@mcampa I don't believe I do, but should anything go into the changelog for this?

No changelog is required for this change.


Remove mm_config and mm_license global state from webapp (PR #3)

Summary
This is a subset of the changes in the MM-8589-remove_mm_global_license branch, broken up into multiple PRs to facilitate easier review. See any linked PRs for other changes if you're interested.
These changes, as a whole, move towards removing our dependence on the global mm_config and mm_license objects. These changes will ultimately facilitate resolving https://mattermost.atlassian.net/browse/MM-8604, allowing us to stop hard refreshing the application whenever someone makes a configuration change.
Ticket Link
https://mattermost.atlassian.net/browse/MM-8589
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)

Related PRs
https://github.com/mattermost/mattermost-webapp/pull/816
https://github.com/mattermost/mattermost-webapp/pull/819
https://github.com/mattermost/mattermost-webapp/pull/821
https://github.com/mattermost/mattermost-webapp/pull/822
https://github.com/mattermost/mattermost-webapp/pull/823

@lieut-data also this seems to need a rebase.
let me know if you want me to review the other related PR's

Noted re: rebase -- it's going to be hard to avoid these given the scope of these six PRs, but once approved, I'll quickly rebase and merge to avoid more conflicts.

@enahum changes as requested, including the requisite rebase.


Remove _complete variables removal part

In production, this block should no longer be required:
https://github.com/AMP-SCZ/utility/blob/34c10622c5be8f060b72bc55ddf5a0cca7399e69/import_records.py#L16-L36

Currently, it is required for https://redcapynh-p11.ynhh.org/ because of incorrect mapping between instruments and events. For example, for ./PronetGA/raw/GA00001/surveys/GA00001.Pronet.json, some of the following variables do not exist in baseline_arm_1:
past_pharmaceutical_treatment_complete 0
family_interview_for_genetic_studies_figs_complete 0
perceived_discrimination_scale_complete 0
pubertal_developmental_scale_complete 0
chrpenn_complete 0
penncnb_complete 2
bprs_complete 0
nsipr_complete 0
cdss_complete 0
cssrs_baseline_complete 0
oasis_complete 0
promissd_complete 0
pgis_complete 0
pss_complete 0
reading_task_premorbid_iq_complete 0
iq_assessment_complete 0
nart_premorbid_iq_complete 0
waisiv_complete 0
wiscv_complete 0
wrat5_complete 2
wasi_current_iq_complete 2
eeg_run_sheet_complete 2
mri_run_sheet_complete 0
digital_biomarkers_mindlamp_onboarding_complete 0
current_health_status_complete 0
blood_sample_preanalytic_quality_assurance_complete 0
cbc_with_differential_complete 0
daily_activity_and_saliva_sample_collection_complete 0
speech_sampling_complete 2
global_functioning_social_scale_complete 0
global_functioning_role_scale_complete 0
gcp_current_health_status_complete 0
gcp_cbc_with_differential_complete 0

And so REDCap API raises error during importing.

This issue is quite irrelevant now. It was stemming from incorrect form-event mapping.