data read

0%|                                                                                | 0/291 [00:00<?, ?it/s]/home/caffe/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/functional.py:2539: UserWarning: Default upsampling behavior when mode=bilinear is changed to align_corners=False since 0.4.0. Please specify align_corners=True if the old behavior is desired. See the documentation of nn.Upsample for details.
"See the documentation of nn.Upsample for details.".format(mode))
TRAIN (1) | Loss: 0.334 | Acc 0.92 mIoU 0.69 | B 0.62 D 0.03 |:  27%|███████▋                    | 80/291 [00:49<01:51,  1.90it/s]Traceback (most recent call last):
File "train.py", line 61, in
main(config, args.resume)
File "train.py", line 42, in main
trainer.train()
File "/home/caffe/语义分割/pytorch_segmentation/base/base_trainer.py", line 101, in train
results = self._train_epoch(epoch)
File "/home/caffe/语义分割/pytorch_segmentation/trainer.py", line 49, in _train_epoch
for batch_idx, (data, target) in enumerate(tbar):
File "/home/caffe/anaconda3/envs/pytorch/lib/python3.6/site-packages/tqdm/_tqdm.py", line 1017, in iter
for obj in iterable:
File "/home/caffe/语义分割/pytorch_segmentation/base/base_dataloader.py", line 81, in iter
self.preload()
File "/home/caffe/语义分割/pytorch_segmentation/base/base_dataloader.py", line 64, in preload
self.next_input, self.next_target = next(self.loaditer)
File "/home/caffe/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 582, in next
return self._process_next_batch(batch)
File "/home/caffe/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 608, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
FileNotFoundError: Traceback (most recent call last):
File "/home/caffe/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/caffe/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/caffe/语义分割/pytorch_segmentation/base/base_dataset.py", line 126, in getitem
image, label, image_id = self._load_data(index)
File "/home/caffe/语义分割/pytorch_segmentation/dataloaders/voc.py", line 37, in _load_data
image = np.asarray(Image.open(image_path), dtype=np.float32)
File "/home/caffe/anaconda3/envs/pytorch/lib/python3.6/site-packages/PIL/Image.py", line 2770, in open
fp = builtins.open(filename, "rb")
FileNotFoundError: [Errno 2] No such file or directory: 'data/VOC-200-200-0/VOCdevkit/VOC2012/JPEGImages/.jpg'

the picture is *.jpg,but i do not know the problem

My guess is that one of the files names is empty, remove it from the variables in self.files
self.files = [i for i in self.files if len(i) > 0]

sorry，i do not know that mens.i should remove the one  of the pictures names or not.em …the picture name have the space，that couse the code can not read the name of picture
---Original---
From<EMAIL_ADDRESS>Date: Fri, Dec 6, 2019 23:39 PM
To<EMAIL_ADDRESS>Cc: "Author"<author<EMAIL_ADDRESS>Subject: Re: [yassouali/pytorch_segmentation] data read (#34)
My guess is that one of the files names is empty, remove it from the variables in self.files
self.files = [i for i in self.files if len(i) > 0]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Was the issue solved ?


Restrict access to home to staff users (all pages now require login)

@glemson yea or nay?

I think it's safer to just lock everything down - we can always re-explore this another day.


[Bug] Inconsistent Metadata Read

Search before asking

[X] I searched in the issues and found nothing similar.

Read release policy

[X] I understand that unsupported versions don't get bug fixes. I will attempt to reproduce the issue on a supported version of Pulsar client and Pulsar broker.

Version
pulsar:3.3.0
Minimal reproduce step
public class Data{

   String id;
   Action action;
}
enum Action{
  A,
  B
}

public class TestMetadata extends BaseResources<Data>{

    public static final String PATH = "/my-test-zk-path/data";

    public TestMetadata(MetadataStore configurationStore, int operationTimeoutSec) {
        super(configurationStore, Data.class, operationTimeoutSec);
    }

    public CompletableFuture<Void> createAsync(Data data) {
        return createAsync(joinPath(PATH, data.id()), data);
    }

    public CompletableFuture<Data> readAsync(String id) {
        CompletableFuture<Data> future = new CompletableFuture();
        getAsync(joinPath(PATH, id)).thenAccept(data -> {
            if (!data.isPresent()) {
                future.completeExceptionally(new MetadataStoreException.NotFoundException("Not found"));
            } else {
                log.info("reading id:{}, job:{}", id, data.get());
                future.complete(data.get());
            }
        });
        return future;
    }

    public CompletableFuture<Void> updateActionAsync(String id, Data.Action action) {
        CompletableFuture future = new CompletableFuture();
        readAsync(id)
                .thenCompose(__ -> setAsync(joinPath(PATH, sourceBrokerId), (old) -> {
                    log.info("updating action. id:{}, old:{} to action:{}", id, old, action);
                    old.setAction(action);
                    return old;
                }))
                .thenCompose(__ -> readAsync(sourceBrokerId))
                .whenComplete((job, ex) -> {

                    // somehow this shows the latest value that was written correctly.
                    log.info("updated action. id:{}, job:{}", id, job);
                    if (ex != null) {
                        if (!future.isCompletedExceptionally()) {
                            future.completeExceptionally(FutureUtil.unwrapCompletionException(ex));
                        }
                    } else {
                        future.complete(null);
                    }
                });
        return future;
    }

}

Given above the test classes above, run the following test.

TestMetadata metadata = new TestMetadata(pulsar.getLocalMetadataStore(),
                pulsar.getConfiguration().getMetadataStoreOperationTimeoutSeconds());

Data data = new Data("1", A);
1. metadata.createAsync(data).join();
3. metadata.updateActionAsync("1", B).join();
4. metadata.updateActionAsync("1", A).join();
// keep trying to verify the data.action is set to A. But it continue to show the old value, B.
5. metadata.readAsync("1").join() => data : {"1", B}
...
30. metadata.readAsync("1").join() => data : {"1", B}

What did you expect to see?
The metadata is expected to read the last value that was written.
What did you see instead?
The metadata returned the old value for read op. (it seems like this inconsistent state can happen under some specific conditions)
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Please check to see if #18518 provides a way to resolve this.

Closing this issue. I cannot reproduce this anymore.


Update tty-screen dependency for fastlane

Fixes https://github.com/fastlane/fastlane/issues/10983

@taquitos what do you mean?


fix: make sure iframe background is translucent

some framework ( like next.js ) default css has
html {
    color-scheme: dark;
  }

so iframe background will be white like this ↓

I add color-scheme: light; make sure iframe background translucent

Thanks for this contribution @sanyu1225 ! 🚀
I am hoping you can add a changeset and I'll be happy to merge.
See above or follow below instructions.
from root: npx changeset

select package updated (fcl)
select patch
add a short description of the change

@gregsantos Done, Thanks for the reminder.

Thanks again for your contribution @sanyu1225 !


Removed javax.annotation.Generated occurrences

Fixes #2154
With Vavr 0.9.2 we build fine with JDK9. However, source and target compatibility are set to Java 1.8.
With this change Vavr build fine, even if we set source and target compatibility to Java 1.9 🤩

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2181   +/-   ##
=========================================
  Coverage     97.44%   97.44%
  Complexity     5260     5260
=========================================
  Files            92       92
  Lines         12063    12063
  Branches       1595     1595
=========================================
  Hits          11755    11755
  Misses          151      151
  Partials        157      157

Impacted Files
Coverage Δ
Complexity Δ

vavr/src-gen/main/java/io/vavr/API.java
92.79% <ø> (ø)
193 <0> (ø)
:arrow_down:

vavr/src-gen/main/java/io/vavr/Tuple3.java
100% <ø> (ø)
33 <0> (ø)
:arrow_down:

vavr/src-gen/main/java/io/vavr/Function4.java
95% <ø> (ø)
11 <0> (ø)
:arrow_down:

...vr/src-gen/main/java/io/vavr/CheckedFunction3.java
96.66% <ø> (ø)
14 <0> (ø)
:arrow_down:

vavr/src-gen/main/java/io/vavr/Function2.java
94.44% <ø> (ø)
11 <0> (ø)
:arrow_down:

vavr/src-gen/main/java/io/vavr/Function8.java
95.83% <ø> (ø)
11 <0> (ø)
:arrow_down:

vavr/src-gen/main/java/io/vavr/Function6.java
95.45% <ø> (ø)
11 <0> (ø)
:arrow_down:

vavr/src-gen/main/java/io/vavr/Tuple0.java
100% <ø> (ø)
14 <0> (ø)
:arrow_down:

vavr/src-gen/main/java/io/vavr/Function0.java
93.75% <ø> (ø)
11 <0> (ø)
:arrow_down:

...vr/src-gen/main/java/io/vavr/CheckedFunction7.java
97.05% <ø> (ø)
14 <0> (ø)
:arrow_down:

... and 20 more

Continue to review full report at Codecov.

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


Todo objects with duplicate STATUS can be generated

Even though when generating the data all property names are uppercased, this is not done before iterating the properties which means that all possible cases will generate duplicates. In particular using any other case in the save_todo call will fail since that will be considered a duplicate of the STATUS property inserted as a default by caldav itself.
Note that the error is probably hidden when using more forgiving CALDAV servers than mine: reading the traceback below it seems that the error is indeed raised the caldav library but that it only validates the event on PUT failure.
I use radicale which AFAICT refuses the VTODO because it also validates it with vobject (since I get the exact same error in the radicale logs)
>>> import caldav
>>> client = caldav.DAVClient("xxx", username="foo", password="bar")
>>> cal = client.principal().calendars()[0]
>>> x = cal.save_todo(summary="works")
>>> y = cal.save_todo(summary="workstoo", STATUS="COMPLETED")
>>> z = cal.save_todo(summary="chokes", status="COMPLETED")
ValidateError                             Traceback (most recent call last)
File ~/aaaaa/hh/lib/python3.11/site-packages/caldav/objects.py:768, in Calendar.save_todo(self, ical, no_overwrite, no_create, **ical_data)
    757 """
    758 Add a new task to the calendar, with the given ical.
    759
    760 Parameters:
    761  * ical - ical object (text)
    762 """
    763 t = Todo(
    764     self.client,
    765     data=self._use_or_create_ics(ical, objtype="VTODO", **ical_data),
    766     parent=self,
    767 )
--> 768 t.save(no_overwrite=no_overwrite, no_create=no_create, obj_type="todo")
    769 self._handle_relations(t.id, ical_data)
    770 return t

File ~/aaaaa/hh/lib/python3.11/site-packages/caldav/objects.py:2283, in CalendarObjectResource.save(self, no_overwrite, no_create, obj_type, increase_seqno, if_schedule_tag_match)
   2280     if seqno is not None:
   2281         self.icalendar_component.add("SEQUENCE", seqno + 1)
-> 2283 self._create(id=self.id, path=path)
   2284 return self

File ~/aaaaa/hh/lib/python3.11/site-packages/caldav/objects.py:2155, in CalendarObjectResource._create(self, id, path, retry_on_failure)
   2150 def _create(self, id=None, path=None, retry_on_failure=True):
   2151     ## We're efficiently running the icalendar code through the icalendar
   2152     ## library.  This may cause data modifications and may "unfix"
   2153     ## https://github.com/python-caldav/caldav/issues/43
   2154     self._find_id_path(id=id, path=path)
-> 2155     self._put()

File ~/aaaaa/hh/lib/python3.11/site-packages/caldav/objects.py:2146, in CalendarObjectResource._put(self, retry_on_failure)
   2142 if retry_on_failure:
   2143     ## This looks like a noop, but the object may be "cleaned".
   2144     ## See https://github.com/python-caldav/caldav/issues/43
   2145     self.vobject_instance
-> 2146     return self._put(False)
   2147 else:
   2148     raise error.PutError(errmsg(r))

File ~/aaaaa/hh/lib/python3.11/site-packages/caldav/objects.py:2137, in CalendarObjectResource._put(self, retry_on_failure)
   2134 def _put(self, retry_on_failure=True):
   2135     ## SECURITY TODO: we should probably have a check here to verify that no such object exists already
   2136     r = self.client.put(
-> 2137         self.url, self.data, {"Content-Type": 'text/calendar; charset="utf-8"'}
   2138     )
   2139     if r.status == 302:
   2140         path = [x[1] for x in r.headers if x[0] == "location"][0]

File ~/aaaaa/hh/lib/python3.11/site-packages/caldav/objects.py:2321, in CalendarObjectResource._get_data(self)
   2319     return to_normal_str(self._data)
   2320 elif self._vobject_instance:
-> 2321     return to_normal_str(self._vobject_instance.serialize())
   2322 elif self._icalendar_instance:
   2323     return to_normal_str(self._icalendar_instance.to_ical())

File ~/aaaaa/hh/lib/python3.11/site-packages/vobject/base.py:254, in VBase.serialize(self, buf, lineLength, validate, behavior)
    252     if DEBUG:
    253         logger.debug("serializing {0!s} with behavior {1!s}".format(self.name, behavior))
--> 254     return behavior.serialize(self, buf, lineLength, validate)
    255 else:
    256     if DEBUG:

File ~/aaaaa/hh/lib/python3.11/site-packages/vobject/icalendar.py:1001, in VCalendar2_0.serialize(cls, obj, buf, lineLength, validate)
    999 cls.generateImplicitParameters(obj)
   1000 if validate:
-> 1001     cls.validate(obj, raiseException=True)
   1002 if obj.isNative:
   1003     transformed = obj.transformFromNative()

File ~/aaaaa/hh/lib/python3.11/site-packages/vobject/behavior.py:85, in Behavior.validate(cls, obj, raiseException, complainUnrecognized)
     83 count = {}
     84 for child in obj.getChildren():
---> 85     if not child.validate(raiseException, complainUnrecognized):
     86         return False
     87     name = child.name.upper()

File ~/aaaaa/hh/lib/python3.11/site-packages/vobject/base.py:124, in VBase.validate(self, *args, **kwds)
    120 """
    121 Call the behavior's validate method, or return True.
    122 """
    123 if self.behavior:
--> 124     return self.behavior.validate(self, *args, **kwds)
    125 return True

File ~/aaaaa/hh/lib/python3.11/site-packages/vobject/icalendar.py:1228, in VTodo.validate(cls, obj, raiseException, *args)
   1226     return False
   1227 else:
-> 1228     return super(VTodo, cls).validate(obj, raiseException, *args)

File ~/aaaaa/hh/lib/python3.11/site-packages/vobject/behavior.py:98, in Behavior.validate(cls, obj, raiseException, complainUnrecognized)
     96         if raiseException:
     97             m = "{0} components cannot contain more than {1} {2}"
---> 98             raise base.ValidateError(m.format(cls.name, val[1], key))
     99         return False
    100 return True

ValidateError: 'VTODO components cannot contain more than 1 STATUS'

I think this bug was fixed in master branch yesterday.  Please check.

I found the same problem when writing up tests in #354, so the fix was piggybacked there.

Right, HomeAssistant ... dependent on how you are running it, it may be really difficult to check how it works with the master branch.  I will make a new minor release Real Soon Now.  Possibly tomorrow.

I think this issue has been fixed and will close the issue.  Please correct me if I'm wrong.

Yes I can confirm this issue is fixed in latest caldav.


GH-47 - Removed deprecated getInstanceEthAddress endpoint

The main app does not support this sort of operation anymore. I've removed this endpoint from docs, but let's keep this PR open until the deployment of the main app in prod.
I'll update the PR accordingly.

thanks @fforbeck. i'll approve and merge after the update.

closes #47


User-entered register names can create invalid OpenQASM

Environment

Qiskit Terra version: 0.20.0
Python version: 3.8
Operating system: Ubuntu

What is happening?
There is a regex check for proper register names in register.py, but it uses match instead of fullmatch, which means it really only checks that the first char is a lower-case alpha. This means a user can enter invalid OpenQASM register names.
It can also cause several different problems with the matplotlib and latex drawers.
How can we reproduce the issue?
qr = QuantumRegister(3, "a*$(%@)")
qc = QuantumCircuit(qr)
print(qc.qasm())

OPENQASM 2.0;
include "qelib1.inc";
qreg a*$(%@)[3];

What should happen?
Invalid register names should not be allowed.
Any suggestions?
I think a solution like #7157 for the register names would be good. In that PR, instead of raising if an invalid name is entered, names are converted to a valid form.

I don't think we should require register names to be valid OpenQASM 2 identifiers; QASM is a separate thing, and OpenQASM 3 will make many more identifiers valid anyway, by expanding into various parts of Unicode.  The solution in #7157 is still appropriate, though; we can just require the exporter to escape any names that it outputs.

Actually I was following this from register.py. It doesn't really work since it uses match instead of fullmatch. As long as the first char is lower alpha, it matches, so the rest can be anything. If we don't want to restrict register names, then this code should probably be removed. We'd still need to address the issue with the drawers.
30    # Register name should conform to OpenQASM 2.0 specification
31    # See appendix A of https://arxiv.org/pdf/1707.03429v2.pdf
32    name_format = re.compile("[a-z][a-zA-Z0-9_]*")

102            if self.name_format.match(name) is None:
103               raise CircuitError(
104                    "%s is an invalid OPENQASM register name. See appendix"
105                    " A of https://arxiv.org/pdf/1707.03429v2.pdf." % name
106                )

Oh, my mistake sorry - I didn't realise that we were already restricting the names on creation.  If it were up to me, I'd remove that provision entirely and have the QASM exporters handle escaping register names, but since we've got it for now, I think improving the verification would be better, rather than escaping the names.  If we escape the names, you'd get more confusing errors, where say my register and my_register might both be escaped to the same string, and you'd see "can't have two registers of the same name" rather than "my register is an invalid name".
If we defer escaping to the exporters, they can handle naming collisions by appending a counter or something, because they'll be working with the full context.

I think these are the issues.

Register name restriction doesn't work as is.
If we restrict names now, some existing circuits may not work. Can do deprecation.
If we leave unrestricted, some names can crash the mpl and latex drawers, won't work for qasm, but will work otherwise.
As far as escaping is concerned, I think that should be left to the exporter level.

So I think this just needs a decision as to what is the least worse.


Fix type inference problem in EquationSearch

Hm, not much of a performance difference and hurts readability, so I’m closing this…


template command to generate empty project

init also drops a tsconfig.json... but maybe i need to be more careful about there it goes.


Watch mode not working

Is this a bug report or a feature request?
Please specify whether this is a feature request 🎁 or a bug 🐛: bug
Version Info

Version of Majestic: 1.1.0
Version of Jest: 23.5.0
Version of Node: 8.9.1
Operating System: linux

Reproduction Repo
https://github.com/hoschi/yode

Go to core packages: cd packages/core
Run majestic
Change a assertion in a editor, eg. line 22 expect(result.functions).toHaveLength(0) from 0 to 1

Expected: Tests were run again like in jests watch mode
Actual: Nothing happens

Should be ok with the new release. Have a look again, please. Thanks.

This still exists with the current version :disappointed:

Could you please try again with the latest majestic. In your sample repository, one of the tests takes a while to execute after a change but it did execute.

I am also experiencing this issue. I toggled the Watch mode and if i make changes to a file, nothing happens. I have to click on the Run button for an individual test in order to see the changes
Majestic version: 1.2.22
Jest version: 23.6.0
OS: MacOS

I also felt that the Toggle watch mode button is confusing. It doesn't actually run the tests but passes the --watch flag whenever you run all the tests or a single file. We should rethink the UI of that button to make it clear.

So if the --watch flag has been passed down, and you run subsequent tests, wouldn't it still watch for those new changes? What was the challenge of trying to implement and watch mode that works similarily to jest's watch mode?

We use Jest watch mode as is and this is how the UI works with watch mode:

First, toggle the watch mode button on
Now if you run all the tests by clicking the Run all button,

All the files would be watched which is the equivalent of running jest --watch

But if you click on the Run button of a single test file,

Only the selected file would be watched which is equals to running jest --watch ./my/file/path.js

There is nothing specifically challenging about implementing watch mode. I feel like we could do better with the UX.

Ah ok, now its working as expected. Nice! I think the UX works as is, just needs some documentation on how to enable it correctly.

I was confused by this as well.  I feel like toggling the "watch mode" button should rerun the tests.


Please edit the code instead of spilling to chat

When I ask for something to be rewritten, just edit my code.  I have version control!

Have you tried selecting some code an hitting cmd + i or ctrl + i? You can use copilot inline and ask it to write and edit code, among other things.

We recommend inline chat for editing code

Seems silly that I should have to discover the magic UI for doing what I want when it could just do what I want all the time, but I guess you all know why you don't want to provide that.

We have some work in progress to give a better experience when you want to apply suggestions from the chat view to the editor. But I believe that editing a file automatically from a query in the chat sidebar would be pretty unexpected, and that a chat conversation is not the right UI paradigm for that task. It's hard to know that was your intent, and which file you want edited, etc. We've added a bunch of new entrypoints to inline chat recently since it's been hard to discover. Lmk if you have any other suggestions.

Thanks. If you had just said, “we're not gonna do this” I would have left this issue alone, but you justifying that choice with arguments that don't make any sense to me.  For example:

editing a file automatically from a query in the chat sidebar would be pretty unexpected

When my query explicitly asks copilot to edit the file, that's exactly what I expect to happen.  Thus this bug report.

a chat conversation is not the right UI paradigm for that task

That's a value judgement you are free to make, but the fact remains that people will try it.  “Hard to know the intent” seems par for the course with AI, but in this case the specific query was a request to change the highlighted code in the editor; seems to me which file to edit is obvious. Of course there may be implementation factors that make this hard, but the fact that copilot spits out a bunch of code in chat that I have to copy and paste back into the highlighted region seems utterly silly from a user point of view.


Portfolio project

In this pull request implemented a change by a prevoius code reviewer
I fixed linter error
I added a descriptive branch name

Portfolio
In this pull request implemented a change by a prevoius code reviewer
I fixed linter error
I added a descriptive branch name


SDL Tridion Alchemy Plugin Installation Error

User is currently working on SDL Tridion 2013 SP1 and Alchemy4Tridion Version: <IP_ADDRESS>
The user has also downloaded the Alchemy4Tridion Developer Pack link and trying to install the demo version plugin but getting error

Error in Debugger
http://domain.com/Alchemy/api/Plugins/Install Status Code :500 Internal Server Error
Debugger Log {message: "An error has occurred.",…}
exceptionMessage:"Value cannot be empty.
Parameter name: userName."
exceptionType:"System.ServiceModel.FaultException1[[Tridion.ContentManager.CoreService.Client.CoreServiceFault, Tridion.ContentManager.CoreService.Client, Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=ddfc895746e5ee6b]]" message:"An error has occurred." stackTrace:" Server stack trace: at System.ServiceModel.Channels.ServiceChannel.HandleReply(ProxyOperationRuntime operation, ProxyRpc& rpc) at System.ServiceModel.Channels.ServiceChannel.Call(String action, Boolean oneway, ProxyOperationRuntime operation, Object[] ins, Object[] outs, TimeSpan timeout) at System.ServiceModel.Channels.ServiceChannelProxy.InvokeService(IMethodCallMessage methodCall, ProxyOperationRuntime operation) at System.ServiceModel.Channels.ServiceChannelProxy.Invoke(IMessage message)
Exception rethrown at [0]: at System.Runtime.Remoting.Proxies.RealProxy.HandleReturnMessage(IMessage reqMsg, IMessage retMsg) at System.Runtime.Remoting.Proxies.RealProxy.PrivateInvoke(MessageData& msgData, Int32 type) at Tridion.ContentManager.CoreService.Client.ISessionAwareCoreService.Impersonate(String userName) at Alchemy4Tridion.Plugins.Clients.AlchemySessionAwareCoreServiceClient.Impersonate(String username) at Alchemy4Tridion.Core.Web.RequestPropertiesContext.get_SessionAwareCoreServiceClient() at Alchemy4Tridion.Core.Web.RequestPropertiesContext.get_User() at Alchemy4Tridion.Core.Security.AlchemyAuthorizeAttribute.OnAuthorization(HttpActionContext actionContext) at System.Web.Http.Filters.AuthorizationFilterAttribute.OnAuthorizationAsync(HttpActionContext actionContext, CancellationToken cancellationToken) --- End of stack trace from previous location where exception was thrown --- at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at System.Web.Http.Filters.AuthorizationFilterAttribute.d__2.MoveNext() --- End of stack trace from previous location where exception was thrown --- at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
at System.Web.Http.Dispatcher.HttpControllerDispatcher.d__1.MoveNext()"

Closing as no other info has been added and issue has not came up.
Marking as complete as of 0.8.2.


Fix Enumerable allocations from AvailableInputHandlers

Before:

After:

Blocking pending re-profiling.

ImmutableArray works fine, as long as we are okay with also changing the next PR to also be ImmutableArray (#3745). Have tested with both and allocs look good.


Update LOCALIZATION_CONTRIBUTORS

I am Berksu Ant Solmaz,I would like to contribute Turkish Translation as a Translator

Thank you for your interest in contributing to Qiskit translations! We opened the Turkish translation project this week, so you can now request to join the Qiskit-docs translation project in Crowdin.
Please follow steps 4 and 5 in the Contributing to Qiskit documentation to register for an account and submit your request.

Can you please move this PR here: https://github.com/Qiskit/qiskit-translations


[DATA] Duplicate data on 24 and 25 May 2021

Which Dataset
covid19za_provincial_cumulative_timeline_confirmed.csv
Error Description
25 May 2021 numbers are duplicates of 24 May 2021.
Suggested fixes

Unsure of fix as NICD report for 25 May 2021 still reports the figures of 24 May 2021 (https://www.nicd.ac.za/latest-confirmed-cases-of-covid-19-in-south-africa-25-may-2021/).

Issue has been fixed in the source data, thank you to the person responsible.

My automated process captured what was in the report for that day which was a repeat.  I fixed this by using data from another dashboard.  See 95225c35066ba43ee22c2391c1c7781ddbb03947 for an explanation.


Adding fluid main package in experimental to combine our primary dev packages

Fixes #4778

What's the advantage of the file per import format? (possibly, it makes it easier to add/modify extend things?)

We probably want to refine the list of stuff from each package to export, then there will be more lines for each files?

What's the advantage of the file per import format? (possibly, it makes it easier to add/modify extend things?)

We probably want to refine the list of stuff from each package to export, then there will be more lines for each files?

@curtisman yeah, that's why I created a file for each package. Right now we're passing everything through as is, but I could see us limiting the exports to a specific subset before this moves from @fluid-experimental to @fluidframework.
As for naming and inclusion of loader/driver, I'm happy to take further suggestions, and recognize that this is still an experimental package and is meant to mutate over time to figure out the exact API's exposed.

@curtisman yeah, that's why I created a file for each package. Right now we're passing everything through as is, but I could see us limiting the exports to a specific subset before this moves from @fluid-experimental to @fluidframework.
As for naming and inclusion of loader/driver, I'm happy to take further suggestions, and recognize that this is still an experimental package and is meant to mutate over time to figure out the exact API's exposed.

Any other changes we want to make before merging this in? Folder location and/or package name are open for suggestions. I'm in no way tied to either.

Any other changes we want to make before merging this in? Folder location and/or package name are open for suggestions. I'm in no way tied to either.

Going to verify where this package is going to be published to before merging. Also considering changing package name to something obviously temporary like @fluid-experimental/experimental-core-package

Going to verify where this package is going to be published to before merging. Also considering changing package name to something obviously temporary like @fluid-experimental/experimental-core-package

Eslint plugins returned
Sent from Outlookhttp://aka.ms/weboutlook

From: Tyler Butler<EMAIL_ADDRESS>Sent: Thursday, January 14, 2021 12:24 PM
To: microsoft/FluidFramework<EMAIL_ADDRESS>Cc: Micah Godbolt<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/FluidFramework] Adding fluid main package in experimental to combine our primary dev packages (#4783)
@tylerbutler commented on this pull request.

In packages/utils/fluid-main/package.jsonhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FFluidFramework%2Fpull%2F4783%23discussion_r557676723&data=04|01|mgodbolt%40microsoft.com|3c640f9aa70b40a5e2cd08d8b8ca64f4|72f988bf86f141af91ab2d7cd011db47|0|0|637462526717427030|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=HxzVgjMc6sAVQAcgEvtK9YhDYeqOEJ300uJQSkMU9Hs%3D&reserved=0:

"eslint-plugin-eslint-comments": "~3.2.0",

"eslint-plugin-import": "~2.22.0",
"eslint-plugin-no-null": "~1.0.2",
"eslint-plugin-prefer-arrow": "~1.2.2",
"eslint-plugin-react": "~7.21.2",
"eslint-plugin-unicorn": "~22.0.0",

I was wrong about this! Leave them in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FFluidFramework%2Fpull%2F4783%23discussion_r557676723&data=04|01|mgodbolt%40microsoft.com|3c640f9aa70b40a5e2cd08d8b8ca64f4|72f988bf86f141af91ab2d7cd011db47|0|0|637462526717427030|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=HxzVgjMc6sAVQAcgEvtK9YhDYeqOEJ300uJQSkMU9Hs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAK6KTGWR6FSFSGBM6V2AETSZ5HHZANCNFSM4V6I5QDQ&data=04|01|mgodbolt%40microsoft.com|3c640f9aa70b40a5e2cd08d8b8ca64f4|72f988bf86f141af91ab2d7cd011db47|0|0|637462526717437027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=6tf6Fr9ffc835K8t59Bgkp1Kj7yrwz1gt0mC12yBalI%3D&reserved=0.

Eslint plugins returned
Sent from Outlookhttp://aka.ms/weboutlook

From: Tyler Butler<EMAIL_ADDRESS>Sent: Thursday, January 14, 2021 12:24 PM
To: microsoft/FluidFramework<EMAIL_ADDRESS>Cc: Micah Godbolt<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/FluidFramework] Adding fluid main package in experimental to combine our primary dev packages (#4783)
@tylerbutler commented on this pull request.

In packages/utils/fluid-main/package.jsonhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FFluidFramework%2Fpull%2F4783%23discussion_r557676723&data=04|01|mgodbolt%40microsoft.com|3c640f9aa70b40a5e2cd08d8b8ca64f4|72f988bf86f141af91ab2d7cd011db47|0|0|637462526717427030|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=HxzVgjMc6sAVQAcgEvtK9YhDYeqOEJ300uJQSkMU9Hs%3D&reserved=0:

"eslint-plugin-eslint-comments": "~3.2.0",

"eslint-plugin-import": "~2.22.0",
"eslint-plugin-no-null": "~1.0.2",
"eslint-plugin-prefer-arrow": "~1.2.2",
"eslint-plugin-react": "~7.21.2",
"eslint-plugin-unicorn": "~22.0.0",

I was wrong about this! Leave them in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FFluidFramework%2Fpull%2F4783%23discussion_r557676723&data=04|01|mgodbolt%40microsoft.com|3c640f9aa70b40a5e2cd08d8b8ca64f4|72f988bf86f141af91ab2d7cd011db47|0|0|637462526717427030|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=HxzVgjMc6sAVQAcgEvtK9YhDYeqOEJ300uJQSkMU9Hs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAK6KTGWR6FSFSGBM6V2AETSZ5HHZANCNFSM4V6I5QDQ&data=04|01|mgodbolt%40microsoft.com|3c640f9aa70b40a5e2cd08d8b8ca64f4|72f988bf86f141af91ab2d7cd011db47|0|0|637462526717437027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=6tf6Fr9ffc835K8t59Bgkp1Kj7yrwz1gt0mC12yBalI%3D&reserved=0.

Appears the build is failing due to unrelated errors. I'll take a look again on Tuesday.

Appears the build is failing due to unrelated errors. I'll take a look again on Tuesday.


Add export const config support and make withAmp a no-op

This adds support for export const config = { amp: true | 'hybrid' } and removes the need for withAmp(() =>
hello world
)

Does it need to be explicitly exported, or would it work when it's a static property as well?
like:
export default function MyPage () {
  // ...
}
MyPage.amp = true;

It needs to be an explicit export so that we can optimize it.


Table 5902 Service Line updates Service Ledger Entry

There is a function that updates Service Ledger Entry (local procedure UpdateServiceLedgerEntry()) but it has no Permissions granted in code, despite we define them in User Permissions we need this in code as well
BEFORE
table 5902 "Service Line"
{
Caption = 'Service Line';
DrillDownPageID = "Service Line List";
LookupPageID = "Service Line List";
AFTER
table 5902 "Service Line"
{
Permissions = tabledata "Service Ledger Entry" = rm;
Caption = 'Service Line';
DrillDownPageID = "Service Line List";
LookupPageID = "Service Line List";

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Build: Improve image build

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other non-breaking change (enhancement, tests, documentation, chore and the like)

And

[ ] This Pull Request DOES NOT target the main (production) branch on this repository (if so it must point to a feature branch or to the develop (staging) branch).
[ ] I have read the CONTRIBUTING document.
[ ] I have added/updated tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] There are no other open Pull Requests for the same update/change.

Fixes

closes #

Proposed Changes

closes #

Codecov Report

Merging #80 (3122966) into develop (c68eb44) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop      #80   +/-   ##
==========================================
  Coverage      97.49%   97.49%
  Complexity       264      264
==========================================
  Files             38       38
  Lines            519      519
  Branches          32       32
==========================================
  Hits             506      506
  Misses             1        1
  Partials          12       12

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 c68eb44...3122966. Read the comment docs.


fatal error for very long gbq queries- 40k characters

I'm using the Google API PHP client to query Google Big Query (GBQ) and am passing in very large automatically generated queries.  The queries have about 40k characters in them.
For smaller queries, I can make an API request and receive a response without any problems.  I receive a fatal error message from PHP:
Allowed memory size of 134217728 bytes exhausted (tried to allocate 74 bytes) in .../vendor/google/apiclient/src/Google/Http/REST.php
on line 126
I've checked this line and it appears related to pulling the entire request body into memory, shortly after that, there is a function called decodeBody which mentions not decoding the body because "it's probably a really long string".  This appears to be related to my issue.
If I load the 40k-long query into the GBQ gui interface, I can run the query without problems.

Try upgrading to v2.2.0 of this library, which will use Guzzle HTTP instead of the class above. This will resolve your issue:
composer require "google/apiclient:^2"


General: Multilayer EXRs support

Description
Add support for multilayer exrs for review, burnins and thumbnail creation.
Changes

moved transcoding related functions into openpype/lib/transcoding.py
do multilayers check per representation instead of per instance
multilayer check was merged with dwaa and dwab check

decompress related functions are included in new ones
should_convert_for_ffmpeg replaces should_decompress
convert_for_ffmpeg replaces decompress

the check is currently done only for .exr
oiiotool must be available to be able do the check

this is limiting MacOS as we don't have oiio build for darwin so all exr conversions are skipped

process methods of extractors were split into 2 or 3 methods (a little bit less spaghetti code)

Questions

should we "try use ffmpeg" without conversion on MacOs?

Task linked: OP-2045 Preview for multilayer exrs

About mac 🍎 - I am wondering if we can use Homebrew of oiio as we already are using Homebrew for the building process... But this is probably note for another issue.

I am wondering if we can use Homebrew of oiio as we already are using Homebrew for the building process... But this is probably note for another issue.

It is offtopic and I think we can. But I'm afraid it won't be compatible across all versions of OSx systems.

It does work as far as I could test it. I had to make some fixes in Maya / Deadline part so it could be tested there. So I am approving it and if someone else can test it in Maya, I think we should merge it.


Error NETWORK_NOT_PEERED

Hi all,
When creating a MatchingEngineIndexEndpoint using the code below:
my_index_endpoint = aiplatform.MatchingEngineIndexEndpoint.create(     display_name=DISPLAY_NAME,     description=DISPLAY_NAME,     public_endpoint_enabled=False,     network=VPC_NETWORK_FULL, )
I keep getting this error:
Error NETWORK_NOT_PEERED
How can I properly set up network peering to fix this issue?
Thank you

Is anyone else facing this issue?
I understand that this is more of a Google Cloud issue (rather than a code issue), but I am wondering if anyone else faced a similar problem and how they resolved it.

@alirezaomneky please see if this doc helps: https://cloud.google.com/vertex-ai/docs/general/vpc-peering.


Appstudio update test-component-pac-hggy

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-hggy-on-pull-request has successfully validated your commit.


Collapsing PHP function with heredoc strings fails

VSCode Version: Code 1.12.2 (19222cdc84ce72202478ba1cec5cb557b71163de, 2017-05-10T13:20:36.315Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

php-debug
felixfbecker
1.10.0

crane
HvyIndustries
0.3.6

Steps to Reproduce:

Copy paste the function snippet below.
Collapse it (click on the '+' sign on the left)
Collapsing fails.

Code example:
funtion onefunction()
{
string heredocString = <<<BATMAN
I love bats.
BATMAN;
$superman = "I love bragging.";

}

Code folding is naive and indentation-based right now. Issue tracking more sophisticated folding: #3422


Add an Advanced Section to the Software

There are two groups of users with radically different needs. Typically these groups are the general population and the UI/process should very easy and straightforward. However, (especially in bitcoin), there are advanced users who want more customization and advanced features.
Much like other signing devices and wallets/apps, I'm proposing adding a Danger Zone or Advanced section to the software where you could house many of the features that don't fit the large portion of users.
It could even be a "secret" or un-shown step to access, (similar to the skip engraving function) by holding down a specific button. You should add warnings and disclosures that the user has to accept, but this would keep the two groups of users happy.
This option keeps SeedHammer highly relevant to many users so the primary use case of multi signature isn't lost, but flexible use cases are added.

This is not a bad idea in general, but an issue should have a clear way of being resolved. I suggest you create a new issue with the (first) particular advanced feature you would like to see, and let's discuss it from there. Some general considerations:

Even if hidden under advanced, a feature should be made fail-safe or at least very hard to misuse in an insecure way.
Even if hidden under an advanced section, the feature should be designed well and as easy to use as possible.
Someone needs to implement the feature. Some features may be good ideas, but may not fit into our development priorities.
Someone needs to maintain the feature. Implementing a feature is one thing, keeping it working is another. Once you have a feature, it's hard to remove again.

The above may be too high a bar for features desired by few advanced users. If there is community interest in implementing and maintaining a feature, we could also consider publishing guides to make custom builds with advanced features included.


Texture packer with grayscale image give strange result

I'm using TexturePacker with some grayscale images like this one :

When I run TexturePacker command line, with the input folder as parameter, with only the image above in it (no pack.json), I get this result :

As you can see, the packed texture is too bright.
There is below the pack.atlas file :

pack.png
size: 128,128
format: RGBA8888
filter: Nearest,Nearest
repeat: none
HideGrayscale
  rotate: false
  xy: 2, 2
  size: 72, 72
  orig: 72, 72
  offset: 0, 0
  index: -1

I use the latest snapshot.

Not sure what is happening. It uses AWT, so if that can't read/write it
then there may not be anything we can do.
On Tue, Feb 3, 2015 at 10:51 PM, Aurélien<EMAIL_ADDRESS>wrote:

I'm using TexturePacker with some grayscale images like this one :
[image: hidegrayscale]
https://cloud.githubusercontent.com/assets/3535019/6030201/2aee7a52-abf6-11e4-90a9-0273d9c7396a.png
When I run TexturePacker command line, with the input folder as parameter,
with only the image above in it (no pack.json), I get this result :
[image: pack]
https://cloud.githubusercontent.com/assets/3535019/6030263/82e0d610-abf6-11e4-98a0-c52bcc8b3ef7.png
As tyou can see, the packed texture is too bright.
There is below the pack.atlas file :
pack.png
size: 128,128
format: RGBA8888
filter: Nearest,Nearest
repeat: none
HideGrayscale
rotate: false
xy: 2, 2
size: 72, 72
orig: 72, 72
offset: 0, 0
index: -1
—
Reply to this email directly or view it on GitHub
https://github.com/libgdx/libgdx/issues/2804.

I could solve this by saving the original image as a PNG-24 file. Your image file has a 32 bit color depth. As I understand, the last 8 bits are used for transparency. Maybe the file contained some misinformation or could not be read properly? I am no expert on this topic, so I cannot tell you exactly what happend there.

Looks like that solves it, thanks!


MINOR: Remove unused hadoop version

All dependencies on hadoop were removed with MiniKDC. This removes the left over version entry.

LGTM

cc @gwenshap @ewencp

huh. why is it even here?

@gwenshap I forgot to remove that when I removed the dependency on minikdc.


In order to effectively be a batch/stream processor, Calc needs to be able to "read" data from stdin

Right now, "read" will read from a file, but maybe a special file name (like "@"??) could signal a read from the input stream.

To be fair, Calc can already read a script (meaning the program / expressions) from stdin. It's just that if you were to specify a script on the command line then having that script read more data from stdin -- that's the missing piece right now.

So, I think the best approach for this is the following:

Change "getFileContents" in Calc.java to return some kind of Reader object
Change consumers of this to read from the stream and produce a string
Move the recognition of "@file" and just "@" into this function as well
Would have to change the way "readFile" works also, to fit this paradigm

So, I think the best approach for this is the following:

Change "getFileContents" in Calc.java to return some kind of Reader object
Change consumers of this to read from the stream and produce a string
Move the recognition of "@file" and just "@" into this function as well
Would have to change the way "readFile" works also, to fit this paradigm

This is done, although there are other issues in REPL mode.


Netlify CMS - non-alphanumeric redirects URLs giving an error

Hi team,
when editing a rule that have a redirect with non-alphanumeric characters, I can't save because of this error:

Please fix

Note: The example was taken from this rule: https://www.ssw.com.au/rules/do-you-know-how-to-describe-the-work-you-have-done-while-avoiding-the-word-bug - I have removed that redirect so I didn't lose my changes
AB#62890

As per my conversation with @taineriley1

Was there an old URI with parentheses or other characters that needs to be added as a redirect, or is this just desired functionality?

In the past we had a bunch of URLs with special characters... when we decided to remove them (during migration from SharePoint to GitHub) we setup these redirects, so the old links would not give a 404.

What characters in particular would you like to be able to use in URIs? I think the simpler we can keep them the better.

Currently we don't want any final URL to have those special characters...so keep only letters, numbers, and dashes

Do you want these characters just for redirects or for current URIs as well? We can fully customise the validation for each field if needed.

Only for redirects... it's OK to keep the validation error if someone adds a special character in the URI (A UNIQUE IDENTIFIER FOR THE URL) field

Hi @tiagov8,

Please fix

Done - see screenshot

Figure: URL with parentheses saved successfully
Note: This redirect was added to the rule. See PR https://github.com/SSWConsulting/SSW.Rules.Content/pull/1566


Model selection.

Is there a reason for pre-selecting possible models instead of just give a selection of installed models?

Is there a reason for pre-selecting possible models instead of just give a selection of installed models?

Hello, so the idea was to make some models available as soon as the person installs the custom node, because otherwise the list will appear blank, now with the custom_model it would be possible to do the installation by putting a value in that field and then in the model field list the models that are already installed, other than that there is the issue of classifying which models are vision models and which ones do not have this feature, there is no way for me to know other than leaving this information previously set in the code.

I see you already did it :)
Awesome thank you!

I see you already did it :) Awesome thank you!

So, I added the possibility of selecting a custom_model, but it is still something that you need to type the name of the model, but with that you can already use your models, you just need to pay attention to the issue that for Ollama Image Describer you need to use a model that is multimodal (with vision), I will see a way to extract the information directly from ollama to find out if a model supports vision and list the ones installed on the machine.

Yeah i was just about to write as i saw the update. Nice. Everything there i need. But sure. For some it may be better to know which models they can use.


[Research] Gas settings in mainnet

For some more context: I believe I was concerned about on mainnet you can't easily submit txs that take > 25M gas.
How much gas do contract uploads / proposals take? If more than this, we need to start asking validators / nodes to increase this config param

I think let's look at biggest code stored on terra / juno and try submitting that on our testnet, monitor gas consumption? That should give us good hints. The gas consumption calc logic is consistent across the chains (local/testnet/mainnet), am I correct?

I can give some insight on this. On Terra, the max_contract_size parameter was set to 614400 bytes. We were constantly bumping up against this and needed to do a bunch of optimizations and workarounds to keep one of our contracts below this size. Here's an example of a MsgStoreCode where we upload a contract that is 612Kb: https://finder.terra.money/classic/tx/F9A0CD5BB24F378176822C4F7C7063B315ACCB4A6CB1C517B68CD0FBA6FBBD62
The gas used was 7,437,249. At the standard Terra gas price of 0.15uusd this should have cost $1.11 if unused gas would have been refunded (any insight on if gas refunds are being worked on?). This feels like an okay price for uploading a contract, but I would say that the cost of uploading contracts is not very important since it won't be done very often. Instead it makes more sense to look at how much gas a typical CosmWasm contract execution takes. This of course varies a lot depending on how complex the execution is, but as an example here is a transaction where someone is exiting an Apollo vault. This used 3,994,206 gas and would have cost $0.59, which I personally feel is somewhat high. What I've discovered is that most code in CosmWasm use almost no gas at all, and that almost all the gas is used by doing SmartQueries and returning CosmosMsgs from ExecuteMsg handlers. @ethanfrey has told me that it needs to be this way because loading those called contracts into memory takes time for the validator's and thus if this were cheap to do the block time could increase (Ethan: please correct me if I misremember your explanation). He suggested a workaround for this would be to flag certain contracts as always staying in the validators' memory. This seems like it could be a viable solution on Osmosis since it is permissioned and likely won't have many contracts.

The osmosis gas cost is about 100x cheaper today then were Terra was at, so in terms of gas USD costs I don't think we need to be super worried.
On Osmosis, the current max contract size is 800kB as far as I understand: https://github.com/CosmWasm/wasmd/blob/main/x/wasm/types/validation.go#L11-L12
Looks like for contract uploads we maybe don't need to be worried? (Frey mentioned at one point that theres is a notably gas overhead for the governance proposals over a normal upload. Can someone post once they try that) We run into 25M limits at the moment, which seems fine.
There is no unused gas refund work going on, its blocked on subsequent SDK releases right now.
Those gas counts do look excessive for exiting an apollo vault, we should look into we can tune down that gas amount. Or get those contracts in a cache, especially as in Osmosis we have bounded amounts of wasm code. Right now, 4M gas / contract call  would lead to Osmosis blocks getting filled rather quickly.

Those gas counts do look excessive for exiting an apollo vault, we should look into we can tune down that gas amount. Or get those contracts in a cache, especially as in Osmosis we have bounded amounts of wasm code. Right now, 4M gas / contract call would lead to Osmosis blocks getting filled rather quickly.

Yeah as mentioned I think almost all of the gas is due to SmartQuery and returning CosmosMsgs from ExecuteMsg handlers. This can be optimized somewhat, but often leads to more complex code. Pinning/caching seems like the best option to me.


Fixed Typo

Fixed typo #4

@gopavasanth : Henceforth make a PR from a branch of your fork and not the master branch. Also try to follow the PR template

Good job! Squash your commits into one with a suitable commit log. Then it will be merged!


New sample Archive inactive teams, Closes #563

Closes #563

Hey @nicodecleyre, really nice sample,  thank you 👏 for your contribution - it is highly appreciated. This should be live shortly 😊
Have you registered for the PnP Recognition program, your submission will qualify for a badge :-)
Its opt in so please register here: https://pnp.github.io/recognitionprogram


Add security contacts

Closes #52

@spiffxp @timothysc Please note that I just randomly selected you as security contacts. Please let me know if you do not want to or cannot be on that list. Also let me know if I am missing someone who should be on this list.

That being the case, why are security contacts necessary?

https://groups.google.com/d/msg/kubernetes-dev/codeiIoQ6QE/o3os3NbFDAAJ :
Matt Farina:

Does this apply to all repos on all actively used orgs owned by Kubernetes?

Jessie Frazelle:

Yes indeed :)

/lgtm

Pointless administrivia, yay!


TE-6.2: Route Removal In Non Default VRF : Commented out EnabledAddre…

TE-6.2: Route Removal In Non Default VRF : Commented out EnabledAddressFamilies variable .
Reference : https://github.com/openconfig/public/pull/738

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1656

Relevant Lines:
2387

💛  - Coveralls

Pull request closed refer to : https://github.com/openconfig/featureprofiles/pull/856


GRUCell is not compatible with its own initial state

System information

Have I written custom code: Yes
OS Platform and Distribution: Ubuntu 16.04
TensorFlow installed from: binary
TensorFlow version: 2.1.0rc0
Python version: 3.5.2

Describe the current behavior
The initial state returned by tf.keras.layers.GRUCell.get_initial_state() can not be passed to the first cell call without error. It raises an InvalidArgumentError error.
Describe the expected behavior
RNN cells should accept their own initial states.
Code to reproduce the issue
import tensorflow as tf
batch_size = 4
cell = tf.keras.layers.GRUCell(20)
initial_state = cell.get_initial_state(batch_size=batch_size, dtype=tf.float32)
output, state = cell(tf.random.uniform([batch_size, 10]), initial_state)

Other info / logs
Traceback (most recent call last):
  File "test/gru_incompat.py", line 5, in <module>
    output, state = cell(tf.random.uniform([batch_size, 10]), initial_state)
  File "/lib/python3.5/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 822, in __call__
    outputs = self.call(cast_inputs, *args, **kwargs)
  File "/lib/python3.5/site-packages/tensorflow_core/python/keras/layers/recurrent.py", line 1846, in call
    matrix_inner = K.dot(h_tm1, self.recurrent_kernel)
  File "/lib/python3.5/site-packages/tensorflow_core/python/keras/backend.py", line 1678, in dot
    out = math_ops.matmul(x, y)
  File "/lib/python3.5/site-packages/tensorflow_core/python/util/dispatch.py", line 180, in wrapper
    return target(*args, **kwargs)
  File "/lib/python3.5/site-packages/tensorflow_core/python/ops/math_ops.py", line 2797, in matmul
    a, b, transpose_a=transpose_a, transpose_b=transpose_b, name=name)
  File "/lib/python3.5/site-packages/tensorflow_core/python/ops/gen_math_ops.py", line 5631, in mat_mul
    _ops.raise_from_not_ok_status(e, name)
  File "/lib/python3.5/site-packages/tensorflow_core/python/framework/ops.py", line 6598, in raise_from_not_ok_status
    six.raise_from(core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: In[0] is not a matrix. Instead it has shape [20] [Op:MatMul] name: gru_cell/MatMul/

@guillaumekln hi, I met this error too, and I think here is a bug. I changed two line of codes, everything is ok.
https://github.com/tensorflow/tensorflow/blob/4235c011cf48daf64953895e844dba7b72b3edc0/tensorflow/python/keras/layers/recurrent.py#L1773
should be h_tm1 = states  # previous memory
https://github.com/tensorflow/tensorflow/blob/4235c011cf48daf64953895e844dba7b72b3edc0/tensorflow/python/keras/layers/recurrent.py#L1871
should be return h, h

Sorry for the breakage, will fix it very soon.


Add authentication keys to the wallet and include them in the bloom filters

This is already a part of #40, though there is one more commit to be added.


chore(db) better error messages on postgres ttl cleanup timer

Summary
The postgres ttl cleanup timer used to error with:
[error] runtime error: ./kong/db/strategies/postgres/connector.lua:343:
  attempt to perform arithmetic on local 'num_queries' (a nil value)
  stack traceback:
  coroutine 0:
	./kong/db/strategies/postgres/connector.lua: in function
	<./kong/db/strategies/postgres/connector.lua:336>,

When it had issues on for example connecting to Postgres database.
This PR changes it to this:
[error] [lua] connector.lua:360: unable to clean expired rows from PostgreSQL database
                                (connection refused)

Need to make sure tests are good but change LGTM.


Expose toolbelt::FileDescriptor?

I'm not 100% sure what you prefer as a solution to this. Here is the problem:
I'm trying to use subspace subscribers and publishers, collect their fds from GetPollFd() (which returns struct ::pollfd), I then use them with coroutine::Wait (which takes a vector of struct ::pollfd), so far so good. But then, I also need to send them over UnixSocket::SendFds (which takes a vector of toolbelt::FileDescriptor). You see the issue, I cannot convert a struct ::pollfd into a toolbelt::FileDescriptor since that would close the fd on destruction.
There are a few solutions:

Have subspace pub/sub give out their toolbelt::FileDescriptor from GetPollFd (or some differently named functions).
Have a "release" function in toolbelt::FileDescriptor similar to std::unique_ptr::release that gives up ownership.
Overload UnixSocket::SendFds to take int fds, rather than demanding toolbelt::FileDescriptor.

Of course, all three could be done too. I think the last one is probably the most annoying and inefficient to implement.

I like the first option best.  I implemented the same as you did except I added 'const' to the functions and added a test.  Hope you don't mind that I check my version in (which is almost identical to yours).  I implemented it before I noticed you had also done it.

Yeah, good idea.  I've done that and released a new version of toolbelt.  Thanks.


Rename the Plugin definitions hookname

Rename the Plugin definitions hookname since it no longer only returns the plugin model's extensions.
https://github.com/jondavid-black/AaC/blob/2e36baa437857c4b3a591c70b9e2a997e8ef42ff/src/aac/hookspecs.py#L16-L23
AC:

[ ] Change get_base_model_extensions to get_plugin_aac_definitions
[ ] All existing plugins are updated to use the new function name
[ ] The genplugin template plugin.py.jinja2 to use the new function name
[ ] Update the genplugin test to check for the new function name in the generated files.
[ ] Update all get_plugin_aac_definitions function docstring summaries from Return data and ext definitions to apply to the AaC base.  to Returns the plugin's AaC definitions.

Hi! Can I try this one :) ?

Hi! Can I try this one :) ?

Hi Gustavo! 👋
We're currently using some Agile ceremonies like sprints and sprint planning to align the team on issues and issue refinement. I haven't had any other team members look over this issue yet, but I can arrange that.
We're happy to hear that you found the project interesting enough to want to contribute! Do you mind if I ask how you heard about the project and what drew you into it?

Hi Alex! hope everything is fine with you. I Wanted to start my journey contributing on open source projects and I liked what it's beeing implemented here. I will work on this today and try to make the PR as soon as I can! Cheers :)


DFA filter

It would not be that hard to add an execution mode which always uses a DFA to filter input, then lets the skip regex engine loose. Depending on benchmarks, this could actually be a useful implimentation.
If the benchmark results are mixed, I bet we could use a heuristic that checks match length after the DFA runs and then compares match length with the average length of zero-repetition paths through the regular expression. A big delta along with the possibility for the e*l optimization would indicate that it is probably worthwhile to disbatch into the skip engine.

Just pushed to the skip-regex branch


Clarify the suffices and prefixes of setting.AppSubURL and setting.AppURL

In more than a couple of places we're slightly confused about whether AppURL and AppSubURL have preceding or trailing slashes.
This PR adds comments to the definition of the setting.App*, removes an old unused setting variable, uses the clarity obtained to simplify several uses of AppURL and AppSubURL and fixes a potential issue with a missing QuoteMeta in regexp.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

setting.AppSubURLDepth has been removed as it is not used.

Codecov Report

Merging #12999 into master will decrease coverage by 0.00%.
The diff coverage is 35.00%.

@@            Coverage Diff             @@
##           master   #12999      +/-   ##
==========================================
- Coverage   42.59%   42.59%   -0.01%
==========================================
  Files         671      671
  Lines       73625    73619       -6
==========================================
- Hits        31363    31357       -6
+ Misses      37183    37181       -2
- Partials     5079     5081       +2

Impacted Files
Coverage Δ

routers/admin/admin.go
11.45% <0.00%> (ø)

routers/api/v1/org/member.go
19.54% <0.00%> (+0.22%)
:arrow_up:

routers/user/notification.go
40.17% <0.00%> (ø)

models/notification.go
65.76% <100.00%> (ø)

models/repo.go
55.15% <100.00%> (ø)

models/user.go
53.58% <100.00%> (ø)

modules/markup/html.go
78.61% <100.00%> (+0.26%)
:arrow_up:

modules/setting/setting.go
47.50% <100.00%> (-0.12%)
:arrow_down:

routers/repo/search.go
69.23% <100.00%> (-1.14%)
:arrow_down:

modules/util/timer.go
42.85% <0.00%> (-42.86%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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

Imho both appurl and appsuburl should end with / otherwise it will still be confusing and inconsistent

The reason that AppSubURL doesn't have a terminal / is because it allows it to be empty when there is no suburl.
That's quite nice actually and leads to very compact relative URLs and easy handling of AppSubURL with relative addresses.
AppURL will have a terminal slash for other similar reasons in that some proxies will likely only proxy the slash terminal address and I suspect that means that it's easier to and simpler to just write AppURL instead of AppURL+"/"
This PRs intention was not change behaviour but rather to clarify what is actually happenin, but if we were to change anything I'd argue we should not have the terminal / on the AppURL.
That would make creating the "relpath" and making the absolute URL the same: clean the path to root it on the AppSubURL then either prefix with suburl or the URL as appropriate.

I would prefer that non of them should have / at the end imho this way AppURL + AppSubURL + /path is always valid

I would prefer that non of them should have / at the end imho, this way AppURL + AppSubURL + /path is always valid

AppURL has to contain the AppSubURL so I think you mean AppURL + "/path".

I would prefer that non of them should have / at the end imho, this way AppURL + AppSubURL + /path is always valid

AppURL has to contain the AppSubURL so I think you mean AppURL + "/path".

yes, I meant (AppURL || AppSubURL) + /path :)

As a (partially relevant) aside:
Do you know if there is ever a reason for Gitea to be producing non-FQDN urls that point outside of its own root? My suspicion is that there is not and that there should not be.

I don't know what did you mean by that but there should not be such urls

Codecov Report

Merging #12999 into master will increase coverage by 0.28%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master   #12999      +/-   ##
==========================================
+ Coverage   41.96%   42.24%   +0.28%
==========================================
  Files         683      683
  Lines       75279    75375      +96
==========================================
+ Hits        31589    31843     +254
+ Misses      38524    38306     -218
- Partials     5166     5226      +60

Impacted Files
Coverage Δ

routers/admin/admin.go
11.41% <0.00%> (ø)

routers/api/v1/org/member.go
19.54% <0.00%> (+0.22%)
:arrow_up:

routers/repo/pull.go
33.29% <0.00%> (ø)

routers/user/notification.go
40.17% <0.00%> (ø)

models/notification.go
65.76% <100.00%> (ø)

models/repo.go
56.24% <100.00%> (ø)

models/user_avatar.go
31.57% <100.00%> (ø)

modules/markup/html.go
78.61% <100.00%> (+0.26%)
:arrow_up:

modules/setting/setting.go
45.88% <100.00%> (-0.12%)
:arrow_down:

routers/repo/search.go
69.23% <100.00%> (-1.14%)
:arrow_down:

... and 20 more

Continue to review full report at Codecov.

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

@uli-heller would you like to test now ?

@6543 : I'll do so but I'm not sure it it will work out today. Still busy @work...

:rocket:


Do not send headers on 2nd request (--publish)

This is a fix for #1474

@vincent-psarga @vincentcapicotto can you review and merge this please? And release?


Import Aliasing/Lists

It looks like KeyWrapping and SimonSpeck (and TWINE, which doesn't count) are currently the only labs in the entire course that demonstrate import aliasing (import {module} as {alias}).  Perhaps this and import lists (import {module} (as {alias})? (hiding)? ({definitions})) should be added to labs/Language/Basics.md and prerequisites.
Edit: Corrected Markdown location
Edit: as {alias} is optional

OK, I've PR'ed (#98) a section on import lists.

Closed via branch deletion / direct commit


Updating Custom Metadata in a Profile doesn't recognize changes

With the latest release with adding custom metadata to profiles, git2kandji doesn't recognize updates to the metadata for profiles due to how the normalize_xml_content function is configured which causes the comparison to say nothing has changed in the profile.

@disenchant - I just came across this and initially thought it only had to do with the normalize_xml_content function and was working to correct it. I got it corrected, and then found more:
However, it also seems that Kandji strips the comments out from the profile when we're uploading it, so what is returned is a profile without the custom metadata which then causes the compare function to find that every profile needs an update as the local profile in git has the custom metadata and the one returned from Kandji doesn't have the metadata.
Wondering if you have any thoughts on how to solve for that? One thought I had was putting the metadata in the PayloadDescription key as that seems to come through. One upside to moving it here is nothing would need to change in the normalize_xml_content function.

What's interesting is that the XML Preview shown in Kandji matches what is returned via the API GET endpoint. However, if you use the "Download" button for the profile from within Kandji, it downloads the profile with the comments included.

@moojomoore good catch. We can certainly find a way around this but IMHO this is a bug in Kandji (API not being consistent with WebUI) and I'd recommend to open a bug with them and see if they feel like addressing it. The main reason why I'm hesitant to just add it e.g. into the PayloadDescription key is that unlike when using comments to store the information it would actually change the profile.

@disenchant - Agreed. Ticket submitted to Kandji. I'll let you know what they come back with.
I'm assuming that even if they classify it as a bug that it'll be awhile before it's actually corrected. I'll play around with it some and see what other options I can think up that doesn't involve modifying the profile.
This is more for me to remember to come back to, but something to try:
Compare the custom metadata value to the API response as name and active are both returned there. If they don't match, update. If they do match, skip with no changes.

Kandji is saying that this is a "feature request" and not a bug 🙃.
I ended up working out the comparison of the custom metadata to the API Response for profiles, I didn't see a reason to change the way we do comparisons for scripts. While I was at that, I added the runs_on values to the metadata to set what platforms to install the profile on.


#1604 Package manager doesn't auto-update if package installed manually

Same notifications as needed for #2143. Cleaned up and consolidated some duplicate code. Tests pass. Manual test scenarios are welcome.

should the rtvs:::package_uninstall and rtvs:::package_update be changed to use this version of install and remove?

They already end up calling overloaded install.packages anyway

:+1:


Missing pharenteses in serenity-js v3 upgrade documentation

Missing pharenteses in notes<MyNotes>().set() example.

Nice catch, thank you @dwaned!


Add the ability to a API admin to delete (disable & archive) a subscription

FIX with https://github.com/MAIF/daikoku/pull/544


[v3] Move Practice Exercise example files to .meta directory

The example implementation files for Practice Exercises should be located in the .meta directory. Besides being consistent with Concept Exercises, it also has the added benefit that no extra configuration is needed to prevent the example file from being downloaded to the student (as files in the .meta directory are never downloaded).
See the spec.
This PR attempts to detect these example files and move them to the exercise's .meta directory.
Note that this change will likely break your CI, so feel free to either merge this and fix the build later, or push a fix to this PR.
Tracking
https://github.com/exercism/v3-launch/issues/18

Exercise configs have been added since this PR was opened so those would need to get updated with the new example file locations. I made a PR to your branch with that change (+ merge main) so that you don't have to, you can just merge: https://github.com/ErikSchierboom/coffeescript/pull/3

@angelikatyborska Actually, I just had to re-run my script :)


HDFS-15408. Add a new parameter to control the timeout properties of Connection in case that the command ‘HDfs fsck’ would failed with SocketTimeoutException.

HDFS-15408- add a new parameter to prevent the command hdfs fsck SocketTimeOut

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
33m 39s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
27m 38s
trunk passed

+1 :green_heart:
compile
1m  7s
trunk passed

+1 :green_heart:
checkstyle
0m 54s
trunk passed

+1 :green_heart:
mvnsite
1m 15s
trunk passed

+1 :green_heart:
shadedclient
17m 31s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 44s
trunk passed

+0 :ok:
spotbugs
3m  5s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m  2s
trunk passed

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 35s
hadoop-hdfs in the patch failed.

-1 :x:
compile
0m 35s
hadoop-hdfs in the patch failed.

-1 :x:
javac
0m 35s
hadoop-hdfs in the patch failed.

-0 :warning:
checkstyle
0m 47s
hadoop-hdfs-project/hadoop-hdfs: The patch generated 3 new + 477 unchanged - 0 fixed = 480 total (was 477)

-1 :x:
mvnsite
0m 36s
hadoop-hdfs in the patch failed.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
shadedclient
3m 58s
patch has errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 39s
the patch passed

-1 :x:
findbugs
0m 36s
hadoop-hdfs in the patch failed.

_ Other Tests _

-1 :x:
unit
0m 38s
hadoop-hdfs in the patch failed.

+1 :green_heart:
asflicense
0m 25s
The patch does not generate ASF License warnings.

95m 48s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2067

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux d2d61791ac2a 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 93b121a9717

Default Java
Private Build-1.8.0_252-8u252-b09-1~18.04-b09

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/patch-mvninstall-hadoop-hdfs-project_hadoop-hdfs.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/patch-compile-hadoop-hdfs-project_hadoop-hdfs.txt

javac
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/patch-compile-hadoop-hdfs-project_hadoop-hdfs.txt

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt

mvnsite
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/patch-mvnsite-hadoop-hdfs-project_hadoop-hdfs.txt

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/patch-findbugs-hadoop-hdfs-project_hadoop-hdfs.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/testReport/

Max. process+thread count
307 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-2067/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


.github/workflows/revert-testing.yml: new post-push action to remove tmp test commits

This PUSH Github action will revert the last commit of a PR being
merged if its message starts with '[TEST TMP]'.
This allows running custom PR tests in OpenShift CI, without affecting
the routes taken by the nightly CI.

@omertuc good for merging I think, no comment on slack == no objection I assume :)

/lgtm

@omertuc good for merging I think, no comment on slack == no objection I assume :)

Nah that's just my forgetfulness 😅

thanks @omertuc  ;-)

/approve


net: support for the socket option SO_REUSEPORT

Does net.Conn support SO_REUSEPORT in linux platform in plan ?

If you have questions you should check out https://github.com/golang/go/wiki/Questions.

No, we don't currently use SO_REUSEPORT on linux.
Can you please explain the problem you are trying to solve that having SO_REUSEPORT available would let you solve?

Go 1.11 probably will fix #9661 instead of exposing a tangle of platform-specific socket options.

@mikioh Noop, I build with master branch  25c3bb377ac455184cc9870e690df45fe3f6e1ab and it's still missing.
I tried to run src/syscall/mkall.sh on my Debian box to update SO_REUSEPORT but it change const and breaks API checker. Did I miss somethings? :/
-       AF_MAX                           = 0x27
+       AF_MAX                           = 0x2a

@mengzhuo,
The site "godoc.org" is your friend.
https://godoc.org/golang.org/x/sys/unix
https://godoc.org/syscall
If your circumstances have no reachability to the site, unfortunately, https://github.com/golang/go/wiki/Questions could help you.

@mikioh
syscall

This package is locked down. Code outside the standard Go repository should be migrated to use the corresponding package in the golang.org/x/sys repository.

Thanks for your reply, please ignore it :)

@mikioh's previous comment (https://github.com/golang/go/issues/23696#issuecomment-404072922) touches on this obliquely, but more directly:
The syscall.SO_REUSEPORT constant does not exist under linux/amd64, and a few other OS/ARCHes.  You have to use golang.org/x/sys/unix.SO_REUSEPORT.


Update js event name in README

Type of PR (feature, enhancement, bug fix, etc.)
Docs
Description
The js event name was incorrectly documented in the README. Fixed to match the event name emitted by the code.
Why should this be added
Users can properly attach to the emitted js event without having to delve into the source to find the correct name.
Checklist

[x] My code follows the style guidelines of this project
[x] Checks (StandardRB & Prettier-Standard) are passing

You're welcome!


Support for ECC keys

Hello,
First of all thanks for all your hard work on the project, it's at least a time saver if not a lifesaver.
My problem is that the only SSL keys I have for my domain are not RSA keys but ECC keys. From what I know postfix support ECC keys by two config options:
smtpd_tls_eckey_file = smtpd-key-ecdsa.key
smtpd_tls_eccert_file = smtpd-cert-ecdsa.cert

The problem is, from what I see the main.dist.cf file accepts only RSA keys with the smtpd_tls_key_file and smtpd_tls_cert_file directive.
Is there any possibility for ECC key support to be added to the container too?

Hello, thanks for like the project.
I believe it's do-able to support the ECC keys to the SMF container.
PR will be welcome. :-)

Testing it right now with a substituted configuration, I only changed smtpd_tls_key_file to smtpd_tls_eckey_file and smtpd_tls_cert_file to smtpd_tls_eccert_file and it seems to work fine. I'll check a few possibilities (mostly whether you can configure both ECC and RSA keys at the same time without causing postfix a lot of headache.
Gonna get back to you with a proper PR as soon as I test it thoroughly.

Okay the PR is ready: #51
Tested it on my setup, it doesn't throw any error in the logs although the smtp.cert/key is missing, probably cause it sees the smtp.ec.key and smtp.ec.key as a replacement.
Thanks for the mention in the README, it made my day :dagger:


Changing the second element affects the edge of the world

By default, the edge of the world is made up of Wall. If you change Wall, eg: by renaming it to another element, this can cause some unexpected behaviours at the edge of the world.

i wonder what the edge of the world should be made of...

What if... the first two elements can't be renamed? What if they're always Air and Wall or something along those lines? To make it clear what they do. Or perhaps.. Empty and Edge... I don't know though. It would be a shame to not be able to change them.
Perhaps some simple options would be better - that let you pick what elements get chosen for the air and edge?
Like a dropdown below the simulation / next to the reset button.
Or perhaps rename the reset button to make it clearer?
Fill world with Air
Set world edge to Wall

Would also be good to be able to set the edge to 'wraparound'

The world now wraps around. By default, the world has a Wall border, which can be cleared/removed


Can  be made hoverable ?

Now that we have a <v-popover> component, can we reconsider the feature mentionned in this issue ?
On the demo page, you show how to use <v-popover> with an interactive component and a 'click' trigger, but no interaction can be made when using the 'hover' trigger.
What do you think ?

Remove offset from v-popover and offset it in CSS instead
``
.tooltip{
border: none;
background-color: transparent;
&.popover {
.popover-inner {
margin-top: 30px;
}
}
}``

Hmm, I'm having this issue as well: I need the popover content to appear on hover, however it disappears as soon as the mouse cursor moves over the popover content. @tombalev unfortunately your solution isn't working for me as I still need the location to accept the position prop.

@Baloche @benjivm  I am not sure why it hasn't be suggested but popover can accept properites such as: open.
So a simple way of making it working with VueJS,  would be:

hover me !

Adding this as I also found a workaround similar to iconictb.
Following from the CSS example given in the documentation, there was styling to add margins to the tooltips:
.tooltip[x-placement^="top"] {
  margin-bottom: 5px;
}

.tooltip[x-placement^="bottom"] {
  margin-top: 5px;
}

.tooltip[x-placement^="right"] {
  margin-left: 5px;
}

.tooltip[x-placement^="left"] {
  margin-right: 5px;
}

This caused a gap between the target element and the tooltip, so hovering away from the target would hide the tooltip.
Instead, if we remove the margin from the tooltip styling, there won't be a gap and the user can move their mouse over the popover element.
Additionally, if you still want to keep some margins between the target element and the tooltip, you can add margins to the tooltip/popover-inner styling.


Fix warning

When Session#digest_auth_configured? is called:
warning: instance variable @digest_username not initialized

Closing this one out. Sorry that I didn't have time to clean things up 😅


cooliris.swf is still looking for some files hosted on cooliris.com

With all respect, it is not a good answer to say that Yahoo bought Cooliris and therefore you are not responsible for cooliris.swf. I have invested several years worth of web albums created with cooliris and now although I have downloaded cooliris.swf to my web host, none of the albums show up because coolliris.swf is looking for crossdomain.xml on your site. It should be easy for your developers who created this complex software to open it in Flash and change it so that cooliris.swf is looking for crossdowmain.xml on a relative path.
Thanks

I agree - if you are going to give this away, give ALL of it away, source included. That we we can fix issues like the files that the .swf is still searching for that don't exist any more.

Same issue here :-(
Embeded url in the cooliris.swf file search for :
http://apps.cooliris.com/embed/images/c2_logo_black.png
https://e.cooliris.com/embed/v/900/embed.js
http://apps.cooliris.com/crossdomain.xml
http://apps.cooliris.com/embed/images/sharing2.png
Some has the source...it would be easy make some changes...

Not a whole lot of point now that Flash is going to go away in a few years.
I wish there was an HTML5 replacement. My users love the image wall, and
there is nothing out there comparable. Yahoo did us no favors by swallowing
up a great product.
On Fri, Oct 13, 2017 at 5:34 PM, byman64<EMAIL_ADDRESS>wrote:

Same issue here :-(
Embeded url in the cooliris.swf file search for :
http://apps.cooliris.com/embed/images/c2_logo_black.png
https://e.cooliris.com/embed/v/900/embed.js
http://apps.cooliris.com/crossdomain.xml
http://apps.cooliris.com/embed/images/sharing2.png
Some has the source...it would be easy make some changes...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cooliris/embed-wall/issues/4#issuecomment-336573084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADVbWd7SwxySw9fkeEPBxdMWJpVzJFN5ks5sr9dwgaJpZM4JN6tG
.

I have changed my web album to Lightbox. I found this javascript library easy to implement and it has a lot of options. Most importantly, it will not suddenly stop working like cooliris.

I would like just update a my old project, I already migrated to phoswipe.com, another great product.
an example from my site
But there is not another coolires wall3d valid like it :-(