Missing key: 'sshKeyPath'

Hi,
I am trying ssh remote mode on Ubuntu NNI1.3:
machineList:
  - ip: <IP_ADDRESS>
    username: simon
    passwd: 1
    #port can be skip if using default ssh port 22
    port: 22
  - ip: <IP_ADDRESS>
    username: simon
    passwd: 1
    port: 22

Got:
Missing key: 'sshKeyPath'

Could somebody help?
Thanks in advance.

Hi @forhonourlx , could you try using
machineList:
  - ip: <IP_ADDRESS>
    username: simon
    passwd: "1"
    #port can be skip if using default ssh port 22
    port: 22
  - ip: <IP_ADDRESS>
    username: simon
    passwd: "1"
    port: 22

This problem might caused by the value type of passwd, the pure digital numbers in passwd might be recognized as Number type, since NNI only support String type in passwd field.

Hi @SparkSnail,
Thank you for your help, passwd: "1" works.
And another exception is raised, what is attribute 'last_line'?
$ nnictl create --config config_ssh.yml --port 8082
INFO: Starting restful server...
Traceback (most recent call last):
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 349, in parse
    transformer.visit(ast_tree)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 280, in visit
    return self._visit_children(node)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 330, in _visit_children
    self.generic_visit(node)
  File "/home/simon/anaconda3/lib/python3.6/ast.py", line 308, in generic_visit
    value = self.visit(value)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 297, in visit
    return self._visit_children(node)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 330, in _visit_children
    self.generic_visit(node)
  File "/home/simon/anaconda3/lib/python3.6/ast.py", line 308, in generic_visit
    value = self.visit(value)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 293, in visit
    return replace_variable_node(node, annotation)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 233, in replace_variable_node
    assert test_variable_equal(node.targets[0], name), 'Annotated variable has wrong name'
AssertionError: Annotated variable has wrong name

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/__init__.py", line 109, in _expand_file_annotations
    annotated_code = code_generator.parse(src.read(), nas_mode)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/code_generator.py", line 351, in parse
    raise RuntimeError('%d: %s' % (ast_tree.last_line, exc.args[0]))
AttributeError: 'Module' object has no attribute 'last_line'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/simon/anaconda3/bin/nnictl", line 8, in <module>
    sys.exit(parse_args())
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_cmd/nnictl.py", line 217, in parse_args
    args.func(args)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_cmd/launcher.py", line 509, in create_experiment
    launch_experiment(args, experiment_config, 'new', config_file_name)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_cmd/launcher.py", line 437, in launch_experiment
    code_dir = expand_annotations(experiment_config['trial']['codeDir'], path, nas_mode=nas_mode)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/__init__.py", line 94, in expand_annotations
    annotated |= _expand_file_annotations(src_path, dst_path, nas_mode)
  File "/home/simon/anaconda3/lib/python3.6/site-packages/nni_annotation/__init__.py", line 118, in _expand_file_annotations
    raise RuntimeError(src_path + ' ' + '\n'.join(str(arg) for arg in exc.args))
RuntimeError: /home/simon/Desktop/DeepRL-1.1/examples_ddppo.py 'Module' object has no attribute 'last_line'

Hi @forhonourlx , this error is raised by NNI annotation, are you using annotation in NNI?

Yes. I simply tried an annotation as following:
    '''@nni.variable(nni.choice(8, 16), name=num_workers)'''
    config.num_workers = 16#

Add @liuzhe-lz  for this issue, he is more familiar with annotation.

I'm not sure if annotation supports attribute. Please try assigning to a local variable first.
'''@nni.variable(nni.choice(8, 16), name=num_workers)'''
num_workers = 16
config.num_workers = num_workers

Looks strange, but I'm not familiar with current annotation module either...

Thank you both.
I will try or raise a new issue.

Hi @liuzhe-lz,
Fixing annotation name config.num_workers works.
Thank you for your help.


[HUDI-5900] Clean up unused metadata configs

Change Logs
Some of metadata table configs cannot be overridden by users anymore (#7709).  This PR removes the configs:
hoodie.metadata.clean.async
hoodie.metadata.cleaner.commits.retained
hoodie.metadata.enable.full.scan.log.files
hoodie.metadata.populate.meta.fields

Impact
Reduces the number of configs for metadata table and makes configuration of MDT mostly invisible.
Risk level
none
Documentation Update
The configs will be removed from the configuration page after the asf-site branch is updated.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

I'll tackle this in a separate PR (HUDI-5934): Remove archival configs for MDT and align that with data table

CI passes.

CI is green.


Update README.md

Update README as per https://github.com/anubhavmishra/terraform-provider-jira/blob/60df217d4d9420306d1790286c77b2ce1804ecb3/jira/config.go#L21

To close https://github.com/anubhavmishra/terraform-provider-jira/issues/3

@bgiorgini thank you for the fix.


feat(cx-2439): Restrict showing request price estimate banner for D-Index less than nine

The type of this PR is: Enhancement

This PR resolves CX-2439
Description

We want to show the Request For Price Estimate Banner only for Artworks with D-Index of >= 9

D-Index >= 9
D-Index < 9

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Restrict showing request price estimate banner for D-Index less than nine - kizito

iOS user-facing changes

Android user-facing changes

Dev changes

I will merge this since @rajsam003 is off today


Add WEIGHTS_JSON file as cli option

Hi Jay, thanks for this pull request, I've added a minor comment. Could you also please update the doc?

Documentation updated! Ready to go 🚀

@tnicola  I can't merge it

@jayc13 merged!Thanks for your contribution! :)


Hard schema for well-defined entities

The current TDM schema is very generic. DataModel, DataPath, ... This is nice to work with but is also challenging as we need attributes to qualify XPaths which are not necessary for OIDs. It might be worth looking at making a harder schema around these well-defined entities. It would be beneficial in some areas such as reducing search space in some cases, but also increases complexity in other cases and requires treating things less generically.

This wouldn't be as much of a problem but a very specific case - IOS XE wants prefixed XPaths and IOS XR wants XPaths prefixed by modules. This requires maintaining an understanding of the module-prefixed and prefix-prefixed XPaths, and there isn't an accepted way of representing them all at once, breaking the 2-field basis of Human ID and Machine ID because reality isn't consistent.


Test & Deploy Current Version

Hey @AyBruno!
So just want to confirm here that what's currently here in main is not deployed to prod, and that the blocker for deploying is just testing all of the new changes that you pushed 2 months ago... correct or no?

@dvdokkum  That is correct. What's currently on main is not deployed. I do think it would be a good idea to test locally before deploying, but I would also feel comfortable deploying and testing on prod. Deploying itself is quite the blocker. It takes a while and AWS permissions are a pain. If you have any tips on how I could automate deployment of containers to the ec2 instance that would be huge.

So just to circle back to this... there's now a deploy action in Github. By default it will dry-run and do the following:

build the docker image
compare that image to what is currently latest in ECR and indicate whether it is new
show currently running images on the prod server

If you set the dry-run prop to false it will do the deploy as outline in the Readme:

build and push to ECR
pull latest on the prod server and restart

I haven't been able to test the actual deploy yet, so let me know when would be a good time (when we're ready to push main to prod) and I'll make sure it is working for real for real.

@dvdokkum The program at this point should build and compile successfully so we should be good to go with testing the deploy action out.

great, i'll give it a shot now!

ok that's done. what's on main has been deployed


deps: Bump aspect_bazel_lib -> 1.3.1

Fix #21865
Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

this one includes a fix for https://github.com/aspect-build/bazel-lib/issues/162 so will allow a cleanup

/lgtm deps


Direct upgrade from 2.9.2 to 2.9.3 is not available

Describe the bug
The following MSI error appeared when trying to upgrade one version of ModSecurity to another:
"Another version of this product is already installed.  Installation of this version cannot continue.  To configure or remove the existing version of this product, use Add/Remove Programs on the Control Panel."
Looks like ModSecurity need to be uninstalled, server rebooted and then new version of ModSecurity installed again. This looks unacceptable for production server.
Logs and dumps
Attached verbose MSI log prepared by execution of the following command line:
msiexec /i ModSecurityIIS_2.9.3-64b.msi  /l*v c:\modsecurity-2.9.3-install.log ADDLOCAL=ALL ALLUSERS=1 REBOOT=R
modsecurity-2.9.3-install.log
To Reproduce
Steps to reproduce the behavior:

Install ModSecurity 2.9.2 64b IIS. Installed OK.
Download installer for version 2.9.3, trying to install new version by launching new downloaded MSI file.

Expected behavior
ModSecurity upgraded to 2.9.3 version.
Server (please complete the following information):

ModSecurity version: 2.9.2 to 2.9.3
WebServer: IIS
OS (and distro): Windows 2012R2

Rule Set (please complete the following information):
n/a
Additional context

Looks like the issue cause by this line:
https://github.com/SpiderLabs/ModSecurity/blob/v2/master/iis/installer.wxs#L10
<Product Id="22B04FDB-9BAB-46B0-87B8-A39544ECECD3" Name="ModSecurity IIS" Language="1033" Version="2.9.1" Manufacturer="ModSecurity" UpgradeCode="82F09489-1678-4C38-ADCB-08C3757653DB">

You have hardcoded UpgradeCode here, this is good, it shouldn't be changed in a whole line of product. However, hardcoding Product ID means that in fact ModSecurity 2.9.3 technically is the same product as 2.9.2 and 2.9.1.
In general it can be handled by dynamically generated product code (just remove this attribute), and then implement RemoveExistingProducts action during upgrade (to perform removal of previous product having the same upgrade code).
p.s. old version number 2.9.1 hardcoded here. This shouldn't be critical, however it is visible in log file.

Thanks for your report @pvasilevich :)

https://github.com/SpiderLabs/ModSecurity/blob/v2/master/iis/installer.wxs#L10

Your input here makes sense to me. I can regenerate a test version of the installer to check that the issue doesn't happen anymore with this changeset.
But I would suggest that you can submit a pull request to fix this issue and have your name as contributor. How does that sound? :)

This was indeed an issue with v2.9.2 and v2.9.3 where the version update in that file was omitted (it still said 2.9.1 in both of those later versions).
However, it has been properly updated since then (in both v2.9.4 and v2.9.5), so I don't think there's value in keeping this item open any longer.


Update gorilla emoji to full gorilla instead of just face

The gorilla emoji doesn't match any other platform. It should be redrawn to be a full-sized gorilla instead of just a face.

Everybody else had 🦍 as a face originally as well, but meanwhile converged on Apple's design choice, unfortunately.
Unicode should adopt ZWJ sequences to distinguish full-body and facial animal emojis (and their should be gender variants in some cases #164).

Whether the choice was unfortunate or not is not exactly the point, although I totally agree that users should be able to choose to display a head or a body (or in some cases choose the gender) for animal emoji. I bring this issue up purely from a user experience perspective. When a user taps on a full-sized gorilla emoji on any platform, Twemoji displays a gorilla head. That's unexpected and feels like a mistake.

I would even argue that in this case displaying only the head of the gorilla is incorrect, because the emoji is called "Gorilla," not "Gorilla Face".
Examples of this:
Cow Face 🐮 (U+1F42E) vs Cow 🐄 (U+1F404)
Pig Face 🐷 (U+ 1F437) vs Pig 🐖 (U+ 1F416)

No, those a legacy distinctions. Showing either, head or body, is fine.

🤔 How can they be legacy but also still be the current and official name of the character? Can you link me to something that explains which emojis are legacy? On unicode.org, which I assume is up to date, these characters are still called "Cow," "Cow Face," "Pig," and "Pig Face."
https://unicode.org/emoji/charts/emoji-list.html#1f42e
https://unicode.org/emoji/charts/emoji-list.html#1f437
https://unicode.org/emoji/charts/emoji-list.html#1f416
https://unicode.org/emoji/charts/emoji-list.html#1f404

Even if they wait to draw the full body gorilla when Unicode adopts ZWJ sequences for animals, the fact remains that the full body gorilla needs to be drawn, so the work needs to be done regardless. Might as well do it now, switch to a full body gorilla to conform with all other platforms, and when Unicode adopts ZWJ sequences for animals, then Twemoji will already have both ready to go!

What I meant by ”legacy distinction“ is that new animal emojis will not come in pairs of two codepoints for both, head and body. This newer policy does not recommend one glyph design over the other, however.
I think it was U+1F992 🦒 with the character name Giraffe Face that sparked the discussion at the time, which resulted in the emoji name becoming just Giraffe.


Refactoring creds handling

Proposed changes

Codecov Report

Merging #264 (5415cb0) into master (694e98d) will decrease coverage by 0.64%.
The diff coverage is 7.69%.

@@            Coverage Diff             @@
##           master     #264      +/-   ##
==========================================
- Coverage   32.89%   32.24%   -0.65%
==========================================
  Files          34       33       -1
  Lines        2110     2047      -63
==========================================
- Hits          694      660      -34
+ Misses       1290     1273      -17
+ Partials      126      114      -12

Impacted Files
Coverage Δ

cli/command/new/cmd.go
10.44% <0.00%> (ø)

cli/command/run/cmd.go
6.97% <0.00%> (ø)

internal/docker/docker.go
44.97% <100.00%> (ø)

internal/archive/tar/tar.go
41.07% <0.00%> (+1.07%)
:arrow_up:

Continue to review full report at Codecov.

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


Error: Failed to create exchange 'dead.letters-ex.1' on connection 'default' with 'No endpoints could be reached'

i'm using cloudamqp for my rabbitmq instance, and wascally for the setup, but i'm getting this error every time i try to start my app:
W20151022-13:28:49.756(-4)? (STDERR) Error: Failed to create exchange 'dead.letters-ex.1' on connection 'default' with 'No endpoints could be reached'

here's my config object:
{
    connection: {
      user: username,
      pass: password,
      server: server,
      //port: 5672,
      //timeout: 2000
      vhost: vhost
    },
    exchanges:[
      { name: 'dead.letters-ex.1', type: 'direct' }
    ],
    queues:[
      { name: 'dead.letters-q.1', subscribe: false, durable: false }
    ],
    bindings:[{
      exchange: 'dead.letters-ex.1',
      target: 'dead.letters-q.1',
      keys: ['dead']
    }]
  }

it says i have no endpoints, but i'm not sure what that means.  it can't reach cloudamqp?

"No endpoints could be reached" means the client could not contact RabbitMQ (or possibly failed to authenticate — although modern clients have no excuse to not provide a more specific error message).

Yes, that error generally means that the client could not connect to your server based on the information you provided. Out of curiosity, are you still getting this error when you have the "port" attribute of the config object uncommented out?

FYI a previous setup was using RabbitMQ 3.5.4, this deployment is 3.5.6.
@Shawn-LeanKit i tried uncommenting the port but got the same errors.  i'm not provided with a port at cloudamqp

you may need to up the timeout when connecting to cloudamqp. add a timeout: 30000 setting to the connection information. i have had to do that in the past, due to network latency across the interwebs

Thanks @derickbailey! That was going to be my next suggestion haha

jacked it up to 60000 and still no love.  this is going to turn out to be something silly, i can feel it.
i've also tried creating a new rmq instance and i tried using one that has worked previously... both have the same problem.
for sure this is me.  but what?  i will dig through some more config on my end and post when i find it.

my next guess would be vhost name. if your vhost has a / at the beginning of the name, you need to include it in the vhost setting: vhost: "/foo". if there isn't one in the name, don't include it: vhost: "foo"

SOLVED
it was definitely my problem.
i am using meteor for this deployment (which has been working fine).  the difference between this app and others i've used wascally with is that i was trying to set my MONGO_URL and MONGO_OPLOG_URL in a config file like this:
process.env.MONGO_URL="mongodb://....."
process.env.MONGO_OPLOG_URL="mongodb://....."

meteor didn't like that.
apparently because meteor was upset with my attempt to set the MONGO_URL it was blocking wascally from doing its job!
i wrote a short shell script to set the environment vars i need for this app and then run the program, and it worked!
thank you
@michaelklishin @derickbailey @Shawn-LeanKit thank you so much for your attentiveness and SPEEDY suggestions and help.  it made a big difference and drove me toward the solution.


[stable/prometheus-operator]: update rules

What this PR does / why we need it:
updates rules using hack/sync_prometheus_rules.py.
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @anothertobi

/assign @anothertobi

Whoa, these look like they're starting to include changes for kubernetes 1.14, and aren't compatible with previous versions of kubernetes, i.e. 1.13 and lower!
This realistically means that we'll need to have two sets of rules and dashboards, one that's compatible with earlier versions and one that works with later versions.
/hold

I think the best way forward here is to lock down the current rules for kubernetes 1.13 and create a new integration to sync rules for kube 1.14 and up. They could be controlled via chart variables, indicating that we should deploy 1.13 vs 1.14 rules

https://github.com/helm/charts/pull/16230 takes care of this requirement

https://github.com/helm/charts/pull/16230 takes care of the requirement to have multiple versions of rules

this was handled on other PRs AFAIK. Closing. 🙏


chore(main): release sdk 7.2.10

:robot: I have created a release beep boop
7.2.10 (2024-12-17)
Bug Fixes

sdk: Fix Moonbeam balance querying 🛠 (83360eb)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

sdk-v7.2.10
:sunflower:


Migration tool doesn't pick up proxy container args

Custom proxy container args seem to be ignored by the migration tool

are you referring to kaap sidecars ?
aren't they visible in the report or in the crd or both?
Looking in the code it just copies the whole container.


Weight 0 vs not member

Now, closes #154.

Oh, I see, good catch. I reverted it in my branch, but it came back from the rebase. Shoud be fixed now.


HTTP method is empty and URI shows the method

I have an existing Ruby on Rails application which is using the Zipkin for tracing. I used the docker image to start the proxy application locally to send the tracing data to Stackdriver Trace.
It does work, but for some reason the "HTTP method" column remains empty on the UI and the URI shows "get" or "post" instead of the actual URI. When i click on a trace, it shows the URI as zipkin.io/http.path property.
How can configure to see the correct URI and HTTP method?
Ruby 2.3.3, zipkin-tracer gem 0.21.0, gcr.io/stackdriver-trace-docker/zipkin-collector docker image "latest" (2 months old).
Here's a screenshot of what i see on the UI:

Hi Andras,
Do you mind to capture actual Zipkin span that being sent to the adapter?
On Mon, Mar 27, 2017, 1:03 AM Andras Suller<EMAIL_ADDRESS>wrote:

I have an existing Ruby on Rails application which is using the Zipkin for
tracing. I used the docker image to start the proxy application locally to
send the tracing data to Stackdriver Trace.
It does work, but for some reason the "HTTP method" column remains empty
on the UI and the URI shows "get" or "post" instead of the actual URI. When
i click on a trace, it shows the URI as zipkin.io/http.path property.
How can configure to see the correct URI and HTTP method?
Ruby 2.3.3, zipkin-tracer gem 0.21.0,
gcr.io/stackdriver-trace-docker/zipkin-collector docker image "latest" (2
months old).
Here's a screenshot of what i see on the UI:
[image: stackdriver_trace]
https://cloud.githubusercontent.com/assets/394358/24346096/8bd3abee-1305-11e7-8938-1e730f23802d.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/stackdriver-zipkin/issues/34, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABL2ZiurWLhE_GZ25bWhIdv5yYy3jphsks5rp21egaJpZM4Mp6nK
.

--
regards, denys

The "URI" field in the UI is actually showing the span name, which is mapped directly from the Zipkin span name. Looking at the code for the Ruby tracer, it looks like it sets the span name to be the HTTP method which is why you're seeing "post". We also populate the "HTTP Method" column based on the value of the "http.method" annotation, however it looks like that isn't set.
@adriancole In our data model, we typically use the URL or operation name as the span name. Is there any documentation about what values to typically expect for the Zipkin span names?

@adriancole https://github.com/adriancole In our data model, we
typically use the URL or operation name as the span name. Is there any
documentation about what values to typically expect for the Zipkin span
names?

The span name is usually the method name (even in http). The key part is
that it is bounded cardinality as it ends up in a drop-down. URLs typically
are not bounded, so make poor zipkin span names.  Most libraries
default http to method name as that's less likely to result in unbounded
cardinality. Most libraries also allow users to override that convention..
https://github.com/openzipkin/zipkin-api/blob/master/thrift/zipkinCore.thrift#L390

@denyska Here is an actual span sent to the adapter:
[
    {
        "name": "post",
        "traceId": "daa4dacc668ea2a1",
        "id": "daa4dacc668ea2a1",
        "annotations": [
            {
                "value": "sr",
                "timestamp":<PHONE_NUMBER>030831,
                "endpoint": {
                    "ipv4": "<IP_ADDRESS>",
                    "port": 443,
                    "serviceName": "[redacted]"
                }
            },
            {
                "value": "ss",
                "timestamp":<PHONE_NUMBER>078433,
                "endpoint": {
                    "ipv4": "<IP_ADDRESS>",
                    "port": 443,
                    "serviceName": "[redacted]"
                }
            }
        ],
        "binaryAnnotations": [
            {
                "key": "http.path",
                "value": "/[redacted]",
                "endpoint": {
                    "ipv4": "<IP_ADDRESS>",
                    "port": 443,
                    "serviceName": "[redacted]"
                }
            }
        ],
        "timestamp":<PHONE_NUMBER>030789,
        "duration": 47650,
        "debug": false
    }
]


Pytorch-ROCm

Dear Bryce,
I would like to know whether there are any barriers against using ROCm-pytorch?

no idea. I hadn't heard of that till you mentioned it.  If you find some other version of stable diffusion working with ROCm I can look at what's involved.

let me know if you try it out

I just did a test while using rocm-5.5.1. It seems like one needs to install a nightly version for compatibility with rocm (https://pytorch.org/get-started/locally/), but imaginairy doesn't seem to like it, currently:
$ pip install pipenv
$ pipenv install torch --index https://download.pytorch.org/whl/rocm5.4.2
$ imagine ...
✨ Generate images faster using a persistent shell session. Just run `aimg` to start. This makes generation and editing much quicker since the model can stay loaded in memory.

/some/path/imaginairy_venv/lib64/python3.9/site-packages/torchvision/io/image.py:13: UserWarning: Failed to load image Python extension: libc10_cuda.so: cannot open shared object file: No such file or directory
  warn(f"Failed to load image Python extension: {e}")
Received 1 prompt(s) and 0 input image(s). Will repeat the generations 1 times to create 1 images.
Traceback (most recent call last):
  File "/some/path/imaginairy_venv/bin/imagine", line 7, in <module>
    imaginairy.cli.imagine.imagine_cmd()
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/imaginairy/cli/imagine.py", line 144, in imagine_cmd
    return _imagine_cmd(
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/imaginairy/cli/shared.py", line 176, in _imagine_cmd
    filenames = imagine_image_files(
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/imaginairy/api.py", line 61, in imagine_image_files
    for result in imagine(
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/imaginairy/api.py", line 153, in imagine
    check_torch_version()
  File "/some/path/imaginairy_venv/lib64/python3.9/site-packages/imaginairy/utils/__init__.py", line 250, in check_torch_version
    raise RuntimeError("ImaginAIry is not compatible with torch>=2.0.0")
RuntimeError: ImaginAIry is not compatible with torch>=2.0.0

Yes, the images produced when I installed torch 2.0.0 were garbage. I haven't had the time to debug it.


Issue with k8s.io/docs/tutorials/kubernetes-basics/deploy-interactive/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
The Module 2 description shows that the client and server version is V1.5, yet the terminal outputs V1.8.0 for client and V1.5.2 for the server.
Proposed Solution:
I don't know? Maybe having the site fetch the most recent version (I don't know - I'm not a confident developer, hence the tutorial).
Page to Update:
https://kubernetes.io/docs/tutorials/kubernetes-basics/deploy-interactive/

Thank you for letting us know. We thought we had removed all the version information. We'll get this updated ASAP.

Cheers! I'll leave you to close them once you've done that :)

Thanks for the feedback. This has been fixed as part of #7240

/close


Follow up PR-18: need consider more use cases

Include, but not limited to:

IDE key refresh - https://github.com/riscv-non-isa/riscv-ap-tee-io/issues/55
TDI Interface removal - https://github.com/riscv-non-isa/riscv-ap-tee-io/issues/57
Device removal - https://github.com/riscv-non-isa/riscv-ap-tee-io/issues/58
SPDM key update - https://github.com/riscv-non-isa/riscv-ap-tee-io/issues/53
SPDM heartbeat - https://github.com/riscv-non-isa/riscv-ap-tee-io/issues/54
Device SPDM measurement collection - done, https://github.com/riscv-non-isa/riscv-ap-tee-io/pull/36
Device Evidence (SPDM certificate, SPDM measurement) recollection, after device runtime update - done, https://github.com/riscv-non-isa/riscv-ap-tee-io/pull/41
Discrete Device Assignment (DDA) mode to passthru entire device to VM - https://github.com/riscv-non-isa/riscv-ap-tee-io/issues/56

Tracking it separately. Close this one.


Docs-5884 - updated states with info from Nishant.

@nishantkakar - I added all the app states from your list to the docs. Probably not all are applicable. Please review and revise.
See screenshot for what the icons look like:


Add button size modifiers

Great job @C-Lodder! I love the mini class size, as well as the large button fix and the addition of a small class. Just one more task for this would be to add a demo for both the small and mini buttons to the index.html demo site. After that, everything will be great and ready to merge!

@cpdevelops done :)

Great job! I hope to see you contribute again. Merged.


ci: Use go 1.12 on travis and golangci-lint

Update the travis config to use version 1.12.x of Go.
Update the golangci-lint version to v1.16.0, which contains Go 1.12.x
According to https://docs-staging.travis-ci.com/user/languages/go/#specifying-a-go-version-to-use
any tagged version of Go is usable by travis, but doesn't say what
"tagged" means.  It does tell us it uses "gimme" to install go, and
it looks at https://golang.org/dl/ for available versions. So 1.12.x
is a valid version as of now (stable is go1.12.5).
We could drop the version completely and just always get the latest
stable version by default, but we want to ensure that the travis Go
version and the golangci-lint Go version are the same(-ish).
golangci-lint highly recommends a specific version be installed so the
behaviour does not change unexpectedly so builds are repeatable
(https://github.com/golangci/golangci-lint/blob/master/README.md).
golangci-lint v1.16.0 is the current latest version according to the
github releases page for it
(https://github.com/golangci/golangci-lint/releases/tag/v1.16.0)
Fixes #198

Codecov Report

Merging #199 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #199   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          18     18
=====================================
  Hits           18     18

Continue to review full report at Codecov.

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

The version of Go used by golangci.com is still v1.11.5, not v1.12.x which I would like. I'm not sure why yet, but until I can figure that out, I cannot merge this.

It is not possible to get the golangci.com service to use Go 1.12 as far as I can tell. The golangci/build-runner docker image has Go 1.11.5, and that's that. The docker image doesn't even have a tag! It's only latest!
Since we don't require success from golangci.com to merge, we'll just have to deal with the spurious warnings/errors until we can bump that to Go 1.12.
We can update travis though, and travis builds to need to pass to merge, so this PR still has value and should be merged.

@camh-anz This resolved the travis ci failure on my PR.  I will replace the code that is causing the golangci failure so it will not cause issues for others.


⚠ Bofrost.be has degraded performance

In ee43c25, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1713 ms

Resolved: Bofrost.be performance has improved in 805f02e.


[stable/opa] Opa disable side effect webhook

What this PR does / why we need it:
Some users that are using old version of Kubernetes (e.g 1.10.x) needs to deal with this when they try install the stable/opa chart:
Comparing opa stable/opa
********************

	Release was not present in Helm.  Diff will show entire contents as new.

********************
error validating "": error validating data: ValidationError(ValidatingWebhookConfiguration.webhooks[0]): unknown field "sideEffects" in io.k8s.api.admissionregistration.v1beta1.Webhook

in clusters/americas/helmfile.yaml: failed processing release opa: helm exited with status 1:
  Error: error validating "": error validating data: ValidationError(ValidatingWebhookConfiguration.webhooks[0]): unknown field "sideEffects" in io.k8s.api.admissionregistration.v1beta1.Webhook
  Error: plugin "diff" exited with error
  Error: plugin "tiller" exited with error

Which issue this PR fixes
Make the sideEffects optional to the end users, they just need to comment the line in the values.yaml file:
SideEffectClass for the webhook, setting to None enables dry-run

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @tsandall

/ok-to-test

@nullck Please make signoff first.

@nullck Please make signoff first.

sure, thanks

@nullck Please make signoff first.

sure, thanks

@Aisuko  done

@nullck Thanks, git commit -s --amend && git push -f <your branch> the command could trigger the DCO checker again.

@nullck Thanks, git commit -s --amend && git push -f <your branch> the command could trigger the DCO checker again.

git commit -s --amend && git push -f origin opa-disable-SideEffect-Webhook                                                                                [23:50:39]
[opa-disable-SideEffect-Webhook 3fbd339c8] Merge branch 'opa-disable-SideEffect-Webhook' of github.com:nullck/charts into opa-disable-SideEffect-Webhook
 Date: Thu Sep 19 23:01:41 2019 -0300
Enter passphrase for key '/Users/nullck/.ssh/id_rsa':
Enumerating objects: 1, done.
Counting objects: 100% (1/1), done.
Writing objects: 100% (1/1), 286 bytes | 286.00 KiB/s, done.
Total 1 (delta 0), reused 0 (delta 0)
To github.com:nullck/charts.git
 + 53265ac9b...3fbd339c8 opa-disable-SideEffect-Webhook -> opa-disable-SideEffect-Webhook (forced update)

done

Please push to your PR, the PR's DCO status was still failing.


Creating a Simple AST and an Excluder Visitor

Started with making the Visitor interface ... as dart doesn't support overloading methods with different parameters, I used generic types, and the Element interface that will accept this visitor.
abstract class Visitor<T, R> {
  R visit(T node);
}

abstract class Element<T> {
  R accept<R>(Visitor<T, R> visitor);
}

then the simple AST only containing (classes, methods)
class Class implements Element<Class> {
  Class(this.name, this.methods):isExcluded=false;
  String name;
  List<Method> methods;
  // removed fields for simplicity
  // List<Field> fields;
  bool isExcluded;

  @override
  R accept<R>(Visitor<Class, R> visitor) {
    return visitor.visit(this);
  }
}

class Method implements Element<Method> {
  Method(this.name, this.isPrivate, this.isAbstract):isExcluded=false;
  String name;
  bool isPrivate;
  bool isAbstract;
  bool isExcluded;

  @override
  R accept<R>(Visitor<Method, R> visitor) {
    return visitor.visit(this);
  }
}

then implementing the Excluder Visitors
class ExcludeClass implements Visitor<Class, void> {

  @override
  void visit(Class c) {
    if(c.name.startsWith("_")) {
      c.isExcluded = true;
    }
  }
}

class ExcludeMethod implements Visitor<Method, void> {

  @override
  void visit(Method method) {
    if(method.isAbstract) {
      method.isExcluded = true;
    }
  }
}

Thanks for taking this up! We probably want to have them all in one class. It's nicer if we have a single visitor class, with multiple void methods: visitClass, visitMethod, ... So the user doesn't have to think about creating multiple classes.

Thank you for the note! Combining everything into a single visitor class makes the design cleaner and easier to use. Here’s the revised code based on your suggestion:
abstract class Visitor {
  void visitClass(Class c);
  void visitMethod(Method method);
}

abstract class Element {
  void accept(Visitor visitor);
}

class ExcludeVisitor implements Visitor {

  @override
  void visitClass(Class c) {
    if(c.name.startsWith("_")) {
      c.isExcluded = true;
    }
  }

  @override
  void visitMethod(Method method) {
    if(method.isAbstract) {
      method.isExcluded = true;
    }
  }
}

In the Class element, I've updated the accept method to automatically propagate the visitor to each method in its methods list, making it easier for the user to apply the visitor across all contained methods:
// simple AST
class Class extends Element {
  Class(this.name, this.methods):isExcluded=false;
  String name;
  List<Method> methods;
  bool isExcluded;

  @override
  accept(Visitor visitor) {
    for (var method in methods) {
      method.accept(visitor);
    }
    visitor.visitClass(this);
  }
}

class Method extends Element {
  Method(this.name, this.isPrivate, this.isAbstract):isExcluded=false;
  String name;
  bool isPrivate;
  bool isAbstract;
  bool isExcluded;

  @override
  accept(Visitor visitor) {
    visitor.visitMethod(this);
  }
}

Minor adjustment is to visit the class first, and not visit the methods if the class is already excluded by the visitor. But you can go ahead and open a PR for this.


Detect changes for project on network drive without recompile

I'm developing on virtual machine running in Parallels Desktop. My project is located on the Mac drive and VS 2015 opens it via mapped network drive.
There is a feature that projects changes recompiled automatically on browser refresh. But this does not work to project located on the network drive. I've to build project each time I change C# file or Razor view.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


adding shapefiles to repo

as suggested in #42 we might want to add the shapefiles to the repo to facilitate use, since the files are fairly small
They are all downloaded from OA sources but might need to double check the licenses on the sites.

[x] verify licensing
[ ] add shapefiles
[ ] add note & path to file to Guidance (to section starting at line 87)
[ ] add missing info on London geospatial file

Hi @aranas, I am happy to assist with this

@arana, I am not very sure which branch and documentation file you and Ruth are currently working on. Could you confirm this so I can add the geo files & notes there?

documentation branch has been merged into main; @BZ-BowenZhang creating new branch from current main to add new shapefiles


Asana Integration

Provide example(s) of how to integrate with Asana.com
If you need this integration, click the thumbs up emoji.
If you are willing to "sponsor" (ie. support) the development of this integration, click the heart emoji.

https://github.com/keptn-contrib/artifacthub/pull/131


files.sync() does not preserve original file mode.

Describe the bug
Some of my uploaded directories contain code files in mode 750 (-rwx-r-x----), they are being re-mode to 664 (rw-rw-r--). I suspect files.sync() is not checking the original mode of the file on host machine.
Seems like mode is only applied if it is supplied explicitly: https://github.com/Fizzadar/pyinfra/blob/e150ddb4794044418aeabb7d49c8e0ce265163a5/pyinfra/operations/files.py#L695
Expected behavior
files.sync() should preserve the original mode from the host. Otherwise unintentional permission breakage will ensue.

I'm definitely considering this a bug in files.sync, as you say could lead to some sketchy permissions issues for sure!
Will have a think about files.put, there's also files.get which should really replicate in reverse. Quite like the explicit-ness of passing mode= there.

Will have a think about files.put, there's also files.get which should really replicate in reverse. Quite like the explicit-ness of passing mode= there.

Agreed. Honestly not a high priority compared to files.sync()

Now fixed/implemented for files.sync in https://github.com/Fizzadar/pyinfra/commit/40377d98b43aa041661676a1c1e0a58675d46636 and released in v1.4.1! I've added https://github.com/Fizzadar/pyinfra/issues/590 to track potential implementation for files.put/files.get.

Thanks @Fizzadar, works perfectly.


passing secret to ContainerSource

I am building a new source based on container source. It is really awkward I have to plaintext secret in env.
I am contemplating a probable solution that could work this way:

adding a SecretReference to ContainerSourceSpec.
by the time the deployment is created, the secret is revealed and appended into the container's env.
container has to know the name of the env var and get the secret.

That at least gives us a chance to provide and retrieve non-plaintext secret.
@n3wscott @grantr

ContainerSource is not fully featured yet. This is a feature we would like to add for sure to bring new sources to parity to what we had with feeds. Thanks for the issue to track this.

@rootfs if ContainerSource took a pod template spec, would that solve this problem?

@grantr i believe SecretReference + PodTemplateSpec is the solution for most, if all, use cases.

/assign @grantr

The app then needs to get the secret out of the secret volume, this surely works.
A probable limitation is that the secret has to be in the same namespace as the pod. This could be a deal breaker if the container source is in e.g. eventing-system  namespace while secret is in a different namespace.

ContainerSource objects themselves don't go in the eventing-system namespace. They're created in the user's namespace along with the pod. It seems reasonable to disallow cross-namespace secret references, but do you have a counter example @rootfs?

i see, that'll work, thanks!

Duplicate of #55


Python 3.11 issue with productions.py: inspect.getargspec() -> inspect.getfullargspec()

In line 21 and line 61 of productions.py, inspect.getargspec() is depreciated in Python 3.11. It must be changed to inspect.getfullargspec(); however, I think a different error results from that change, and I haven't had the chance to review it yet.

I have a commit from a couple of years ago that fixed this on the previous incarnation of this repository.
It can probably be updated and applied if there's anybody still working on this project.

If you need to use Python 3.11+, my fork of python_actr seems to be working for my use cases.
Here are the changes required to fix this getargspec issue.

Thanks for these
I’m travelling to the middle of the month but I will push all of them!!
cheers
R

On Aug 4, 2023, at 12:02 PM, Andy Maloney @.***> wrote:
If you need to use Python 3.11+, my fork of python_actr https://github.com/asmaloney/python_actr seems to be working for my use cases.
Here https://github.com/asmaloney/python_actr/commit/eb0fb8d906a6d8703a7ec8a4cb7c26b867f1d378 are the changes required to fix this getargspec issue.
—
Reply to this email directly, view it on GitHub https://github.com/CarletonCognitiveModelingLab/python_actr/issues/19#issuecomment-1666052105, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABR4VBIIBTIUGNYRQBX4ZZLXTVBMFANCNFSM6AAAAAAUJIM4L4.
You are receiving this because you are subscribed to this thread.

OK - if you are going to merge PRs, I'll make one for this issue.

@rlwest

I’m travelling to the middle of the month but I will push all of them!!

BTW - there's no "push" involved here. You just merge the PRs using the interface here on GitHub. If you go to a PR there's a green "Merge" (or similarly-named) button.


Firestore plugin [0.8.2+1] online mode only does not work

I want to get results only from the server and not from the cache.
Apparently this setting should do what I want :
Firestore.instance.settings(persistenceEnabled: false)
but I'm still getting results from queries when I turn airplane mode ON, at least for iOS (didn't test yet on Android).

@diegolaballos
The issue at https://github.com/flutter/flutter/issues/25284 has been closed and moved here. Future collaboration on this issue will be done here.

Hi @diegolaballos
If you are still experiencing this issue
with the latest version of Flutter and FlutterFire plugin
can you please provide your updated  flutter doctor -v
and your and your flutter run --verbose/flutter build --verbose ?
Also, to better address the issue, would be helpful
if you could post a self contained app on github
or the steps to reproduce it.
Thank you
https://github.com/FirebaseExtended/flutterfire/issues/856

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Merge master branch to prTesting1_202011081927 branch

Docs Build status updates of commit 28be750:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


add serviceAnnotation to KubernetesResourcesSpec.

resubmit: https://github.com/istio/operator/pull/736 due to https://github.com/istio/operator/pull/713
depends on: https://github.com/istio/api/pull/1230

@morvencao: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test mandiff_operator

test_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test mandiff_operator

test_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test test_operator

e2e_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

lgtm

These failed tests depend on https://github.com/istio/api/pull/1230

/retest

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test test_operator

e2e_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test e2e_operator

mandiff_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test e2e_operator

mandiff_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test mandiff_operator

test_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

mandiff_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test mandiff_operator

test_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test test_operator

e2e_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
ebb8e978cabe016c8663a7e147c9b72b72e8b28f
link
/test e2e_operator

test_operator
52df0ea348082f77159bb6d8fa622d04ecd38d43
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
52df0ea348082f77159bb6d8fa622d04ecd38d43
link
/test test_operator

e2e_operator
52df0ea348082f77159bb6d8fa622d04ecd38d43
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e_operat

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
52df0ea348082f77159bb6d8fa622d04ecd38d43
link
/test test_operator

e2e_operator
52df0ea348082f77159bb6d8fa622d04ecd38d43
link
/test e2e_operator

gencheck_operator
19207453625a1bd790ec506dd9c31d2c4ad73d8b
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_operator
19207453625a1bd790ec506dd9c31d2c4ad73d8b
link
/test gencheck_operator

e2e_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_operator
19207453625a1bd790ec506dd9c31d2c4ad73d8b
link
/test gencheck_operator

e2e_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test e2e_operator

lint_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test lint_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test e2e_operator

lint_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test lint_operator

test_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test e2e_operator

lint_operator
3853168f967733b0aecad6da0ee52b88798b164c
link
/test lint_operator

test_operator
85daebc660d6b59d46456dd9fb99b10609e53265
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operat

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
85daebc660d6b59d46456dd9fb99b10609e53265
link
/test test_operator

e2e_operator
85daebc660d6b59d46456dd9fb99b10609e53265
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operator

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
85daebc660d6b59d46456dd9fb99b10609e53265
link
/test e2e_operator

test_operator
85daebc660d6b59d46456dd9fb99b10609e53265
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operator


cex.io: maker/taker fees, presition, limits added and nonce() fixed

I've hit the wall, really don't know how to fix this transpilation error.

I'm not sure if we have support for the Math.log10 in the transpiler. I think an advice from @kroitor is needed here :)

Hi! Thanks for this submission, @mkutny ! I'll take a look into it and will add missing rules the transpiler for it to pass. Will be there very soon!


Run all.sh components in a subshell

Run each all.sh component in a subshell. Rewrite the error reporting for keep-going mode accordingly. It is now a little more robust than before.
In the long term, I want to rewrite the infrastructure of all.sh in Python, to make it more robust and easier to maintain. But this is too big a leap in one go. This pull request is a step along the way: by making each component a subshell, we're closer to having Python infrastructure that runs shell snippets.
Change summary — user's perspective:

Now uses bash.
There is now an error if you pass an unknown component name. Fix #2783.
Changes to **/Makefile are no longer systematically silently discarded. They are now discarded only if Makefile seems to have been replaced by an automatically generated one (typically after doing an in-tree cmake build).
No longer risks restoring an old config.bak. Fix #3139.
Unanticipated errors (e.g. from config.py or cmake) are no longer hard aborts in keep-going mode.

Change summary — component writer's perspective:

Error detection is more robust. Fix #2570. Note that failures inside conditionals (e.g. between if and then, including in functions called there) are still ignored.
You no longer need record_status or if_build_succeeded. They're now no-ops, kept to facilitate the transition in case there are pending pull requests that use them, to be removed in a follow-up.
Each component runs in a subshell. This reduces the risk that a component will contaminate subsequent ones, for example due to setting an environment variable.
make is no longer a function, so things like VAR=value make … will work as intended. Whether a component will keep trying after a failure in keep-going mode is now determined by can_keep_going_after_failure.

Change summary — internal design:

Error detection in keep-going mode is now based on bash's ERR trap. If you don't fully understand the subtleties of what it does and does not detect, see pseudo_component_error_test for some examples of what it does detect.

Status: ready for review. The CI is failing because the FreeBSD jenkins-mbedtls.oss.arm.com instances don't have bash. Tracked internally as OSSDEVOPS-2330.
Backports: 2.16 (TODO), 2.2x (TODO)

This is a rebase of the 2.7 version. Here are my notes from the rebase (mostly of interest for the two reviewers of the 2.7 version).
410ecc0272 Switch all.sh to bash

Copyright statement change conflict. Abandoned in this branch.

634e3a2d6a Add --error-test option to test error detection and reporting
ac5c9fc796 Run each component in a subshell and handle errors more robustly

record_status: echo "command -> status" message to stderr (from
2b2bdaa793d42d58f7048d46859224041d4ea6a5). Is being done in this branch
anyway.
run_component: there is extra code before running a component in 3.0
(inherited from 2.2x): set MBEDTLS_TEST_CONFIGURATION, create seedfile,
redirection for --quiet. Reconcile with the new code.

aa8a348022 Detect errors on the left-hand side of a pipeline
fe7bf17a6c component_test_cmake_out_of_source: simplify and fix error handling
06e1d09cdb Remove obsolete functions

Re-done as "Stop dispatching through obsolete functions"

88f52fd972 Fix double reporting when the last command of a function fails
c7e15e6850 Better not function

Conflict due to consecutive changes (obsolete functions kept in rebase).

4ac7b02f05 Complain if an unsupported component is explicitly requested
8ea8ea2ad3 Remove code that is useless now that components run in a subshell
1e151e23af Unify ASan options in make builds (continued)

Was already done on this branch

358e5581b7 Don't restore config.h before backing it up

Adjust code and message to the presence of $CRYPTO_CONFIG

d9a5bf1a1e Keep going after a shell "[" a.k.a. "test" fails
New commits:

Remove barely-used redirect functions
Heed --quiet when running make generated_files

Looked at this from user's perspective. I am still able to do what I usually do locally with all.sh: run one or a few specified components. I am ok with the changes from user's perspective and particularly happy with "There is now an error if you pass an unknown component name." and "Changes to **/Makefile are no longer systematically silently discarded.".

There is now an error if you pass an unknown component name

Great, thanks for implementing this! As a side note, it broke some of my personal scripts that were doing tests/scripts/all.sh -k cleanup as a way of cleaning up after / in case of in-tree cmake builds. (Previously those scripts invoked find and so on directly, but were never up-to-date.) I'm absolutely not complaining about this change breaking my scripts because I was clearly abusing all.sh (also I still have the easy work-around of doing tests/scripts/all.sh -q '*nonesuch' for now), but I was wondering if this is something we should support, for example with an explicit --cleanup command-line option. It would also be useful for example after a component fails when running all.sh without -k.
I'm mentioning that now because perhaps you'll feel like doing something about it while you're improving all.sh; OTOH it's out of the original scope of the PR, so absolutely feel free to ignore the suggestion.

an explicit --cleanup command-line option. It would also be useful for example after a component fails when running all.sh without -k.

I've added a --restore option (which I think is a more accurate name: I find --cleanup more ambiguous as to what it does, does it run additional cleanup first? last?). I'm not sure if it's the semantics you wanted though, if it's not I'll take it out or ask you to adjust it as desired.

I'm not sure if it's the semantics you wanted though

It is, thanks for adding that. Actually looking at the implementation I realise a bit late that there was already a clean way of achieving that effect: tests/scripts/all.sh --except '*'. But the new option makes it more clearer I think.

Unsure if this is something we need to backport? Robustness improvements are less useful for branches that are in maintenance mode.

Yes, this does need to be backported. Otherwise every backport that involves changes to all.sh (and this is pretty common) would require significant modifications.

(Removing the "needs: review" label until the CI is fixed.)

(Removing the "needs: review" label until the CI is fixed.)

I'm afraid I disagree. The reason for the CI failures is known and documented: we need bash installed on our FreeBSD images, and there's a ticket for that (other than FreeBSD, all tests are passing). Clearly this PR can't be merged until this is resolved, but in the meantime I don't think that's an obstacle to reviewing and even backporting. So I'm reverting the label, but please tell if you think I'm missing something.

So I'm reverting the label, but please tell if you think I'm missing something.

That is fine, I don't think you are missing anything and the CI will be updated soon and will pass anyway.

The CI now fully passes (the pr-merge job on Jenkins passed even though that's not reflected on github). This is now only waiting for backports. @tom-daubney-arm I assume you'll be working on backporting?

Hi @mpg You are correct, I am working on the backporting today.


Cannot create media services account from command line

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az ams account sp create
Errors:
The command failed with an unexpected error. Here is the traceback:
list index out of range
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 718, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/ams/_exception_handler.py", line 16, in ams_exception_handler
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/ams/operations/sp.py", line 78, in create_or_update_assign_sp_to_mediaservice
    return _update_sp(cmd, graph_client, aad_sp, ams, account_name, resource_group_name,
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/ams/operations/sp.py", line 111, in _update_sp
    app_object_id = _get_application_object_id(graph_client, app_id)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/ams/operations/sp.py", line 237, in _get_application_object_id
    return list(client.application_list(filter="appId eq '{}'".format(identifier)))[0]['id']
IndexError: list index out of range

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ams account sp create -a {} -g {} -n {} --role {}

Expected Behavior
Environment Summary
Linux-5.19.0-32-generic-x86_64-with-glibc2.35, KDE neon 5.27
Python 3.10.8
Installer: DEB

azure-cli 2.45.0

Extensions:
account 0.2.5

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

Additional Context

route to CXP team

Closing as won't fix as Azure Media Services was retired on 6/30/2024 (https://aka-ms/ams-retirement).


packbeam --external requires full path

Surprisingly, it seems that
rebar3 atomvm packbeam -e estdlib/estdlib.avm -e eavmlib/eavmlib.avm

doesn't work and fails with enoent, while:
rebar3 atomvm packbeam -e `pwd`/estdlib/estdlib.avm -e `pwd`/eavmlib/eavmlib.avm

does work.
I guess the current working directory is not where rebar3 is invoked from. It should at least be mentioned in documentation.

Indeed. It is only vaguely implied in the example of packing an external file… But not stated explicitly. I will change the help to say this should be the full path to the avm file, and make it clear in the documentation as well.

Please take a look at PR #36, it enables using relative paths as well for external AVMs.


Add editor icon button tooltip

What's this, then?

Oh no lol, didn't find it until reload my project.


dapper integration

Hi guys,
any forecast for integration with dapper?
I'm doing some tests and using the blocto, what would be the difference using the dapper?

Thanks for the issue.
Dapper wallet integration is not planned. It may happen in the future. Be sure to check out Discord for announcements!
:) onflow.org/discord


Breadcrumb Component

🚀 Feature Proposal
A breadcrumb component
Motivation
Requested component in hospitalrun frontend issue:

Add breadcrumbs #1690

Example
Home > Patients > Patient 1 > Patient 1 Subpage

breadcrumb components has already been created in issue #21 and pr #142

Checkout the storybook here: https://components.hospitalrun.io/?path=/story/breadcrumbs--breadcrumb


Optional feature for openslide images

Hiya, would this project be interested in https://crates.io/crates/openslide-rs support being added, probably as an optional feature?
I suggest optional as the underlying https://openslide.org/ is LGPL, so some people might want to disable that.

Hi!
Thanks for bringing OpenSlide to my attention, it's the first I've come across it. Currently I'm working on support for the RGB JPEG2000 -> JPEG path, but afterwards tiled TIFFs and other colorspaces will be on my radar.
OpenSlide seems like a good candidate to get the ball rolling on that, although from a look over the API it seems like it wants to be responsible for all IO (the only way to get an openslide_t* is by giving the API a path to open and then IO is done transparently on calls to get pixel data, is that right?).
My preference would be for a decoder we have a bit more control over (in terms of allocations and IO), but we'd be happy to carry an implementation that only supports filesystems and pulls stripes of image data from a decoder running in a blocking thread pool for OpenSlide users who'd be interested in that. If we want to get the most out of integrating Laya with OpenSlide (i.e. async IO, using alternative data sources like AWS S3, setting upper bounds on resource limits) then we can also look into patches on the OpenSlide side.
In the long-term we'll probably end up with a separate TIFF decoder, but there are a bunch of novel image formats supported by OpenSlide I haven't seen anywhere else that it's worth carrying the support for.

Hey Gary, this project will definitely want its own TIFF support, and will likely want to use a different strategy for standard formats like OME-TIFF and OME-NGFF, and DICOM.
But unfortunately for the extremely proprietary formats that this type of scanners use, there is only openslide (C, LGPL) and bio-formats (Java; GPL, but there is https://github.com/AzHicham/bioformats-rs , also by @AzHicham who is doing all the Rust stuff in this area).  Due to how much reverse engineering is involved, creating a Rust project for handling these formats is going to be hard to sustain.
Having a custom openslide sounds like a good idea to get the most out of it.  But good ideas are cheap ;-)  IMO we'd want to do this in https://github.com/AzHicham/openslide-sys if at all.


how to specify the region in pyspark?

org.apache.spark.sql.execution.datasources.hbase.InvalidRegionNumberException: Creating a new table sh
ould specify the number of regions which must be greater than 3.
i saw in issue doing like this:
df.write.options(map(catalog=cat, HBaseTableCatalog.newTable -> "5")).mode("append").format(data_source_format).save()
but how to pass param 'HBaseTableCatalog.newTable'  in pyspark?

sorry,  i misspelled the table name.
i works now

@shen-xianpeng so how did you passed the newTable parameter in pyspark finally?

@ser0t0nin
You can refer to this page for passing the demanded parameters.
https://github.com/apache/hbase/blob/master/hbase-spark/src/main/scala/org/apache/spark/sql/datasources/hbase/HBaseTableCatalog.scala
The corresponding parameter of "newTable" is "newtable".
You can assign the number of regions by adding this line :
df.write.options( catalog = your_catalog, newtable = num_of_regions ).format( your_format  ).save()
Work for me.


docs: Add developers guide page about BPF testing framework

This PR adds a new page to the "For developers/Testing" section.
The page covers how to run and create BPF tests within the framework
that was added in #20017

I've got a few suggestions, and tons of nitpicks

Ah, I see my English is failing me. Thanks, all of them good suggestions.


Some admin_level=2 boundary relations not showing up in rendered tiles

Bug description
There are some apparently-properly tagged admin_level=2 relations where borders are not rendered in the browser.
OHM user Alanantos has built these relations, couldn't figure out why they weren't rendering, and after review, I could not either.
I think this has happened before and there was a straightforward fix, but I cannot remember what that was or figure it out.
Here are a couple of sample relations:
https://www.openhistoricalmap.org/relation/2744157#map=7/51.856/21.237&layers=O&date=976-02-20&daterange=900-01-01,1000-01-01

https://www.openhistoricalmap.org/relation/2740036#map=6/51.713/20.391&layers=O&date=964-03-28&daterange=900-01-01,1000-01-01
(Separately, there is enough data (yay!) that https://vtiles.openhistoricalmap.org/#12.6/52.15908/18.76273 is almost useless, because there are too many results in the inspector list to check to see if the relation was making it through on the vector tiles. See below for screenshot.)
Repro Steps
Please provide detailed instructions to reproduce the behavior:

Go to either URL above
Unclick the relation left-nav to remove the relation highlight
Play with the timeslider / zoom in and out to see if the boundaries ever appear
Double-check the tags on the relation to make sure nothing obvious is missing

Screenshots & URL
Overloaded vtiles:

I'm really not sure why these are not making it to the tiles.
One thing I noticed that we have one large, slow-loading tile, https://vtiles.openhistoricalmap.org/maps/osm/4/8/5.pbf, which comes through with thousands of elements:

Including these 6 similar looking Admin 2 boundaries that are coming through:

But there's no land_ohm relation with ID -2744157 in that tile.
Let's see if @geohacker, @batpad, or @Rub21 have any thoughts about why these are not coming through to the tiles

Do this still an issue?  the admin 2 is showing in the vtiles


Driver Detection Issue

When running Blackbone with EAC, it says, "Please close Xenos Injector." I have traced this back to the driver itself being detected by EAC (probably signature, or whatever.) Anyhow, what degree of change in the driver is necessary to remove this signature? Class name changes? Obfuscation? VMProtection?

You probably should start from changing all driver strings.


default.nix: More thoroughly explain the macOS upgrade process

Motivation for this change
The install instructions broke an IRC user, floop's setup and was pretty frustrating.
error: evaluation aborted with the following error message: '
This version of Nixpkgs requires Nix >= 2.0, please upgrade:

- If you are running NixOS, `nixos-rebuild' can be used to upgrade your system.

- If you have a multi-user Nix installation on macOS, update Nix by running:

      sudo -i nix-store -r /nix/store/lc3ymlix73kaad5srjdgaxp9ngr1sg6g-nix-2.1.1;
      sudo -i /nix/store/lc3ymlix73kaad5srjdgaxp9ngr1sg6g-nix-2.1.1/bin/nix-env --uninstall nix
      sudo -i /nix/store/lc3ymlix73kaad5srjdgaxp9ngr1sg6g-nix-2.1.1/bin/nix-env -i /nix/store/lc3ymlix73kaad5srjdgaxp9ngr1sg6g-nix-2.1.1
      sudo launchctl unload /Library/LaunchDaemons/org.nixos.nix-daemon.plist
      sudo launchctl load /Library/LaunchDaemons/org.nixos.nix-daemon.plist

    if you have done that and still see this error message, also run:

      /nix/store/lc3ymlix73kaad5srjdgaxp9ngr1sg6g-nix-2.1.1/bin/nix-env -i /nix/store/lc3ymlix73kaad5srjdgaxp9ngr1sg6g-nix-2.1.1

- If you installed Nix using the install script (https://nixos.org/nix/install),
  it is safe to upgrade by running it again:

      curl https://nixos.org/nix/install | sh

For more information, please see the NixOS release notes at
https://nixos.org/nixos/manual or locally at
/home/grahamc/projects/nixpkgs/doc/manual/release-notes.

If you need further help, see https://nixos.org/nixos/support.html

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

I'm not sure if this is the right place to fix this, shouldn't this be in the installer instead?

@Infinisil the installer and the Nix documentation can't be updated without doing a full Nix release, unfortunately.

@grahamc Is there a problem with doing a 2.1.1 release to fix the macOS installation? It's obviously a problem with Nix, not nixpkgs

I ran into a kind of similar problem today.
A coworker followed https://nixos.org/nix/manual/#ch-upgrading-nix to upgrade, but ended up actually downgrading back to 1.11.6 due to the channel. I had either perform an intermediate upgrade or grep for *fallback* and run something like this to recover because the current message was useless.
find /nix/var/nix/profiles -name '*fallback*.nix'
sudo -i NIX_SSL_CERT_FILE=$NIX_SSL_CERT_FILE nix-env -i /nix/store/lc3ymlix73kaad5srjdgaxp9ngr1sg6g-nix-2.1.1
sudo launchctl remove org.nixos.nix-daemon
sudo launchctl load /Library/LaunchDaemons/org.nixos.nix-daemon.plist

Any updates on this pull request, please?

Is this PR still up-to-date after the macOS Catalina issues (eg https://github.com/NixOS/nix/issues/2925)?


Readme typo for including clicky-menus.js.

@ShaunDychko Thanks so much for catching this and submitting the fix. Merged.


Sorting issue

Hello,
There is an issue with the sorting. It works very well if the fields are strings but it doesn't works with numeric field because all field are sorted like a string so if I try to sort 1, 2, 10, 3 the results will be 1, 10, 2, 3 instead of 1, 2, 3, 10.
I think we just have to check the column type and update the sortBy method depending of this type.
I can submit a patch if you want.
Thanks,
Régis

I have pushed the fix if you want

I has been approved.


missing products

Version of flutter_inapp_purchase
5.4.2
Platforms you faced the error (IOS or Android or both?)
IOS
Expected behavior
Should return all products.  There are 8 products defined in my App Store configuration.

Actual behavior
getProducts only returns 6 of the products.
[log] FINER: 12:5:43:239: PurchasePage: initConnection result: true
[log] FINER: 12:5:46:756: PurchasePage: _getProduct Concise Buddha Path English id_concise_dzogchen_buddha_path_english
[log] FINER: 12:5:46:758: PurchasePage: _getProduct Concise Buddha Path German id_concise_dzogchen_buddha_path_german
[log] FINER: 12:5:46:759: PurchasePage: _getProduct Concise Buddha Path Russian id_concise_dzogchen_buddha_path_russian
[log] FINER: 12:5:46:760: PurchasePage: _getProduct Buddha Path 2nd Ed. English id_second_edition_english
[log] FINER: 12:5:46:761: PurchasePage: _getProduct Buddha Path 2nd Ed. S. Chinese id_second_edition_simplified_chinese
[log] FINER: 12:5:46:762: PurchasePage: _getProduct Buddha Path 2nd Ed. T. Chinese id_second_edition_traditional_chinese

Tested environment (Emulator? Real Device?)
tested on real device
iPad (6th generation)
iOS 17.2
Steps to reproduce the behavior

this was a problem in my code.


Not able to edit code and answers after submission closes

Would you mind elaborating on this? When the submission closes, no more changes can be made and that is the intended behaviour.

Yes! I mean that it will be useful if we have a feature that allows us to make changes to the submitted answers for revision.
For example if the path was not submitted, everything will be red right, but if the student wants to revise, it would help if they can try again and then save the green version for themselves so when they revise they do not have to keep clicking again to check which one is the correct one and be unable to save.
Same goes for the code, for example in the path they have the mini coding questions that need to pass the test cases, they can try to write it after submission and run the test cases, but they have no way to save the code that they have edited

I think the more general topic is "versioning". At the moment, we only keep one copy of student submissions. It would be GREAT if we would keep all saved versions with time stamps. This would by itself be a great CP3108 project, involving frontend and backend.


Add new classes for form label sizing

Add control-label-xxs and control-label-xxl classes to match default custom checkboxes / radio buttons and also very large custom form controls such as toggle switches.

Completed.


Fixed bug in simnet when installing iptables rules in the host namespace

-- Insert rather than append the iptables rules that redirect packets to the simnet sn-FORWARD table.  This ensures packets flow through the simnet rules before any system firewall rules installed in the host namespace.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3795340). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #391   +/-   ##
=========================================
  Coverage          ?   54.48%
=========================================
  Files             ?      341
  Lines             ?    58436
  Branches          ?        0
=========================================
  Hits              ?    31836
  Misses            ?    26600
  Partials          ?        0

Continue to review full report at Codecov.

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


Close outcomeCh in Task

When task ends its run, close outcomeCh

Can't create issue on Github, there are some data races to fix:
go test -trimpath -race -count=1 -timeout 60s ./...
==================
WARNING: DATA RACE
Write at 0x00c00030a090 by goroutine 94:
  github.com/jamestrandung/go-concurrency/async.(*batcher[...]).Shutdown()
      github.com/jamestrandung/go-concurrency/async/batch.go:167 +0x1b4
  github.com/jamestrandung/go-concurrency/async.(*batcher[...]).Shutdown()
      <autogenerated>:1 +0x38
  github.com/jamestrandung/go-concurrency/async.TestBatcher_AutoProcessOnInterval.func4()
      github.com/jamestrandung/go-concurrency/async/batch_test.go:116 +0x44
  runtime.deferreturn()
      runtime/panic.go:476 +0x30
  testing.tRunner()
      testing/testing.go:1446 +0x188
  testing.(*T).Run.func1()
      testing/testing.go:1493 +0x40

Previous read at 0x00c00030a090 by goroutine 95:
  github.com/jamestrandung/go-concurrency/async.NewBatcher[...].func1()
      github.com/jamestrandung/go-concurrency/async/batch.go:82 +0xc8

Goroutine 94 (running) created at:
  testing.(*T).Run()
      testing/testing.go:1493 +0x55c
  testing.runTests.func1()
      testing/testing.go:1846 +0x90
  testing.tRunner()
      testing/testing.go:1446 +0x188
  testing.runTests()
      testing/testing.go:1844 +0x6c0
  testing.(*M).Run()
      testing/testing.go:1726 +0x870
  go.uber.org/goleak.VerifyTestMain()
      go.uber.org/goleak@v1.1.12/testmain.go:53 +0x40
  github.com/jamestrandung/go-concurrency/async.TestMain()
      github.com/jamestrandung/go-concurrency/async/goleak_test.go:10 +0x324
  main.main()
      _testmain.go:145 +0x308

Goroutine 95 (running) created at:
  github.com/jamestrandung/go-concurrency/async.NewBatcher[...]()
      github.com/jamestrandung/go-concurrency/async/batch.go:66 +0x31c
  github.com/jamestrandung/go-concurrency/async.TestBatcher_AutoProcessOnInterval()
      github.com/jamestrandung/go-concurrency/async/batch_test.go:105 +0xe4
  testing.tRunner()
      testing/testing.go:1446 +0x188
  testing.(*T).Run.func1()
      testing/testing.go:1493 +0x40
==================
--- FAIL: TestBatcher_AutoProcessOnInterval (0.11s)
    testing.go:1319: race detected during execution of test
==================
WARNING: DATA RACE
Read at 0x00c0005149f0 by goroutine 827:
  github.com/jamestrandung/go-concurrency/async.TestBurstCapacity()
      github.com/jamestrandung/go-concurrency/async/worker_pool_test.go:128 +0x238
  testing.tRunner()
      testing/testing.go:1446 +0x188
  testing.(*T).Run.func1()
      testing/testing.go:1493 +0x40

Previous write at 0x00c0005149f0 by goroutine 828:
  github.com/jamestrandung/go-concurrency/async.(*WorkerPool).processWaitingQueue()
      github.com/jamestrandung/go-concurrency/async/worker_pool.go:312 +0x180
  github.com/jamestrandung/go-concurrency/async.(*WorkerPool).dispatch()
      github.com/jamestrandung/go-concurrency/async/worker_pool.go:200 +0xd4
  github.com/jamestrandung/go-concurrency/async.NewWorkerPool.func1()
      github.com/jamestrandung/go-concurrency/async/worker_pool.go:80 +0x34

Goroutine 827 (running) created at:
  testing.(*T).Run()
      testing/testing.go:1493 +0x55c
  testing.runTests.func1()
      testing/testing.go:1846 +0x90
  testing.tRunner()
      testing/testing.go:1446 +0x188
  testing.runTests()
      testing/testing.go:1844 +0x6c0
  testing.(*M).Run()
      testing/testing.go:1726 +0x870
  go.uber.org/goleak.VerifyTestMain()
      go.uber.org/goleak@v1.1.12/testmain.go:53 +0x40
  github.com/jamestrandung/go-concurrency/async.TestMain()
      github.com/jamestrandung/go-concurrency/async/goleak_test.go:10 +0x324
  main.main()
      _testmain.go:145 +0x308

Goroutine 828 (running) created at:
  github.com/jamestrandung/go-concurrency/async.NewWorkerPool()
      github.com/jamestrandung/go-concurrency/async/worker_pool.go:80 +0x318
  github.com/jamestrandung/go-concurrency/async.TestBurstCapacity()
      github.com/jamestrandung/go-concurrency/async/worker_pool_test.go:105 +0x8c
  testing.tRunner()
      testing/testing.go:1446 +0x188
  testing.(*T).Run.func1()
      testing/testing.go:1493 +0x40
==================
--- FAIL: TestBurstCapacity (0.50s)
    testing.go:1319: race detected during execution of test
==================
WARNING: DATA RACE
Write at 0x00c000072390 by main goroutine:
  runtime.recvDirect()
      runtime/chan.go:348 +0x7c
  github.com/jamestrandung/go-concurrency/async.ExampleRepeat()
      github.com/jamestrandung/go-concurrency/async/repeat_test.go:60 +0x158
  testing.runExample()
      testing/run_example.go:63 +0x2e8
  testing.runExamples()
      testing/example.go:44 +0x1e8
  testing.(*M).Run()
      testing/testing.go:1728 +0x930
  go.uber.org/goleak.VerifyTestMain()
      go.uber.org/goleak@v1.1.12/testmain.go:53 +0x40
  github.com/jamestrandung/go-concurrency/async.TestMain()
      github.com/jamestrandung/go-concurrency/async/goleak_test.go:10 +0x324
  main.main()
      _testmain.go:145 +0x308

Previous read at 0x00c000072390 by goroutine 2001:
  runtime.chansend1()
      runtime/chan.go:146 +0x2c
  github.com/jamestrandung/go-concurrency/async.ExampleRepeat.func1()
      github.com/jamestrandung/go-concurrency/async/repeat_test.go:49 +0x5c
  github.com/jamestrandung/go-concurrency/async.Repeat.func1()
      github.com/jamestrandung/go-concurrency/async/repeat.go:23 +0x78
  github.com/jamestrandung/go-concurrency/async.Repeat.func2()
      github.com/jamestrandung/go-concurrency/async/repeat.go:34 +0xc8
  github.com/jamestrandung/go-concurrency/async.InvokeInSilence.func1()
      github.com/jamestrandung/go-concurrency/async/task.go:153 +0x44
  github.com/jamestrandung/go-concurrency/async.(*task[...]).doRun.func1()
      github.com/jamestrandung/go-concurrency/async/task.go:311 +0xb0

Goroutine 2001 (running) created at:
  github.com/jamestrandung/go-concurrency/async.(*task[...]).doRun()
      github.com/jamestrandung/go-concurrency/async/task.go:303 +0x28c
  github.com/jamestrandung/go-concurrency/async.(*task[...]).Run.func1()
      github.com/jamestrandung/go-concurrency/async/task.go:238 +0x58
==================
FAIL
FAIL	github.com/jamestrandung/go-concurrency/async	9.839s
FAIL

@TheMickeyMike Thanks for the patch and sorry for the slow reply. I've been the only one working on this repo. Hence, I didn't pay attention to the email notifications when someone were raising an MR.
As for the race conditions, I investigated the code that were mentioned in the logs. I've merged some code to fix the race conditions in the batchers (https://github.com/jamestrandung/go-concurrency/pull/4/files).
As for the other cases, they're just some happening in some tests where we're trying to produce a scenario and do some checkings on the internal fields. It shouldn't have any tangible effects on client codes.
Lemme know what you think :)


Compiling on Xcode 8.3

Has anyone gotten this compiled on Xcode 8.3? It seems to have trouble compiling the diff library.

Hi Michael, can you check which version of Bond and Diff.swift you're using? There were issues compiling Diff.swift under early betas of Xcode 8.3, but I believe the Swift team fixed those a couple of betas ago.

Actually, bear with me - the fixes to Diff.swift aren't in a release yet. We'll get this sorted shortly, but in the meantime you can reference https://github.com/wokalski/Diff.swift/pull/49 for more details.


Release candidate 1.6

Description of PR
This PR adds the highlights for the new release and does some minor docstrings-clean-up of the new lazy_ready_iiasa() function.

Published on test-pypi successfully, see https://test.pypi.org/project/pyam-iamc/1.6.0rc1/


Update issue templates to include flaky test build

Maybe a small section for a general description of how (and how often) it fails? If it fails 1/50 times it's less urgent than one that fails 1/2


PR_END_OF_FILE_ERROR in Firefox on connection bypass

@adguard-bot commented on Tue Jul 13 2021
Issue URL (Incorrect Blocking)
https://www.torproject.org/
Comment
Username: @Morku

The issue PR_END_OF_FILE_ERROR always appears in first visit with Firefox. When I reload the page, it loads fine. It's then temporary fixed for this one AdGuard instance. I have noticed, that the HTTPS certificate is then "Let's Encrypt" instead AdGuard, but it's not in HTTPS Exclusion list. When I completely close AdGuard and reopen it, the issue recovers and I have to reload the page. It happens a second time, when you want start download e.g. https://dist.torproject.org/torbrowser/10.5/torbrowser-install-win64-10.5_de.exe. Disable filtering Ads, Stealth and Extension in AdGuard does not fix the issue. Reinstall HTTPS certificate also doesn't fix the issue. As workaround I had to add torproject.org in HTTPS exclusion list to fix it (or disable AG). This issue only exist in Firefox. I tested Edge Chromium and the page loads fine directly at first visit with AG enabled.

Screenshots

Screenshot 1

Screenshot 2

Screenshot 3

System configuration

Information
value

AdGuard product:
AdGuard for Windows v7.6.1

Browser:
Firefox

AdGuard driver:
WFP

Stealth mode options:
Hide your search queries, Send Do-Not-Track header, Remove X-Client-Data header from HTTP requests, Strip URLs from tracking parameters

DNS filtering:
server: Cloudflare DNS-over-HTTPSfilters: "AdGuard DNS filter" with AdGuard Home

Filters:
Ad Blocking:AdGuard Mobile Ads, AdGuard BasePrivacy:AdGuard Tracking Protection, AdGuard URL TrackingSocial Widgets:AdGuard Social MediaAnnoyances:AdGuard AnnoyancesLanguage-specific:AdGuard Russian, AdGuard German

Userscripts:
https://userscripts.adtidy.org/release/adguard-extra/1.0/adguard-extra.user.js

@sfionov please triage, how important is this one?

I have noticed the same happens with anonym.to redirection pages, e.g. https://anonym.to/?http://www.skype.com
Steps and behavior are exactly the same as first post, only on Firefox.
Screenshot-1:

Another issue with AdGuard for Windows v7.11.2
https://github.com/AdguardTeam/AdguardFilters/issues/135447

Another one ~bite the dust~
https://github.com/AdguardTeam/AdguardFilters/issues/137334


unable to build go-sqlite3 on windows 10

when i use the command
go get github.com/mattn/go-sqlite3

i get error
cc1.exe: sorry, unimplemented: 64-bit mode not compiled in

i have gcc in my system  ( see screenshot below)

my go env output
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\dark knight\go
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\DARKKN~1\AppData\Local\Temp\go-build281473903=/tmp/go-build -gno-record-gcc-switches
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config

install TDM-GCC64
https://sourceforge.net/projects/tdm-gcc/files/TDM-GCC Installer/tdm64-gcc-5.1.0-2.exe/download

please install TDM64-GCC (it's a MinGW-w64) compiler suite, easy to use
[SourceForge](https://sourceforge.net/projects/tdm-gcc/files/TDM-GCC Installer/tdm64-gcc-5.1.0-2.exe/download)

1)enter to console export CC=x86_64-pc-cygwin-gcc
2) try again
It`s help me

What OS you are doing this on? What you seen? log file? go env ?

@hackertron Is this is current ? Please remove CYGWIN and use TDM suggested by @firasmdar
Issue closed; please re-open if required.


major microservice updates

will be major rev 5.0.0
opt-in request logging
opt-in create server
eslint -> standard

@tone81 - technically, this module already has a node 8.9.4 dependency if you check the package.json

Oh, that Jonathan. :shakesfist:


[Android] Integrate with @react-native-firebase/messaging

SDK version: 2.0.1
Environment: Development and Production
Describe the bug
in our application, we already add the @react-native-firebase/messaging for some custom use cases.
we notice that rich push notifications (data-only) are not working on android. with more investigation, we notice
that @react-native-firebase/messaging is overwriting the CustomerIO service, hence, there is no handler registered for the data-only push notification.
P.S: because of the CustomerIO UNNotificationServiceExtension in IOS,  rich push notifications are working properly
To Reproduce

create a react native app
initialize the FCM message using @react-native-firebase/messaging documents.
initialize the CustomerIO base on documents.
notification-only push notifications works, but data-only push notifications are not working (even though we know that they are delivered to the application by looking at the logs)
after removing the @react-native-firebase/messaging package, data-only push notifications work as expected

Additional context
I believe the problem can be solved by using a customer FirebaseMessagingService on android to support both CustomerIO and @react-native-firebase/messaging package.

Hey, @vanenshi thank you for reaching out, we are aware of this, and we should probably document this too. If you are using any other push services, CIO integration might not work as we are intending to take over all the load from creating the token to handling the push and tracking ourselves.
This is a feature on our roadmap that we intend to get to in the next couple of releases.

Hi guys, is this issue getting closed by https://github.com/customerio/customerio-reactnative/pull/117?

Hey @vanenshi, apologies for the delay and yes This issue is resolved now. You can check out the docs for more information on it.
RN-Firebase, will receive the data only payload and you can forward it to our SDK. Thanks for your patience.


variableWidth and Infinite gives a white space issue

View Fiddle here: http://jsfiddle.net/tenold/t8apgqt9/
Basically when you keep clicking next, once it gets to the end there is a white space before you hit next again.
Is there anyway to get rid of this extra spacing that causes an odd experience?
Screenshot:

i have the same Problem.

That issue is already considered as a bug here.

Closing this issue as it is a duplicate. Please follow this issue: https://github.com/kenwheeler/slick/issues/1207


Storybook knob titled "Link" should be "Image"

https://github.com/dpc-sdp/ripple/blob/master/packages/components/Molecules/Card/stories.js#L68
Storybook knob for Card Image Navigation component reads "Link", though it's for the "Image" property.

@rubyllamadora does that relates to the storybook control work you are doing now?

@rubyllamadora does that relates to the storybook control work you are doing now?

Yes. This should go away once we implement control addon.