vllm mode has return ""

Hi. harness developers. I am using vllm mode for evaluation leaderboard benchmarks.
This is my setting for evaluation in jupyter notebook

vllm_settings= ',dtype=auto,gpu_memory_utilization=0.8'

args = argparse.Namespace(
    model="vllm",  #
    model_args="pretrained=/home/jovyan/data-vol-1/models/"
                + 'LGAI-EXAONE__EXAONE-3.0-7.8B-Instruct'
                + vllm_settings, #
    tasks="leaderboard",  #
    num_fewshot=None,
    batch_size="auto",
    max_batch_size=None,
    device="cuda",  #
    output_path=f'result/LGAI-EXAONE__EXAONE-3.0-7.8B-Instruct.json',
    limit=None,
    use_cache=None,
    cache_requests=None,
    check_integrity=False,
    write_out=False,
    log_samples=False,
    system_instruction=None,
    apply_chat_template=False,
    fewshot_as_multiturn=False,
    show_config=False,
    include_path=None,
    gen_kwargs=None,
    verbosity="INFO",
    wandb_args="",
    hf_hub_log_args="",
    predict_only=False,
    seed=[0, 1234, 1234, 1234],
    trust_remote_code=True,
    parallelize=True
)

#
cli_evaluate(args)

Below is the log for
2024-10-21:10:42:03,900 WARNING  [task.py:332] [Task: leaderboard_musr_team_allocation] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,900 WARNING  [task.py:332] [Task: leaderboard_musr_team_allocation] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,901 WARNING  [task.py:332] [Task: leaderboard_musr_team_allocation] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,919 WARNING  [task.py:332] [Task: leaderboard_musr_object_placements] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,919 WARNING  [task.py:332] [Task: leaderboard_musr_object_placements] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,937 WARNING  [task.py:332] [Task: leaderboard_musr_murder_mysteries] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,938 WARNING  [task.py:332] [Task: leaderboard_musr_murder_mysteries] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,959 WARNING  [task.py:332] [Task: leaderboard_ifeval] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:03,959 WARNING  [task.py:332] [Task: leaderboard_ifeval] has_training_docs and has_validation_docs are False, using test_docs as fewshot_docs but this is not recommended.
2024-10-21:10:42:04,883 WARNING  [model.py:422] model.chat_template was called with the chat_template set to False or None. Therefore no chat template will be applied. Make sure this is an intended behavior.
2024-10-21:10:42:04,893 INFO     [task.py:422] Building contexts for leaderboard_mmlu_pro on rank 0...
100%|██████████| 12032/12032 [00:02<00:00, 4647.58it/s]
2024-10-21:10:42:08,200 INFO     [task.py:422] Building contexts for leaderboard_bbh_boolean_expressions on rank 0...
100%|██████████| 250/250 [00:00<00:00, 528.44it/s]
2024-10-21:10:42:08,679 INFO     [task.py:422] Building contexts for leaderboard_bbh_causal_judgement on rank 0...
100%|██████████| 187/187 [00:00<00:00, 538.46it/s]
2024-10-21:10:42:09,031 INFO     [task.py:422] Building contexts for leaderboard_bbh_date_understanding on rank 0...
100%|██████████| 250/250 [00:00<00:00, 535.24it/s]
2024-10-21:10:42:09,504 INFO     [task.py:422] Building contexts for leaderboard_bbh_disambiguation_qa on rank 0...
100%|██████████| 250/250 [00:00<00:00, 535.03it/s]
2024-10-21:10:42:09,977 INFO     [task.py:422] Building contexts for leaderboard_bbh_formal_fallacies on rank 0...
100%|██████████| 250/250 [00:00<00:00, 534.61it/s]
2024-10-21:10:42:10,451 INFO     [task.py:422] Building contexts for leaderboard_bbh_geometric_shapes on rank 0...
100%|██████████| 250/250 [00:00<00:00, 537.51it/s]
2024-10-21:10:42:10,923 INFO     [task.py:422] Building contexts for leaderboard_bbh_hyperbaton on rank 0...
100%|██████████| 250/250 [00:00<00:00, 543.99it/s]
2024-10-21:10:42:11,388 INFO     [task.py:422] Building contexts for leaderboard_bbh_logical_deduction_five_objects on rank 0...
100%|██████████| 250/250 [00:00<00:00, 532.50it/s]
2024-10-21:10:42:11,864 INFO     [task.py:422] Building contexts for leaderboard_bbh_logical_deduction_seven_objects on rank 0...
100%|██████████| 250/250 [00:00<00:00, 531.48it/s]
2024-10-21:10:42:12,341 INFO     [task.py:422] Building contexts for leaderboard_bbh_logical_deduction_three_objects on rank 0...
100%|██████████| 250/250 [00:00<00:00, 531.48it/s]
2024-10-21:10:42:12,817 INFO     [task.py:422] Building contexts for leaderboard_bbh_movie_recommendation on rank 0...
100%|██████████| 250/250 [00:00<00:00, 537.73it/s]
2024-10-21:10:42:13,288 INFO     [task.py:422] Building contexts for leaderboard_bbh_navigate on rank 0...
100%|██████████| 250/250 [00:00<00:00, 546.45it/s]
2024-10-21:10:42:13,751 INFO     [task.py:422] Building contexts for leaderboard_bbh_object_counting on rank 0...
100%|██████████| 250/250 [00:00<00:00, 523.34it/s]
2024-10-21:10:42:14,235 INFO     [task.py:422] Building contexts for leaderboard_bbh_penguins_in_a_table on rank 0...
100%|██████████| 146/146 [00:00<00:00, 544.57it/s]
2024-10-21:10:42:14,507 INFO     [task.py:422] Building contexts for leaderboard_bbh_reasoning_about_colored_objects on rank 0...
100%|██████████| 250/250 [00:00<00:00, 534.53it/s]
2024-10-21:10:42:14,981 INFO     [task.py:422] Building contexts for leaderboard_bbh_ruin_names on rank 0...
100%|██████████| 250/250 [00:00<00:00, 538.14it/s]
2024-10-21:10:42:15,451 INFO     [task.py:422] Building contexts for leaderboard_bbh_salient_translation_error_detection on rank 0...
100%|██████████| 250/250 [00:00<00:00, 533.15it/s]
2024-10-21:10:42:15,927 INFO     [task.py:422] Building contexts for leaderboard_bbh_snarks on rank 0...
100%|██████████| 178/178 [00:00<00:00, 539.38it/s]
2024-10-21:10:42:16,261 INFO     [task.py:422] Building contexts for leaderboard_bbh_sports_understanding on rank 0...
100%|██████████| 250/250 [00:00<00:00, 542.76it/s]
2024-10-21:10:42:16,727 INFO     [task.py:422] Building contexts for leaderboard_bbh_temporal_sequences on rank 0...
100%|██████████| 250/250 [00:00<00:00, 544.27it/s]
2024-10-21:10:42:17,193 INFO     [task.py:422] Building contexts for leaderboard_bbh_tracking_shuffled_objects_five_objects on rank 0...
100%|██████████| 250/250 [00:00<00:00, 540.94it/s]
2024-10-21:10:42:17,661 INFO     [task.py:422] Building contexts for leaderboard_bbh_tracking_shuffled_objects_seven_objects on rank 0...
100%|██████████| 250/250 [00:00<00:00, 536.05it/s]
2024-10-21:10:42:18,134 INFO     [task.py:422] Building contexts for leaderboard_bbh_tracking_shuffled_objects_three_objects on rank 0...
100%|██████████| 250/250 [00:00<00:00, 542.32it/s]
2024-10-21:10:42:18,601 INFO     [task.py:422] Building contexts for leaderboard_bbh_web_of_lies on rank 0...
100%|██████████| 250/250 [00:00<00:00, 547.28it/s]
2024-10-21:10:42:19,064 INFO     [task.py:422] Building contexts for leaderboard_gpqa_diamond on rank 0...
100%|██████████| 198/198 [00:00<00:00, 1859.83it/s]
2024-10-21:10:42:19,265 INFO     [task.py:422] Building contexts for leaderboard_gpqa_extended on rank 0...
100%|██████████| 546/546 [00:00<00:00, 1865.76it/s]
2024-10-21:10:42:19,803 INFO     [task.py:422] Building contexts for leaderboard_gpqa_main on rank 0...
100%|██████████| 448/448 [00:00<00:00, 1867.20it/s]
2024-10-21:10:42:20,249 INFO     [task.py:422] Building contexts for leaderboard_math_algebra_hard on rank 0...
100%|██████████| 307/307 [00:00<00:00, 530.82it/s]
2024-10-21:10:42:20,846 INFO     [task.py:422] Building contexts for leaderboard_math_counting_and_prob_hard on rank 0...
100%|██████████| 123/123 [00:00<00:00, 533.15it/s]
2024-10-21:10:42:21,090 INFO     [task.py:422] Building contexts for leaderboard_math_geometry_hard on rank 0...
100%|██████████| 132/132 [00:00<00:00, 529.71it/s]
2024-10-21:10:42:21,352 INFO     [task.py:422] Building contexts for leaderboard_math_intermediate_algebra_hard on rank 0...
100%|██████████| 280/280 [00:00<00:00, 537.31it/s]
2024-10-21:10:42:21,891 INFO     [task.py:422] Building contexts for leaderboard_math_num_theory_hard on rank 0...
100%|██████████| 154/154 [00:00<00:00, 536.92it/s]
2024-10-21:10:42:22,192 INFO     [task.py:422] Building contexts for leaderboard_math_prealgebra_hard on rank 0...
100%|██████████| 193/193 [00:00<00:00, 537.16it/s]
2024-10-21:10:42:22,566 INFO     [task.py:422] Building contexts for leaderboard_math_precalculus_hard on rank 0...
100%|██████████| 135/135 [00:00<00:00, 538.78it/s]
2024-10-21:10:42:22,829 INFO     [task.py:422] Building contexts for leaderboard_ifeval on rank 0...
100%|██████████| 541/541 [00:00<00:00, 146432.53it/s]
2024-10-21:10:42:22,886 INFO     [task.py:422] Building contexts for leaderboard_musr_murder_mysteries on rank 0...
100%|██████████| 250/250 [00:00<00:00, 3876.81it/s]
2024-10-21:10:42:22,962 INFO     [task.py:422] Building contexts for leaderboard_musr_object_placements on rank 0...
100%|██████████| 256/256 [00:00<00:00, 3773.71it/s]
2024-10-21:10:42:23,041 INFO     [task.py:422] Building contexts for leaderboard_musr_team_allocation on rank 0...
100%|██████████| 250/250 [00:00<00:00, 3798.49it/s]
2024-10-21:10:42:23,118 INFO     [evaluator.py:489] Running loglikelihood requests

...
-> This is repeated. But, this issues is not happen in hf mode.

Hi! For loglikehood tasks, we are actually not generating anything but calculating the conditional logliklehood of a given answer string conditioned on some context (but vllm requires at least one output token, but we don't actually use it). I guess you can check if the results are similar to HF, to check if the results are correct. (also optionally you can use a chat template for your instruct model with apply_chat_template=True)


Add freehan and dnardo as wg member

Adding Minhan Xia and Daniel Nardo as wg member

@freehan good deal. Which organization are these respective members representing?

@dnardo and I are on the kubernetes networking team at Google.

	Thank you.

On Fri, Apr 7, 2017 at 6:38 PM -0500, "Minhan Xia"<EMAIL_ADDRESS>wrote:
@dnardo and I are on the kubernetes networking team at Google.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Acc test lockability

PR description
Just SpotlessApply changes.
Fixed Issue(s)

This branch was merged into the circle ci branch. As such, all the changes here should now be integrated into master.

So, is this Pull Request no longer valid? If so, we should close it and delete the branch
(by the way, we are accumulating a ton of old branches in our main fork!)

So, is this Pull Request no longer valid? If so, we should close it and delete the branch
(by the way, we are accumulating a ton of old branches in our main fork!)

I agree - this pull request has been superseded. I think it should be closed. Do you agree @Raghavendra-PegaSys

Agreed

@Raghavendra-PegaSys , I closed the PR but didn't delete the branch because I'm not sure if it has unmerged work in it. If it doesn't please remember to delete it.

Deleted


improve prompt input for yes

At the end of the user flow to complete an entry, faereld will ask:
Is this correct? (y/n) ::

This will require the user to press "y" and "y" only to accept the
input. This pull-request changes the display (and behaviour) to:
Is this correct? (Y/n) ::

I.e. uppercase "Y" is displayed. This is a common convention in many
command line tools to present user input like this to indicate what the
default value will be. In this case, if user simply presses enter the
data will be accepted and added to the database. This will save a
keystroke everytime a user inputs data to faereld. The characters "Y" and
"y" are also acceptable inputs to accept the data.

woohoo!


build(deps): CRA-v5, webpack-v5, router-v6, react-live-v3

Multiple dependencies updated.
React-scripts v5
Webpack v5
React-router v6
React-live v3

Coverage seemed to have dropped significantly. Did all tests pass?

Yes the coverage is odd. I will look into it. All the tests seems to be running though.

I found the issue. The dist folder is being included when running the tests and the coverage is reported on those instead of the src files. I need to find a way to exclude dist from resolution.
It's yet again something webpack related so I will put this PR in draft until I have resolved the issue.

@thaestrup finally got his working!  Ultimately it had to do with the dist files not being cleaned out. npm prepare appears to be running which builds the code, but it should have also been running npm clean.
In any case, fixed that issue, and added React 18 to the test matrix. Also pointed this at next and got it updated and resolved and conflicts. I'm going to merge this, then we can a release out (finally).


Add stream bodies to generated response models

This is debating if we want to add a Body field to the response envelope.  It would look something like this.
type FooResponse struct {
    Body io.ReadCloser
    HeaderOne *string
    RawResponse *http.Response
}

For cases where there's a response body and header, there is at present no Body field in the response envelope so you have to know to access it via the RawResponse.  If there's no header (just body) then there's no response envelope, just the *http.Response.

Per discussion, we don't want to do this as it introduces two places to get the same data.


Searching motionbuscard causes a blank screen and breaks further searches

Describe the bug
After searching for motionbuscard in the Mobility Database web interface, the application displays a blank screen. Subsequent searches return the same blank screen, indicating an unresolved error.
Steps/Code to Reproduce

Open mobilitydatabase.org.
Search for motionbuscard.
Return to the home page.
Attempt to search for anything else.

Expected Results
The search should yield results or indicate "no results" if none are found, with the ability to perform further searches.
Actual Results
A white (blank) page appears after the initial search for "motionbuscard," persisting for any subsequent search attempts. Errors are displayed in the console.
Additional notes
I believe that mdb-1920 (having motionbuscard provider) is causing issues because a lot of its fields are set as unknown in the catalog repo

The API returns location null, and the UI is not protected against null values.


Pass CI on main branch

Merging 9c74f77 introduced a TypeScript build error, which is
failing our CI build. This commit replaces the old formSubmitted
helper with the newer formSubmitEnded.
Attempt to resolve flaky tests
Tests involving successive this.goBack() and this.goForward() calls
are flaky due to timing issues:
× firefox on linux 5.11.0-1020-azure - FrameTests - navigating back then forward after pushing URL state from a turbo-frame[data-turbo-action=advance] restores the frames next contents (0.629s)
367
    StaleElementReference: [GET http://localhost:4444/wd/hub/session/01c3334a-e303-4d29-b392-1e4f1d4d7f5c/element/a97830bd-56a4-4b7a-86a0-9543cf697716/text] The element reference of <h2> is stale; either the element is no longer attached to the DOM, it is not in the current frame context, or the document has been refreshed
368
    For documentation on this error, please visit: https://www.seleniumhq.org/exceptions/stale_element_reference.html
369
    Build info: version: '3.141.59', revision: 'e82be7d358', time: '2018-11-14T08:25:53'
370
    System info: host: 'fv-az214-396', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'amd64', os.version: '5.11.0-1020-azure', java.version: '11.0.11'

In order to resolve that, this commit is more explicit with timing:
navigating forward and back will wait on turbo:load events to fire,
which won't occur until the page renders the cached version and
re-fetches the contents. Hopefully, this addresses the stale element
failures, since the referenced element won't be replaced while being
read from.
This commit introduces the this.getVisibleTextForSelector() helper to
combine the access-and-read action to a single Promise chain, instead of
sequential access and read commands.

@dhh this passes CI and resolves a test suite regression.

Solved via https://github.com/hotwired/turbo/pull/444


Check auth onEnter

Hi,
I try to implement check authorization with a router. I can see that before there was support for onEnter method which we can add to route
<Route name="main" onEnter={(route) => {console.log('check Auth - isLogin?')}}>
The current version doesn't support this, any ideas how I can implement autch check on Router level?
Thanks!

I was thinking about that too. Here's my idea (I'm new to react-native and to native programming, btw.)
How about setting up a view, let's call it "launch", which calls a log in view. On this view, there is a check if the token is in the storage. If true, the user is redirected to the "home" route, or whatever. On the log in view you could also, e.g., show a loading spinner before rendering the email and password form, etc.
I'm not sure if this is the right way to do it, would be glad to hear other/better ideas.

This might work if your app requires login in for all your components, a problem is when you have routes which can be accessed without login in.
Generic solution is when:

you have onEntry method which you can add to your route
be able to create nested routes and root will check auth for you

3.0 release should solve this problem using Switch renderer, check README.
However it has a little bit different API comparing with 2.x release. Anyway I'm closing this ticket because 2.x release will not be supported except critical bugs.


Environment Variable and Volume Mount

What I am trying to achieve: read a value from the KV, present it as an environment variable for the deployment, via a Kubernetes secret. The two secrets are a username & password
Exactly Similar to : https://github.com/Azure/secrets-store-csi-driver-provider-azure/issues/133
Where In  I am using

A cluster with Managed Identity & enabled with Pod Identity
Azure AD pod identity - Managed Mode

What works:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: "2048-deployment"
  namespace: my-app
  labels:
    aadpodidbinding: my-pod-identity
spec:
  replicas: 1
  selector:
    matchLabels:
      app: "2048"
  template:
    metadata:
      labels:
        app: "2048"
        aadpodidbinding: my-pod-identity
        version: v1
    spec:
      containers:
      - image: *****
        imagePullPolicy: Always
        name: "2048"
        ports:
        - containerPort: 80
        volumeMounts:
        - name: secrets-store-inline
          mountPath: "/mnt/secrets-store"
          readOnly: true
        - name: config-volume
          mountPath: /app/appsettings.json
          subPath: appsettings.json
      volumes:
        - name: secrets-store-inline
          csi:
            driver: secrets-store.csi.k8s.io
            readOnly: true
            volumeAttributes:
              secretProviderClass: "azure-csi-sample"
        - name: config-volume
          configMap:
            name: azkv-csi-int-test-configmap
      imagePullSecrets:
      - name: acrsecret

With this i can see the secrets Mounted on Pod

What does not work :
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx0
  labels:
    app: nginx
spec:
  replicas: 1
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
        aadpodidbinding: my-pod-identity
    spec:
      containers:
      - name: nginx
        image: *****
        env:
         - name: SQL_USER
           valueFrom:
             secretKeyRef:
               name: loginsecret1
               key: sql-user-id
        volumeMounts:
          - name: secrets-store-inline
            mountPath: "/mnt/secrets-store"
            readOnly: true
      volumes:
        - name: secrets-store-inline
          csi:
            driver: secrets-store.csi.k8s.io
            readOnly: true
            volumeAttributes:
              secretProviderClass: "azure-csi-sample"
      imagePullSecrets:
      - name: acrsecret

SecretproviderClass
apiVersion: secrets-store.csi.x-k8s.io/v1
kind: SecretProviderClass
metadata:
  name: azure-csi-sample
spec:
  provider: azure
  secretObjects:                                 # [OPTIONAL] SecretObject defines the desired state of synced K8s secret objects
  - secretName: loginsecret1
    type: Opaque
    data:
    - objectName: sql-user-id                     # name of the mounted content to sync. this could be the object name or object alias
      key: sql-user-id
  parameters:
    usePodIdentity: "true"
    keyvaultName: "aks-csi-azkv"
    objects: |
      array:
        - |
          objectName: sql-user-id
          objectType: secret
          objectVersion: ""
    tenantId: ""

I deploy my application pods don't start as they don't find the secret. The deployment yaml should mount the volume and create the secret as per the deployment.yaml
Events:
  Type     Reason     Age                  From               Message
  ----     ------     ----                 ----               -------
  Normal   Scheduled  35m                  default-scheduler  Successfully assigned my-app/nginx0-6f6b59dcff-b4fdf to aks-nodepool1-28764885-vmss000000
  Warning  Failed     33m (x12 over 35m)   kubelet            Error: secret "loginsecret1" not found

thanks for the troubleshooting guide... After installing the required clusterrole and clusterrolebinding via
secrets-store-csi-driver.syncSecret.enabled=true  the issue is ressolved.
Thanks

Issue ressolved after installing required clusterrole and clusterrolebinding


Fix #4504

Fixes issue https://github.com/kivy/kivy/issues/4504.

According to answers and comments at Determine how many decimal digits of a float are precise, there are at least 15 significant digits, (that is, the error always occurs on the 16th significant digit or more).
That means that significant decimals can will be less that 15. For example, 443*.2 results in a float 88.60000000000001 with 15 significant digits, 2 digits being ints and 13 being decimals. Subtracting 1e-12 fixes the issue the PR targets, in this case.
However, if the int significant digits are 4, then subtracting 1e-12 is ineffective since the error could occur on the 11 decimal.
Another way for 1e-12 to be ineffective is if we have multiple widgets, e.g. 10 widgets each having an error on 12 decimal, would accumulate with a total error on 11th decimal.
Conclusion:
We need a value higher than 1e-12. In case of 10000.xxxx then the subtraction has to occur on the 15-5=10th decimal or earlier, therefor i ll change the subtracted value to 1e-10.


how to set the install path of cpp-netlib?

The default install path is /usr/local. How to change the install path to /usr?
The message below is the details of cmake output, it seems that the install path was config successfully firstly, but changed in the next loop:
cmake ../cpp-netlib-0.11.0-final/ -DCMAKE_INSTALL_PREFIX=/usr -DCMAKE_BUILD_TYPE=Release -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++
INSTALL_CMAKE_DIR /usr/lib/CMake/cppnetlib
CMAKE_INSTALL_PREFIX /usr                       (####marked####)
-- Boost version: 1.56.0
-- Found the following Boost libraries:
--   unit_test_framework
--   system
--   regex
--   date_time
--   thread
--   filesystem
--   program_options
--   chrono
--   atomic
-- Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the system variable OPENSSL_ROOT_DIR (missing:  OPENSSL_LIBRARIES OPENSSL_INCLUDE_DIR)
INSTALL_CMAKE_DIR /usr/lib/CMake/cppnetlib
CMAKE_INSTALL_FULL_INCLUDEDIR /usr/include
-- Configuring done
You have changed variables that require your cache to be deleted.
Configure will be re-run and you may have to reset some variables.
The following variables have changed:
CMAKE_C_COMPILER= clang
CMAKE_CXX_COMPILER= clang++
-- The C compiler identification is Clang 3.4.2
-- The CXX compiler identification is Clang 3.4.2
-- Check for working C compiler: /home/hanh/workspace/tools/clang/clang+llvm-3.4.2-x86_64-fedora20/bin/clang
-- Check for working C compiler: /home/hanh/workspace/tools/clang/clang+llvm-3.4.2-x86_64-fedora20/bin/clang -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /home/hanh/workspace/tools/clang/clang+llvm-3.4.2-x86_64-fedora20/bin/clang++
-- Check for working CXX compiler: /home/hanh/workspace/tools/clang/clang+llvm-3.4.2-x86_64-fedora20/bin/clang++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
INSTALL_CMAKE_DIR /usr/local/lib/CMake/cppnetlib
CMAKE_INSTALL_PREFIX /usr/local                              (####marked####)
-- Boost version: 1.56.0
-- Found the following Boost libraries:
--   unit_test_framework
--   system
--   regex
--   date_time
--   thread
--   filesystem
--   program_options
--   chrono
--   atomic

Huh, that's interesting. @infinity0 -- have you ever tried doing anything here?

Is this still a problem in 0.11.2 @tianchao-haohan?


Add New Brunswick shields

Adds shields for New Brunswick. The work here was mostly already done for me, as this mostly square province was made even more square when it got distorted for the provincial highway shield. I kept the original geometry of New Blobswick almost intact, but removed some islands in the northeast and some inlets in the south.

New Brunswick's three shields have identical designs with colour palette swaps. One- and two-digit routes are Arterial highways (CA:NB:primary), which have green backgrounds. The 100-series is Collector highways (CA:NB:secondary), with blue shields. And the remainder of routes, 200-999, are Local highways (CA:NB:tertiary), with black shields. The designs introduced here adjust the colours to match the existing palette.

Screenshots look good, and I'm fine with the 3 SVGs approach.  We can always go back later and make improvements to the recoloring code to obviate the need for that duplication, but at this point, a too-large sprite sheet is not a significant issue.  I have not yet run this code to verify functionality.


Improvements

Here are some improvements you can look into:
https://github.com/jordanhandy/obsidian-cloudinary-uploader/blob/2cab25efd73dc752de314b17af8408e9314dbf55/src/main.ts#L42
This logic seems faulty - if files.length is equal to 0 then it shouldn't be able to access files[0].
https://github.com/jordanhandy/obsidian-cloudinary-uploader/blob/2cab25efd73dc752de314b17af8408e9314dbf55/src/main.ts#L47
Need to move this up one line.
https://github.com/jordanhandy/obsidian-cloudinary-uploader/blob/2cab25efd73dc752de314b17af8408e9314dbf55/src/main.ts#L79
You don't want to do this - because editor-paste runs for every paste event. Doing so will break regular pasting text.
https://github.com/jordanhandy/obsidian-cloudinary-uploader/blob/2cab25efd73dc752de314b17af8408e9314dbf55/src/main.ts#L90
Instead of using getValue() and split('\n'), you should use getLine() and lineCount.

Recommendations should be implemented with 277ddf389dd4a188ac41cab3f1a297856f4ebdc7


-(enable|disable)-only-one-dependency-file causes only the first frontend job to emit a real dependency file.

Fixes a problem with incremental performance: Since each frontend job contains the same info in its emitted make-style dependencies file, the build system slows down the build when the it reads all the redundant info. Disabled-by-default, this feature causes only one frontend job to emit the real info, the rest create dummy files.
rdar://57882772

@swift-ci please test

Explanation: A small fix to .rst formatting that prevents docs from building under sphinx.
Scope of issue: Build breakage when sphinx is present.
Origination: Typo when written, as near as I can tell.
Risk: Minimal, because off by default.
Reviewed by: Robert Widman
Testing: Normal regression tests.
Radar: rdar://57882772

@swift-ci please nominate

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 3ab6e0ef48b153f1cfa67df0af21dc4130b4b4ab

Build failed
Swift Test OS X Platform
Git Sha - 3ab6e0ef48b153f1cfa67df0af21dc4130b4b4ab

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 6db25e618f71a379bc544b74e29a7d355ea36f69

Build failed
Swift Test OS X Platform
Git Sha - 6db25e618f71a379bc544b74e29a7d355ea36f69

Ignore those failure notices, CI sometimes emits them before it reschedules your job onto a different builder.

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 523096c582df0f113c49ae106e3ddb638adc2b3b

Build failed
Swift Test OS X Platform
Git Sha - 523096c582df0f113c49ae106e3ddb638adc2b3b


F4discovery

STM32F407VGT6 (F4 discovery) (Id 0x0413) has 192Ko of RAM

@lhondareyte Were you able to make the corrections I raised previously? I’ll merge this PR once those corrections are made.

The issue of missing MCU IDs and revision IDs for STM32F407xx devices has been resolved with commit 7359a0310a7e73ba5f4f49d0177c4f3fcdca3852 in a different way than this PR. Specifically, it separates out the errata from the standard IDs, and also adds all the currently known revisions of STM32F407xx devices.
The target.mk diff for gcc-arm-embedded on FreeBSD was done in PR #10.
Closing this PR.


[C++] Move fsspec FileSystem to a separate module

Describe the enhancement requested
https://github.com/apache/arrow/issues/38309 adds the capability to register FileSystems not defined in libarrow. Python objects which satisfy fsspec are suitable for registration as such a filesystem.
Component(s)
C++, Python

But is it actually useful to make it a separate C++ DLL? The C++ code footprint for it is probably minimal.


fix sniff http ipv6 address

What are you trying to do? Can you give an example?

IPv6 address will not be handled correctly when sniffing destOverride be enabled,
For example,  when visit [2401:1bc0:51f0:ec08::1]:80
debug log
2018/12/09 20:37:42 [Info] [693541750] v2ray.com/core/proxy/socks: TCP Connect request to tcp:[2401:1bc0:51f0:ec08::1]:80
2018/12/09 20:37:42 [Info] [693541750] v2ray.com/core/app/dispatcher: sniffed domain: [2401
2018/12/09 20:37:42 [Debug] v2ray.com/core/app/dns: querying DNS for: [2401.

[2401:8bc0:51f0:ec08::1]:80 -> [2401
should be [2401:8bc0:51f0:ec08::1]:80 -> [2401:8bc0:51f0:ec08::1]

It would be better to use net.SplitHostPort then.


ImportError: No module named core.config

Torch v0.5, requirements satisfied with pip install -r requirements.txt and trying to run valid.py with the suggested model/config for Coco
python pose_estimation/valid.py --cfg experiments/mpii/resnet50/256x256_d256x3_adam_lr1e-3.yaml     --flip-test     --model-file models/pytorch/pose_coco/pose_resnet_50_256x256.pth.tar
Traceback (most recent call last):
  File "pose_estimation/valid.py", line 25, in <module>
    from core.config import config
ImportError: No module named core.config

I saw in the paper that this was based on the latest Mask R-CNN and a similar issue was raised here so is the Facebook detectron required to run this code?

you can check your sys.path, usually it is because that your sys path has another module also named as core

@ss32 you should use python3.

@luohuan2uestc that did it.  Solved.

Could you please tell me how to solve it.


features: check for owner_active? instead of rollout

This adds support for feature flagging organizations and not just users.

Let's fix the specs! :-D


Update logo requirements

Update logo requirements, include recommendations for optimizing file size.

[x] Follows the developer docs style guide.
[x] Follows the best practices and guidelines.
[x] New documentation pages include document tags.
[x] New documentation pages include SEO keywords.
[x] New documents are in the .mdx file format to support the previous two components.
[x] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[x] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

Could you elaborate on why you want to remove WebP? It’s actually the smallest format of the three. If anything, I’d consider removing PNG instead and adding a max file size (KB) requirement to ensure consistency.
That said, keep in mind that not all tools export WebP natively, which might pose challenges for some contributors. Also, SVGs can be tricky since their size depends heavily on the complexity of the logo.

Could you elaborate on why you want to remove WebP? It’s actually the smallest format of the three.
My bad, didn't intend to remove webp, adding it back.

Since it's regarding logos, in most cases SVG would be a smaller file size and also scale when used in various sizes.

@sea-snake I would typically agree but we had a few cases where the SVGs were huge.
My proposal would be:
screenshots: jpeg / webP (specify the PX size and a max KB size maybe max 300?)
logo: webP or svg (50kb max?)

@sea-snake I would typically agree but we had a few cases where the SVGs were huge.
logo: webP or svg (50kb max?)
50kb is already too much right now with the II canister WASM size limitations, we're aiming for 4kb or less with max 10kb.

I added recommended online tooling for file size optimizations in this PR that saves roughly 70% in most cases, compared to explicit file size limits this seems like a more proactive approach to enable users to keep file sizes optimal.

@sea-snake I don't feel strongly about it. LGTM


make it better.

let user choose how many processes they want to run, also tell them what the output of crossystem wpsw_cur even means.
also remove useless things (just dont clear terminal...)


White Blank Page in home page and admin/products page after adding product

I got an error with white blank page after adding product, this error showing in home page (http://localhost:3000) and product administration (http://localhost:3000/admin/products), but other page showing correctly.
This console error log in homepage (http://localhost:3000)
Uncaught TypeError: Cannot read property 'replace' of undefined
    at Object.formatCurrency (helper.js?9e86:36)
    at FormattedCurrency (itemPrice.js?a117:24)
    at mountIndeterminateComponent (react-dom.development.js?cada:8032)
    at beginWork (react-dom.development.js?cada:8221)
    at performUnitOfWork (react-dom.development.js?cada:10224)
    at workLoop (react-dom.development.js?cada:10288)
    at HTMLUnknownElement.callCallback (react-dom.development.js?cada:542)
    at Object.invokeGuardedCallbackDev (react-dom.development.js?cada:581)
    at invokeGuardedCallback (react-dom.development.js?cada:438)
    at renderRoot (react-dom.development.js?cada:10366)
formatCurrency @ helper.js?9e86:36
FormattedCurrency @ itemPrice.js?a117:24
mountIndeterminateComponent @ react-dom.development.js?cada:8032
beginWork @ react-dom.development.js?cada:8221
performUnitOfWork @ react-dom.development.js?cada:10224
workLoop @ react-dom.development.js?cada:10288
callCallback @ react-dom.development.js?cada:542
invokeGuardedCallbackDev @ react-dom.development.js?cada:581
invokeGuardedCallback @ react-dom.development.js?cada:438
renderRoot @ react-dom.development.js?cada:10366
performWorkOnRoot @ react-dom.development.js?cada:11014
performWork @ react-dom.development.js?cada:10967
requestWork @ react-dom.development.js?cada:10878
scheduleWorkImpl @ react-dom.development.js?cada:10732
scheduleWork @ react-dom.development.js?cada:10689
enqueueSetState @ react-dom.development.js?cada:6212
Component.setState @ react.development.js?99ee:237
(anonymous) @ custom.js?78cb:70
Promise resolved (async)
CustomProducts._this.fetchProducts @ custom.js?78cb:66
componentDidMount @ custom.js?78cb:85
commitLifeCycles @ react-dom.development.js?cada:8770
commitAllLifeCycles @ react-dom.development.js?cada:9946
callCallback @ react-dom.development.js?cada:542
invokeGuardedCallbackDev @ react-dom.development.js?cada:581
invokeGuardedCallback @ react-dom.development.js?cada:438
commitRoot @ react-dom.development.js?cada:10050
performWorkOnRoot @ react-dom.development.js?cada:11017
performWork @ react-dom.development.js?cada:10967
requestWork @ react-dom.development.js?cada:10878
scheduleWorkImpl @ react-dom.development.js?cada:10732
scheduleWork @ react-dom.development.js?cada:10689
enqueueSetState @ react-dom.development.js?cada:6212
Component.setState @ react.development.js?99ee:237
onStateChange @ connectAdvanced.js?4805:205
dispatch @ createStore.js?6413:173
(anonymous) @ index.js?e2e7:14
dispatch @ applyMiddleware.js?be0b:35
(anonymous) @ actions.js?0e6b:386
Promise resolved (async)
(anonymous) @ actions.js?0e6b:376
(anonymous) @ index.js?e2e7:11
setCurrentPage @ app.js?5c8e:74
componentWillReceiveProps @ app.js?5c8e:26
callComponentWillReceiveProps @ react-dom.development.js?cada:6389
updateClassInstance @ react-dom.development.js?cada:6575
updateClassComponent @ react-dom.development.js?cada:7848
beginWork @ react-dom.development.js?cada:8225
performUnitOfWork @ react-dom.development.js?cada:10224
workLoop @ react-dom.development.js?cada:10288
callCallback @ react-dom.development.js?cada:542
invokeGuardedCallbackDev @ react-dom.development.js?cada:581
invokeGuardedCallback @ react-dom.development.js?cada:438
renderRoot @ react-dom.development.js?cada:10366
performWorkOnRoot @ react-dom.development.js?cada:11014
performWork @ react-dom.development.js?cada:10967
batchedUpdates @ react-dom.development.js?cada:11086
batchedUpdates @ react-dom.development.js?cada:2330
dispatchEvent @ react-dom.development.js?cada:3421

react-dom.development.js?cada:9747 The above error occurred in the <FormattedCurrency> component:
    in FormattedCurrency (created by ItemPrice)
    in span (created by ItemPrice)
    in div (created by ItemPrice)
    in ItemPrice (created by Item)
    in div (created by Item)
    in a (created by Link)
    in Link (created by Route)
    in Route (created by NavLink)
    in NavLink (created by Item)
    in div (created by Item)
    in Item (created by ProductList)
    in div (created by ProductList)
    in ProductList (created by CustomProducts)
    in CustomProducts (created by IndexContainer)
    in div (created by IndexContainer)
    in section (created by IndexContainer)
    in IndexContainer (created by Connect(IndexContainer))
    in Connect(IndexContainer) (created by Route)
    in Route (created by withRouter(Connect(IndexContainer)))
    in withRouter(Connect(IndexContainer)) (created by SwitchContainers)
    in SwitchContainers (created by Connect(SwitchContainers))
    in Connect(SwitchContainers) (created by Route)
    in Route (created by App)
    in SharedContainer (created by Connect(SharedContainer))
    in Connect(SharedContainer) (created by Route)
    in Route (created by withRouter(Connect(SharedContainer)))
    in withRouter(Connect(SharedContainer)) (created by App)
    in App
    in Router (created by BrowserRouter)
    in BrowserRouter
    in Provider

Consider adding an error boundary to your tree to customize error handling behavior.
Visit https://fb.me/react-error-boundaries to learn more about error boundaries.

This console error log in product administration (http://localhost:3000/admin/products
Uncaught TypeError: Cannot read property 'replace' of undefined
    at Object.formatCurrency (helper.js?65d1:20)
    at ItemPrice (item.js?9569:22)
    at mountIndeterminateComponent (react-dom.development.js?cada:8032)
    at beginWork (react-dom.development.js?cada:8221)
    at performUnitOfWork (react-dom.development.js?cada:10224)
    at workLoop (react-dom.development.js?cada:10288)
    at HTMLUnknownElement.callCallback (react-dom.development.js?cada:542)
    at Object.invokeGuardedCallbackDev (react-dom.development.js?cada:581)
    at invokeGuardedCallback (react-dom.development.js?cada:438)
    at renderRoot (react-dom.development.js?cada:10366)
formatCurrency @ helper.js?65d1:20
ItemPrice @ item.js?9569:22
mountIndeterminateComponent @ react-dom.development.js?cada:8032
beginWork @ react-dom.development.js?cada:8221
performUnitOfWork @ react-dom.development.js?cada:10224
workLoop @ react-dom.development.js?cada:10288
callCallback @ react-dom.development.js?cada:542
invokeGuardedCallbackDev @ react-dom.development.js?cada:581
invokeGuardedCallback @ react-dom.development.js?cada:438
renderRoot @ react-dom.development.js?cada:10366
performWorkOnRoot @ react-dom.development.js?cada:11014
performWork @ react-dom.development.js?cada:10967
requestWork @ react-dom.development.js?cada:10878
scheduleWorkImpl @ react-dom.development.js?cada:10732
scheduleWork @ react-dom.development.js?cada:10689
enqueueSetState @ react-dom.development.js?cada:6212
Component.setState @ react.development.js?99ee:237
onStateChange @ connectAdvanced.js?4805:205
dispatch @ createStore.js?6413:173
(anonymous) @ index.js?e2e7:14
dispatch @ applyMiddleware.js?be0b:35
(anonymous) @ actions.js?c740:218
Promise resolved (async)
(anonymous) @ actions.js?c740:216
(anonymous) @ index.js?e2e7:11
onLoad @ index.js?b852:20
componentDidMount @ list.js?5f5e:18
commitLifeCycles @ react-dom.development.js?cada:8770
commitAllLifeCycles @ react-dom.development.js?cada:9946
callCallback @ react-dom.development.js?cada:542
invokeGuardedCallbackDev @ react-dom.development.js?cada:581
invokeGuardedCallback @ react-dom.development.js?cada:438
commitRoot @ react-dom.development.js?cada:10050
performWorkOnRoot @ react-dom.development.js?cada:11017
performWork @ react-dom.development.js?cada:10967
requestWork @ react-dom.development.js?cada:10878
scheduleWorkImpl @ react-dom.development.js?cada:10732
scheduleWork @ react-dom.development.js?cada:10689
scheduleTopLevelUpdate @ react-dom.development.js?cada:11193
updateContainer @ react-dom.development.js?cada:11231
(anonymous) @ react-dom.development.js?cada:15226
unbatchedUpdates @ react-dom.development.js?cada:11102
renderSubtreeIntoContainer @ react-dom.development.js?cada:15225
render @ react-dom.development.js?cada:15290
(anonymous) @ index.js?75f6:23
(anonymous) @ app-903cbf6cf509543f5099.js:2963
__webpack_require__ @ vendor-b900ab5d221f5037eb80.js:56
webpackJsonpCallback @ vendor-b900ab5d221f5037eb80.js:27
(anonymous) @ app-903cbf6cf509543f5099.js:2
react-dom.development.js?cada:9747 The above error occurred in the <ItemPrice> component:
    in ItemPrice (created by ProductItem)
    in div (created by ProductItem)
    in div (created by ProductItem)
    in div (created by ProductItem)
    in ProductItem (created by ProductsList)
    in div (created by List)
    in List (created by ProductsList)
    in div (created by ProductsList)
    in ProductsList (created by Connect(ProductsList))
    in Connect(ProductsList) (created by Route)
    in Route (created by withRouter(Connect(ProductsList)))
    in withRouter(Connect(ProductsList))
    in div
    in div
    in Unknown (created by Route)
    in Route (created by App)
    in Switch (created by App)
    in div (created by App)
    in div (created by App)
    in MuiThemeProvider (created by App)
    in Router (created by BrowserRouter)
    in BrowserRouter (created by App)
    in App
    in Provider

Consider adding an error boundary to your tree to customize error handling behavior.
Visit https://fb.me/react-error-boundaries to learn more about error boundaries.

Hi rizalibnu,
in Settings -> General Settings
ADD   currency formatting     USD or EUR


Build error in Defold 1.2.186

Build Errors when building in Windows 10. I don’t actually use imgui in this project, I just have it fetched as a dependency.
/imgui/ext.manifest
	clang-10: warning: treating 'c' input as 'c++' when in C++ mode, this behavior is deprecated [-Wdeprecated]
	lld-link: error: undefined symbol: __cpuid

>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:1091
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void * __cdecl stbi__load_main(struct stbi__context *, int *, int *, int *, int, struct stbi__result_info *, int))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:1091
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void * __cdecl stbi__load_main(struct stbi__context *, int *, int *, int *, int, struct stbi__result_info *, int))
	lld-link: error: undefined symbol: _mm_setr_epi16
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2541
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2542
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2543
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2544
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2545
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2546
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2547
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2548
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
	lld-link: error: undefined symbol: _mm_set1_epi32
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2551
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2552
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
	lld-link: error: undefined symbol: _mm_load_si128
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2555
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
>>> referenced by /tmp/job5884567677308494677/upload/imgui/src/imgui/stb_image.h:2556
>>>               DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))

(etc)

https://www.sizeofvoid.com/2018/12/02/get-the-default-include-paths-from-clang/

Closed by https://github.com/britzl/extension-imgui/pull/19

I've fetched the latest version of this repo and I'm still getting issues in 1.2.187. Builds fine in 1.2.184. Have I missed something or is the issue still active?
clang-10: warning: treating 'c' input as 'c++' when in C++ mode, this behavior is deprecated [-Wdeprecated]
`lld-link: error: undefined symbol: __cpuid

referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:1148
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void * __cdecl stbi__load_main(struct stbi__context *, int *, int *, int *, int, struct stbi__result_info *, int))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:1148
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void * __cdecl stbi__load_main(struct stbi__context *, int *, int *, int *, int, struct stbi__result_info *, int))`

`lld-link: error: undefined symbol: _mm_setr_epi16

referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2581
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2582
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2583
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2584
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2585
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2586
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2587
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))
referenced by /tmp/job8480497724863692534/upload/imgui/src/stb/stb_image.h:2588
DefoldImGui_1.lib(extension_imgui.cpp_0.o):(void __cdecl stbi__idct_simd(unsigned char *, int, short *const))`

etc etc

Excellent! Closing the issue.


Big problems with GAN and keras 2.0 because of new batchnorm

Hi,
I work with Conditional GANs and in the last days I'm working with Conditional Wasserstein GAN keras implementation, starting from this code:
https://github.com/tdeboissiere/DeepLearningImplementations/tree/master/WassersteinGAN
the problem is that Conditional WGAN (and GANs and Conditional GANs too) don't produce correct output when batchnorm is not set on batchnorm_mode=2
I tried mode=0 before keras 2.0 and the generated images by the generator become indipendent by the noise, so I do not get any variability.
So, in keras 2.0 I cannot work with WGAN/GAN anymore, being mode=2 removed by new batchnorm implementation.
I think this is a very big issue and I hope the old batchnorm modes will be supported again, otherwise keras would not work on most of recent generative models.
Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[ X] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[ X] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ X] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ X] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Lots of issues around this. The real question is, what should BN do for a GAN?

Batchnorm real and generated together?
Batchnorm each separately?
Batchnorm the real, and use those stats to batchnorm generated?

There are a lot of ways I could imagine using BN in a GAN, and I can't be sure about the best until someone tests them out and writes a paper.
Cheers

I think the several options you listed could be referred to the discriminator, and in fact we still don't know the appropriate way to use BN on it - I agree with your considerations.
Anyway, the problem here refers to the generator - its BN influence heavily the generated images. And beside the theoretical analysies, still remains the main issue: in the new keras 2.0 have been removed a BN mode that is pivotal for the correctness of GAN models, and this should be addressed as soon as possible if we wanna see a broad adoption of keras 2.0.

So you're having issues using BN in the generator? Using BN in the discriminator has tons of considerations but in the generator should be fine. Do you have something simple to demo the problem?

I have done some experiments about this issue:

The problem is not on discriminator BatchNorm, but in the generator Batchnorm: the generative model is working nicely both with batchnorm mode=0 and mode=2 on Discriminator. When I try mode=0 on generator, i get garbage as output.

This issue happens both on Wasserstein GAN and conditional Wasserstein GAN on a simple dataset generation task, that is on MNIST dataset.

I'm confident it happens also on regular GAN.

Step to easily reproduce the problem:

Use keras 1.2, the code need to be updated on few parts in order to run on keras 2.0 (still the problem happens on keras 2.0 as well)

clone the github repo tdeboissiere/DeepLearningImplementations:
git clone https://github.com/tdeboissiere/DeepLearningImplementations.git

modify the file DeepLearningImplementations/WassersteinGAN/src/utils/data_utils.py adding those lines at the very beginning of the file:
import matplotlib matplotlib.use('Agg')
in order to let the code be able to save some samples of the current training on disk.

run the main.py file located into DeepLearningImplementations/WassersteinGAN/src/model/main.py with default parameters (bn_mode is 2)
python main.py

look at the nice output after some epochs (10 is enough to see the issue) in file DeepLearningImplementations/WassersteinGAN/figures/current_batch.png
The first two rows are the generated MNIST images at the current training step (using random input noise), the other two are some real MNIST images taken at random from the target dataset.

-repeat the experiments with bn_mode=0:
python main.py bn_mode=0
and you will see that the output will be garbage: in particular the 8 output images are always the same (beside a very very little variation), while each of those should be different - maybe it could be explained as a total GAN collapse.

@engharat Do you mean the sample-wise normalization?

Exactly! Feature-wise normalization seems to be crucial for correct GAN output at test time

Has this issue been resolved?  I ran into exactly the same problem after using Keras 2.0

I suppose batchnorm has not received any update, so the problem is not solved.
Anyway, It can be solved pretty easily by cloning BatchNorm class into a new class, like BatchNormGAN or BatchNormMode2, and modifying the following code:
        return K.in_train_phase(normed,                                 normalize_inference,                                 training=training)
to:
        return K.in_train_phase(normed,                                 normalize_inference,                                 training=True)
so the batchnorm behaviour should be the same of the old bn with mode = 2.
At least, it worked for me!

Thank you for the quick response. After some digging, it seems the issue is with sharing discriminator in GAN and  model.trainable does not work as expected.  My loss curve also suggests that generative loss does not decrease. I tried several ways suggested elsewhere but didn't work.

I solved the problem of .trainable with the following function:def make_trainable(net, value):     net.trainable = value     for l in net.layers:         l.trainable = value
Where net is the model that you wanna freeze, and value is True/False if you wanna make it trainable/not.

Here is my loss curve.  I tried pretty much everything and still not able to converge.

We are going to off-topic, anyway: I tried with my proposed batchnorm and my proposed .trainable function on WGAN and it worked flawlessly. The loss doesn't not have to converge to zero, i think 0.70 and 0.60 are quite reasonable. I invite you to try again,to look at produced output, and if anything fails to look at some errors in code somewhere else, because this approach is actually working for me (and for several GAN researchers I suppose... ;)

Appreciate your response. My problem here is that in pre-training mode, the discriminator does not function as expected. The accuracy is only roughly 50%. There is something wrong with the architecture.

Finally,  identified the problem as model insufficiency and unnecessary BatchNormalization. I removed BatchNormalization, which turned out unnecessary anyway.  Here is result from WGAN-GP.  I did not use make_trainable. Simply set/reset tradable property. Very nice and fast convergence with GP.


Backward pass algo description correction

I think it should be this way. Correct me if I am wrong :P

You're correct - thanks!


Make ci_infrastructure.yml jobs execute in series

Describe the task
There are some github actions that should be in series but are currently in parallel
Acceptance Criteria

[ ] Checkin Terraform Plan checks are in run in series

Looks like it was previously running in series but commented out for some reason. Code is in ci_infrastructure.yml

Did this as part of #421 investigation as terraform changes were needed and running in parallel caused a failure due to inavailability of terraform cloud.


Create new page on CC wiki

Create new page on CC wiki where basic information about the internationalization project, its funding, our goals and time table, and additional information about reversionary rights around the world that is not yet part of the new tool, etc. will be set out. Text and layout to be sent by email to @robmyers.

completed by Rob, off main website legal projects page.


VictimCrasherCompact Overlay text are misaligned when values are higher than 10

this image should explain everything https://i.imgur.com/bxyegdo.png
this also what it looks like when playing in-game: https://i.imgur.com/FRIlhFY.png

this also what it looks like when playing in-game: i.imgur.com/FRIlhFY.png

What mods do you have enabled when playing? just HDDT?

this image should explain everything i.imgur.com/bxyegdo.png

I'll have to check this myself later today

What mods do you have enabled when playing? just HDDT?

Generally, +DT can make the values go higher than 10. I don't think it applies to +HR also

Yes, I'm aware of that :). I was asking because of this image with invalid URL being displayed

Yes, I'm aware of that :). I was asking because of this image with invalid URL being displayed

Ah, that was with +Auto mod, it didn't show because the keyword written by default is "AT". I fixed it by changing it to "AU"


Missing support for special surfaces?

I can't find bindings for surfaces for various image types, like PNG or PDF. It would be very helpful to be able to render to image, as with cairo_surface_write_to_png. Are these bindings expected to be supported anytime soon? (I can try to lend a hand, but I'll need to get a rubber stamp from my employer first.)

It all depends what state is the gtk-rs binding generator.
@gkoz: Any info on this ?

All surfaces types should be supported eventually. Sadly the generator won't be of help here, there's no gobject metadata.
As to difficulty, the biggest challenge to implementing various backend bindings might be detecting them in the build script.

@dstu FYI, I'm working on PNG support, see #60

Thanks! This will be useful for headless use, like testing the rendering of custom widget components, capturing visualizations produced by code that also produces an interactive widget, and (maybe eventually?) Web backends.

I believe this is fixed and can be closed. See, among others:

https://gtk-rs.org/docs/cairo/struct.ImageSurface.html#method.write_to_png
https://gtk-rs.org/docs/cairo/pdf/struct.Writer.html

Indeed!


Problem calling debugModule more than once

The function debugModule uses the global variable eventCounter to make sure everything stays in sync.
If debugModule is called on two different modules, they will share the same eventCounter, ensuring that both will mess up in fantastic and creative ways.

No idea if this still makes sense, so going to close this since only past me knew what I was talking about.


Remove integration from state if it was removed manually (#186)

Description
Currently, if integration is removed manually, terraform plan fails with "no integrations exist in specified model". This PR  changes the behavior to remove it from the state instead.
Type of change
Please mark if proceeds.

[ ] New resource (a new resource in the schema)
[ ] Changed resource (changes in an existing resource)
[ ] Logic changes in resources (the API interaction with Juju has been changed)
[ ] Test changes (one or several tests have been changed)
[X ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Other (describe)

Environment

Juju controller version: 2.9.42
Terraform version: 1.4.4

QA steps
Manual QA steps should be done to test this PR.
The same that was reported in issue https://github.com/juju/terraform-provider-juju/issues/186
Additional notes
It might be worth doing this for all the resources and also creating a specific error for that instead of checking the error message.

The situation reported in #186 cannot be reproduced after #189 was landed. This is a good check.
Thanks for the PR!!

@amandahla I'm OK with the changes. Failing tests are not due to these changes. Ready to merge if you're OK with the PR.

Fix #186
Fix #187

@amandahla I'm OK with the changes. Failing tests are not due to these changes. Ready to merge if you're OK with the PR.

Awesome! Thanks.


Fixed type "lauch" to "launch"

(You can replace all of this text with your description.)
Before creating your pull request, please check your content against these quality criteria:

Did you consider search engine optimization (SEO) when you chose the title in the metadata section and the H1 heading (i.e. the displayed title that starts with a single #)?
For new articles, did you add it to the table of contents?
Did you update the "ms.date" metadata for new or significantly updated articles?
Are technical terms and concepts introduced and explained, and are acronyms spelled out on first mention?
Should this page be linked to from other pages or Microsoft web sites?

For more information about creating content for docs.microsoft.com, see the contributor guide at https://docs.microsoft.com/contribute/.

@manu-p-1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off @manu-p-1 Thanks!


add repr for ChannelTuple

Which is different from the repr of the ChannelList

[x] Add a changelog entry

bors merge

bors merge


rules code question

in rule doc , input_type permitted values ( string num bool ).  don't understand   this code.


[ Loss Scale ]  Enable to define different type of Variable and Gradient for Loss Scale

In this PR
This PR enables separate definition of variable and gradient of Weight. It's because we have to support different type considering Mixed Precision Training or Quantization.
For example, Variable should be FP16 and Gradient should be FP16 for FP16-FP16 Mixed Precision.
Or Variable should support BCQ3 and Gradient FP16 for BCQ3-FP16 QAT.
Therefore, we need to set update the Variable weight as it is requested.. and for Gradient, we have to set same type of Activation.
This PR enables to set different type between Variable and Gradient.
Commits to be reviewed in this PR
[Property] Add loss scale property
It add loss scale property as model common property.
Signed-off-by: Jiho Chu<EMAIL_ADDRESS>
[ Weight ] Add Loss Scale factor in Weight
This PR enables the loss scale factor in Weight.
. Change the WeightSpec to incluide the loss factor
. Add LossScaleForMixed Property as an layer common property, so that
it can set the scale factor in initContext.
. Add Loss Scale in initContext
. Set the LossScaleForMixed Property when there is LossScale Model
Property
Resolves:
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>
[ Weight ] split variable dim and grad dim to set separately
This PR split the Variable and Gradient Dim in Var_Grad and Weight.
By this way we can set different Variable Type and Gradient in Wegiht.
. add dim_g for gradient in WeightSpec.
. manager need to update to support WeightSpec.
. Create Tensors according to dim_v and dim_g
. Create Weight chaged in Weight.h
Resolves:
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>
[ NEURALNET ] change the loss scale property to Rigid Property
Loss Scale is more like Rigid Property of model, rather than flexible
property.
Resolves:
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2562. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Music titles in other languages

Was wondering if it would be possible to keep a version of xiv_bgm.csv for other languages (or at least, Japanese) for our non-Anglophone friends.
And for me, as I would like to be able to have both at the same time, à la XIVAlexander's merging of UI and dialogue languages to show double languages (I can merge the files myself if need be but since there's no list in Japanese that I know of, it's not possible right now).

Please note that this repository for the plugin is not being maintained, and this issues page is not monitored. Issues should be brought up here: https://github.com/lmcintyre/OrchestrionPlugin
I am not fluent in anything other than English, and compiling track lists in other languages wouldn't be something I'd be able to proofread or verify, so it's not on my list at the moment.


R.swift doesn't generate resources for xibs and storyboards in SPM

I need to transfer all resources to SPM. Strings and assets of images are fine, but xibs and storyboards are a problem. At the same time, information about them appears in the R.generated file in the form of messages:
"// This _R.nib struct is generated, and contains static references to 50 nibs."
"/// This _R.storyboard struct is generated, and contains static references to 17 storyboards."
But opposite the lines generated for xibs and storyboards there is an error like "Cannot find type 'SomeView' in scope", "Cannot find type 'SomeViewController' in scope"
and accordingly, due to these errors, R.nib, R.Storyboard and R.reuseidentifier are not available.
Please advise what can be done in such a situation.

I apologize, I figured out the problem myself. It is likely that the library works with storyboard with swift-file or xib with swift-file file pairs. If you transfer them in pairs to SPM, then resources are generated.


Should the result of assignment, +=, -=, ..., ++, and -- be a left-value?

Or more precisely, should e++ (or e = 42, etc..) have the exact same type as e, or the right-value type of e.
Currently it is the later, which matches C/C++ as far as I can tell.
Moving to the former would be trivial and allow things like x++ ++;, which makes sense to me, but it would also allow things like x++ = 4; which is a lot weirder (but would be well-defined, as we have an explicit left-to-right order of execution).
In any case, this is a fairly trivial change, that can easily be made later.

We've agreed to make these right-values, and reject the chaining of things like ++.


Middle-click title bar to move windows between zones

DisplayFusion provides a very useful functionality where you can move windows between zones simply by middle clicking the Titlebar. This will move a window across to the next zone. I believe this is a good feature for rapidly moving windows between different zones.

Yes - that is correct.


-10 bytes

Instead of having a separate wtime and btime variable, use a single variable time_left.
Remove variables found and allocated_time.
-10 bytes
No functional change

Thanks.


RequirementMachine: Conditional requirement inference

If a type parameter is subject to both a conformance requirement
and a concrete type requirement, the concrete type might conform
conditionally.
In this case, introduce new requirements to satisfy the conditional
conformance.
Since this can add new hitherto-unseen protocols to the rewrite
system, restrict this feature to top-level generic signatures, and
not protocol requirement signatures. Allowing this to occur in
protocol requirement signatures would change the connectivity of
the protocol dependency graph (and hence the connected components)
during completion, which would be a major complication in the
design. The GSB already enforces this restriction.

@swift-ci Please smoke test

@swift-ci Please test source compatibility


custom action url setted in the client

Hi, right now we are exploring the possibility to add form.io into our app, but we need to set the form endpoint dynamically (based on some id) within out client side application, and we were wondering if it is posible set the form endpoint or action in the client side app using this library?

Currently this can be set as part of the form definition passed to the form.
If you are using the src attribute () then set the Custom Action Url in the form.io interface.
If you are passing the form directly into the form definition () then set myformObject.action to the url you want to submit to. You can see the code for this at https://github.com/formio/react-formio/blob/master/src/Formio.jsx#L210-L213

@randallknutson thank you so much man, this really helps me a lot.

No problem!


[Vaults UI] Add parameter to parents endpoint to retrieve full parents hierarchy

As discussed IRL with @flvndvd

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Not needed anymore


Feature request: CI for Windows build and test

https://www.appveyor.com/
Free build and test in cloud for Windows. I think this should be implemented for TensorFlow to make sure the windows builds are as well tested as the Linux builds.
Motivation: https://github.com/tensorflow/tensorflow/issues?utf8=✓&q=is%3Aissue is%3Aopen windows

@gunan interesting idea.

We do have windows continuous tests setup and running on our own test infrastructure.
http://ci.tensorflow.org/job/tf-master-win-gpu-cmake/
http://ci.tensorflow.org/job/tf-master-win-bzl/
http://ci.tensorflow.org/job/tensorflow-master-win-cmake-py/
We might be lacking firsthand windows development machines for quickly debugging these. but the test infra is there.
Closing the issue.


Uncomment & fix failing test from #3535.

Resolves #3419 and #3628.
I just saw #3628 after I started working on this and I opened another PR (wrong) about the same bug #3628 looks good to me but adds an extra property, which I personally tend to dislike. This might have some side effects for cases that are untested, but I thought it's worth opening as an alternative to #3628.

I've looked at this while stepping through the tests and am feeling it's a better direction to go than #3628.
Interestingly, this PR moves the code back towards a state prior to changes made via ae2888b42598d18052328a2f4308fbf49a8b2d3e, 806681efa3c61ff5c9a591cfa489857946a5277c.  It's not clear why those changes were neccessary, perhaps @GoodBoyDigital can shed some light, however now that we have tests over this it's possibly ok.
Additionally, because of ac743bb34e0fdaf01bc3d033af6863713fb8ef4c and e46b3652be6922464007b75210271ddaf4531365, all children are being partially inspected anyway, but I can write some further tests to investigate what should/should not be triggered.

This also passes #3665

Hey @uorbe001, if you can make that variable name change I'll approve this :v:

Ping @uorbe001

@staff0rd sorry, I didn't see the notification for this. Do you have a better name for that variable in mind? I suck at naming things lol

updated the name, but I'm not sure it's a good one either, if someone has a better idea I'm all for it.

not entirely sure why that build failed, it doesn't look like anything I changed?

Build failing was an unrelated (now fixed) issue.


(Discussion) new package keytheorems

I've written a package keytheorems that builds on much of the thmtools code, at some points blatantly plagiarizes it (see, for example, the code for \listofkeytheorems), and even allows for identical syntax with the thmtools-compat option. As far as I can tell all of this is allowed under the LPPL, however I wanted to make sure the maintainer of thmtools had a chance to voice any concerns or suggestions before I upload the package to CTAN. If you have none, please feel free to close this issue.
For context, I wrote the package as an exercise in using expl3, but over time I've added most of the functionality of thmtools and in the process fixed a few nagging thmtools bugs. See the keytheorems README for a list of open issues fixed by keytheorems. There are also sections titled Differences with thmtools and Notes/issues on thmtools, not on Github.

Well I have to say, I'm not a competent maintainer of thmtools. Very happy you create a new theorem package which can be used as a replacement of thmtools.


Setup issue in ubuntu 20.04

Hi Murat,
I followed all steps shared at below link for setting up tools on Ubunut 20.04.
https://reveda.eu/pages/ubuntuInstallation
But it is ending with error:  podman-compose: command not found
Screenshot is attached for reference. Please suggest how to resolve it.
Regards
Vipul

Hello,
I don't have access now to Ubuntu machine but I could setup in a VM. Have you installed podman-compose? If not, could you try to install it or alternatively docker-compose? Then you could change the command to docker-compose.

Good news. Let me know if you have any further problems.


[WIP] Headless Services: Adding option to specify None for PortalIP

Some quick changes to gather early feedback on whether this is acceptable as a solution. I will constantize the "None" value of PortalIP. I am currently thinking about including the constant in every api version.

@bgrant0607 @smarterclayton would appreciate some early feedback to determine if PortalIP=None is something we can agree on.

Ok, I think I'm ok with this if Brian is.  We need to make it a constant, and ensure we add a few tests.

Ref #1607

Also need to cover kube2sky:
https://github.com/GoogleCloudPlatform/kubernetes/blob/89f18f6fe1b541523e7f944f6f2ac45cfc0704fd/cluster/addons/dns/kube2sky/kube2sky.go#L55

And ValidateServiceUpdate:
https://github.com/GoogleCloudPlatform/kubernetes/blob/668e2dbcd83a0ff84684fc6734312dd3a9a4c8c8/pkg/api/validation/validation.go#L740
It would be useful to add a case to ValidateService, also, that confirmed the value was empty, None, or a parseable IP address.

And we'd need to update the PortalIP field descriptions and docs/services.md.
And, tests, as @smarterclayton mentioned.
ResourceLocation still works, since it targets endpoints directly.
Some thought should be put into interaction with other options, such as createExternalLoadBalancer and PublicIPs.
I'm ok with the approach, since I proposed it. As expected, it's a pretty straightforward change.
A mode where we populated (or backed) DNS directly from pod IPs would be orthogonal to this, I'd think.
Thanks for doing this.
cc @thockin

@bgrant0607 @smarterclayton This was just hurriedly shared as a WIP PR since I was looking for guidance on whether this approach is even acceptable (based on Clayton's concerns). I will implement the feedback and have the PR up for review later today/tomorrow.

----- Original Message -----

And we'd need to update the PortalIP field descriptions and docs/services.md.
And, tests, as @smarterclayton mentioned.
ResourceLocation still works, since it targets endpoints directly.
Some thought should be put into interaction with other options, such as
createExternalLoadBalancer and PublicIPs.
I'm ok with the approach, since I proposed it. As expected, it's a pretty
straightforward change.
A mode where we populated (or backed) DNS directly from pod IPs would be
orthogonal to this, I'd think.

I have a todo to talk with Tim on DNS once we both get some time.

Thanks for doing this.
cc @thockin

Reply to this email directly or view it on GitHub:
https://github.com/GoogleCloudPlatform/kubernetes/pull/5556#issuecomment-82766815

My concerns are addressed / resolved.
----- Original Message -----

@bgrant0607 @smarterclayton This was just hurriedly shared as a WIP PR since
I was looking for guidance on whether this approach is even acceptable
(based on Clayton's concerns). I will implement the feedback and have the PR
up for review later today/tomorrow.

Reply to this email directly or view it on GitHub:
https://github.com/GoogleCloudPlatform/kubernetes/pull/5556#issuecomment-83057061

At this point most of the feedback is implemented and I am working on adding some documentation for headless services in docs/services.md

Looks pretty good to me. The description strings need to be updated. I'm fine with services.md being updated in a separate PR, though.
Failed gofmt:
./hack/verify-gofmt.sh
./pkg/api/helpers.go:89:6: expected '(', found 'IDENT' IsServiceIPSet
./pkg/api/helpers.go:94:6: expected '(', found 'IDENT' IsServiceIPRequested
./pkg/api/helpers.go:96:3: expected '}', found 'EOF'

Duh :(
Got a conflict while pushing the commit and didn't resolve correctly.

@bgrant0607 Initially, was planning to include the doc changes in this PR as well. However, I believe the doc changes themselves would be likely to go through a few iterations and we can let this PR get reviewed/merged in the meantime.
I will submit a separate PR for the services.md doc changes.

@bgrant0607 Alright, tests are passing now.

LGTM. Awesome!


Fixed ORDER BY ... DESC support

I see, now you can have different sortings for different fields. Nice.

No, DESC setting had been totally ignored.

Oh, you're right.
--
Sent using a cell-phone, so sorry for the typos and wrong auto-corrections.
On May 13, 2016 1:03 PM, "Phil Krylov"<EMAIL_ADDRESS>wrote:

No, DESC setting had been totally ignored.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/versae/qbe/pull/53#issuecomment-219101467


Can you forward some useful information about the response

So body have this information
{
  success: false,
  error: message,
  status: 0-999,
  statusText: 'status text'
  body: '',
  headers: [
    [key, value] // 2D array with key/value (some headers can be set multiple times)
  ]
}

Even if the request fails i would still want to be able to get the text of a failed page response
for example i want to be able to read this body: https://httpbin.org/status/418

@jimmywarting You can send PR with proposed change. Thank you.


rtmp_watch returned from a reactor doesn't seem to actually be watchable

Using a workflow similar to this one: https://github.com/KallDrexx/mmids/blob/08110c3269c913bdaef85246ecc8a7d3bee1609a/mmids-app/mmids.config#L70-L73 https://github.com/KallDrexx/mmids/blob/08110c3269c913bdaef85246ecc8a7d3bee1609a/mmids-app/mmids.config#L8-L10
with the reactor returning something similar to: https://github.com/KallDrexx/mmids/blob/08110c3269c913bdaef85246ecc8a7d3bee1609a/reactor-test-server/workflows/test/abc.mmids#L1-L4
If I actually try to watch/play any of the streams with either VLC or ffplay (e.g. ffplay rtmp://localhost/abc/test) it fails to play it. If I try the same with a more basic workflow without reactors, similar to: https://github.com/KallDrexx/mmids/blob/08110c3269c913bdaef85246ecc8a7d3bee1609a/mmids-app/mmids.config#L15-L18
Then it can play just fine.
I've tried this on 1.1.1 and master with the same result

Wow this is embarrassing, sorry about that :(.  I extensively tested reactors and not sure how that got missed.
I checked in a quick fix into master, and will do a more extensive look to see why tests didn't catch that later on.

amazing, thanks for the quick fix, things seem to be working as expected now 👍

Thanks for the confirmation!  Going to close this as I created #22 to address the test coverage holes.


Improvement for k8s.io/docs/setup/production-environment/tools/kubeadm/high-availability/

There's a potentially missing step at the end of bootstrapping a HA cluster using kubeadm, the kube-apiserver, etc. do not get a updated manifest that points to all etcd nodes (using a stacked arch), so a k get componentstatuses may only display a single etcd node, the default localhost etcd.
I'd suggest to update the doc to explicitly state that kube-apiserver, etc. manifests need updating to point to all etcd nodes provisioned so that componentstatuses actually retrieves and provides the correct status/es.
Example:
NAME                 STATUS    MESSAGE             ERROR
controller-manager   Healthy   ok
scheduler            Healthy   ok
etcd-0               Healthy   {"health":"true"}

vs
NAME                 STATUS      MESSAGE                                                                                ERROR
controller-manager   Healthy     ok
scheduler            Healthy     ok
etcd-1               Healthy     {"health":"true"}
etcd-0               Healthy     {"health":"true"}
etcd-2               Unhealthy   Get https://<IP_ADDRESS>:2379/health: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

so a k get componentstatuses may only display a single etcd node, the default localhost etcd.

componentstatuses is being deprecated. there was an issue about that in the kubernetes/enhancements repository.
please use etcdctl to get the status of your etcd cluster.

There's a potentially missing step at the end of bootstrapping a HA cluster using kubeadm, the kube-apiserver, etc. do not get a updated manifest that points to all etcd nodes

this step is not really required for the cluster to function and amending the manifests is a further complication post etcd bootstrap.

I'd suggest to update the doc to explicitly state that kube-apiserver, etc. manifests need updating to point to all etcd nodes provisioned so that componentstatuses actually retrieves and provides the correct status/es.

i don't think the kubeadm docs should be updated to comply with componentstatuses (as it might get removed at some point).
/sig cluster-lifecycle
/priority awaiting-more-evidence

It's true that the step isn't required for a functioning cluster, but the page is about setting up a production cluster so I thought it should be part of this doc?
FWIW, there is https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/high-availability/#set-up-the-first-control-plane-node that creates an example kubeadm-config.yaml when one isn't using a stacked approach.
Either way, I thought it's worth mentioning as when that one configured local etcd node would stop functioning, then so would that kube-apiserver (instead of continuing to work using the other 2 or more etcd nodes).

It's true that the step isn't required for a functioning cluster, but the page is about setting up a production cluster so I thought it should be part of this doc?

i don't think we should mention componentstatus in the kubeadm docs.
one alternative is to move the contents of 8. Optional: Check the cluster health from here:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/setup-ha-etcd-with-kubeadm/#setting-up-the-cluster
as a new item under common tasks here:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/high-availability/#common-tasks-after-bootstrapping-control-plane
PRs are welcome!
/help

/kind cleanup
/remove-priority awaiting-more-evidence
/priority backlog

i don't think we should mention componentstatus in the kubeadm docs.
You're probably right, since it's being removed anyway, there's no point.

Though moving the 8. Optional: Check the cluster health from the ha etcd setup won't actually tell admins they should update the kube-apiserver.yaml to point to the other etcd instances, if that makes sense? That was the original "issue" and I believe that simply mentioning this in _ Common tasks after bootstrapping control plane_  with perhaps an example "snippet" would do.

no, we should not do that. for stacked, an api-server instance should only know about the etcd server instance that was deployed on this same host.

no, we should not do that. for stacked, an api-server instance should only know about the etcd server instance that was deployed on this same host.

Hmm, I'm curious, what's the reasoning behind this?

this changes the topology defined here:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/ha-topology/#stacked-etcd-topology
it now needs arrows from each api-server to individual etcd members.
by adding it to the docs we will also recommend it as a best practice and it's more of a workaround for a componentstatus issue (please don't use it). a stacked-etcd node should be self-contained and the etcd cluster is "stacked" on top of the kubernetes cluster.

Actually, it's not a workaround for a componentstatus issue, I mentioned componentstatuses because that's what pointed me at the fact that the kube-apiservers aren't configured to use all etcd nodes.
However, you're right, based on that diagram, stacked topology seems to mean that the components on a host do not cross connect / only use the localhost etcd endpoint.
FWIW - It still seems wrong to me that an issue (or maintenance, or whatever) on only the local etcd instance should effectively take out the kube-apiserver too.

FWIW - It still seems wrong to me that an issue (or maintenance, or whatever) on only the local etcd instance should effectively take out the kube-apiserver too.

that is a tradeoff for stacked, one component failing, fails the whole node.
and stacked nodes are considered self-contained and swappable - i.e. the LB should give you a new node if the current leader's storage layer fails.
what you are suggesting can be done on demand, but i still don't think we should include it in the docs as an optional step.
for something that is more HA, we suggest using external etcd where all api-servers know about all the etcd members.

Fair enough I guess. Thanks!


posh-git hangs (30+% CPU usage) when navigating to a large repo

After enabling posh-git and reloading my profile, when I navigate to a large (>100MB) repository, the prompt / powershell freezes.
I'm not sure what other info I can give you.
Name                           Value
----                           -----
PSVersion                      5.1.14393.206
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.206
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Apologies - looks to be a duplicate of https://github.com/dahlbyk/posh-git/issues/50
Unfortunately it seems like I can't use posh-git for now (40 000+ lines from $GitPromptSettings.Debug = $true!)

@voltagex keep an eye on https://github.com/dahlbyk/posh-git/pull/208, I'm going to try to finally review and merge that Very Soon™.