IndexOutOfBoundsException when reading multiple documents

When I try to read the 52 MITRE documents with ExtractFromDirectory, I run into the following error after a certain number of documents are processed.
java.lang.IndexOutOfBoundsException: 4
	at scala.collection.immutable.Vector.checkRangeConvert(Vector.scala:123)
	at scala.collection.immutable.Vector.apply(Vector.scala:114)
	at org.clulab.wm.eidos.EidosActions.$anonfun$mergeAttachments$4(EidosActions.scala:190)

(I have suppressed the rest of the stack trace for readability)
This occurs whether I keep or remove the parallelism in ExtractFromDirectory. The issue seems to be in the mergeAttachments function in EidosActions - in this line:
https://github.com/clulab/eidos/blob/676944d2afde1a4fa75298b98dc34117aca88923/src/main/scala/org/clulab/wm/eidos/EidosActions.scala#L190
the index is out of bounds. Was it @ZhengTang1120 who wrote the function? If so, could you (@ZhengTang1120 ) take a look at it and see what's going on?

Please continue to pound on it.  Can you remind me where these 52 documents are stored?  Thanks.

It's in the shared Google Drive folder called WorldModelers: https://drive.google.com/open?id=1cHJIfQTr0XE2CEqbo4POSm0-_xzrDP-A
it's under evaluations/6month-eval/mitre-17Feb2018/DocumentSubsetArchive.zip

@adarshp are you running on the txt files that I uploaded here? https://drive.google.com/drive/folders/1ECQNUej2O3_W9NmlA-wyPkwY1UhW_qnD
I have read all of those with Eidos via INDRA (which calls Eidos classes directly rather than the CLI you are using here) and it worked well, though it takes a bit of time.

@bgyori Yes, I am running it on those. When did you run Eidos on the documents? I was able to do it a couple of weeks ago just fine, but when I tried to do it again today I was not able to. The mergeAttachment function where the error seems to be popping up was  first added 6 days ago and has undergone more recent modifications.

I ran them on 3/27 so yeah, it's possible that something changed since then.

That should fix the crashing issue.  In the future I'll run through the 52 documents locally before accepting any suspicious code changes, my own in particular.  @adarshp deserves a bug bounty.  Thanks.


Cannot remove the file

It seems that our rm have some troubles deleting ordinary files from the file system.
PASH /tmp> touch 123
PASH /tmp> ls 123

Exists            : True
Name              : 123
IsReadOnly        : False
...

PASH /tmp> rm 123
The item at path '/tmp/123' has child items. Use recursion to remove it
  +CategoryInfo: NotSpecified, Reason: PSInvalidOperationException
  +FullyQualifiedErrorId: PSInvalidOperationException,

sh works in this case:
PASH /tmp> sh -c 'rm 123'

This seems to result from wrong logic in the FileSystemProvider's HasChildItem method. Feel free to try to fix it and add your test :) (I'm currently on long-term vacation, so I have no time for this ;))

@sburnicki no problems. I've just installed Pash from our shiny new official nixos package and started to document issues I face while using it in my environment.
I think I'll start fixing that issues in a couple of days.


Need to handle native function outputs in a consistent way.

Is your feature request related to a problem? Please describe.
Currently, native function results like _transfer are written to db without explicit permissions.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Merged to #195


Not seeing "Show Inspector" on Outlook for Mac

I think few more instructions are missing
https://github.com/OfficeDev/office-js-docs-pr/blob/master/docs/testing/debug-office-add-ins-on-ipad-and-mac.md
Did all that, those commands and when opening Outlook for Mac and attempt to right-click on my custom add-in, I see no such "Show Inspector" context menu entry.
Probably something else missing?

hi there - the instructions on the github docs work for me as-is. could you please share what version of OS you are running + the Office version?
-Sudheer

Office 2016 on Mac OS

can you be a bit more specific? I'd like the versions from Apple Logo > About this Mac
and Outlook > About Outlook

Great - thanks. It should work for your build. I just tested the instructions again on a new macbook. you should try restarting Outlook [if you havent already].

No @smaremanda I still do NOT see such a context menu entry with "Show Inspector" inside Outlook Native. Can you do a screenshot on your side and show me here?

sure here it is ...

@smaremanda looks weird. Is this in Outlook Native? Can I see the whole app? Or is this a web page?

its outlook for Mac. version 16.14.

Weird. Why am I not seeing that context-menu at all?
Really ran the command defaults write com.microsoft.Outlook OfficeWebAddinDeveloperExtras -bool true and restarted my machine.
Can you suggest other steps to find out what else is missing?

I do not see a context menu at all. And what's an add-in frame? And yes I'm in read mode.
My office add-in is using <ExtensionPoint xsi:type="MessageReadCommandSurface"> as an entry point by the way.

Ok. what happens when you click on that button? does it launch a taskpane or a "frame" like you saw in my screenshot above?

No, clicking on my add-in button doesn't show any additional UI elements. It only moves the message around and forwards it to someone else.

Ok. so your add-in is a UI-less command. You will only see the context menu if you have UI displayed as part of your add-in [the context menu belongs to the web control that displays your add-in]. Are you able to display a blank page on your dev box to debug your add-in?

Yes able to debug on my dev box but only via Office 365 online. Not on Outlook Native.
I think documentation should be corrected, saying elements in Outlook Native can only be inspected when add-in is NOT UI-less.

that's fair. I can follow up on that and make sure its updated.

The documentation has been updated. Thank you for pointing out this issue.
I'll proceed to close this issue now because we resolved it. Thanks!


AttributeError: 'Settings' object has no attribute 'ONCE'

I am always receiving this error in celery logs after execution like
[2017-07-10 15:40:33,053: CRITICAL/MainProcess] Task provider.KENO.tasks.add[1da681d2-2815-4257-b139-01facaaef67d] INTERNAL ERROR: AttributeError(u"'Settings' object has no attribute 'ONCE'",)
Traceback (most recent call last):
  File "/Users/deanchristianarmada/Desktop/projects/radars/radar/lib/python2.7/site-packages/celery/app/trace.py", line 299, in trace_task
    state, retval, uuid, args, kwargs, None,
  File "/Users/deanchristianarmada/Desktop/projects/radars/radar/lib/python2.7/site-packages/celery_once/tasks.py", line 129, in after_return
    self.once_backend.clear_lock(key)
  File "/Users/deanchristianarmada/Desktop/projects/radars/radar/lib/python2.7/site-packages/celery_once/tasks.py", line 52, in once_backend
    return import_backend(self.once_config)
  File "/Users/deanchristianarmada/Desktop/projects/radars/radar/lib/python2.7/site-packages/celery_once/tasks.py", line 48, in once_config
    return self.config.ONCE
  File "/Users/deanchristianarmada/Desktop/projects/radars/radar/lib/python2.7/site-packages/celery/datastructures.py", line 353, in __getattr__
    type(self).__name__, k))
AttributeError: 'Settings' object has no attribute 'ONCE'

My celery version is 3.1.24. I tried it in 4.02 and the error still exists
My code is:
from celery import Celery
from celery_once import QueueOnce

celery = Celery('tasks', broker='amqp://guest:guest@localhost:5672//')
celery.conf.ONCE = {
  'backend': 'celery_once.backends.Redis',
  'settings': {
    'url': 'redis://<IP_ADDRESS>:6379',
    'default_timeout': 60 * 60
  }
}

@celery.task(base=QueueOnce)
def add(x, y):
    return x + y

I run  it through terminal celery -A proj -l info and coded in python manage.py shell like:
>>> from app import add
>>> x = add.apply_async(args=[6, 74])
>>>

Hey Dean-Christian-Armada thanks for the bug report.
I've been trying to re-create this on my side following your instructions but haven't been able to.
The code example you provided, is it in one file?
One thing to check is that you are importing the declared celery and not directly via import celery.

yes, only in one file app/prov/KEN/tasks.py
I'm using django by the way
settings.py
APP_TASKS = [
    'prov.KEN',
]
INSTALLED_APPS += APP_TASKS

Then I have a celery.py in the same folder as my settings.py the code is:
from __future__ import absolute_import
import os
from celery import Celery
from django.conf import settings

# set the default Django settings module for the 'celery' program.
os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'radar.settings')
app = Celery('app')

# Using a string here means the worker will not have to
# pickle the object when using Windows.
app.config_from_object('django.conf:settings')
app.autodiscover_tasks(lambda: settings.INSTALLED_APPS)

Hey Dean-Christian-Armada, from that settings.py and app.config_from_object('django.conf:settings') you posted above, it seems to be missing any ONCE configuration options.
In your settings.py, could you try adding
ONCE = {
  'backend': 'celery_once.backends.Redis',
  'settings': {
    'url': 'redis://<IP_ADDRESS>:6379',
    'default_timeout': 60 * 60
  }
}

or
CELERY_ONCE = {
  'backend': 'celery_once.backends.Redis',
  'settings': {
    'url': 'redis://<IP_ADDRESS>:6379',
    'default_timeout': 60 * 60
  }
}

And seeing if that works?

I encountered the same problem in one of my projects using Celery 4.1. I can confirm that adding the first block to my celery configuration resolved the problem.
Would it make sense to set some "sane defaults" for celery_once's config?

@codekoala For another django project? If so, it's maybe worth mentioning in the docs someone.
I'm against adding defaults, I like following Python's mantra of "Explicit is better than implicit."

Fair enough.
My project does not use Django at all. It's just standard Python.

And @codekoala  how are you set the config file?

For those that still encounter the same issue in using Django and celery_once, here's how I've set this up:
celery.py
...
app = Celery('yourapp')

app.conf.ONCE = settings.CELERY_ONCE  # force CELERY_ONCE to load settings
app.config_from_object('django.conf:settings')
app.autodiscover_tasks(lambda: settings.INSTALLED_APPS)
...

settings.py
...
CELERY_ONCE = {
      'backend': 'celery_once.backends.Redis',
      'settings': {
        'url': 'redis://<IP_ADDRESS>:6379/0',
        'default_timeout': 60 * 60
      }
    }
...

For those that also run Django tests on their tasks that use QueueOnce, here's how I mock the redis backend.
test_something.py
....
from django.test import TestCase
from fakeredis import FakeStrictRedis
from mock import patch

...

class AsyncTaskTest(TestCase):

    def setUp(self):
        self.get_redis = patch('celery_once.backends.redis.get_redis')
        self.mocked_redis = self.get_redis.start()

        self.redis = FakeStrictRedis()
        self.mocked_redis.return_value = self.redis

    @patch('yourapp.tasks.call_command')
    def test_your_task(self, mockCall):
        """
        Assert that your_task
        calls your_command only once.
        """
        your_task.delay()

        # Set redis key with TTL 100 seconds from now
        # so subsequent tasks won't run
        self.redis.set('qo_your_app.tasks.your_task', int(time()) + 100)

        your_task.delay()
        your_task.delay()

        mockCall.assert_called_with('your_command')
        self.assertEqual(mockCall.call_count, 1)

    def tearDown(self):
        self.get_redis.stop()


Strengthen an assertion in AccessSummaryAnalysis.

Mark Lacey caught this bug in the assertion logic that looks for expected SIL
patterns involving non-escaping closures. I broadened it to allow multiple use,
but it was only verifying the first use. This is NFC w.r.t. our automated
testing because that never hits the multiple-use case.

@swift-ci smoke test and merge.

@swift-ci smoke test OS X platform.


Context aware suggestions for optional fields

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #32262
Approach
https://user-images.githubusercontent.com/1329674/131994144-533dbfcd-a554-4219-80b3-2184813e5832.mp4
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@kaviththiranga @NipunaMarcus please note this behaviour change.


Add profile picture

It would be nice to have the possibility to add a profile picture on the left of the name and short description.
I managed to get an okay result with the following:

My Name
=======

---
> Some description
---

...

Nice, but I'm guessing that wouldn't work for pdf output?

For anyone stumbling on this one, there is a stupidly simple work around for this, you just Ctrl+p in your browser. It will keep the pictures but also the colors


Theme on github pages not working

Currently I try to publish a website with this theme to github pages.
Unfourtaly the statics (css & js) files will not resolved correctly.
I think the issue is because of using relURL here instead of absURL:
https://github.com/matcornic/hugo-theme-learn/blob/9ed9958316b419d9820dae850d93dfc762b79e4b/layouts/partials/header.html#L12-L19
&
https://github.com/matcornic/hugo-theme-learn/blob/9ed9958316b419d9820dae850d93dfc762b79e4b/layouts/partials/footer.html#L59-L69
GitHub project pages are located in pages.github.com/user/project.
When we use relURL it points to pages.github.com/[css|js]. This is wrong.
It should be pages.github.com/user/project/[css|js]. I think with the change to absURL would fix this...

I just found out that we have the same issue with the navigation...

I think you want to be running your build with the -b flag
-b, --baseURL string         hostname (and path) to the root, e.g. http://spf13.com/

I have set the baseUrl via the config.toml:
baseURL = "https://pages.github.com/user/project/"

Isn't this the same? 🤔

You're welcome! 👍


Template stacktraces

This seems to be all it takes to get the correct filename and line number to show up when there is an error in a .tt template.
Stacktrace before:
(erb):7:in `block in template': undefined method `each' for nil:NilClass (NoMethodError)
	from (erb):5:in `each'
	from (erb):5:in `template'
	from /opt/boxen/rbenv/versions/2.1.0/lib/ruby/2.1.0/erb.rb:850:in `eval'
	from /opt/boxen/rbenv/versions/2.1.0/lib/ruby/2.1.0/erb.rb:850:in `result'
	from /opt/boxen/rbenv/versions/2.1.0/lib/ruby/gems/2.1.0/gems/thor-0.19.1/lib/thor/actions/file_manipulation.rb:116:in `block in template'

Stacktrace after:
/Users/Thoughtworker/repos/opensource/swagger/swagger-thor/resources/templates/grape/files/app/api.rb.tt:7:in `block in template': undefined method `each' for nil:NilClass (NoMethodError)
	from /Users/Thoughtworker/repos/opensource/swagger/swagger-thor/resources/templates/grape/files/app/api.rb.tt:5:in `each'
	from /Users/Thoughtworker/repos/opensource/swagger/swagger-thor/resources/templates/grape/files/app/api.rb.tt:5:in `template'
	from /opt/boxen/rbenv/versions/2.1.0/lib/ruby/2.1.0/erb.rb:850:in `eval'
	from /opt/boxen/rbenv/versions/2.1.0/lib/ruby/2.1.0/erb.rb:850:in `result'
	from /Users/Thoughtworker/repos/opensource/thor/lib/thor/actions/file_manipulation.rb:118:in `block in template'

Very nice. Would you mind adding a spec for this so it doesn’t regress in the future?

Done.


fix the Scala URL issue for downloading data in scripts

Description
Some of the downloading links are not valid anymore for the Scala package. Redirect them into more stable servers.
@nswamy @piyushghai
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@lanking520 Thanks for the contribution!
@mxnet-label-bot [pr-awaiting-review]


[0.9.1] [Linux] Implementing new weather szenario "night", CarlaWeather.ini

Hello everyone,
I would like to create a night szenario in Carla.
I tried to create a new weather setting inside "CarlaWeather.ini". But I couldn't select it while i was using the manual_control.py.
Is this the right way or is there a better alternative?
And If this is the right way, does anyone have already the the right weather setting for a night szenario?

Great! Thank you very much.
While changing a few settings inside the editor it is possible to get a real night feeling.

@Atlunch - Which settings did you change to get above night effect in UE4 editor?

Great! Thank you very much.
While changing a few light settings inside the editor it is possible to get a real night feeling.

Hey! I'm really interested on this feature as well; would you mind sharing the assets/how to do that please?

@rikardomarenzzi  Can you mention the settings please


Update DS version to v46 and tidy up front end code

What and why?
This PR will update RASRM frontstage to v46 of the Design System. It also fixes the breaking changes and makes some changes to tidy up the front end code
How to test?
Trello

 Update ROPS DS version to 46


Fixes #38

Adds additional trail attributes 'type' and 'interval'.

@dube1503 the shapes mutator looks good. Could you please add interval to be an option to the variable blocks?

@dube1503 also can we change the input on make trail to a drop down with just true or false like the picture in #38

@codestar12 is it a dropdown field or is it a shadow block created with the addition of the make trail attribute?

lets make it a dropdown for now

making it a dropdown "for now" is actually not a very reasonable solution I am coming to discover. All other attributes take inputs and the get_variable block iterates through lists and determines its own type based off the connection of the block in the input space. Therefore, in order to return a type for the value of "make_trail" means rewriting the way the variable blocks work for this only instance, which is possible, but if we want to have a block input for assigning the attribute in the same vain as all the other shape attributes work then it doesnt seem like a good use of time to rewrite this functionality only for us to refactor it back to its original manor if we decide to populate attributes with default shadow blocks as discussed in #57 .
That being said, adding the menu options for the trai_type and interval are necessary regardless.

The newest commit now included the ability to access the new trail features and also updates the block type and color to reflect the type that the block either expect to receive or that it outputs (set and get block respectively).
It also fixes an undiscovered bug that texture had no type assigned to it in the variable block. When testing the typeof a texture assignment in glowscript it returns a type of string so the texture block and variable blocks for textures now reflect this type assignment (uses the string color type).


Null reference exception while decrypting stream to stream

I've been trying to encrypt and decrypt files in memory using streams, but there is a null reference exception on
DecryptAsync(Stream inputStream, Stream outputStream, Stream privateKeyStream, string passPhrase)

method.
My code:

Key generating:

public static void CinchooPGP_GenerateKeys()
{
    using (ChoPGPEncryptDecrypt pgp = new ChoPGPEncryptDecrypt())
    {
        pgp.GenerateKey(@"C:\TEMP\Keys\public.asc", @"C:\TEMP\Keys\private.asc", null, "password");
    }
}

Encryption:

        public static async Task<MemoryStream> CinchooPGPEncryption()
        {
            var ContentToEncryptStream = await FileToStreamMethod(@"C:\TEMP\Content\content.txt");
            var PublicKeyStream = await FileToStreamMethod(@"C:\TEMP\Keys\public.asc");
            MemoryStream encryptedContent = new MemoryStream();
            using (ChoPGPEncryptDecrypt pgp = new ChoPGPEncryptDecrypt())
            {
                //this accepts 3 streams: stream to encrypt, stream to decrypt, public keys
                await pgp.EncryptAsync(ContentToEncryptStream, encryptedContent, PublicKeyStream);
            }
            return encryptedContent;
        }

Decryption:

        public static async Task<MemoryStream> CinchooPGPDecryption(MemoryStream encryptedContent, string passPhrase)
        {
            var PrivateKeyStream = await FileToStreamMethod(@"C:\TEMP\Keys\private.asc");
            MemoryStream decryptedContent = new MemoryStream();

            using (ChoPGPEncryptDecrypt pgp = new ChoPGPEncryptDecrypt())
            {
                await pgp.DecryptAsync(encryptedContent, decryptedContent, PrivateKeyStream, passPhrase); //exception is thrown here
            }
            return decryptedContent;
        }

Stacktrace:
   at Cinchoo.PGP.ChoPGPEncryptDecrypt.Decrypt(Stream inputStream, Stream outputStream, Stream privateKeyStream, String passPhrase)
   at Cinchoo.PGP.ChoPGPEncryptDecrypt.<>c__DisplayClass47_0.<DecryptAsync>b__0()
   at System.Threading.Tasks.Task.InnerInvoke()
   at System.Threading.Tasks.Task.<>c.<.cctor>b__277_0(Object obj)
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)
--- End of stack trace from previous location ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Cinchoo.PGP.ChoPGPEncryptDecrypt.<DecryptAsync>d__47.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()

No stream is null while I was debugging the app, so I think it's thrown somewhere inside the lib. Or maybe I am doing something wrong?

I had the same issue when decrypting. Make sure that you set the input stream's position to zero. ChoPgp doesn't do it.


CREATE A NEW USER FLOW TO IMPROVE THE USER EXPERIENCE

Explanation
To improve the user experience, we thought of dividing the flow into 2 parts:
Part 1. The user has the options to enter what they want to see, for example: Issues or pull request?, User or organization, Repository name
Part 2. The user has the options to select the date.
This flow is with the intention that the user has to select only once the data that points to where he wants to search, and can change the times that the date requires without having to see the previous data. If you want to change that data, there will be a button that takes you to that view again.
Screenshots

Checklist
This issue can be closed when the following tasks are completed:

[ ]  The first thing the user will see will be the part of the form where they can select only the address data:
[ ] Issues

[ ] User / Organization.
[ ] Repository name.
[ ] Next button.

[ ] The next thing the user will see when pressing the "Next button" button is the part of the filters:

[ ] Select with the options: After of, Before of, Equals to.
[ ] An input type "Date" that allows you to select the date from which you want to start searching.

Related Issues

#6

References
https://www.figma.com/file/Pw5EfnvCLnXox2mFS40kdW/Azordev?node-id=0%3A1

I want to be assigned to this task


Identifying equivalent variables when importing constraints into a parent model

Closely related to #442, here is a sketch of a test I would love to see working:
from gpkit import Model, Variable

class Model1(Model):
    def setup(self):
        x = Variable('x')
        y = Variable('y')
        objective = x
        constraints = [x >= y, y >= 1]
        constraints += Model2().constraints
        return objective, constraints

class Model2(Model):
    def setup(self):
        y = Variable('y')
        objective = y
        constraints = [y >= 2]
        return objective, constraints

sol = Model1().solve()
assert sol['cost'] == 2

Hmm, interesting.


On Collection Deposits page, display title up to 150 characters and show full title on mouseover

Design Doc: https://docs.google.com/document/d/1fcr2DYo7OrX-qTdeUOTWKSS1rlq_47sehESmbvgUsrk/edit#heading=h.di4p5wjzg5xm
This has been recently refined. Titles of deposits should display up to 150 characters. If there are more than 150 characters, display 150 characters followed by an ellipse.
Example of one of these pages with a title over 150 characters: https://sul-h2-stage.stanford.edu/collections/105/works

I have filed ticket # for mouseover to display the entire title so that it's visible somewhere.
I have filed ticket # for this same issue on the dashboard.

I have filed ticket # for mouseover to display the entire title so that it's visible somewhere.
I have filed ticket # for this same issue on the dashboard.


Stronger wording for non-UK assesment only

Trello card
https://trello.com/c/hlUs8iJf
Context
The International team have had contact from a number of UK assessment only providers as they’ve been contacted by non-UK applicants about QTS.
Changes proposed in this pull request
Stronger wording in the non-UK section to make it clear only the providers listed under non-UK will be able to help.
Guidance to review

@philspray Looks fine to me - one suggestion re the anchor link but otherwise good to deploy Weds first thing!


Consider not displaying continue results if query matches start of command name

ie, currently, if the user wants to tgl reports, the plugin will fuzzy filter against tgl r, tgl re, ..., tgl report until it detects the tgl reports command.
This leads to a pretty jittery user experience.
Perhaps check if the query is an exact match for the start of a command name, and suppress continue results if so.
This would also make selecting commands much easier, as the results will not be crowded with past time entries.

Perhaps also consider hiding the start result, so you can quickly trigger a command without needing to type its name out in full (as it would then be the top result, and you can tab/enter it)


Gulp tasks seen as error on VS.

those gulp task are not an error, but VS show as errors and also (when publishing) it says on status bar: published failed, but app was published successfully

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.


feat(history): add getOrigin method

Per https://github.com/aurelia/router/issues/88 we want to be able to generate an absolute URI with the router.generate() method. That method relies upon the history module. getOrigin allows the router to generate an absolute URI with the protocol, hostname, and port.

@jedd-ahyoung If you can address the comment from @bryanrsmith then pending any other discussion we can probably merge this.

This has been updated with a corrected error message.

@bryanrsmith I'll let you decide when to merge this since there are several other PRs that are part of the same logical improvement.

Where are we at with this? We want to get to an RC, so I'd like to get things like this resolved in the next couple of weeks.

@EisenbergEffect, I will look into this personally in the coming week. This
hasn't moved due to my lack of time. I may be able to schedule some time to
work on this specifically with @bryanrsmith.
2016-03-19 12:41 GMT-04:00 Rob Eisenberg<EMAIL_ADDRESS>
Where are we at with this? We want to get to an RC, so I'd like to get
things like this resolved in the next couple of weeks.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/aurelia/history/pull/7#issuecomment-198746852

@jedd-ahyoung ping

After the modal stuff is done, we really need to get this wrapped up fast.

Status?
We need this and dialog issues wrapped in the next week or two at most. You weren't in the meeting...but we are trying to get to RC and this is holding us up....

@EisenbergEffect This can be merged, along with https://github.com/aurelia/history-browser/pull/22 and https://github.com/aurelia/router/pull/294. The test is failing because of a missing karma entry in package.json. Previously, I think we had an issue with the hash or something like that - something about options.root - but I can't find the original problem, and it looks as though that's been resolved. Testing this in the skeleton-navigation app yields positive results.

Ok, great. @bryanrsmith Do you have time to do a quick second look?

lgtm


Retrieving headers from csv using nodejs

Hi I need to list all files from a S3 bucket and I need to list the headers of that file for that I used the below code it is not working is there any other way to achieve it
const AWS = require('aws-sdk')
var csvHeaders = require('csv-headers')
const s3 = new AWS.S3({
accessKeyId:'----------------',
secretAccessKey:'------------------------------',
region:'ap-south-1'
})
const params = {
Bucket:'abcd-1'
}
s3.listObjects(params,(err,data)=>{
if(err){
return console.log(err)
}
var options = {
file      : data.Contents.Keys,
delimiter : ','
};
csvHeaders(options, function(err, headers) {
if(!err)
console.log(headers);
})
});

Looks like you have the wrong library. this is csvtojson. If using this library you can use the .on('headers', (headers: string[]) => {})

@Sethu16 please close the issue


Fixes markdown header levels

...to have only a single H1, and other headers properly nesting by level.

I'd also like to edit this readme to add info about the required env vars to get the local build running. Let me know if you are open to that, I'll pop up another PR to address that specific issue.


Migrated concreate descriptor to be object based

Now every important part of the descriptor is treated as object. With
this we can easily extend our schema and use object api instead of
plain dictionaries inside code.
This also enables to have multiple schemas per object type, so we can
easily introduce incompatible changes to our schema without breaking
old images. Each object is resposinble to handling its schema and
converting it to compatible object state.

@goldmann can you please review again?


SAK-40808 Lessons update Turkish my workspace info HTML

@eemirtekin please refrain from putting information about the branch on the commit message

@eemirtekin please refrain from putting information about the branch on the commit message

Thanks for your interest.
I will do it so.

Hi @mpellicer,
Can you add label to this commit far Sakai 19.x?
Thanks in advance.


Small input decorator clean up

This PR adds small style fixes to input decorator and its template.
Came up in review #120910, pointed out by @QuncCccccc
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

test-exempt: code refactor with no semantic change


Nested SVG positioned incorrectly

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 256 256">
	<rect x="0" y="0" width="256" height="256" fill="red" />
	<svg x="0" y="0" viewBox="0 0 256 256">
		<rect x="0" y="0" width="256" height="256" fill="lime" />
		<circle cx="128" cy="128" r="32" fill="blue" />
	</svg>
</svg>

This SVG should appear like this:

But Ladybird renders it like this:

The red rect shouldn't appear at all since it's covered by the lime rect in the nested SVG, but that seems to be in the wrong place

Reproducible as of d4d335ebdadbb0364913f50725dafbd1d07feaf5


add SD_APP_EXTENSION macro for ios8 extensions

define the SD_APP_EXTENSION preprocessor macro when you are building an extension that includes SDWebImage. This avoids compile time errors about the user of [UIApplication sharedApplication]
Addresses: #859

Any reason this hasn't been merged yet? It won't affect existing users at all. I'd say a simple note in the documentation that the developer is responsible for calling cleanup within an extension should suffice for now.

Note: this also happens when you build the library as a framework with the new version of CocoaPods, so it's likely about to become a much bigger problem for people.

+1

+1

Any plans on when this will be added? As AppBoy has a direct dependency to SdWebImage, we are forced to stay behind on a buggy cocoa pods version as SDWebImage cannot build for extensions. Would be great to hear from you!

closing this. this https://github.com/rs/SDWebImage/commit/91b425111518b3a818fcea5a775b8fc6e606cee9 was already merged


[BUG] Receiving: Error buildAgentGraph - TypeError: Cannot read properties of undefined (reading 'includes') after including Conditional Agent.

Describe the bug
The flow stopped working after including the Conditional Agent.
To Reproduce

Load the provided JSON Agent file.
Observe that the flow stops working.

Agent JSON File
You can download the JSON file here: https://github.com/user-attachments/files/16683760/Sequential.Agents.v2.Agents.json
Expected behavior
The flow should continue working as expected after including the Conditional Agent.
Screenshots

Flow
I was following Leon's tutorial from this YouTube video:: https://www.youtube.com/watch?v=6LbvgTbS0BE&t=1573s.
Setup

Flowise Version 2.0.3 (also occurs on version 2.0.5)
Running Docker locally. The problem also occurs in an online version.

Error Trace
2024-08-20 21:20:16 Error: Error buildAgentGraph - TypeError: Cannot read properties of undefined (reading 'includes')
2024-08-20 21:20:16     at buildAgentGraph (/usr/src/packages/server/dist/utils/buildAgentGraph.js:345:15)
2024-08-20 21:20:16     at async utilBuildAgentResponse (/usr/src/packages/server/dist/utils/buildChatflow.js:363:31)
2024-08-20 21:20:16     at async utilBuildChatflow (/usr/src/packages/server/dist/utils/buildChatflow.js:159:20)
2024-08-20 21:20:16     at async createInternalPrediction (/usr/src/packages/server/dist/controllers/internal-predictions/index.js:7:29)

In your Condition node, change it to $flow.output.route:

It should works fine now.


Switchbot Restore State

Home Assistant release with the issue: 0.91.3

Last working Home Assistant release (if known): 0.90
Operating environment (Hass.io/Docker/Windows/etc.): hassbian

Component/platform: switchbot

Description of problem:
After restarting home assistant the switch is always on.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
switch:
  - platform: switchbot
    mac: 'my_mac_address'

Same issue here the switch is always on.
@luca-angemi how did you connect your switchbot to HA? Did you pair it with bluetooth or?

Hi.
No, just configured first with my iPhone then in configuration.yaml

Hi
Same here I think the reason why the state is always on of the switch is due to the switch gets disconnected to bluetooth and is not always connected to bluetooth

Not sure about that. Might be related to #22087

Will this be resolved in 0.92? I can't see it in the release of 0.92 as resolved?

hi, it seems it didn't go in last night.

0.93

Hi
I keep getting this error with switchbot:
2019-04-25 22:12:42 ERROR (SyncWorker_5) [switchbot] Cannot connect to switchbot. Retrying
Traceback (most recent call last):
File "/config/deps/lib/python3.7/site-packages/switchbot/init.py", line 48, in _sendpacket
hand.write(binascii.a2b_hex(key))
File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 200, in write
return self.peripheral.writeCharacteristic(self.valHandle, val, withResponse)
File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 543, in writeCharacteristic
return self._getResp('wr')
File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 407, in _getResp
resp = self._waitResp(wantType + ['ntfy', 'ind'], timeout)
File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 362, in _waitResp
raise BTLEDisconnectError("Device disconnected", resp)
bluepy.btle.BTLEDisconnectError: Device disconnected
Can you guys please look into it?
Thanks

@bachoo786 Could you try this fix: https://github.com/home-assistant/home-assistant/commit/95bbea20a87dc321b252a2be19cee06c5dcbfcdb ?
It will be part of 0.93

How do I do that in docker? I am running HA in a docker container

ok managed to change all the files except for requirements_all.txt as I couldn't find it in docker container. The problem still remains same errors and the state of the switch is always on after HA restart.

Then I think you should try again after 0.93 is released

thanks.
can I just point out I get errors of "cannot connect to switchmate" but I am using switchbot. I think the switchbot component looks into 2 devices ie. switchmate and switchbot.
is it possible to keep the 2 devices separate?

That is just a typo in the error message, fixed it now

after doing your changes except for the requirements_all.txt I had issues with the component and at times it would work and at times it wouldnt when I tried to switch on or off. The errors are many but this is the latest one:
[Errno 32] Broken pipe
Traceback (most recent call last):
  File "/config/deps/lib/python3.7/site-packages/switchbot/__init__.py", line 48, in _sendpacket
    hand.write(binascii.a2b_hex(key))
  File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 200, in write
    return self.peripheral.writeCharacteristic(self.valHandle, val, withResponse)
  File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 542, in writeCharacteristic
    self._writeCmd("%s %X %s\n" % (cmd, handle, binascii.b2a_hex(val).decode('utf-8')))
  File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 302, in _writeCmd
    raise BTLEInternalError("Helper not started (did you call connect()?)")
bluepy.btle.BTLEInternalError: Helper not started (did you call connect()?)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/app/homeassistant/components/websocket_api/commands.py", line 121, in handle_call_service
    connection.context(msg))
  File "/usr/src/app/homeassistant/core.py", line 1138, in async_call
    self._execute_service(handler, service_call))
  File "/usr/src/app/homeassistant/core.py", line 1160, in _execute_service
    await handler.func(service_call)
  File "/usr/src/app/homeassistant/helpers/entity_component.py", line 194, in handle_service
    required_features
  File "/usr/src/app/homeassistant/helpers/service.py", line 316, in entity_service_call
    future.result()  # pop exception if have
  File "/usr/src/app/homeassistant/helpers/service.py", line 336, in _handle_service_platform_call
    await getattr(entity, func)(**data)
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/homeassistant/components/switchbot/switch.py", line 58, in turn_off
    if self._device.turn_off():
  File "/config/deps/lib/python3.7/site-packages/switchbot/__init__.py", line 63, in turn_off
    return self._sendpacket(OFF_KEY)
  File "/config/deps/lib/python3.7/site-packages/switchbot/__init__.py", line 50, in _sendpacket
    if retry < 1 or not self._connect():
  File "/config/deps/lib/python3.7/site-packages/switchbot/__init__.py", line 36, in _connect
    bluepy.btle.ADDR_TYPE_RANDOM)
  File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 391, in __init__
    self._connect(deviceAddr, addrType, iface)
  File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 433, in _connect
    self._writeCmd("conn %s %s\n" % (addr, addrType))
  File "/config/deps/lib/python3.7/site-packages/bluepy/btle.py", line 305, in _writeCmd
    self._helper.stdin.flush()
BrokenPipeError: [Errno 32] Broken pipe

I have therefore gone back and undone the changes and I will hope it would continue to work as before bar the 'on state' issue.
Thanks for your help. I will wait till this is all resolved in 0.93

I can report after reverting back to the way things were the component has broken for me :( and I get the same error as in my previous post: [Errno 32] Broken pipe

I apologise even after all this I still get broken pipe error


[WIP] Add basic plugin support

This PR is a proposal for adding basic plugin support in Traefik using the new Golang 1.8 plugin mechanism.
Right now, only Middleware plugin type is available. Here is an example of plugin:
package main

import (
	"github.com/containous/traefik/log"
	"github.com/containous/traefik/plugin"
	"net/http"
)

var _ plugin.Middleware = (*Sample)(nil)

type Sample struct {
}

func Load() interface{} {
	return &Sample{}
}

func (s *Sample) ServeHTTP(w http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
	log.Debug("Sample plugin begin")
	next(w, r)
	log.Debug("Sample plugin end")
}

Build your plugin using:
go build -buildmode=plugin -o sample.so sample.go

Then add in you Traefik toml file:
[[plugins]]
path = "sample.so"

Please, give your feedback on this basic implementation :)
/cc @containous/traefik
Fixes #1336
Fixes #1048
Part of #30
Signed-off-by: Emile Vauge<EMAIL_ADDRESS>

@timoreimann thanks for your review :)

Should we expand the docs?

Of course! That's why I added [WIP] in the description, just wanted to validate the implementation first :)

We have no single test. Should we add some?

I will add an integration test.

I wonder if this will affect building Traefik locally on Mac machines (which I use) where plugins are not supported yet. Will need to test.

Hum. I hope this will not be an issue... Looking forward to get the results of your tests.

OK, bad news... We can't load an external plugin if Traefik is built with CGO_ENABLED=0, and we really need this to build a statically linked golang executable to run in a Docker container.
https://github.com/golang/go/issues/19569

To be continued #1865


Remove crazyegg from the preview-index.html

It looks like it is more than 6 years old https://github.com/openstreetmap/iD/blame/372bb7d892048d67415087f474688f85391bab2a/dist/index.html#L28 and the linked URL seems to be broken:

$ curl http://dnn506yrbagrg.cloudfront.net/pages/scripts/0013/6714.js
"undefined"==typeof CE2&&(CE2={}),CE2.uid=136714,CE2.status="no data available";

So to me it looks like it is save to remove it(?)

Yup  🔪


Seperate Key Down and Key Up

First of all thanks for this plugin. It's pretty awesome.
One thing that would be very useful would be seperate buttons for key down and key up events.
This could be used in multi actions using a delay. Then one could for example for 'exit seat' add a key down event, a delay and a key up event. This way the player could simply press the button instead of holding it down. Additionally this could be combined with other actions like switching to a on-foot profile after leaving a seat.
As an alternative a button that allows to set a delay would be great.
Thanks!

I'll consider that for a future update.
I don't want to make the dropdown three times the size (normal, up, down)
So, the button  template would have to be redesigned with 3 extra radio buttons. (normal, up, down)

Or a seperate button with one more option: down time in ms

@garv3
I tested the current static button type with the multi-action button
and it works fine for me.
the streamdeck software takes care of the key down/up events, for each step in the multi-action list.
Do you have a use case, where it is not working for you ?

@garv3
see version 1.,0.7


Ability to raise type checking error using something like a typeerror type.

Search Terms
typeerror type, user-defined type checking
Suggestion
Conditional type brings a useful method for composing types, but there are many scenarios that conditional type works while losing type safety. It would be nice to have a special type like typeerror to notify the compiler of incorrect types:
let foo: typeerror;
    ^^^ error

let bar: {yo: string; ha: typeerror};
    ^^^ error

Examples
interface Foo {
  yo: string;
  ha: number;
  pia: boolean;
};

type Bar = {
  [K in keyof Foo]:
    Foo[K] extends string ? string[] :
    Foo[K] extends number ? number :
    typeerror;
};

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. new expression-level syntax)

This has come up, but the biggest question is really just a matter of where and how to give the error when the type appears in an arbitrarily complex type.

Looks like a duplicate of #23689.

@mattmccutchen Thanks for referring.


color support for more than one argument

Hi,
color support works fine when using
$log.warn("test") // printed correctly in orange

however, with multiple parameters it doesn't work:
$log.warn("test", "test") // printed incorrectly in black

All the best,
Robin

Second field is usually reserved for color overrides.
$log.log('%c Oh my heavens! ', 'background: #222; color: #bada55');

arg. I thought since you have examples like $log = $log.getInstance(...), that the API would actually stay the same as in normal angular $log, which you do use like $log.log(p1, p2, p3, ...).
Too bad, this is actually a show-stopper for us, since we are not going to use a custom API. I would expect the styling to be overwritten as it is used on the provider, with $log.setLogMethodColor.

I get what your saying, but it's not really an limitation of the library per say, but the browsers. If you take a look at the documentation for coloring in chrome/ff the second parameter always defines color
console.log("%cThis will be formatted with large, blue text", "color: blue; font-size: x-large");

https://developer.chrome.com/devtools/docs/console#styling-console-output-with-css.


unbound: does not compile on macOS 11.1-x86_64 host

Copy from OpenWrt issues, https://github.com/openwrt/packages/issues/14573 as reported by @k-ronny
Description: cross compiling of unbound on a macOS 11.1-x86_64 host is failing in the configure phase:
checking whether CONNECT_RESUME_ON_READ_WRITE is declared... no
configure: error: TCP Fast Open is not available for client mode: please rerun without --enable-tfo-client

with this patch it compiles on macOS and Debian Buster:
--- a/configure.ac	2020-12-03 09:31:18.000000000 +0100
+++ b/configure.ac	2021-01-26 01:16:58.000000000 +0100
@@ -120,6 +120,9 @@
 AC_SUBST(LIBUNBOUND_REVISION)
 AC_SUBST(LIBUNBOUND_AGE)

+# AC_CANONICAL_HOST is needed to access the 'host_os' variable
+AC_CANONICAL_HOST
+

 cmdln="`echo $@ | sed -e 's/\\\\/\\\\\\\\/g' | sed -e 's/"/\\\\"/'g`"
 AC_DEFINE_UNQUOTED(CONFCMDLINE, ["$cmdln"], [Command line arguments used with configure])
@@ -1226,15 +1229,15 @@
 AC_ARG_ENABLE(tfo-client, AC_HELP_STRING([--enable-tfo-client], [Enable TCP Fast Open for client mode]))
 case "$enable_tfo_client" in
 	yes)
-		case `uname` in
-			Linux) AC_CHECK_DECL([MSG_FASTOPEN], [AC_MSG_WARN([Check the platform specific TFO kernel parameters are correctly configured to support client mode TFO])],
+		case "${host_os}" in
+			linux*) AC_CHECK_DECL([MSG_FASTOPEN], [AC_MSG_WARN([Check the platform specific TFO kernel parameters are correctly configured to support client mode TFO])],
 			                     [AC_MSG_ERROR([TCP Fast Open is not available for client mode: please rerun without --enable-tfo-client])],
 			                     [AC_INCLUDES_DEFAULT
 #include <netinet/tcp.h>
 ])
 					AC_DEFINE_UNQUOTED([USE_MSG_FASTOPEN], [1], [Define this to enable client TCP Fast Open.])
 			  ;;
-			Darwin) AC_CHECK_DECL([CONNECT_RESUME_ON_READ_WRITE], [AC_MSG_WARN([Check the platform specific TFO kernel parameters are correctly configured to support client mode TFO])],
+			darwin*) AC_CHECK_DECL([CONNECT_RESUME_ON_READ_WRITE], [AC_MSG_WARN([Check the platform specific TFO kernel parameters are correctly configured to support client mode TFO])],
 			                      [AC_MSG_ERROR([TCP Fast Open is not available for client mode: please rerun without --enable-tfo-client])],
 			                      [AC_INCLUDES_DEFAULT
 #include <sys/socket.h>

@wcawijngaards could you comment on this as a potential build env fix?

Thanks for this @EricLuehrsen and @k-ronny!
Didn't include AC_CANONICAL_HOST as this is already used by acx_nlnetlabs.m4


CHP: "browse course materials" button styling

We missed a couple of details in the mobile styling of course pages
Designs and Mockups
https://projects.invisionapp.com/d/main?origin=v7#/console/18552410/442839508/preview?scrollOffset=213
Acceptance Criteria:

[ ] white text, blue background
[ ] the arrow should be on the left, pointing left (because there's where the drawer will come from)

I need access of the invision design

Try again?

Try again?

Got it. Thanks!


api types: Update InitialDataRealm and RealmDataForUpdate to FL 237

We can use this for #5804 and #5805.

Four of the nine added /register-response fields seem to have a small documentation issue: https://chat.zulip.org/#narrow/stream/412-api-documentation/topic/.60.2Fregister.60.3A.20fields.20missing.20.22Present.20if.20.5B.E2.80.A6.5D.22.3F/near/1707782
For three of them, I didn't find them mentioned in #api design:
realm_can_access_all_users_group
realm_create_multiuse_invite_group
realm_move_messages_between_streams_limit_seconds

Thanks for the review! Revision pushed.

Thanks! Reply above on that one thread. Otherwise all LGTM.

Thanks for the review! Merged, after addressing that reply.


Error when loading data

Bash script train_base.sh throws the following error:
Global seed set to 42 Traceback (most recent call last): File "train.py", line 540, in <module> main(args) File "train.py", line 502, in main checkpoint_callback=get_checkpoint_callback( File "/export/home/dialogue-sum/sa-bart/src/callbacks.py", line 100, in get_checkpoint_callback checkpoint_callback = ModelCheckpoint( TypeError: __init__() got an unexpected keyword argument 'filepath'
Could be caused by an inconsistent pytorch_lightning version.
The README does not specify which version the authors used.

Another error when setting up the model:
Traceback (most recent call last):   File "train.py", line 540, in <module>     main(args)   File "train.py", line 498, in main     trainer: pl.Trainer = generic_train(   File "/export/home/dialogue-sum/sa-bart/src/lightning_base.py", line 689, in generic_train     trainer = pl.Trainer.from_argparse_args(   File "/root/miniconda3/lib/python3.8/site-packages/pytorch_lightning/trainer/properties.py", line 212, in from_argparse_args     return from_argparse_args(cls, args, **kwargs)   File "/root/miniconda3/lib/python3.8/site-packages/pytorch_lightning/utilities/argparse.py", line 53, in from_argparse_args     return cls(**trainer_kwargs)   File "/root/miniconda3/lib/python3.8/site-packages/pytorch_lightning/trainer/connectors/env_vars_connector.py", line 40, in insert_env_defaults     return fn(self, **kwargs) TypeError: __init__() got an unexpected keyword argument 'early_stop_callback'
Seems to be a version problem, can you please specify the version of pytorch_lightining you used, or even better create a requirements.txt file?


Update API version from 2021-04-01 to 2022-09-01 for resources module

Related command
No response
Resource Provider
Microsoft.Resources/resources
Description of Feature or Work Requested
We need to update the API version from 2021-04-01 to 2022-09-01 for the resources module. The Swagger changes have been merged in https://github.com/Azure/azure-rest-api-specs/pull/21203.
Minimum API Version Required
2022-09-01
Swagger PR link / SDK link
https://github.com/Azure/azure-rest-api-specs/pull/21203
Request Example
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/examples/PutTagsSubscription.json, https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/examples/PatchTagsSubscription.json, https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/examples/DeleteTagsSubscription.json, https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/examples/PutTagsResource.json, https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/examples/PatchTagsResource.json, https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/examples/DeleteTagsResource.json
Target Date
01-31-2022
Additional context
No response

update API for resources

@meghanagupta95 I'd like to confirm with you whether we only need to upgrade the api-version, or we also need to support some new commands and parameters? I see that in Request Example , there are some examples related to tag, does this need us to update the tag related commands?

@zhoxing-ms the new API version introduces a new HTTP response code of 202 for existing tags PUT/PATCH/DELETE operations (The examples are related to these pre-existing tags commands.) I don't think that should qualify as a new command or parameter, but please correct me if I'm wrong?

the new API version introduces a new HTTP response code of 202 for existing tags PUT/PATCH/DELETE operations (The examples are related to these pre-existing tags commands.

@meghanagupta95 Yes, they do not generate new commands and parameters. So you don't need us to add new commands and parameters, right?

@zhoxing-ms No, it is not needed.

@meghanagupta95 Sorry, I have to tell you that there are risks in completing the bumping version of Microsoft.Resources in this sprint. The main reasons for the risk are as follows:

The bumping version for RP Microsoft.Resources affects too many modules, and 850 files need to be re-recorded. Unfortunately, other modules are also doing the bumping version, resulting in a large number of conflicts in PR.
The bumping version pipeline we used recently is being upgraded, this leads to more time than usual to create PR.
Our Chinese team has just experienced a long Chinese Spring Festival holiday, so some works were interrupted during the holiday.

So I'd like to confirm with you in advance whether this feature request is very urgent? Could you accept that we plan to postpone it's release to the next sprint (03-07)? Before that, we can provide the private package through edge build to let customers with urgent needs use it in advance.
If this feature request is really very urgent and important, I will consider completing it in time through working overtime in the last few days

Hi @zhoxing-ms, to clarify, do you mean the changes will be out by 03-07? That should be fine.

do you mean the changes will be out by 03-07? That should be fine.

@meghanagupta95 Yes, we will release it in 03-07, thanks for your understanding~

@meghanagupta95 since we have some urgent security related feature to deal with in this sprint and this resource version bump needs more time to collaborate with other module owners and test, we have to postpone this to the next sprint (04-04). Sorry for the inconvenience caused.


WebAuthenticator in PlatformIntegrationDemo ends (crashes?) App

I'm trying to use the WebAuthenticator Sample.
I chose navigate to the page and chose either service (Microsoft, Google, ...). I wait a while, the browser will stay open but the App will end (crash?).
If it does not end by just waiting I can change to another App, change back and the App has ended.
I can only sign in if I'm fast enough to do it before the App will end (crash?). I can also sign in while the app has crashed. Although I can't go back to the app the login will be cashed and when I try to login again the browser will return instantly and I'll be signed in.
So the App seems to randomly end while logging in ... What can I do?
EDIT: It seems my Smartphone is the reason. It's a Samsung M23. No issues in the Simulator, no matter how long tha login screen is opened.

Moved to MAUI project: https://github.com/dotnet/maui/issues/20980


Added SQS queue policy for SNS subscriptions.

@mhahn @phobologic Take a look at this when you can, please.


Perserve directory in output with --path

Bug Report
Looking at here https://github.com/AlexDaSoul/nestjs-proto-gen-ts/blob/dc64e4a38d0abec30fd96db5729ba72176f12f53/src/compiller/index.ts#L112
Current behavior
When adding an output path in a cli command that also includes a --target e.g.
--output /proto/services the expectation is the I will get proto/services/hero.ts however I get /proto/services/ + {--path} where {path} is the argument passed into the path.
From the documentation it seems that --path is used to reference rootDir which I think is misleading in cases where protobuf resides in a separate directory.
In proposing a solution, without breaking existing applications, Is there a way to keep the options for a root and also a path or somehow extend target to read directories?

Hi. I don't quite understand the problem. --target is used to list the .proto files to be compiled. And --path lists the paths to those files - there can be several of them too, not just root.

--path is not added to --output, they are not connected at all and this is clearly visible in the code: https://github.com/AlexDaSoul/nestjs-proto-gen-ts/blob/master/src/compiller/index.ts#L109
I could not reproduce your problem. Maybe you could make a small repository with an example?

у меня так же генерирует, прото файл находится в node_modules/grpc_proto/proto/order.proto, запускаю комманду tsproto --path ./node_modules/grpc_proto/proto --output src/proto и получаю результат в  /src/grpc_proto/proto/order.ts


Remove Default Actions from Package

Any Salesforce org comes with default Actions for things like New Case, New Contact, New Note, etc. At some point, these defaults also got included in our package. So, customers see both the packaged version of these Actions, as well as the ones included in their org. They can't edit our Packaged Version. Can we remove/deprecate them? Existing customers can then delete them from their orgs and new customers will only have the de-duped defaults.
Defaults in Orgs:

What Our Customers See:

**lurch: add

Tracking W-037059

Salesforce has determined that this issue will not be explored further.


Crash in StickyHeaderHelper class as if ViewGroup for Sticky Headers is not included in layout

I have implemented sticky headers in multiple places and everything is working well for a large majority. I am unable to reproduce the following bug (got it in from Crashlytics from a SAMSUNG-SM-G900A running 5.1.1). It seems like the crash would be caused by the sticky header view container not being included in the layout (which would make mStickyHolderLayout null), but I am definitely including the requisite code  directly after the RecyclerView in my layout. Seems to be pretty hard to debug since I can't reproduce it - any ideas?
Fatal Exception: java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.ViewPropertyAnimator android.view.ViewGroup.animate()' on a null object reference
       at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper.clearHeaderWithAnimation(StickyHeaderHelper.java:240)
       at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper.updateOrClearHeader(StickyHeaderHelper.java:119)
       at eu.davidea.flexibleadapter.FlexibleAdapter$AdapterDataObserver.updateOrClearHeader(FlexibleAdapter.java:4445)
       at eu.davidea.flexibleadapter.FlexibleAdapter$AdapterDataObserver.onChanged(FlexibleAdapter.java:4452)
       at android.support.v7.widget.RecyclerView$AdapterDataObservable.notifyChanged(RecyclerView.java:10769)
       at android.support.v7.widget.RecyclerView$Adapter.notifyDataSetChanged(RecyclerView.java:6347)
       at eu.davidea.flexibleadapter.FlexibleAdapter.postUpdate(FlexibleAdapter.java:4627)
       at eu.davidea.flexibleadapter.FlexibleAdapter.updateDataSet(FlexibleAdapter.java:497)
       at eu.davidea.flexibleadapter.FlexibleAdapter.updateDataSet(FlexibleAdapter.java:466)
       at com.winering.wine.base.BaseAdapter.updateDataSet(BaseAdapter.java:139)
       at com.winering.wine.channels.channeldetails.ChannelDetailsFragment.swapData(ChannelDetailsFragment.java:155)
       at com.winering.wine.channels.channeldetails.ChannelDetailsPresenter$1.handleMessage(ChannelDetailsPresenter.java:44)
       at android.os.Handler.dispatchMessage(Handler.java:102)
       at android.os.Looper.loop(Looper.java:145)
       at android.app.ActivityThread.main(ActivityThread.java:6938)
       at java.lang.reflect.Method.invoke(Method.java)
       at java.lang.reflect.Method.invoke(Method.java:372)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1404)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1199)

Sorry - I see now another ticket that mentions that setAdapter must be calling enableStickyHeaders()

@nbortolussi, probably there's still something out of our control, I will release a new snapshot now, with a null check being sure of not crashing the execution. However, it was not supposed to be null that layout in that chain of call.

@nbortolussi, do you see this warning log WARNING! ViewGroup for Sticky Headers unspecified! You must include..... when enablingStickyHeaders()?

@davideas no I don't see the warning, but also I have not been able to reproduce the crash. I have since moved all the enableStickyHeaders() calls to after setAdapter() is called. you don't think that will fix the problem?

Interesting - I actually did just get the crash. It was only upon doing 'Instant Run' from Android Studio however. The warning you mentioned did show in the logs first.
``11-17 19:09:22.101 31482-31482/com.winering.wine W/StickyHeaderHelper: WARNING! ViewGroup for Sticky Headers unspecified! You must include @layout/sticky_header_layout or implement FlexibleAdapter.getStickySectionHeadersHolder() method

Yes, calling after Adapter is attached to RV should be enough, however I don't know if the warning is enough, maybe an IllegalStateException should block the developer before deploying the App.

Agreed - thanks for the help. great library by the way, very well done 👍

@nbortolussi thank you!! I'm glad you resolved without any fix.
However, for the new checks I need to think a little, there are some others things linked, so I won't publish a new Snapshot now.


Test NB with multiple MDLs.

This issue is created in xdp-for-windows extension to:
- Test handling NB (Net Buffer) with multiple MDLs.
Reference of NB: https://www.codemachine.com/articles/ndis6_net_buffer_lists.html

Closing as duplicate; tests are always created for supported features.


add PyAny::is_exact_instance and PyAny::is_exact_instance_of

Split off from #2881. I'll rebase after that merges.

bors r+

Thanks for finishing this up @adamreichold 👍


time deltas

https://www.kaggle.com/c/talkingdata-adtracking-fraud-detection/discussion/53361
https://www.kaggle.com/aharless/talkingdata-time-deltas

merged


Added repo badges

Hi Amila, thanks for your PR ! Looks like there are leftovers from another module in the commit though !

Fixed!


[FEATURE] Ability to reorder notes on a single day.

This can be done in various ways, perhaps the best is a drag item in a list to reorder them, but can be added a "time of event" field that is used to order in ascending chronological way.

This makes sense, I like the drag myself but if that is too tricky to get working correctly I can for sure add a time field and order by that.


3Dslicer fail to respond and VRAM issue

It's is really impressive work! However, my 3dslicer fail to respond and can't do anything after "Start Mask Sync" in Samm Base. My 3dslicer version is 5.2.2

Thanks for trying it out. We are continuously improving it. This week has been hectic, but I will release a new version lastest this weekend.
Could you send a more detailed bug report? Or, could you send a demo video showing how you were using it? Please note issue #13 (also found in ReadMe, Known Issue section). This is a pending problem plaguing the extension. I hope a new version over the weekend will help you more.

Thanks for your reply! I run sam_server.py and output "Starting to Wait for Message". But here's nothing happen when I load image in 3dslicer, even made some points and start sync

You would need to compute the embeddings of the image first in the "Initialization" tab, and wait for the embeddings finishing. Let me know if it doesn't work.

I click the button "Compute Embedding", and the console show the info:
Initializing SAM server ...
SAM server initialized ...
Starting To Wait for Messages ...
Computing embedding...
Processing slc0...
slc0 shape: (421, 512, 3)

the last two line is debug info which I added(I just add some print to debug).

Thanks for the debugging info. @anxingle Can you also attach the GPU information of your machine? We recently received a bug report from 4GB Nvidia GeForce RTX 3050Ti, even though on our test machine, the VRAM usage is at around 3GB.

Thanks a millon for your reply.
My GPU is RTX 3060 Laptop GPU, and GPU memory is  6143MB.

98129af38eff0fa422b5b5842134e6dbe770b869
Thanks for your report.  Could you please report back when you get time to test it out?

ok. I update the latest code, then run , it show out of memory:
RuntimeError: CUDA out of memory. Tried to allocate 1024.00 MiB (GPU 0; 6.00 GiB total capacity; 4.60 GiB already allocated; 0 bytes free; 4.67 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Unfortunately, this is something we don't have a solution at the moment, if we  want to keep the segmentation process local. This is essential for the real-time-ness. A future option could be using their web interface (not sure if they provide it or not), which will limit the speed.
VRAM usage is also relevant to data size.

Get it. Thank you very much. I will try to decrease the CT scans.

If I don't have GPU, can this function work?

If I don't have GPU, can this function work?
Unfortunately SAM depends on GPU to work. Theoretically we can make a CPU running model, but I don't know how much worth in it, given in medical image cases there are other tools that are computationally less-intensive (and probably better performance).

Get it. Thank you very much. I will try to decrease the CT scans.

I doubt reducing scans would improve. Single image slice resolution is probably the key.

I bought a 4070, it is installed in my computer, I gonna test it!

Closing the issue.

The repository now supports the vit_b which is significantly smaller.


⚠ Bofrost.at has degraded performance

In ad87eef, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1808 ms

Resolved: Bofrost.at performance has improved in bc4fdb3.


Unsafe ESLint errors

Bug summary
I am getting eslint errors
const Router = createRouter({
    Home: '/',
    About: '/about',
    UserList: '/users',
    UserDetail: '/users/:userId',
})

// Unsafe assignment of an `any` value @typescript-eslint/no-unsafe-assignment
// Unsafe call of an `any` typed  value @typescript-eslint/no-unsafe-call

Library version
latest
Environment info
nextjs page

Steps to reproduce

…
…

Reproducible sample code
...

@ahoyahoy You don't provide a lot of informations here, but I guess you just need to restart the eslint server here (Cmd + Shift + P in VScode):

I'm sorry! Everything works perfectly now.


Add bash completions for forc

[ ] Add forc --list that lists commands, similar to cargo --list
[ ] Add forc locate-project that prints out manifest file location, similar to cargo locate-project
[ ] Port https://github.com/rust-lang/cargo/blob/47b869c107fbe0aa42444f46d59f4ba03ffa15fe/src/etc/cargo.bashcomp.sh over

With regards to --list -- we have forc --help. However, cargo --list lists plugins and extensible commands that have been installed. We don't really have that functionality in forc. Do we want that?

Eventually we probably want plugins, but for now simply listing commands just like cargo --list is probably good enough IMO.

Since the default shell for MacOS is zsh, we should also consider a zsh plugin similar to the rust completion one: https://github.com/ohmyzsh/ohmyzsh/tree/master/plugins/rust

Going to start working on implementing a forc -list

Hey @ControlCplusControlV , i'm working on the shell autocompletions FYI. But perhaps you could have a go at forc locate-project along with forc --list

Closed by #864

Since the default shell for MacOS is zsh, we should also consider a zsh plugin similar to the rust completion one

Yes, please! An Oh My Zsh plugin for Forc would be great.


Rancher 2.5 Best Practices Guide

Our best practices guide needs separate guidance for Rancher 2.5 and for Rancher 2.4 and below.
In the existing BPG, in the short term, we can add a note that says a BPG for 2.5 is coming.
The best practices guide should have a section describing changes in rancher 2.5
The parts of the existing BPG that are really geared toward Rancher v2.0-v2.4 are:

The "tips for scaling and reliability" section assumes you are installing Rancher on an RKE cluster.
The Deployment strategy section assumes that Rancher will manage downstream clusters.
The "Tips for running rancher" section assumes Rancher will manage downstream clusters.

Closing because the BPG was updated.


Error "Exception in thread "main" brut.androlib.AndrolibException:"On windows

Information

Apktool Version (apktool -version) -2.4.0
Operating System (Mac, Linux, Windows) -windows
APK From? (Playstore, ROM, Other) -playstore

Stacktrace/Logcat
I: Using Apktool 2.4.0 on gdzzyb.apk
Exception in thread "main" brut.androlib.AndrolibException: brut.directory.DirectoryException: java.util.zip.ZipException: error in opening zip file
	at brut.androlib.ApkDecoder.hasResources(ApkDecoder.java:307)
	at brut.androlib.ApkDecoder.decode(ApkDecoder.java:103)
	at brut.apktool.Main.cmdDecode(Main.java:167)
	at brut.apktool.Main.main(Main.java:76)
Caused by: brut.directory.DirectoryException: java.util.zip.ZipException: error in opening zip file
	at brut.directory.ZipRODirectory.<init>(ZipRODirectory.java:55)
	at brut.directory.ZipRODirectory.<init>(ZipRODirectory.java:38)
	at brut.directory.ExtFile.getDirectory(ExtFile.java:52)
	at brut.androlib.ApkDecoder.hasResources(ApkDecoder.java:305)
	... 3 more
Caused by: java.util.zip.ZipException: error in opening zip file
	at java.util.zip.ZipFile.open(Native Method)
	at java.util.zip.ZipFile.<init>(ZipFile.java:225)
	at java.util.zip.ZipFile.<init>(ZipFile.java:155)
	at java.util.zip.ZipFile.<init>(ZipFile.java:169)
	at brut.directory.ZipRODirectory.<init>(ZipRODirectory.java:53)
	... 6 more

APK
If this APK can be freely shared, please upload/attach a link to it.
https://www.taptap.com/app/134346

I can't seem to download a binary from that location. Do you have another uploaded area I can take a look at?

I see the user hasn't yet replied, and since I'm having the exact same issue, here's the file I'm trying to decompile, which comes from MIUI.

The file you are trying to download is no longer available.

Sorry I never got to it. Closing.


Make temp_location an attribute of the StandardOptions class

Currently, the --temp_location argument is parsed by the GoogleCloudOptions class. However, --temp_location or a similar argument may be useful for other runners as well.
For example, in the case of DirectRunner, temp_location may be the place where the runner stores its temporary files, if it ever supports out of process shuffles, combines, etc. In the case of InteractiveRunner, temp_location may be a good default location for the runner to store its cache files. In the case of the WriteToFiles transform, setting temp_location to p.options.view_as(GoogleCloudOptions).temp_location makes it appear as if temp_location has to be on GCS, which from my understanding, is not the case unless we are using the DataflowRunner?

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

retest this please

Should we have a gcp_temp_location which defaults to temp_location (as in Java : https://github.com/apache/beam/blob/39dab79b7bd56a47c0f2a02033ab4ca3bd4a67e2/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/options/GcpOptions.java#L245)
IIRC @udim had some taught on this.

retest this please

retest this please

retest this please

@aaltay @udim I updated the PR to include both --temp_location and --gcp_temp_location. However, I am not sure if the division of concerns between those to options is the same as it is in the Java SDK?
I would think that most people would continue to use --temp_location even when using GCP, unless they want to separate their Dataflow temp files from the SDK temp files?

Another issues with this change is that, it is backward incompatible to move an option from its current class. This will break user pipelines that are setting these options in their code.
Unless there is a workaround we may want to delay this until beam 3.0.
@ostrokach are you blocked on this change?

Is it possible to have this option settable/gettable from both places? In
the end, they're just wrappers around a common dictionary.
On Fri, Aug 2, 2019 at 2:36 AM Ahmet Altay<EMAIL_ADDRESS>wrote:

Another issues with this change is that, it is backward incompatible to
move an option from its current class. This will break user pipelines that
are setting these options in their code.
Unless there is a workaround we may want to delay this until beam 3.0.
@ostrokach https://github.com/ostrokach are you blocked on this change?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/8801?email_source=notifications&email_token=AADWVAKGFIYZQOY7T3DVCFLQCN6QPA5CNFSM4HV6QYRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3MH47A#issuecomment-517504636,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADWVAMBL5VU7BA52QV6DGTQCN6QPANCNFSM4HV6QYRA
.

@aaltay I am not blocked on this change; I just figured that it is strange to use one GoogleCloudOptions for things that might not be GCloud-related, like cache location. But there are things around it, like having a separate --cache_location argument for example.
@robertwb The PipelineOptions class collects the arguments defined by every subclass into a single _BeamArgumentParser, which is basically just an argparse.ArgumentParser. We could over-write the add_argument method of _BeamArgumentParser in order to skip duplicate arguments, but there does not seem to be a clean API for doing this without relying on "private" attributes, and then there are edge cases where maybe the type or the destination of the new argument is different from that of the old, and those cases may end up being hard to debug.

We could also just add a property to GoogleCloudOptions that gets/sets the
value in the underlying dict (possibly with a deprecation warning) for
backwards compatibility. It would only be parsed by StandardOptions.
On Fri, Aug 2, 2019 at 6:23 PM Alexey Strokach<EMAIL_ADDRESS>wrote:

@aaltay https://github.com/aaltay I am not blocked on this change; I
just figured that it is strange to use one GoogleCloudOptions for things
that might not be GCloud-related, like cache location. But there are things
around it, like having a separate --cache_location argument for example.
@robertwb https://github.com/robertwb The PipelineOptions class collects
the arguments defined by every subclass
https://github.com/apache/beam/blob/7fe54a0e178acf6957a797ac17edc4ec74e4bd42/sdks/python/apache_beam/options/pipeline_options.py#L179
into a single _BeamArgumentParser, which is basically just an
argparse.ArgumentParser. We could over-write the add_argument
https://github.com/python/cpython/blob/3.7/Lib/argparse.py#L1322 method
of _BeamArgumentParser in order to skip duplicate arguments, but there
does not seem to be a clean API for doing this without relying on "private"
attributes, and then there are edge cases where maybe the type or the
destination of the new argument is different from that of the old, and
those cases may end up being hard to debug.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/8801?email_source=notifications&email_token=AADWVALBZQYCBGUKJ44GNUDQCRNRRA5CNFSM4HV6QYRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3OG5XI#issuecomment-517762781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADWVAPC4HKZLEUOQQQV4EDQCRNRRANCNFSM4HV6QYRA
.

@robertwb suggestion sounds good. We can change it so that it will be
StandardOptions.temp_location
GcpOptions.gcp_temp_location (defaulting to StandardOptions.temp_location)
Also, how does this options, their defaults, and which ones are being required works in Java SDK?

Run Portable_Python PreCommit

@aaltay @robertwb I updated the PR so that:

Move temp_location attribute to StandardOptions.
Calling GoogleCloudOptions.temp_location falls back to StandardOptoins.temp_locatioin and emits a deprecation warning.
GoogleCloudOptions.gcp_temp_location is used for Dataflow related tasks requiring a GCS bucket. If GoogleCloudOptions.gcp_temp_location is not provided, we fall back to StandardOptoins.temp_locatioin.

retest this please

Run Portable_Python PreCommit

@ostrokach Can you rebase this on the latest master?

retest this please


[FR] Custom Serial Number Mapping

Problem
Our company currently uses an alphanumeric Serial Number (S/N) system ex. L4Z, L51 ... L59, L5A etc. This alpha numeric system grants us over 30,000 permutations of S/Ns with very few characters (We exclude the letters O, I and the number 0). This is preferable as we make small PCBs and these serial numbers easily fit on them. I understand that the newest version of Inventree now supports manual entry of these non-integer S/Ns but not the auto-generation of them. To eliminate the need to generate these S/Ns elsewhere we would like to have it all done in Inventree.
Suggested solution
Create a S/N "mapping" that takes the integer Inventree auto-generated number and maps it to a company-specific one. So perhaps in the admin tab, there is a simple CSV file that has integers in one column (0,1,2,3...) and company-specific S/Ns in the other column (AB1, AB2, AB3 ...). Inventree can still use its default integer auto generation on the backend but display the company S/N on the front end. This is useful for organizations looking to move to Inventree but who want to maintain their current S/N format and start point.

Add a "Serial Number Mapping" tab in the Inventree admin panel
Default mapping contains a file with a 1-to-1 mapping with integers in both columns
Create the ability to then upload a custom mapping with company-specific serial numbers

Examples of other systems
I do not have examples of this elsewhere.
Do you want to develop this?
I do not have the know-how to develop this.

@NPaes12 intresting idea!
Sounds plausible but with the current backlog I am not sure if/when a dev of the core team will have time for this.
@SchrodingersGat would this be a go from you and should we tag it help wanted? I have no real use so won't commit at this time.

@NPaes12 thanks for the input. We recently refactored the order reference fields to support some level of customization. A similar approach could be extended to serial number fields.
However it's a pretty complex proposition to provide a "general purpose" generation tool which meets the particular requirements of every company. Perhaps this would be a good use case for a custom plugin, which takes an existing serial number and generates the next one in the sequence.
The current "increment" function looks like this:
https://github.com/inventree/InvenTree/blob/4d5e267753cf123bfad8117c058dba7f6e52ca59/InvenTree/InvenTree/helpers.py#L283
I would suggest that for users looking for "custom" incrementing here, a custom plugin (or some other way to hook in a custom method) would be ideal. This would require extension of the existing plugin architecture.
@matmair would there be a way (besides plugins) to allow custom function hooks? A dangerous idea would be to allow the user to upload raw python code and run that.

@matmair would there be a way (besides plugins) to allow custom function hooks? A dangerous idea would be to allow the user to upload raw python code and run that.

@SchrodingersGat I am going to just ping @dievus to make discovery easier for him. Joke aside thanks for your disclosures again - made us a lot more aware of security in the app.
I think there are 2 obvious solutions:

importing a list of serial numbers and using them is a good low barrier method
a simple PluginMixin should cover more complex

You could solve this with pattern matching, a custom syntax like we have with serial number entry etc. but I think if anyone wants that they can submit a FR. Solutions 1 and 2 should cover more than 80% so I would suggest doing them in that order.

These solutions should cover numbers in general. We need this for at least serials and IPNs, maybe also BO, SO, PO and so on. Suggested name for solution 1: pre-calculated reference.

Definitely had a giggle when I read that.
A couple of suggestions. First, if you want to implement that, maybe make it needs based rather than being available or enabled by default (looks like that's a possibility if it's added as a plugin). Security principles should always ensure that the least amount necessary is available. I would definitely avoid exec(), eval(), and input(), as well as any other function that might allow backend execution beyond the application itself.
If you all want me to poke at it before doing a full release just let me know and I'll see if there's anything I can find.

Appreciate all the feedback. I am new to Github so was quite surprised with how responsive everyone here is!

@NPaes12 we are generally fast to repsonse arounde here ;-)
I reopened this as the solution is not ready for endusers as of now. I would also like to look into solution 1 at some point as it is easier for non-programmers.


Fix broken link on configure your profile

Description & motivation

Someone mentioned this in dbt Slack so I thought thought I'd fix
To-do before merge

Review 😄

@clrcrl

😬 my bad!


10.2 - General Function Failure when running OSM Attribute Selector

When I run the OSM attribute selector tool on a GDB created through the Load OSM File tool (originating .osm file downloaded from overpass xapi), I get warnings for a "General Function Failure" for various tag values (e.g., barrier, bicycle, tiger fields, etc...). This happens for point, line and polygon layers. I'm using the ALL parameter for which additional tags to extract.

I encountered this bug as well. It would be nice to have the option to do background geoprocessing so that we could do other things while it's executing in ArcGIS. Can someone at ESRI be assigned to fix this? Thanks.

Form me, this happens when i try to select an attribute that has a ":" symbol on the name, like "addr:suburb".
Luckly in my case geofabrik.de has a shapefile download and i can do my processing it with the places.shp filtering the features by "type" = 'suburb'.

I added additional checks into the selector tool and this issue should be fixed with the next release.


Building Objective-Git for accessing Github via SSH fails

I am currently trying to build the Objective-Git master branch to be able to connect to Github again via SSH in an iOS Xcode project as there seems to be no updated binary as of now.
When I try to build via Carthage, Cartfile:
github "libgit2/objective-git" "master"

via:
carthage update --platform iOS

I get:
xcodebuild: error: Scheme ObjectiveGit iOS is not currently configured for the archive action.

I also tried building via submodule / project embedded in Xcode as described in https://github.com/libgit2/objective-git#subproject , which seems to work fine, I also see OpenSSH+libssl being built, but at runtime I get an "unsupported URL protocol" error (I guess libgit2 doesn't pick up the ssh library?)
Any hints how to build and embed the updated Objective-Git currently in an iOS Xcode9 project?

Hmm, I got it to build using Carthage by manually changing the scheme and enabling the iOS framework for the archive build, but I still get the "unsupported URL protocol" error at runtime even though the SSH library was built. What's the trick to build Objective-Git with SSH support?

hmm, libssl and libssh2 were built successful, but build-libgit2.log says:
-- Checking for module 'libssh2'
--   Package 'libssl', required by 'libssh2', not found
-- LIBSSH2 not found. Set CMAKE_PREFIX_PATH if it is installed outside of the default search path.

Greetings, anyone know how is this happening?

We've just released 0.14.1, which should fix this issue, so I'm closing this. Feel free to reopen if applicable though.


Action is slow

I'm using this action for publishing JUnit report generated by 'ctest'.
JUnit report with 176 tests, from them 84 skipped; and it takes about 35 minutes, mostly in "Process test results" step.
Is there a way to improve action performance?
Thanks, Vitaly

@vitalyk-multinarity 35 minutes? That seems intense.
Which configuration do you use?

Given that annotateOnly has to iterate over all annotations, and uses the GitHub interface to annotate each and all of them: https://github.com/mikepenz/action-junit-report/blob/main/src/annotator.ts#L31-L45 (with annotateNotice) it wouldn't surprise me to take longer.
As far as I am aware there is no batch API for these available.
Ways to speed up in that case:

annotateNotice = false
use the checks instead of annotate only

I use this action with thousands of test-cases in different projects and it usually completes within seconds.
For these cases we do have annotateOnly = false and use the normal checks + summary.

@mikepenz  - thank you!
I re-read the Readme, but I'm still not sure what is effect of "annotate_notice = true/false".
My configuration is
uses: mikepenz/action-junit-report@v3         if: always()         with:           report_paths: '/tmp/ctest-*-report.xml'            fail_on_failure: true           detailed_summary: true               include_passed: true
which converts to:
Run mikepenz/action-junit-report@v3   with:     report_paths: /tmp/ctest-*-report.xml     fail_on_failure: true     detailed_summary: true     include_passed: true     token: ***     exclude_sources: /build/,/__pycache__/     annotate_only: false     update_check: false     check_name: JUnit Test Report     require_tests: false     check_retries: false     transformers: []     job_summary: true     annotate_notice: true

@vitalyk-multinarity ok given the output you are not using annotate_only: true so that's not the source of the problem.
Given the other issue you had created I understand that you will want to use include_passed: true, but would it possible to compare speed with it disabled?

Without "include_passed: true" it takes just 2 seconds :-)

@vitalyk-multinarity if I remember correctly. you use include_passed mainly to get them also included in the summary. but you actually don't mind the annotations at the specific file locations. correct?
If so, I'll look into it and check optimisations we can apply to skip any file matching and similar.

Could you please test how it performs using f4441d735cac5cebc8f7c6dadf8eef97865b4d32?
See the related PR here: https://github.com/mikepenz/action-junit-report/pull/658

ultimately disabling the resolving of the path on the filesystem unless annotation for passed tests is requested.

(still need to adjust tests for this PR due to the adjusted handling for the annotations)

Many thanks -  with this commit it takes just one second, as far as I added 'annotate_notice: false':
 uses: mikepenz/action-junit-report@f4441d735cac5cebc8f7c6dadf8eef97865b4d32         if: always()         with:           report_paths: '/tmp/ctest-*-report.xml'            fail_on_failure: true           detailed_summary: true               include_passed: true           annotate_notice: false

Thank you for testing this.

This will be in the next release, which will probably be released later this week.
You can use the git commit SHA pinned version until then

Mike, many thanks!
Vitaly


unable to find local dependency with version x.y.z-dev

I've got two crates in a repository. One references the other. When I'm working, I've got both crate's version configuration set to x.y.z-dev, withz being one higher than the currently released version.
This does not work when building the crate:
$ cargo build
error: no matching package named `sm_macro` found
location searched: file:///Users/jean/code/sm/sm_macro
did you mean: sm_macro
required by package `sm v0.6.2-beta (file:///Users/jean/code/sm/sm)`

As soon as I change the crate versions from x.y.z-dev to x.y.z, again with z being a not-yet-released version, it works as expected.
The same happens when using anything other than dev after the dash. From my understanding, this is valid SemVer syntax, and in fact this is also what's used in the cargo-release crate.
Is this a bug, or did I miss some part of the documentation that mentions that this is not how it's supposed to work?

We're seeing exactly the same quirky behaviour. I am almost sure this is a regression.

I believe this is because a version requirement of x.y.(z-1) doesn't match x.y.z-dev, you have to explicitly have a version requirement of x.y.z-dev to get it to match.
Is that perhaps the issue here?


Update zio-prelude to 1.0.0-RC28

About this PR
📦 Updates dev.zio:zio-prelude from 1.0.0-RC21 to 1.0.0-RC28
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-prelude" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-prelude" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #164.


Move composer files to move on the first place

During the upgrade, file update / deletion is made is several ajax calls (400 files per call).
Sometimes, depending on the number of file in the release package, all the composer files are not copied in the same call. This trigger errors about undefined class etc, and this breaks the whole process.
The objective is to move these files at the beginning of the array instead of the end. We then make sure all these files are handled in the same call.
Fixes http://forge.prestashop.com/browse/BOOM-4742

Thank you @Quetzacoalt91


Publish the latest updates to Nuget

Would you be able to publish the latest updates to Nuget?  Right now, Admin.EntityFramework uses the Nuget package which doesn't have fixes for things like Client Secret, deleting clients, etc.
If I download the latest version of IdentityServer3.Admin and add it's Core project as a reference to Entity Framework Admin, everything works as expected.

@brockallen, is it possible to publish a new nuget package?

NuGet now updated


Fix: add ASF category B validation and modify category X entries for validating HugeGraph 1.2.0 release

This PR is a subtask of https://github.com/apache/incubator-hugegraph-doc/pull/307, and it includes the following changes:

Add ASF category B license validation
Modify some incorrect entries for existing category X license validation based on (https://www.apache.org/legal/resolved.html#category-b)

@imbajin @VGalaxies PTAL~

can also consider the discussion here: #307 (comment) in this PR

get it.

@Pengzna Once the script modifications stabilize, synchronize the changes with .github/workflows/validate-release.yml.

@Pengzna Once the script modifications stabilize, synchronize the changes with .github/workflows/validate-release.yml.

Fixed. And have tested it in local repo, looks good.

related https://github.com/apache/incubator-hugegraph-doc/pull/197


[WIP] Migrate tests to Mocha

@mathiasbynens this is a proof of concept of how we can move to Mocha by replacing the custom test runner with a few small helpers.
I have migrated cookies.spec.js and all the tests pass.
I'm keen to plough ahead and start migrating more test files, but wanted to raise this PR to check:

do you have any comments on how the new cookie tests look? Before I make similar changes across all files!
Using the cookies.mocha.spec.js renaming we can run one test at a time rather than migrate in bulk. Would you rather one PR for one test file or I just do them all in bulk? The changes are mechanical mostly so I think one PR might be OK.

I love the way the tests look, thanks for taking this on Jack! I can't wait to drop the custom test runner code. To answer your question, I'd prefer a single large PR in this case since the changes are mostly mechanical.


I want to know how you convert the EE pose into the 6d action of the arm.

My current research work is based on your article. I read your paper, and the model in the paper generates the EE relative posture of the gripper. However, the paper does not mention how to convert the relative EE posture into 6d action. I would be grateful if you could provide reference articles or relevant code.

@noooob-coder FIrst, end effector poses converted from the world global frame to relative frame.  Then matrices with relative poses converted to vector where first 3 values are linear transformations in relative frame and other 6 - 6D orientation transforms . Details of convertation from SO3 to 6D you can find here.


Adding the BAU College of Arts and Design of Barcelona

Institution/School Name
BAU College of Arts and Design of Barcelona
Instiution/School Website
https://www.baued.es
More information:
https://es.wikipedia.org/wiki/BAU_Centro_Universitario_de_Diseño
https://www.baued.es/en/studies/masters-and-postgraduate-degrees/master-in-digital-experience-design
Email Users
Please place an x between the square brackets if yes

[x] Students
[ ] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@rafatga Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Plugin Not Responding on FCPX

Just bought it on App Store and then when testing everything worked just fine on toolbox window. But when exporting media to my library it turns red (effect offline) and then this appears:

2016 MacBook Pro

What should I do?

Seems like a FCPX restart solved the problem, but It still feels like something is not working like expected. I'll close it for now. Thanks. If I need any assistance, I'll open another case.