Deprecate NavGroup and move all functionality into NavItem

I started out this library in a private repo and went with the simple naive approach of having a NavGroup and NavItem, one being able to have sub-items. But as this library has evolved, it's making less sense to keep them apart.
Recent changes (that have not been published in a new version, yet) have brought them even further together, with NavGroup only differing in a few ways.

I think this is probably the correct way forward, but it will need to be handled gracefully. It's probably too much of a change for the upcoming release (it already has a ton of changes), so we can plan it for the next release.
Let's split it across two releases though:

Move all the functionality within NavItem and have a shell NavGroup that logs a warning about the deprecation when used.
Remove from the library after a bit of time has passed.

If time permits, a Django check might be nice. But that may be overkill and a warning on the NavGroup would probably suffice.

This approach would not work if we went down the road of using widgets (#) so one or the other should be chosen


Tag new version to include RN 0.47 breaking change?

Could you tag a new version of this package that includes the PR that was merged a while back? https://github.com/WhatAKitty/react-native-bottom-sheet/pull/8

OK, I will

https://github.com/WhatAKitty/react-native-bottom-sheet/releases/tag/1.0.3

Thank you!


Confusing Request Response

Hi,
I have one annotated java source file and when I try to load the api-docs the "fetching" message displays and yet the browser dev tool tells me that 3 requests were made: 1 to api-doc, 1 to api-docs/ and 1 to api-docs/2016/services. All return 200s but the final one has this for response content: CXF - Service listAvailable SOAP services:Available RESTful services:Endpoint address: http://localhost:8080/api-entitlement/WADL : http://localhost:8080/api-entitlement/?_wadl&type=xmlEndpoint address: http://localhost:8080/api-entitlement/2020WADL : http://localhost:8080/api-entitlement/2020?_wadl&type=xml
Seems to be a generic web service response and not my annotated code.
Is my source not annotated correctly?
Thanks,
John Harter

@jharter - not really sure how you got to this point. How did you configure swagger-core in the first place? I'd highly recommend following the guides in the wiki for integration.
If you provide me with more details on your environment (such as the JAX-RS implementation), I can point you to the specific guide.

Thanks for your reply Ron.  We should probably close this issue until I have a better understanding of the problem and can ask intelligent questions, not waste your time. Thanks, johnh


⚠ [Vue warn]: Component has already been registered in target app.

Thank you for providing an example for Nuxt 3!
The following messages appear when launching the application:
[Vue warn]: Component "Link" has already been registered in target app.
[Vue warn]: Component "Title" has already been registered in target app.

What does this mean and how can it be fixed?

遇到同样的问题。。。


FunctionEmitter incorrectly classifies ArraySelect as side-effect free

Discovered while working on #3824.
https://github.com/scala-js/scala-js/blob/dacf08e14b0a856b89204d63d2b9450fa55a7e65/linker/shared/src/main/scala/org/scalajs/linker/backend/emitter/FunctionEmitter.scala#L1147-L1148
This is incorrect, as it can throw an ArrayIndexOutOfBoundsException.

Indeed, that should be conditioned on semantics.arrayIndexOutOfBounds == Unchecked

Fixed in https://github.com/scala-js/scala-js/commit/f18a9b4c8a35bf3646643b919ed8c720eb3f2528


Using get_queryset with modelForm

I'm using ModelAdmin with a form that subclasses autocompletelight's ModelForm.
I having some issues that I need to solve with prefetch_related and select_related to fetch tables in a single batch.
Looks like the problem is with the choices field. So that can't be done using the get_queryset of admin.ModelAdmin.
How can I make the choices field use the right queryset?

Did you try overriding get_choices ?
I having some issues that I need to solve with prefetch_related and
select_related to fetch tables in a single batch.
Looks like the problem is with the choices field. So that can't be done
using the get_queryset of admin.ModelAdmin.
How can I make the choices field use the right queryset?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/yourlabs/django-autocomplete-light/issues/657

I'm on v2, yeah. Precisely: 2.2.5.
I didn't know there was a get_choices. Is it a django ModelForm method? Because I can't find any docs on it.

My bad, that would be choices_for_request and choices_for_values:
http://django-autocomplete-light.readthedocs.org/en/2.3.1/api.html#autocompletebase

Seems like you might have missed the part about security:
http://django-autocomplete-light.readthedocs.org/en/2.3.1/autocomplete.html#overriding-the-queryset-of-a-model-autocomplete-to-secure-an-autocomplete

So, I can't use this with dal.ModelForm, right?
I have to remove dal.ModelForm(or inherit from django.ModelForm) and work my way through that choices_for_request method?
Why you say I missed the security part? Mind explaining that?

Absolutely not, perhaps it would help if you pasted your code.

These are the models I'm trying to make DAL work with:
class CourseTopic(models.Model):
    course = models.ForeignKey("Course")
    topic = models.ForeignKey(Topic)

class Course(models.Model):
    topics_required = models.ManyToManyField(Topic, related_name="courses", verbose_name=u'Tópicos necessários', through="CourseTopic")

My admin models:
class CourseTopicInline(admin.TabularInline):
    model = CourseTopic
    form = CourseTopicForm

@admin.register(Course)
class CourseAdmin(admin.ModelAdmin):
    inlines = (CourseTopicInline,)

And this is my model form:
class CourseTopicForm(autocomplete_light.ModelForm):
    class Meta:
        model = CourseTopic
        exclude = ()

class CourseForm(autocomplete_light.ModelForm):
    class Meta:
        model = Course
        exclude = ()

So, do you have any Autocomplete class registered ?

class CourseAutoComplete(autocomplete_light.AutocompleteModelBase):
    search_fields = ['^name', ]
    choices = Course.objects.none()
    widget_attrs = {
        'data-widget-maximum-values': 4,
        'class': 'modern-style',
    }

class CourseTopicAutoComplete(autocomplete_light.AutocompleteModelBase):
    search_fields = ['^name', ]
    choices = CourseTopic.objects.none()
    widget_attrs = {
        'data-widget-maximum-values': 4,
        'class': 'modern-style',
    }

autocomplete_light.register(Course, CourseAutoComplete)
autocomplete_light.register(CourseTopic, CourseTopicAutoComplete)

I set them as none for now just to see if the choices ran faster. But they didn't. Meaning this is being ignored.

Could you tell what happens in choices_for_request and choices_for_values please ?
http://django-autocomplete-light.readthedocs.org/en/2.3.1/api.html#autocompletebase
Perhaps you can use a debugger such as pdb.

Sure. What you wanna know?
You want me to return my queryset, there?

I'd like to know what happens exactly, line by line, in your case, in choices_for_request, please, because that's the method that's used for rendering of the choices in the autocomplete.
Also, this reminds me that we don't even know where your problem happens, is it in initial form rendering, is it in the autocomplete ajax, is it during validation ... please provide more information and research !

It happens on rendering when calling this line:
autocomplete_light/widgets.py in render(165)
  values = [autocomplete.choice_value(c) for c in choices]

Did you see the line above that ? Here:
choices = autocomplete.choices_for_values()

Perhaps all you have to do is override choices_for_values to return the proper queryset then ?

Well, just a follow up on this:
Looks like django's widget class queries one id at a time to build the widget(in that case the DAL widget).
This stacktrace tells us DAL can't do anything about since it's on django's end. Here, DAL receives the widget's value and builds itself with it. And this is the id of the model. Which makes DAL make one query for each one. Sad, but true story.
 /Users/patrickbassut/Programming/logus/lib/python2.7/site-packages/django/forms/forms.py(560)as_widget()
-> return force_text(widget.render(name, self.value(), attrs=attrs))
  /Users/patrickbassut/Programming/logus/lib/python2.7/site-packages/django/contrib/admin/widgets.py(271)render()
-> output = [self.widget.render(name, value, *args, **kwargs)]
  /Users/patrickbassut/Programming/logus/lib/python2.7/site-packages/autocomplete_light/widgets.py(164)render()
-> choices = autocomplete.choices_for_values()
> /Users/patrickbassut/Programming/logus/core/forms.py(31)choices_for_values()
-> assert self.choices is not None, 'choices should be a queryset'

Let me know if you have a workaround for this.

So, my problem is that when executing the batch that gets the Course objects, I can't find a way to also get the CourseTopic objects(along with the course and topic, because it's a foreign key through another model) in a single batch.
Any ideias?
I'm on this issue for days now.
If upgrading for v3 will solve my issue, let me know cause I'm willing to make the effort.

I don't think the issue is in DAL now, it's just that it doesn't get the right queryset.
Please take this to stackoverflow.


Snowflake Usage | Queries with "DB"."TABLE" aren't being parsed correctly for Usage & Lineage

Affected module
Ingestion Framework
Describe the bug
Looks like the Query parser for the Snowflake Usage connector isn't correctly parsing the table names when the below format is used "DB_NAME"."TABLE_NAME"
I see a similar error as below:
WARNING: Table does not exist, skipping usage publish TABLE_NAME, DB_NAME
To Reproduce
Not sure the below steps would reproduce this correctly:
-- CREATE TEST SCHEMA
CREATE SCHEMA TEST_SCHEMA;

CREATE TABLE "TEST_SCHEMA"."TEST_TBL1" AS
SELECT
    'TEST' COL
FROM
    DUAL;

CREATE TABLE "TEST_SCHEMA".TEST_TBL2 AS
SELECT
    *
FROM
    "TEST_SCHEMA".TEST_TBL1;

Expected behavior
The query parser  need to be able able to correctly parse and identify the table to which usage & lineage can be attributed
Version:

Python version: Python 3.8
OpenMetadata version: [e.g. 0.8]: 0.9.1

Currently table names are not logged on the snowflake_usage connector. Also, the --debug mode wouldn't log the queries that are being parsed making it more harder to identify if the usage connector is deriving usage & lineage correctly.

I was not able to reproduce the same error. It would be a great help, if you could provide more precise information.

@sidhreddy if you don't mind can you give us a sample query where we might be missing this.

Hi @sidhreddy, could you please share an example with us where this is failing?


fix: fast refresh is enabled if FAST_REFRESH environment variable is not defined

As pointed out here: https://github.com/facebook/create-react-app/issues/9913#issuecomment-716879695
The comment in the code clearly states // which is why it's disabled by default. suggesting react-refresh should be disabled by default.
But if the environment variable FAST_REFRESH is not set at all, the condition process.env.FAST_REFRESH !== 'false' evaluates to true (as undefined !== 'false'), enabling react-refresh.

@martinfrancois I think Fast Refresh is enabled by default per https://github.com/facebook/create-react-app/pull/9350.
Maybe @iansu forgot to update the comment.

@n3tr true! In this case, let's change the comment, thanks!

@petetnt thanks for approving! Will this PR be merged at one point?


Convert Lambda to Method Group whenever it's possible

Replaces
input.Where(item => Test(item));

with
input.Where(Test);

Being worked on by @sqdavid
This is a Hidden dianostic.
Category is Style.
Diagnostid id is CC0020

Closed on #72.


Use Topiary logos

This PR replaces the stock photo and the header text with logos, on the website.
I've used the text logo, rather than the banner logo, in the header, as the shadow under the tree doesn't really work on a dark background. It looks like this in my browser:

You are of course welcome to switch to a header with light background. But it works like this too.

You are of course welcome to switch to a header with light background. But it works like this too.

Dark looked better than the stock Bootstrap light background, imo.


Allow to print response

From<EMAIL_ADDRESS>on January 13, 2011 09:53:39
E.g.
print().get("/something");
print should set "text/plain" parsing and simply print the reponse to the console. print() should return TestSpecification directly.
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=12

From<EMAIL_ADDRESS>on January 16, 2011 12:53:46
Status: Fixed


python3 command is just python sometimes

Summary
On Archlinux and some other OS (maybe Windows as well?), python 3.x is ran using the python command and python 2.x using the python2 command. Here, we're assuming it's always python3: https://github.com/axa-group/Parsr/blob/dc3bec7ae62b0db12a075f31c26ba8951a63c2f7/server/src/modules/TableDetectionModule/TableDetectionModule.ts#L35
Steps To Reproduce
Steps to reproduce the behavior:

Have your path with python pointing on Python 3.x
Remove python3 from your path.

Expected behavior
Table extraction should run.
Actual behavior
A node error: Error: spawnSync python3 ENOENT.
Environment

Reference commit/version: dc3bec7ae62b0db12a075f31c26ba8951a63c2f7
Other platform details: Python 3.x is just python, not python3
OS: Arch Linux, maybe others?

I'm unsure but we may have the same issue on the Docker container.

Commit 387707b should resolve this but the docker container still needs a look.


[TYPO] in README.md

In the first section of the README, suggested projects include "personalised weedings invitations". I believe this should read "personalised wedding invitations".

thanks for spotting it. don't hesitate to click on "edit" directly in the readme next time :)


Attribute 'valueGroup' is not allowed to appear in element 'pattern'.

I'm getting a strange error when attempting to use TextPatternTagger.  I'm hoping it's just something I'm doing wrong, but it seems like a strange one.  My goal is to extract a thumbnail image for the page to display in search results.  These images are identified by itemprop="image" followed by the URL source.  I'd like to save to the field called thumbnail_url.  I have it in my preParseHandlers section.
<tagger class="com.norconex.importer.handler.tagger.impl.TextPatternTagger" >
    <pattern field="thumbnail_url" valueGroup="1"><![CDATA[itemprop="image" src="(.*?)">]]></pattern>
</tagger>

A quick check of the config file shows me some errors about the TextPatternTagger:
derek@solr:/var/norconex-collector-http-2.7.1$ ./collector-http.sh --checkcfg  -c myredacteddomain-config.xml
INFO  [AbstractCrawlerConfig] Reference filter loaded: ExtensionReferenceFilter[onMatch=EXCLUDE,extensions=zip,gif,jpg,jpeg,png,caseSensitive=false]
ERROR (XML Validation) TextPatternTagger: cvc-complex-type.3.2.2: Attribute 'valueGroup' is not allowed to appear in element 'pattern'.
ERROR [XMLConfigurationUtil$LogErrorHandler] (XML Validation) TextPatternTagger: cvc-complex-type.3.2.2: Attribute 'valueGroup' is not allowed to appear in element 'pattern'.
INFO  [AbstractCollectorConfig] Configuration loaded: id=MySite Config HTTP Collector; logsDir=./myredacteddomain-output/logs; progressDir=./myredacteddomain-output/progress
There were 1 XML configuration error(s).

I'm referring to the documentation here: http://www.norconex.com/collectors/importer/latest/apidocs/com/norconex/importer/handler/tagger/impl/TextPatternTagger.html.   What did I miss?

Oh, shucks... it's a version issue.  I'm on version 2.7.1 and I need 2.8 in order to use groups.


Basic authentication not working in TaskModule for Android

Hi,
I created a simple website which is protected using basic authentication. Now when i try to open it in a task module using the fallback URL on an Android device, i do not get a popup for Basic authentication. (I do get a popup when i load it in a task module in the browser or the desktop app)
Kindly look into this issue.
PS: You can try out a simple basic authentication web page from here
https://jigsaw.w3.org/HTTP/Basic/
User/Pass: guest/guest

@register9091 : Task module will not support mobile device yet. for now it will work desktop app and browser.

It's under active development.

@billbliss: I do understand you have not gone GA with this. I was just bringing this to your notice that in case of fallback URL basic auth does not work on the app browser being opened up. If the same link were to be opened in chrome we do get a popup.
I will also verify if SSO login has an issue, and update this issue accordingly.
@arun-msft: I was referring to the fallback URL of task modules. Which does open up the task module on a mobile device.

It's under active development.

i have a behavioral report for this topic.
@billbliss i tested your sample:
https://github.com/OfficeDev/microsoft-teams-sample-task-module-nodejs
The links are being showed in mobile app, but these are not clickable. Buttons are clickable, but they are not working.
Do you have any feedback, when "Task Module" feature will be supported on Teams Mobile App?

We don't have a date to share but it will be soon. I'm sorry I can't be more specific.


When passing yaml file to conftest gatekeeper, that is bigger than 10kB, it returns an error

Error: read yaml document: short buffer
suspected source https://github.com/clover/gatekeeper-conftest/blob/949f3730a1e537fcbad751954e2bdb6d8f87b866/cmd/root.go#L143

it's being solved by a fork https://github.com/nerzhul/gatekeeper-conftest/commit/6c9fd1b0047f7f8030f744ced3ea5531880d570f


coursework cleanup

Drop all discussions in course tables and clean up table of contents.

Are we gonna keep the part about trello?

ah... crap. tell ya what, you wanna merge and I'll fix that?

Sounds good!


fix(client types): handling of the Node interface

I started setting up typescript code generation for my resolvers and I noticed that Prisma.node() was returning an incompatible Node type. So I've checked my Prisma client:
// myapp/src/generated/prisma-client/index.ts
// ...
export interface Node {}

export interface Prisma {
  // ...
  node: (args: { id: ID_Output }) => Node;
}

Shouldn't Node have an id property? What's the reasoning behind the empty interface? It seems to be this way since @timsuchanek introduced TypescriptGenerator.ts.
I noticed there was a NodeNode interface which wasn't used anywhere:
// ...
export interface NodeNode {
  id: ID_Output;
}

After making some changes to prisma-client-lib to fix the Node interface, I got this error from running compilation.test.js:
prisma/cli/packages/prisma-client-lib/dist/codegen/artifacts/generated_ts.ts (507,18):
  Interface 'PostPromise' incorrectly extends interface 'Node'.
  Types of property 'id' are incompatible.
    Type '() => Promise<string>' is not assignable to type 'string'.

It seems ModelPromise interfaces shouldn't be extending Node directly, only Promise<Model>.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Dialog closes on mouseup outside dialog after mousedown inside dialog

Current Behavior
The dialog closes when the mouse is released outside the dialog after clicking inside the dialog.
Expected Behavior
The dialog should only close when the click occurred outside the dialog. For example, if the user selects text with the mouse inside the dialog and the mouse is released outside the dialog, closing the dialog is not the expected behavior.
Steps To Reproduce
https://www.shadcn-svelte.com/docs/components/dialog
In the dialog example, click "Edit profile", select text in one of the input elements, and release the mouse outside the dialog.
Link to Reproduction / Stackblitz (reproduction template)
https://www.shadcn-svelte.com/docs/components/dialog
More Information
No response

This issue should be raised with https://github.com/melt-ui/melt-ui rather than here.


stages: add org.osbuild.chrony stage

I'm not sure about naming of this stage... org.osbuild.timeservers sounds more like the other stages, but org.osbuild.chrony is imho more flexible and maintainable... What do you think?

/packit copr-build

Packit seems asleep. Merging anyway.

/packit build


Allow .LU files to be dropped into Composer and referenced

Is your feature request related to a problem? Please describe.
I work for an enterprise company and we are doing some R & D on Microsoft Bot Kit.  We have many bots that re-use many of the same intents and entities.  I dropped a .LU file into the folder structure for Composer, but it seems to not load it in correctly, nor can I reference it from the Dialog GUI.
It is frustrating to have this many intents and entities, and have to go into every single trigger/dialogue in order to edit them.  As well as having to copy/paste any intents that I would want to re-use.
Describe the solution you'd like
I would like to be able to drop in or load intents/entities from LUIS and reference them in my bot dialogs from triggers. I also want them to be able to be referenced from the workflow GUI inside the triggers. This will allow re-use of the intents/entities as well as a better way to edit them.
Describe alternatives you've considered
I have considered doing this outside of the Bot framework composer, but this is not ideal as a long term solution.
Additional context
Here is what shows up if I drag in a custom .LU file that I have created and dropped into the language-understanding folder.

Notice how all the defined entities end up in only the bottom intent.

Here I try to reference these intents/entities, but am unable to.

Missing intent condition in workflow GUI.

Hello @UAWILSO and thank you for engaging with the Composer project. We'll take this issue as a feature request and add it to the backlog for prioritization. We do not have an ETA at this time for when this feature will be available.


Add me to merchats

I accept Monero as payment.
Programmer and Web Developer
Website: marcosaguayo.com
Thanks

Hi Marcos, I don't see anything on your website mentioning Monero.
+merchant

@mattcode55 Sorry I've restarted the cache. Take a look now please.
Thanks

It'll get added when #452 gets merged :)


Target .NET Standard 1.1

This makes the project usable in .NET Core apps and .NET Standard libs

This would fix #8

@David-Desmaisons Thanks for this great project! We're making use of it in our app and we're migrating everything over to .NET Core and this has become a blocker.   Would appreciate a timely review/publish if at all possible.  Thanks again.

Note, I made the Test project target the recently released .NET Core 2.1, so you will need VS 2017 15.7 to build/run the tests I assume.

Ha, I did not notice all the other PR's for this already.  Anyhow, many of them are out of date, and/or not complete.  This one targets 1.1 which is most compatible with previous target of .NET Framework 4.5.  The test project has been upgraded as well to use all latest packages, and all tests are passing.


Create SentinelAnalyticRulesMassiveCreationScript.ps1

Required items, please complete
Change(s):

New PowerShell script for massively creating Analytic Rules in Microsoft Sentinel starting from Templates installed in the workspace through Content Hub Solutions

Reason for Change(s):

Such a script was missing. Previous scripts with a similar objective were not considering the new deployment model of the Analytic Rule Templates, based on Content Hub Solutions. The core part of this script uses the same  ARM calls used by the Sentinel All-In-One solution.

Version Updated:

N.A.

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

N.A.

@microsoft-github-policy-service agree

@v-atulyadav  @sreedharande @v-prasadboke I'm about to complete a newer version with a much clearer code and with additional capabilities like the massive update of the rules requiring an update. I'll create a new pull request once finished and tested

Thanks for the update @stefanpems, Will be waiting for the new PR.


Implemention of href in tweets

How do i add hyperlink in tweet. For instance my tweet will be b="<a href="' + "http://www.google.com" + '">" + "google" + "". So google will be the link, on clicking it redirects to a new page.

Twitter doesn’t support hypertext, only text. If you want to tweet a link, just include the URL somewhere in the tweet body.


set-output  command is deprecated and will be disabled soon warning

When running this GitHub Action in a workflow, I'm receiving the following warning:
Warning: The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
https://github.com/Azure/appservice-settings/blob/233d3bfd27b273c844ec829676c77c16ab478117/src/main.ts#L61

Action authors who are using the toolkit should update the @actions/core package to v1.10.0 or greater to get the updated saveState and setOutput functions.

Is there an alternative to this action that would be simple to implement in my CI, or will this action be updated before the deprecation is final?

A note of urgency. If this is not resolved, this action will fail on June 1st, in just a few weeks.
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

GH actions have postponed the June 1st expiry date, but this won't be forever, and we do still have annoying deprecation messages.
I have a feeling this project has been abandoned so might need forking.

As stated in the linked github changelog, this error can be resolved by updating @actions/core to version 1.10.0.
I have forked this repo and confirmed that updating this package does not introduce any breaking changes.
Not entirely sure what the process is, as I have never done anything like this before. But I will create a pull request with the change and hopefully it gets picked up by the Azure team.

Is there a date for an updated version of this action?

I have forked the action and published a Node 20 version to the marketplace: Azure App Service Settings for Node20.

Still facing this issue with azure/appservice-settings@v1
Thanks @rajbos published Azure App Service Settings for Node20. It works like a charm.


Java 9 HttpClient has been moved to jdk.incubator.httpclient

Thank you for the tutorials, they are great!  I got here from the Intro to Project Jigsaw post.
HttpClient is no longer available under java.net.http.  Do you plan to update the corresponding examples for how to properly include the incubator module?  I can compile a module and run it if I have requires jdk.incubator.httpclient properly defined.  But I can't wrap my head around how that dependency should be handled for unit tests.
Getting it to work with a test like the following would really help out.
SimpleHttpRequestsTest

Hey James - sorry I missed this one for a while.
Having a look now.
Cheers,
Eugen.

@eugenp no worries.  I'm guessing that unit tests will need their own module-info.java files as well?  But now, with all the module drama from redhat and ibm :/ I'm not sure it is even worth thinking about anything module related.


card is not working with fixed top navbar

Contrary to some other components like e.g. the hero components, the daisyui caird component will, when the page is scrolled, happily scroll over/above a fixed top navbar with appropriate z-index etc. in the same html page.
Interesting enough, I do not see in your css code any z-index below 0 or above 100, so it should theoretically not be an issue, but it is.
Also interesting is that the bug does not happen on your https://daisyui.com/components/card website with a home grown fixed topbar (may be an idea for another tailwinds daisyui component? especially if compatible with all your components).
Thanks
Kai

Can you share an example code (GitHub repo or Tailwind Play page) with me so I can see the issue?

I'm closing this for now. Please provide a reproduction page so I can see this issue.


Update disk icon

Updates the base disk icon to use the latest version, and tweaks positioning and sizing constants of the app icon insert accordingly.
Before:

After:

(of note: the first two commits of this PR belong to #72. I'll remove them once that PR gets closed)

Can you fix the merge conflict?

Done, the merge conflict is fixed.
I also fixed the icon—something weird happened when committing it the first time, and it was partially corrupted.

Thanks :)


Cloth compute demo refers to non-existent texture

That texture is part of the additional asset pack and not included in the repository itself.


Added plugin to allow marking a function as runnable.

This creates a wrapper for running it, deserializing the inputs and
serializing the outputs.

Stack:

#6576 ⬅
#6575
#6555

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


feat: support customized sections

Description
Right now we support 14 sections ('work', 'education', 'awards'), which should be able to cover 90% cases for resume composition:
export type SectionID =
  | 'basics'
  | 'location'
  | 'profiles'
  | 'work'
  | 'education'
  | 'volunteer'
  | 'awards'
  | 'certificates'
  | 'publications'
  | 'skills'
  | 'languages'
  | 'interests'
  | 'references'
  | 'projects'

However, in some rare cases, these pre-defined sections may not enough, in these cases people may need 100% customized sections.
[Optional] Possible Solutions
Supporting 100% customized sections is pretty complicated task, it is in essence a new no-code product to build some PDF. We need to think it carefully.
Acceptance Criteria
TBD
Todo list
TBD

And it will be nice if support sections with other language like  ‘工作经历' as 'work' and '竞赛奖项' as award'.

And it will be nice if support sections with other language like ‘工作经历' as 'work' and '竞赛奖项' as award'.

I am going to support CJK resumes in the near future, whether the section titles will be localized according to the main language of the resume: https://github.com/ppresume/community/issues/4
On the other hand, if you want more customizable, I am planning to get a feature for section alias, where you can rename the section titles to anything you like: https://github.com/ppresume/community/issues/11

And it will be nice if support sections with other language like ‘工作经历' as 'work' and '竞赛奖项' as award'.

I am going to support CJK resumes in the near future, whether the section titles will be localized according to the main language of the resume: #4
On the other hand, if you want more customizable, I am planning to get a feature for section alias, where you can rename the section titles to anything you like: #11

Glad to hear this, looking forward to these new features.


Using testcontainers-hs with Podman instead of Docker

Hey guys,
Just wondering if anyone has had any luck running testcontainers-hs with Podman instead of Docker and if there are any helpers for it? I'm trying to run some tests with Podman but unfortunately I am getting permission errors when calling it from haskell, despite things working when using Golang (testcontainers-go).
The go module for testcontainers also has helpers for specifically targeting (I'm guessing they are setting some things that aren't being set when I run the tests from haskell), and I'm wondering if testcontainers-hs has something similar?
https://golang.testcontainers.org/system_requirements/using_podman/

@sigrdrifa What kind of issues are you getting specifically? Testcontainers generally does not want to add Podman-specific code. If Podman is setup to provide the Docker Engine API through its systemd service, anything not working would be considered an upstream incompatibility issue with Podman and should be raised there (the Podman team has been receptive in helping to improve compatibility once reported).


Evaluate Apple Silicon for CI

Pinged (pung?) a message to Chris S to see whether we can easily get access to some chonky Windows runners, but it really sucks that compiling Android on Apple Silicon doesn't work...

Ready for review.


Unhandled error, publishing to an exchange using an invalid exchange type

Hi there,
I seemed to have been getting panic when writing to a closed (go not rabbit) channel, which completely makes sense.  However, I got in this state due to a typo in my exchange type, x-consistent-hash was spelled x-constistent-hash.  I think I see a couple ways this error state can be handled gracefully, if this change is welcomed, I'd be more than happy to create a pull request for this.  Returning an error for this on declare of the exchange seems to make the most sense to me, but I'm certainly open to discussing.

This https://github.com/streadway/amqp/pull/196, seems related to what happens when this isn't caught.

Specifying an incorrect exchange type will result in an unrecoverable (connection) exception. What kind of improvements do you have in mind?

#196 does look relevant but I don't remember what the decision was (if any). Maybe @gerhard or @streadway do :)

I'm certainly open to suggestions if this is not ideal, but you could validate the exchange type, possibly using a map[string]bool, where the keys are valid exchange types, though I can understand if that's not ideal due to the need to keep it up to date every time valid exchange types are either removed or added.

Exchange types can be provided by plugins. We cannot limit types to the list of built-in or known ones.

figured so, hmm, I'll have to look into what happens when declaring an exchange of a wrong type a little further then.

@cpg1111 it would be much easier to suggest something if you could post your code. The issue is more about connection.close handling than exchanges or exchange types. In fact, I wouldn't be surprised if it's a different manifestation of #196.

Yes it definitely is https://github.com/streadway/amqp/pull/196 that causes the panic, tracing my code shows that.  My issue is more about handling my cause of that beforehand.  I am saying handle the wrong exchange type, not the race condition (in this issue specifically). Here's my code:
for <condition arbitrary> {
    go func(ex, exType string, c *amqp.Conn){
        var err error
        channel := c.Channel()
        err = channel.ExchangeDeclare(ex, exType, true, false, false, false, nil)
        if err != nil {
           log.Error(err)
           return
        }
        err = channel.Publish(ex, exT, "comeFromConfig",  true, false, amqp.Publishing{
            ContentType: "someMSGType",
            ContentEncoding: "utf-8",
            DeliveryMode: 2,
            Body: <some body>,
        })
        if err != nil {
             log.Error(err)
        }
    }("exchange", "x-consistent-hash", conn) // originally mispelled --> "x-constistent-hash"
}

@cpg1111 how can the library know what exchange type is "wrong" if it can be any string that begins with an x-?
Some other clients, e.g. Bunny, have helpers for known exchange types: topic, fanout, direct, etc.
We could introduce additional functions that declare exchanges of that type but I have a feeling @streadway won't be happy about the explosion of functions with more or less identical purposes.
Another thing we can do is to introduce constants for known exchange types. I think this should be an easier sell.

I do like the constants idea personally.  I was thinking of handling a failed attempt after it's sent to rabbitmq in Channel's sendOpen function, but after further inspection, I see you won't get an explicit error from rabbit of a bad exchange type.

Token needed for integrate system and syncronoused

@cpg1111 constants is an easy and no risk solution. @gustamora sorry, can you elaborate?

@cpg1111 we believe registering an error handler on the connection is what you want. @streadway @gerhard and I agree that introducing constants for known exchange types is a good idea.

@michaelklishin yeah I've come to that inclusion, thanks for pointing me there. And yeah I think the constants would would be a great idea.


Speed up validation of polygons with a mix of holes.

Schema validation could be parallelized to speed it up further.


New Language: Sweave

Language name
Sweave
A prose language for literate programming with the R statistical software language. Precursor to Rmarkdown, a supported language. Mixes R code with TeX documentation. Present in documentation of many R language repositories.
URL of example repository
17.9k files
https://github.com/search?q=path%3A*.rnw+NOT+randomstring154769&type=code
https://github.com/rstudio/rstudio
https://github.com/lme4/lme4
URL of syntax highlighting grammar
https://github.com/textmate/sweave.tmbundle
Most popular extensions

.Rnw
.Snw
.rnw
.snw

Detected language
none

Numerous example documents can be found here: https://github.com/yihui/knitr/tree/master/inst/examples


Not Working

Is there anything else that needs to be changes in the config/config.php? It still defaulting to Session even after I specifiy
'store.type'                    => 'redis:Redis',

No that should be it. Add the module, enable the module,copy the config file into the config directory ant set the correct host.
What version of SSP are you using?


fix mismatch length in 44100.json

Issue: #428

That works for you? Because even with your pull request in my case training in 44k doesn't work

I know and I'm sorry I forgot to convert it to the draft.
I'm just helping Blaise for now-

  loss_mel = F.l1_loss(y_mel, y_hat_mel) * hps.train.c_mel
Process Process-1:
Traceback (most recent call last):
  File "C:\Users\X\Downloads\Applio\env\lib\multiprocessing\process.py", line 315, in _bootstrap
    self.run()
  File "C:\Users\X\Downloads\Applio\env\lib\multiprocessing\process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\X\Downloads\Applio\rvc\train\train.py", line 258, in run
    train_and_evaluate(
  File "C:\Users\X\Downloads\Applio\rvc\train\train.py", line 465, in train_and_evaluate
    loss_mel = F.l1_loss(y_mel, y_hat_mel) * hps.train.c_mel
  File "C:\Users\X\Downloads\Applio\env\lib\site-packages\torch\nn\functional.py", line 3297, in l1_loss
    expanded_input, expanded_target = torch.broadcast_tensors(input, target)
  File "C:\Users\X\Downloads\Applio\env\lib\site-packages\torch\functional.py", line 73, in broadcast_tensors
    return _VF.broadcast_tensors(tensors)  # type: ignore[attr-defined]
RuntimeError: The size of tensor a (23) must match the size of tensor b (13) at non-singleton dimension 2


token exchange

is there still no support for this feature?
https://github.com/ThreeMammals/Ocelot/issues/101

typo

I came across this solution which could be a promising option
https://github.com/clawrenceks/ReferenceTokenExchange

Which kind of advice do you need?
This is ancient issue!

If token exchage is implemented at "PreAuthorisationMiddleware" step then it might work.

Sounds like a success story!

I came across this solution which actually has the code to do token exchange
https://github.com/clawrenceks/ReferenceTokenExchange

I guess this solution is outdated because of .NET Core 2.0 version. I consider this repo as useless one now!
Identity Server 4 perfectly supports JWT because it is default auth scheme for IS4.
Also, Ocelot doesn't expose/forward a JWT token if you define authenticated route.
From the 2nd side, you can forward any tokens to downstream for anonymously defined routes.


path='model/{id}' does not show @EndpointsAliasProperty(required=True) in API Manager

Hello,
For example I have code like this
class SomeClass(EndpointsModel):

    def auth_id_set(self, value):
        ....

    @EndpointsAliasProperty(required=True, setter=auth_id_set)
    def auth_id(self):
         ....

@Mymodel.method(request_fields=('id', 'auth_id'), path='models/{id}')
def model_get_id(self, model):
    ...

So In API Manager, I could not see auth_id field and everytime I execute the request it will throw error because the auth_id field is required. Once I remove {id} from the path, it becomes how it is supposed to be.
Is this the intended behaviour?
Thanks

The way you have it structured, id should go in the path and auth_id in the body. Are you referring to the API manager for a deployed app or with the dev appserver?

oh sorry i should have put http_method='GET', that one was in dev app server, havent checked in the deployed app

In a GET request your arguments can only come through the path. This is part of endpoints and isn't specific to this library.
I'd recommend going through
https://cloud.google.com/appengine/docs/python/endpoints/
Also, a question like this may be more appropriate for StackOverflow. Have you used that site?

Ah sorry, I'll do it in StackOverflow then

No need to be sorry, I'm happy to help.


Add ability to have multiple select for prompts / conversations to remove it

Description
Add ability to have multiple select for prompts / conversations to remove it
https://www.figma.com/design/lwL5jneveZczAj5RLcLFH2/DIAL-Chat?node-id=9500-14264&t=oMlxlW2015uh7s0g-1
Use case/motivation
No response
Related issues
No response
Confidential information

[X] I confirm that do not share any confidential information

'Select all' button was implemented on 'Conversation list' and 'Prompt list' panels.
'Select' option in context menu was implemented for chats/prompts/folders.
Reported the following issues: #1689 #1691 #1692 #1694 #1698 #1699 #1704 #1710

Added to the conversations and prompts documentation


Getting created 201 uploading is not working..

I am tring to upload the file using this package. i am getting <IP_ADDRESS>:53810 - "POST /files/ HTTP/1.1" 201 Created upload is not started on this API. i am using tus-client-js in frontent.

okay, let me see it. I will fix it now

@c0d33py please try out the latest:
https://pypi.org/project/tuspyserver/1.1.0/
or you can clone the package and run locally that way you can see how it works the best.
let me know if it fixes the issue or not

you can run the app and go to /static/uppy.html and try it out

Closing this as solved due to inactivity. I did test though..., if you have any issues feel free to reopen


Winner - Budibase hackathon

Hi Chungchun
I'm happy to inform you, you have won the best component award at the budibase hackathon. Please could you email me at :
joe at budibase dot com

Hi @joebudi, thank you very much, it's an honor to receive this award! I have sent you an email, it should be from<EMAIL_ADDRESS>


Z-height for zero material thickness appears wrong

Copied conversation from pull request:

jadatkins commented 38 minutes ago
In your code comments you state that the correct Z value for a material of thickness 0 is 17, but you also say that empirically you found it to be closer to 19. What made you think it was 17? When I perform a "score" using xTool Creative Space and set the material as 3mm basswood plywood, in Wireshark I can see that it is setting a Z height of 17. This makes me think that the value for a material of thickness 0 would be 20. Is it possible you got mixed up somehow?

fritzw commented 28 minutes ago
I thought, that the G-code from the XCS software was cutting at Z=17 for a material thickness of zero, that's where I got this value from. Also, the maximum material thickness I can set in XCS is 16. However, i may have made a mistake somewhere (seems likely, since the focus was that far off for me).

jadatkins commented 23 minutes ago
Here's a capture I made scoring a short line on 3mm plywood. The Z value of 17 is set at the start when all the other defaults are set, and then it doesn't get set again.
https://lightburn.fider.io/posts/2086/xtool-m1-support

I'll do a few tests in XCS with different material thicknesses tomorrow.

So, tomorrow turned into "next month", but anyways... xTool Creative Space outputs G1 Z17 for Thickness=0mm, G1 Z14 for T=3mm, and G1 Z7 for T=10mm.
Still, that does not explain why the focus point of my M1 appears to be about 2 mm lower than what XCS assumes. Maybe my measurements of focus were off. I recently got a good mobile phone microscope lens, so I could re-test focus in more detail.

Or it could be that XTool is just plain wrong.


Add TELEGRAF_DEBUG_ENABLED env var

Just to make telegraf debugging easier

Tested with my app and seen that it's working


Package request: Safing Portmaster

Project description
Portmaster is a free and open-source application firewall that does the heavy lifting for you. Restore privacy and take back control over all your computer's network activity.
Metadata

homepage URL: https://safing.io/
source URL: https://github.com/safing/portmaster
license: AGPL-3.0 license
platforms: unix, linux

#213735

duplicate of #174194


bower.json issue with "main" path

The library fails to automatically integrate with a grunt build process. The "main" path provided in bower.json and .bower.json points to "google-places.js" instead of "src/angularjs-google-places.js".
I can see the right path in the repository, but when I bower install an version 0.1.0 gets installed with the old paths in "main". Did you register the update with bower?
Could you please correct the path?
Thanks

This version of bower.json below fixes the issue:
 {
  "name": "angularjs-google-places",
  "version": "0.1.0",
  "homepage": "https://github.com/arunisrael/angularjs-google-places",
  "authors": [
    "Arun Israel<EMAIL_ADDRESS>  ],
  "description": "An angular.js wrapper around the Google Places API",
  "main": "src/angularjs-google-places.js",
  "keywords": [
    "google places",
    "angular.js"
  ],
  "license": "MIT",
  "ignore": [
    "**/.*",
    "node_modules",
    "bower_components",
    "test",
    "tests"
  ],
    "dependencies": {
    "angular": "~1.x"
  },
  "devDependencies": {
    "angular-mocks": "~1.x"
  }
 }

forked and it's available on bower via angularjs-google-places-service
If you want collaborate on that project, send me a message


fix: icon alignment for horizontal bar

Description of Change

Fix for the improper icon alignment for horizontal bar
Motivation and Context

Fixes #2041
The sidebar horizontal height issue is already fixed as part of #2169
Checklist

[x] My pull request is properly named
[x] The changes respect the code style of the project (npm run prepare-code)
[x] npm test passes
[x] I tested/previewed my changes locally

Screenshots of fix
Normal Icon and normal sidebar

Normal Icon and very slim sidebar

Very small Icon and normal sidebar

@balajiv113 - is this ready for review?

Yes, it is ready for review

Can you please post screenshots of the vertical layout? That would enable me to approve this PR as-is.

Screenshots of vertical layout. Sure, Will join the gitter channel as well.
Normal Icon and normal sidebar

Normal Icon and very slim sidebar

Very small Icon and normal sidebar


dev branch: add a password resource for local email/password login

Kubernetes changes tested locally.

@kans @dghubble @squat ready for review


Fix for stack overflow on Linux.

The DateTime.Now static property was being visited
which may return a unique value every time, allowing
unbounded recursion.
Fixes #447.

Thank you for the pr.


Could Encounter be an appointment?

In https://bits.nictiz.nl/browse/ZIB-1472 the question is asked if zib Encounter could be an appointment as well. The answer seems to be "yes for the prepublication". However, the question for the current publication remains, so a new question is asked: https://bits.nictiz.nl/browse/ZIB-1872
Based on the outcome, we should discuss profiling this zib on Appointment as well.

Related: https://jira.hl7.org/browse/HN-51


chore: add bootstrap address for Validator.wiki

Description
Briefly describe the changes introduced by this pull request.
Related issue(s)
If this Pull Request is related to an issue, mention it here.

Fixes #(issue number)

Please provide one address at the same time.


Filter out webpacked workers before parsing the scripts in the ASBG

Problem:
As explained in https://github.com/NativeScript/android-runtime/issues/778 - webpack will bundle essential code inside the worker chunks, and that will cause the static binding generator's parser to traverse and generate the same classes as many times as there are workers. The logic in the ASBG prevents a class with the same name from being generated more than once to guarantee predictable runtime behavior, and will crash build-time.
Proposed Solution:
Read a list of the worker scripts output by the webpack worker loader, and exclude the script chunks from being javascript-parsed. Should there be extending of classes the runtime binding generator will take care of handling it the first time per app installation.
Depends on a custom implementation of the worker-loader, and existing webpack user configurations importing the custom plugin.
Addresses: #778

run CI

run ci

@Plamen5kov thanks for your input. Please note that besides the list of workers to exclude from parsing, Gradle doesn't know if its building a webpacked, or a regular project, thus adding a warning message on every build which doesn't involve webpacking, I think will confuse users more than it would help others who webpack.


Unexpected TRY-EXCEPT behavior in teardown

Robot Framework version: 5.0
I was migrating from Run Keyword And Ignore Error to TRY-EXCEPT when I encountered the following problem. A developer cannot necessarily know whether their keyword is being run normally or in a teardown but this can dramatically affect how the keyword works.
I can understand the benefit of continuing in teardown to other keywords to make sure that teardown is done as completely as possible but I think that should not occur within a TRY block.
Test code:
*** Test Cases ***
Running TRY Normally
    Keyword With TRY

Running TRY In Teardown
    Log    Test content here.
    [Teardown]    Keyword With TRY

*** Keywords ***
Keyword With TRY
    TRY
        Fail    First failure
        Fail    Second failure
    EXCEPT    First failure
        Log    In EXCEPT
    END

Execution log:
> robot .\try.robot
==============================================================================
Try
==============================================================================
Running TRY Normally                                                  | PASS |
------------------------------------------------------------------------------
Running TRY In Teardown                                               | FAIL |
Teardown failed:
Several failures occurred:

1) First failure

2) Second failure
------------------------------------------------------------------------------
Try                                                                   | FAIL |
2 tests, 1 passed, 1 failed
==============================================================================
Output:  C:\Git\Lifecare-Imaging-Workflows\e2e\output.xml
Log:     C:\Git\Lifecare-Imaging-Workflows\e2e\log.html
Report:  C:\Git\Lifecare-Imaging-Workflows\e2e\report.html

I encountered multiple problems with keywords which are used both normally and in teardown. After thinking this over for a while I'm now hesitant in using TRY-EXCEPT at all and instead relying on older Run Keyword * keywords instead. It seems so dangerous to implement a keyword which works differently based on the context it is being run in.

It's true that execution automatically continuing in teardowns makes using TRY/EXCEPT in them complicated. Need to think is there something we could do to that. Notice that you get the same behavior with Run Keyword And Except Error and others as well.

The underlying problem here is that in teardowns execution continues also after there are failures. This is done to make sure all cleanup activities are run and cannot be changed. Changing the behavior in TRY block could be considered, but I think it is too magic. We'd also need to decide should the mode change affect possible child keywords. A better solution is making to explicitly turn the continue-on-failure mode off when needed. I submitted #4303 about that and hope it could be implemented already in RF 5.1.
The current "unexpected TRY-EXCEPT behavior" is considered to be by design and I close this issue.


fix(swc-575, picker): update background and border color

Description
This PR fixes SWC-575. Updates the background color and border color values for picker in spectrum-two.

How and where has this been tested?
Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.
Validation steps

Regression testing
Validate:

The documentation pages for at least two other components are still loading, including:

[x] The pages render correctly, are accessible, and are responsive.

If components have been modified, VRTs have been run on this branch:

[ ] VRTs have been run and looked at.
[ ] Any VRT changes have been accepted (by reviewer and/or PR author), or there are no changes.

To-do list

[x] I have read the contribution guidelines.
[x] I have updated relevant storybook stories and templates.
[x] I have tested these changes in Windows High Contrast mode.
[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] ✨ This pull request is ready to merge. ✨

@TarunAdobe This was quite out of date so I pulled out the update & merged it in here: https://github.com/adobe/spectrum-css/pull/2786/commits/da9a90adb1b46abc7f3078d2b7cfcf84f7a41026


random generator (random()) doesn't work

VTL version:
master
OS:
win10
What did you do?
import vtl

fn main() {
	println(vtl.random(0,1,[2,2]))
}

What did you expect to see?
a 2*2 matrix with elements 0.0~1.0
What did you see instead?
[[0.949985009492209,             0e+00],
[            0e+00,             0e+00]]

Hey! thanks for this great catch! I have a fix right now, it was due to some issues on the iterator. I'll test it more and then I'll push it to master.

Solved on latest commit 👌🏻


[BrokenLinksH2] Fixed the link path

Fixed the link path

Docs Build status updates of commit 17e3fb2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/administration-tools.md
:bulb:Suggestion
View
Details

biztalk/core/administration-tools.md

Line 37, Column 154: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /rest/api/overview/biztalk/?view=rest-biztalk-2020

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

Docs Build status updates of commit 763caf2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/administration-tools.md
:white_check_mark:Succeeded
View

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


Not working in feature tests

This gives me a validation error in feature tests. But working with artisan serve
  $response = $this->json("POST",'/login', ['username'=> 'testusername', 'password'=> 'any']);
  $response->assertSessionHasNoErrors();
  $response->assertRedirect('/');

We fixed it in our fork. https://github.com/arimacdev/laravel-request-mapper/commit/756e72b55ddc046a7bf921d04f0922f0e2c40248


feat: Complete implementation of IOT partner Registrar APIs for atsign reservation and activation

Follow-on from #785 and #786

Hi Ashish - I've created this as a new issue because I think we can close the 'design and implementation spike' tickets from last sprint

In addition to @athandle's work to wrap up the registrar implementation, we also need to update the registration / activation parts of the Java and Dart Registration / Onboarding CLIs @srieteja we will need your help on that portion

Certainly! Let's schedule a call and discuss what needs to be done so we can have a clear picture of everything moving forward. @gkc and @athandle when do you prefer we schedule this call?

any day 12 noon IST works with me

@gkc atSign generated by 1 API key should be allowed to be activated by another super API key or not?
As of now any super API key can activate atSigns generated by other super API keys

@gkc atSign generated by 1 API key should be allowed to be activated by another super API key or not? As of now, any super API key can activate atSigns generated by other super API keys

(1) I would say the answer is no, but (2) I thought we had agreed activation would only require atSign and activation key, not the API key that registered the atSign. Intent being that an IOT device would only need to be provisioned with its atSign and the activation key for that atSign?

Secondly, do we need to put any rate limiter to these APIs like 1 key can generate these many atsigns in a certain amount of time?

Two separate questions here (1) rate limiting based on whatever capacity we have available at the time (2) quotas within a period. I think the answer to both is yes but they are different things.

@gkc atSign generated by 1 API key should be allowed to be activated by another super API key or not? As of now, any super API key can activate atSigns generated by other super API keys

(1) I would say the answer is no, but (2) I thought we had agreed activation would only require atSign and activation key, not the API key that registered the atSign. Intent being that an IOT device would only need to be provisioned with its atSign and the activation key for that atSign?

Secondly, do we need to put any rate limiter to these APIs like 1 key can generate these many atsigns in a certain amount of time?

Two separate questions here (1) rate limiting based on whatever capacity we have available at the time (2) quotas within a period. I think the answer to both is yes but they are different things.

Thanks @gkc for clarification,  I missed the first part. It's fixed now. The only concern is that we have a restriction of max atSigns an API key can activate. Either we have to remove this restriction or I can do reverse engineering. I can use from which API key this atSign was generated and then will add restriction. Please confirm.
Rate limit based on capacity is already in place. By capacity I mean no. of atSigns an API key is allowed which is set by admin.
Can you confirm what should be the restriction like no. of atSigns reservation per hour and no. of activation per hour. Can you please confirm the numbers.

a restriction of max atSigns an API key can activate
The max number needs to be changeable, by us, for a given API key.

I can use from which API key this atSign was generated and then will add restriction
sounds good except the max number for the restriction for a given API key needs to be changeable by us

Can you confirm what should be the restriction like no. of atSigns reservation per hour and no. of activation per hour. Can you please confirm the numbers.
I can't confirm. We should discuss on the next architecture call. However I believe reservation per hour should be a very high number, since the reservation operation isn't doing very much, and reservations are likely to be 'bursty'. Activations of previously reserved atSigns will likely be spread out over a much longer period of time. All of these numbers will change over time, and over time we can scale up our infrastructure to cope with load

I believe the client-side API work is largely complete, moving to PR43 for PR reviews and merging

Thanks @stulluri - I have approved the PR. @athandle please feel free to release the v3 API to prod at your convenience - thank you!

@gkc moved to prod, can we close this now?

Yep!


"Apply business process flow while creating an entity record" doesn't applicable for cloud version

The legacy process-related attributes (ProcessId, StageId, and TraversedPath) are not available in the cloud version.
How this behavior can be achieved without these fields?

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

ID: d232372e-022f-0c2e-65fa-8a70c8517499
Version Independent ID: e800d280-3d2c-b584-b8c1-4daffb8a4a5f
Content: Work with business process flows using code (Developer Guide for Dynamics 365 Customer Engagement)
Content Source: ce/customerengagement/on-premises/developer/model-business-process-flows.md
Product: d365ce-op
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

Thanks for your comment, @webyhwh. This issue has been assigned to the doc author for review.

any news?


Logo for duct - suggested designs

If a logo for the duct project would be of interest, are any of the designs here meaningful to the maintainers of this project.
https://github.com/practicalli/graphic-design/blob/master/duct-logo-designs.svg
Adjustments to the designs can be made if any of them are of interest.
Logos are created with the Open Source Inkscape software. If any of these designs are interested they would be donated to the project under a creative commons license, along with original SVG assets.
Thank you.

Thank you for the logo suggestions, but I'm afraid these aren't the style of design I'd want for Duct.

Thank you for considering these designs.  If you wish to share the style of designs you are looking for, I can review and see if I can help.
Thanks again


Overview of git commits

Hi,
I really like to have a full overview of my git commits I'm going to make.
Instead of just this:

I like to have an overview of all inline changes in one single page. That would be awesome!

Can you be more elaborate? What do you mean?

@joaomoreno what i mean is something like this (taken from gitlab)
I see the changes for every file right all listed on one view.


Question about "Using Trained Model"

Hi Team,
I used the "Using Trained Model" code from your GitHub repository. When running "gears_model.predict([['VEGFR3']])", I encountered an error stating that "VEGFR3 is not in the perturbation graph. Please select from GEARS.pert_list!" As a result, I created a new gene list and passed it to the gene_set_path parameter. Do I need to retrain the model in this case?

Yes, for genes that were not present in the perturbation graph at the time of training, you would have to have to include them in the perturbation graph and re-train the model. Otherwise, the prediction will probably just be random.


Add Kazakh/kk language

Hi @jacobras, thanks for the library. Here's the localization. I'd appreciate it if you could merge the request.

Looks great, thanks! This will be in 1.10.0. Release date for that one is set at next week, but I'll try to move it forward.


Allow linking against system-provided libjq

Distributions that provide jq typically also offer development packages that can be discovered by using pkg-config. Adding optional pkg-config configuration to j9-sys would be a welcome step towards easier packaging of jnv without having to build a vendored libjq version.

I've drafted the necessary changes to make this happen, I can make a PR if you are interested

@tranzystorekk Wow. Please let me ask.

Is this a mechanism that allows skipping the build of libjq by referring to an already installed libjq?
Is the choice between using it or not controlled by the LIBJQ_NO_VENDOR environment variable?

@tranzystorekk Wow, thanks for your suggestions. Please let me ask.

Is this a mechanism that allows skipping the build of libjq by referring to an already installed libjq?
Is the choice between using it or not controlled by the LIBJQ_NO_VENDOR environment variable?

Is this a mechanism that allows skipping the build of libjq by referring to an already installed libjq?

Yes, as long as pkg-config is installed (so only on unix platforms) and the jq installation contains the libjq.pc configuration file (typically provided on distros by development subpackages like jq-devel).

Is the choice between using it or not controlled by the LIBJQ_NO_VENDOR environment variable?

LIBJQ_NO_VENDOR=1 forces j9-sys to use the system version or fail to build otherwise, this is useful for distro packages that need to ensure they're using the system provided version. With LIBJQ_NO_VENDOR=0 or unset, we try to grab the system version first, and fallback to vendored if not found (or if pkg-config is unavailable).
Typically similar -sys crates also expose a vendored feature that if enabled always select the vendored version even if system is available, but I wanted to make a minimal proposal first.
The pkg-config crate also has its own set of config variables to control its behavior: https://docs.rs/pkg-config/latest/pkg_config/#environment-variables

Closing as jnv has picked the jaq route instead.


DBZ-650 Fix the test breaking the build

Weird, seems that got lost during my merge. Thought I had added it. Thanks!


[backport] Fix argmax/argmin in CUB block reduction for F-order arrays with ndim > 1

Backport of #4096

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 9773c5831f845f9b75e1afba8cd59c2c74844d03, target branch v8) succeeded!


Add navbars to appointments page

Status:
:rocket: Ready
Description :star2:

Basic frontend design for global navbar (sidebar and header)
Import and incorporate Ant Design and Router
Create placeholder pages
Clean up some unused placeholder/setup code

Some code stolen from YMCA, resolves #19

TODOs :star:

Add in functionality for header
Actually make the other pages lol
Integrate with backend

small changes, extend content to the bottom of the page, right now it stops like 7/8 way through
Also top left drop down needs to move to the right a bit.
Overall very cool! thanks Michael 👍


Rework model io utilities

Fixes #102
This PR is another cleanup and changes the model io to an (in my opinion) easier and more consistent API. There are now basically three functions that are located in the io file: save, load, export
The load function will depend on the file ending and how to load the serialized model. I don't know if this is the most clever way. I thought also we could check the correct models with try-except blocks. I am open to changing this.
Now it is also possible to re-export already exported models. For now, these are just dumped again to disk but maybe in the future, we allow users to change the model's metadata during this process.
I remove unhelpful warnings. These warnings were in the past hidden by cryptic error messages (see #102). Now they are clearer. i.e.
metatensor-models eval eval.yaml which means I forgot the model and provided a yaml file instead I now get
$ metatensor-models eval eval.yaml
usage: metatensor-models eval model options [-h] [-o OUTPUT]
metatensor-models eval: error: argument model: invalid load value: 'eval.yaml'

and if I try to use a checkpoint for evaluation I now get
$ metatensor-models eval bpnn-model.ckpt eval.yaml
ERROR: The model must already be exported to be used in `eval`. If you are trying to evaluate a checkpoint, export it first with the `metatensor-models export` command.

Finally, I also added check_atomistic_model to verify that exported models are valid when tried to load from disk.

📚 Documentation preview 📚: https://metatensor-models--144.org.readthedocs.build/en/144/

I've tried to play around with this and the errors are good when the ckpt and pt suffixes are used properly. For silly renaming (e.g., checkpoints named model.pt), the error messages are worse, but not terrible

Thanks for the testing. We can keep it for now like this and if we get a lot of complains I will rethink again!


Upgrade with pytorch 1.9

Hey @zecloud, thanks for collaborating. Why is this upgrade necessary?

Hey @sborquez thanks for your  great work.
When i tried your image in dockerhub the generate_images fails unless i upgrade pytorch. I use a Tesla K80 in Azure Container Instance.
When i rebuild the image, i discovered that a new package was needed by an upgrade of taming-transformer.


Please remove the snow effect. It wastes a lot of resources.

Please remove the snow introduced in https://github.com/open-amt-cloud-toolkit/sample-web-ui/releases/tag/v3.5.0. Here the browser is wasting precious resources showing all the  192 snow balls around.

Hey @rgl, this will be removed in the next release. Until then, please use the previously tagged release to avoid the winter wonderland. Happy new year!


Support drop-in replacement of MIT libraries for managed platforms

This library should be drop-in compatible with MIT Kerberos within a (managed) application.
This requires supporting the following MIT features:

Configuration file parsing and generation for client, server, and potentially KDC (https://github.com/dotnet/Kerberos.NET/issues/126)
Cache file parsing and generation for client (https://github.com/dotnet/Kerberos.NET/issues/160)
GSS-API (https://github.com/dotnet/Kerberos.NET/issues/54)

Some optional work:

Command line tooling for manipulating above? Kinit/klist?
Windows interop with LSASS for loading keys?

PR #176 adds cache file support and would close #160.
#126 has the ability to read the configuration, but at the moment the library doesn't actually use the configuration. Whatever additional work thats added to glue this together should tap into the cache support to make it a complete e2e configurable solution.


Encountered error while generating package metadata

Collecting torch==2.0.1 (from -r requirements.txt (line 4))
Downloading torch-2.0.1-cp311-cp311-manylinux1_x86_64.whl (619.9 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 619.9/619.9 MB 3.0 MB/s eta 0:00:00
Collecting flash-attn==0.2.8 (from -r requirements.txt (line 5))
Using cached flash_attn-0.2.8.tar.gz (1.8 MB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [6 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "/tmp/pip-install-tjhjvhnh/flash-attn_d4c66fc1204a49cf9a44de9ebbba6e8d/setup.py", line 10, in
import torch
ModuleNotFoundError: No module named 'torch'
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details

Hi, sorry for the late response.
There seems to be an issue with the torch installation.
Could you follow the installation guideline specified in https://pytorch.org/  (based on your environment setting)?
Thank you.


Azure OIDC docs point to kfctl 1.3.0 which does not exist

On Authentication using OIDC in Azure, users are prompted to download kfctl 1.3.0 which does not exist. 1.2.0 appears to be the latest version.

@RFMVasconcelos I guess we need to change kfctl version in the doc, as the latest available version is 1.2.0. Do you know the team/people to which I can talk to confirm/rectify the issue?

I believe @brettkoonce or @aronchick might be good people to ask about Azure :)


Go to file: prefix with / to prefer results in root

I'd like to open index.html in root. Using the "go to file" feature for that is currently not friendly because it's a 14th match. In general, the sorting feels totally random.

Feature request: prepend the search with /, e.g. /index.html to indicate which index.html I want.

With https://github.com/microsoft/vscode/commit/72a381699f549b3f1c7c38b2b9a6d9a1de06f603 the sorting is more predictable now, previously items with same score would be sorted by recency but that would always result in a different sort order depending on what file was opened last. We now simply use the same sorting as for normal file searches and prefer shorter matches over longer.
Nevertheless, a new setting "search.quickOpen.history.filterSortOrder": "recency" allows to strictly sort by recency in the history list (https://github.com/microsoft/vscode/issues/35610).

Verification
Have a workspace with many files of the same name in different folders and verify that the one with the shorted path appears to the top when you search by name in the "recently opened" bucket (might require that you open each file first once).


Traffic Light Detection. Detection.

What?
Updated TLClassifier Class.
Added models with weights for detection and classification of TL Lights.
Added JSON model for TL Detection because of Keras bug with model_load with custom objects
[INFO] [1521385727.398925]: [TL_DETECTOR] Loading TLDetector model
/usr/local/lib/python2.7/dist-packages/keras/models.py:291: UserWarning: Error in loading the saved optimizer state. As a result, your model is starting with a freshly initialized optimizer.
  warnings.warn('Error in loading the saved optimizer '

and Adam optimizer. Because Adam Optimizer has been added in the most recent version of Keras, had to update requirements list to remove version for Keras. This will allow to instal the most recent one.
model = load_model('./....h5')  File \"/usr/local/lib/python2.7/dist-packages/keras/models.py\", line 257, in load_model    custom_objects=custom_objects)  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 689, in deserialize    printable_module_name='optimizer')  File \"/usr/local/lib/python2.7/dist-packages/keras/utils/generic_utils.py\", line 141, in deserialize_keras_object    return cls.from_config(config['config'])  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 127, in from_config    return cls(**config)  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 403, in __init__    super(Adam, self).__init__(**kwargs)  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 63, in __init__    'passed to optimizer: ' + str(k))TypeError: Unexpected keyword argument passed to optimizer: amsgrad

Encountered a problem with Pillow
File "/opt/ros/kinetic/lib/python2.7/dist-packages/cv_bridge/core.py", line 248, in cv2_to_imgmsg
   img_msg.height = cvim.shape[0]

had to upgrade it to the version >4.3. Remove requirements version constraint for Pillow.
Updated TLDetector Class to call initialize and load Classifier.
Got another bug from keras, when loading a model and weights Predict function does not recognize one of the layers from the model.
[ERROR] [1521471510.962707]: bad callback: <bound method TLDetector.image_cb of <__main__.TLDetector object at 0x7f5106fab250>>
Traceback (most recent call last):
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/rospy/topics.py", line 750, in _invoke_callback
    cb(msg)
  File "/capstone/ros/src/tl_detector/tl_detector.py", line 170, in image_cb
    light_wp, state = self.process_traffic_lights()
  File "/capstone/ros/src/tl_detector/tl_detector.py", line 334, in process_traffic_lights
    state = self.get_light_state(light)
  File "/capstone/ros/src/tl_detector/tl_detector.py", line 306, in get_light_state
    tl_image = self.detect_traffic_light(cv_image)
  File "/capstone/ros/src/tl_detector/tl_detector.py", line 278, in detect_traffic_light
    image_mask = self.detector_model.predict(resize_image[None, :, :, :], batch_size=1)[0]
  File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 1832, in predict
    self._make_predict_function()
  File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 1029, in _make_predict_function
    **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/keras/backend/tensorflow_backend.py", line 2502, in function
    return Function(inputs, outputs, updates=updates, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/keras/backend/tensorflow_backend.py", line 2445, in __init__
    with tf.control_dependencies(self.outputs):
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 3936, in control_dependencies
    return get_default_graph().control_dependencies(control_inputs)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 3665, in control_dependencies
    c = self.as_graph_element(c)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2708, in as_graph_element
    return self._as_graph_element_locked(obj, allow_tensor, allow_operation)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2787, in _as_graph_element_locked
    raise ValueError("Tensor %s is not an element of this graph." % obj)
ValueError: Tensor Tensor("conv2d_2_1/Sigmoid:0", shape=(?, 96, 128, 1), dtype=float32) is not an element of this graph.

To fix this I have to call self.detector_model._make_predict_function() after weights and model have been initialized and loaded. Found the solution here link.
Before detecting TL I am doing image augmentation to resize it to the proper size for the model.
After the TL was found I pass in data into TL Classifier to detect the light color.
Function extract_image does all the work with predicted TL Light. It takes in an image and a predicted mask and return cropped image of the TL.
Why?
Traffic Light Detection: This can be split into 2 parts:
Part1: Detection: Detect the traffic light and its color from the /image_color. The topic /vehicle/traffic_lights contains the exact location and status of all traffic lights in simulator, so you can test your output.
[x] - works on simulator
[x] - tested locally
[x] - local env version Used Docker container with Mac OS HIgh Sierra 10.13.3

@jae1e You've ashed how does the detection model predicts TL.
In def detect_traffic_light(self, cv_image): we predict traffic light on the image.
image_mask = self.detector_model.predict(resize_image[None, :, :, :], batch_size=1)[0] returns first predicted TL on the image. Later function returns a cropped image with TL color lights only.
This cropped image will be used to predict color in state = self.light_classifier.get_classification(tl_image).


Usage with image data

How can we use this with an image database instead of using it with citation databases?

First you would have to convert your image into a graph. For images you must decide what do your nodes and connections represent. For example, each pixel could be a node, with edges to its adjacent pixels, or the nodes could be groups of pixels or objects detected in the image.
Then you could store your graphs following the same format as Cora https://github.com/PetarV-/GAT/tree/master/data

can you suggest any method to generate the format as Cora https://github.com/PetarV-/GAT/tree/master/data
after generating the graph? Because I tried to convert the graph to this format, but I didn't get success till date.


Unknown parameter in CommonAttributes: "Version", must be one of: Dimensions, MeasureName, MeasureValue, MeasureValueType, Time, TimeUnit

When running the python sample app for the upsert using the version...I am getting the error:
Unknown parameter in CommonAttributes: "Version", must be one of: Dimensions, MeasureName, MeasureValue, MeasureValueType, Time, TimeUnit

Hi Murphy,
It might be that a SDK that you have is an older version. Could you make sure that when you do pip3 list you see a boto3 version at least 1.6.25?
If you see anything like 1.6.24 or older than you can run pip3 install -r requirements.txt --upgrade again to upgrade.
Best regards,
Tiratat

Thanks...I'm using the code in a Lambda with python 3.8 using the built in boto3. Looks like the boto3 version: 1.15.16

Please upgrade to newer Boto3. 😄

Thanks @tiratatp. I was confused at first because 1.15.16 is > 1.6.25...but I'm guessing you meant 1.16.25. After doing some testing, it looks like >= 1.16.0 works. Thanks again.

However, my lambda is still throwing the error even with the boto3 version.
START RequestId: 7cf7c524-ded6-4349-9a29-242a2d69b821 Version: $LATEST boto3 version: 1.16.25 Writing records with upsert Error: Parameter validation failed: Unknown parameter in CommonAttributes: "Version", must be one of: Dimensions, MeasureName, MeasureValue, MeasureValueType, Time, TimeUnit END RequestId: 7cf7c524-ded6-4349-9a29-242a2d69b821 REPORT RequestId: 7cf7c524-ded6-4349-9a29-242a2d69b821	Duration: 4.20 ms	Billed Duration: 5 ms	Memory Size: 128 MB	Max Memory Used: 71 MB	Init Duration: 325.87 ms

I got the same error @murphycrosby
Were you able to resolve this?

@tiratatp I can confrim this is still an issue with 1.16.31

I got the same error @murphycrosby
Were you able to resolve this?

Unfortunately no. I just moved on to Timescaledb.


Save stock to file branch

Hey guys so the file that is being generated has all the correct stock items, but I don't know how to test it.

For testing, you can iterate through each map given a key and do an AssertEquals with the values in the object.


Provide the ability to know if any database upgrades are needed for an upgrade.

🔖 Feature description
Provide the ability to know if any database upgrades are needed for an upgrade.
🎤 Context

To know if a specific upgrade will be not backwards compatible/break my integration env
To possibly have rollback scripts. (that can be an enhancement)

✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Cross linking with #6366 as the issue is very much related.

Gonna close this as a duplicate of #6366


Fix user guide typos

Findcc prefixes are outdated, p/ and c/ has been replaced by pi/ and ci/ respectively
Findcc command purpose is inaccurate, findcc finds cc associated to both person and concert
Addcc command format: addcc pi/INDEX should be replaced by pi/PERSON_INDEX
Example commands are missing a few commands. E.g. help, listp, listc, listcc, editp, editc
Reorder command summary to follow the features list

Help command purpose has a typo explaning -> explaining

Help page image will be updated in #134


Prevent survey popup

Signed-off-by: Nikhil Modem<EMAIL_ADDRESS>Type of change

Bug fix

Description

Removed Segment code
Removed triggerSurvey code and references to it in other files
Modified the dev:apollo command to work with the new esbuild change. This also fixes the lerna run dev:apollo command

@modemn looks like the build is failing due to not updating detect secret. Please run the following command and checkin the files that changed related to detect secret.
lerna run detect-secrets


Binary limits

Description
Organize better several limits...
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

@aschran can you take a look at the protocol config changes?
sorry I merged before you could take a look


Update set-up-client-policies-for-your-organization.md

Added Notes
Replaced SkypeOnlineConnector with MicrosoftTeams

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@tonysmit PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Wrong formatting for type spec and body

First of, thank you Philip for tremendous work and effort you took to understand the formatter and implement custom formatting settings.
I've downloaded latest version of settings and tried to apply formatter on some of utPLSQL sources.
I think utPLSQL can be a good battlefield for formatter as we use quite a lot of PL/SQL and SQL language features.
The only setting I have changed was to keep keywords as lowercase.
When formatting type spec, the methods are not aligned properly.
Some examples below:
create or replace type ut_realtime_reporter force under ut_output_reporter_base (
  /*
  utPLSQL - Version 3
  Copyright 2016 - 2019 utPLSQL Project

  Licensed under the Apache License, Version 2.0 (the "License"):
  you may not use this file except in compliance with the License.
  You may obtain a copy of the License at

      http://www.apache.org/licenses/LICENSE-2.0

  Unless required by applicable law or agreed to in writing, software
  distributed under the License is distributed on an "AS IS" BASIS,
  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  See the License for the specific language governing permissions and
  limitations under the License.
  */

  /**
   * Cached XML header to be used for every XML document
   */
   xml_header             varchar2(4000),

  /**
   * Total number of all tests in the run (incl. disabled tests).
   */
   total_number_of_tests  integer,

  /**
   * Currently executed test number.
   */
   current_test_number    integer,

  /**
   * Current indentation in logical tabs.
   */
   current_indent         integer,

  /**
   * Buffers lines to be printed.
   */
   print_buffer           ut_varchar2_rows,

  /**
   * The realtime reporter.
   * Provides test results in a XML format, for clients such as SQL Developer interested in showing progressing details.
   */ constructor function ut_realtime_reporter (
      self in out nocopy ut_realtime_reporter
   ) return self as result,

  /**
   * Provides meta data of complete run in advance.
   */ overriding member procedure before_calling_run (
      self   in out nocopy ut_realtime_reporter,
      a_run  in ut_run
   ),

  /**
   * Provides meta data of a completed run.
   */ overriding member procedure after_calling_run (
      self   in out nocopy ut_realtime_reporter,
      a_run  in ut_run
   ),

  /**
   * Indicates the start of a test suite execution.
   */ overriding member procedure before_calling_suite (
      self     in out nocopy ut_realtime_reporter,
      a_suite  in ut_logical_suite
   ),

  /**
   * Provides meta data of completed test suite.
   */ overriding member procedure after_calling_suite (
      self     in out nocopy ut_realtime_reporter,
      a_suite  in ut_logical_suite
   ),

  /**
   * Indicates the start of a test.
   */ overriding member procedure before_calling_test (
      self    in out nocopy ut_realtime_reporter,
      a_test  in ut_test
   ),

  /**
   * Provides meta data of a completed test.
   */ overriding member procedure after_calling_test (
      self    in out nocopy ut_realtime_reporter,
      a_test  in ut_test
   ),

  /**
   * Provides the description of this reporter.
   */ overriding member function get_description return varchar2,

  /**
   * Prints the start tag of a XML node with an optional attribute.
   */ member procedure print_start_node (
      self          in out nocopy ut_realtime_reporter,
      a_node_name   in  varchar2,
      a_attr_name   in  varchar2 default null,
      a_attr_value  in  varchar2 default null
   ),

  /**
   * Prints the end tag of a XML node.
   */ member procedure print_end_node (
      self    in out nocopy ut_realtime_reporter,
      a_name  in varchar2
   ),

  /**
   * Prints a child node with content. Special characters are encoded.
   */ member procedure print_node (
      self       in out nocopy ut_realtime_reporter,
      a_name     in  varchar2,
      a_content  in  clob
   ),

  /**
   * Prints a child node with content. Content is passed 1:1 using CDATA.
   */ member procedure print_cdata_node (
      self       in out nocopy ut_realtime_reporter,
      a_name     in  varchar2,
      a_content  in  clob
   ),

  /**
   * Prints a line of the resulting XML document using the current indentation.
   * a_indent_summand_before is added before printing a line.
   * a_indent_summand_after is added after printing a line.
   * All output is produced through this function.
   */ member procedure print_xml_fragment (
      self                     in out nocopy ut_realtime_reporter,
      a_fragment               in  clob,
      a_indent_summand_before  in  integer default 0,
      a_indent_summand_after   in  integer default 0
   ),

  /**
   * Flushes the local print buffer to the output buffer.
   */ member procedure flush_print_buffer (
      self         in out nocopy ut_realtime_reporter,
      a_item_type  in varchar2
   )
) not final
/

In above, the member procedure is not placed on new line.
It is even more visible here:
create or replace type ut_xunit_reporter under ut_junit_reporter (
  /*
  utPLSQL - Version 3
  Copyright 2016 - 2019 utPLSQL Project

  Licensed under the Apache License, Version 2.0 (the "License"):
  you may not use this file except in compliance with the License.
  You may obtain a copy of the License at

      http://www.apache.org/licenses/LICENSE-2.0

  Unless required by applicable law or agreed to in writing, software
  distributed under the License is distributed on an "AS IS" BASIS,
  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  See the License for the specific language governing permissions and
  limitations under the License.
  */
  /**
   * The XUnit reporter.
   * Provides outcomes in a format conforming with JUnit4 as defined in:
   *  https://gist.github.com/kuzuha/232902acab1344d6b578
   */
   constructor function ut_xunit_reporter (
      self in out nocopy ut_xunit_reporter
   ) return self as result, overriding member function get_description return varchar2
) not final
/

When formatting type body the behavior is also odd with some keywords:
create or replace type body ut_xunit_reporter is
  /*
  utPLSQL - Version 3
  Copyright 2016 - 2019 utPLSQL Project

  Licensed under the Apache License, Version 2.0 (the "License"):
  you may not use this file except in compliance with the License.
  You may obtain a copy of the License at

      http://www.apache.org/licenses/LICENSE-2.0

  Unless required by applicable law or agreed to in writing, software
  distributed under the License is distributed on an "AS IS" BASIS,
  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  See the License for the specific language governing permissions and
  limitations under the License.
  */
   constructor function ut_xunit_reporter (
      self in out nocopy ut_xunit_reporter
   ) return self as result is
   begin
      self.init($$plsql_unit);
      return;
   end;
overriding
   member function get_description return varchar2 as
   begin
      return 'Depracated reporter. Please use Junit.
    Provides outcomes in a format conforming with JUnit 4 and above as defined in: https://gist.github.com/kuzuha/232902acab1344d6b578';
   end;
end;
/

All was done running SQLDeveloper 19.4 (JDK 8 included) and Windows 10 Pro x64.

I try to simplify the example.

Apply the settings according this repository
Change advanced settings Keyword case to lower.
Then the following code should be formatted as shown below.

Object Type Specification
create or replace type ut_xunit_reporter under ut_junit_reporter (
   constructor function ut_xunit_reporter (
      self in out nocopy ut_xunit_reporter
   ) return self as result,
   overriding member function get_description return varchar2
) not final
/

Object Type Body
create or replace type body ut_xunit_reporter is
   constructor function ut_xunit_reporter (
      self in out nocopy ut_xunit_reporter
   ) return self as result is
   begin
      self.init($$plsql_unit);
      return;
   end;
   overriding member function get_description return varchar2 as
   begin
      return 'Depracated reporter. Please use Junit.
    Provides outcomes in a format conforming with JUnit 4 and above as defined in: https://gist.github.com/kuzuha/232902acab1344d6b578';
   end;
end;
/

Is this correct?

That is correct.
It seems like member procedures functions are not prefixed with newline.
I'm not sure if parsing tree is distinguishing between:

overriding
final
not final
static

final/not final and member can be combined.
I noticed that IntelliJ is formatting those in an interesting way (more Java-like), so that the static, final, member, overrriding keywords are in separate lines (like java annotations).
Example:
create or replace type ut_realtime_reporter force under ut_output_reporter_base(
  /**
   * Provides meta data of complete run in advance.
   */
  overriding
  member
  procedure before_calling_run(
    self in out nocopy ut_realtime_reporter,
    a_run in ut_run
    )
)
  not final
/

At first I really didn't like it, but after a moment I understood the concept and it is an interesting approach.
Not sure if that would be your (or mine) preference though.
Your suggested fix is definitely valid.

Thanks @jgebal . Yes we have to to ensure that a line break is added in all the cases you've mentioned. This should not be a problem when using the parent nodes, method_specification (spec) and body_adt_field (body).
My default Java formatter does not add line breaks between method modifiers, so I'm not used to this formatting variant. So maybe that's why I do not like it and therefore will not add it.

Thank you, Jacek for reporting this. As a side-effect of this change, the empty lines before the comments are preserved 😃.


Token with wall attached sometimes throw permission error when DM deletes them

How to reproduce:
Make a token, attach a tile and some walls to it.
Save these token settings as a new actor so you can drag new ones on the map.
Drag some copies of it onto the map.
Now as the DM, delete the tokens.
Every now and then the system seems to bug out and the player screen will show:
Error: User [Player name] lacks permission to delete Walls in parent aC6KsUiJuwnkA8ma
This will also result in the attached tile not getting deleted, resulting in a desync where the player will continue seeing the tile, while it is already gone on the DMs screen

Should be fixed in 4.0.2


Open social media links in new tab and add Medium icon

Changes

[x] Opens social media links in a new tab
[x] Adds an svg file of the Medium logo (source)
[x] Updates the assets

Considerations

Given that the changes are generated, I'm not sure how you'd like to see the static/dist files updated. Lmk if you'd like to see any changes.
I used the same :hover color as for LinkedIn.

Hi @hannahhenderson

Given that the changes are generated, I'm not sure how you'd like to see the static/dist files updated.

Not entirely sure what you mean. You updated the CSS and it appears the file is showing up in the dist folder.

Fine by me. I don't think Medium even has a color.

Lastly though I think it would be a good idea to add an accessibility affordance here. Perhaps what Scott Vinkle outlines in this post (skip down to "Example of what I’m talkin’ about"). https://medium.com/@svinkle/why-let-someone-know-when-a-link-opens-a-new-window-8699d20ed3b1
Thanks very much!
Bud

Ah, great. I've been using .proto files a ton lately and don't commit the compiled code. Been awhile since my brain CSS-ed 😋
Great call-out on the accessibility affordance. Officially updated.

This looks great, @hannahhenderson Thank you.


Add note clarifying context hash usage when passing resources through the ResourceSerializer

Description:
I found the documentation to be lacking in explaining how context is passed in when using the ResourceSerializer.
Only after playing around with the ResourceSerializer for some time that I found that the second argument for initializing a resource was the context itself. In the documentation, this second argument was simply nil without explanation that this was actually the context the resource initializes with:
JSONAPI::ResourceSerializer.new(PostResource).serialize_to_hash(PostResource.new(post, nil))
Changes Proposed:

Add a note clarifying how to pass in a context hash when using the ResourceSerializer.

Add a code example that shows how it works, building on top of the first example, which leads into the results in the ResourceSerializer section below it.

Hopefully this will help newcomers who might need context for their resource while using ResourceSerializers. Hope this helps @lgebhardt!

@kevinzwh Thanks!


#grip-class anchor is undefined in the README

Many places in the README.md link to https://github.com/joeyespo/grip#grip-class, but the #grip-class anchor doesn't seem to exist.

Oh wow, nice catch!
Fixed in 0dcad7657d2bb3078201b83eb4d6e0ab257f7a2a.
Thanks, @jacebrowning!


SAA-1857 turn on adjudications source of truth

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.12%. Comparing base (ec6ae47) to head (ad03846).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #973   +/-   ##
=========================================
  Coverage     84.12%   84.12%
  Complexity     3658     3658
=========================================
  Files           379      379
  Lines         12226    12226
  Branches       1067     1067
=========================================
  Hits          10285    10285
  Misses         1458     1458
  Partials        483      483

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.