ci: dependabot should be present to keep dependencies and GHA up to date

once merge, a weekly "cronjob" will be launched to bump version that needs to be updated

@ccoVeille Thank you once again! I have been meaning to look into this :smile:


Error output in v0.5.0-dev is misplaced

Here is the output I got:

This has been fixed. Not 100% sure of the version but in at least v0.7.0.


post MIxS 6 merge -> master -> main notes

Some of this could be reiteration of existing issues. I'll try to link issues as needed.

I checked in .idea, which contains some minimal configuration for PyCharm or other JetBrains IDEs. Some would argue that this is unnecessary or even undesirable
I don't think I've contributed anything to any changelogs
we might need new, improved documentation of the repo contents. For example "what's .idea? what's doc_templates?"
doc_templates contains jinja templates for the LinkML auto-generated documentation. @sujaypatil96 and I need to work on the contents and aesthetics some more. We should look for opportunities to integrate with other MIxS or GSC (static?) documentation, like the GH pages maintained by @only1chunts
I haven't done anything to push the LinkML auto generated documentation to this repo's GH pages
Should we get rid of the downloads fodler? That's where the MIxS Google Sheets were downloaded for processing by mixs_converter.py
Should we remove or somehow bury mixs_converter.py etc.? (Since the LinkML that mixs_converter.py generated is now the source of truth, not the Google Sheets?)

if so, we should probably remove no-longer-relevant portions of the Makefile. I think I have at least commented them out.

Let's make sure the distinction between generated and release is clear (to us and our collaborators!)

I think I have a new PR for Makefile rules that move artifacts from (https://github.com/GenomicsStandardsConsortium/mixs/tree/main/generated) to release. It also attempts to set prepare the repo for poetry build; poetry publish but I haven't gotten that to work yet. See #378

CC @ramonawalls

Actually, before we nuke mixs_converter.py, we should get several people's eyes on the logs. I have been git-ignoring the logs... maybe they should get pushed up to GH?

Representattive entries from XXX:

ERROR:root:Multiple slots with URI

from owl generation, because of combinatorial classes?

ERROR:root:No PK for quantity_value

from sql ddl generation. probably a reasonable design that quantity values like 15000 meters aren't getting their own identifiers
I think there was some discussion of silencing that kind of error in future versions of LinkML

INFO:root:Default_range not specified. Default set to 'string'
INFO:root:Skipping excel as it is in exclusion list
INFO:root:Skipping java as it is in exclusion list
INFO:root:Skipping markdown as it is in exclusion list

confirm we're using the right (and consistent) licenses

@turbomam have these questions been resolved? The only one I know is still relevant is

Let's make sure the distinction between generated and release is clear (to us and our collaborators!)

and I think we are addressing it in #692.


Update requirements.txt and config.yaml.template

The latest version of urillib3 still reports an error. It is suggested that the specific version of urllib3 not be defined.

During my tests, I discovered that the classpath in the config template differs from the actual folder. It can be tested normally after modification.


A typo in wiki - "scripting guide"

See How to make each thread act differently
The description talks about "curNumber", but it is written as "threadId" in the example > beforeThread()

@soongjamm could you make a pull-request?

I think I don't have permission to access wiki, and can't make a PR for wiki page.
or can I ?

Right.. @donggyu04 plz fix it

@soongjamm I fixed. thx for reporting
https://github.com/naver/ngrinder/wiki/How-to-make-each-thread-act-differently


feat(Parser) Add monadic parser combinator

Since one type-level parser implementation is worth a thousand words, here is how to implement a JSON parser:
type JSON = P.OneOf<[NullLit, StrLit, NumLit, Obj, Arr]>;

type NumLit = P.Number;

type NullLit = P.Do<[P.Literal<"null">, P.Ap<Constant<null>, []>]>;

type StrLit = P.Do<
  [
    P.Literal<'"'>,
    P.Let<"key", P.Word>,
    P.Literal<'"'>,
    P.Ap<Identity, ["key"]>
  ]
>;

type Obj = P.Do<
  [
    P.Literal<"{">,
    P.Optional<P.WhiteSpaces>,
    P.Let<"key", StrLit>,
    P.Optional<P.WhiteSpaces>,
    P.Literal<":">,
    P.Optional<P.WhiteSpaces>,
    P.Let<"value", JSON>,
    P.Optional<P.WhiteSpaces>,
    P.Literal<"}">,
    P.Ap<
      Compose<[Objects.FromEntries, Objects.Create<[arg0, arg1]>]>,
      ["key", "value"]
    >
  ]
>;

type CSV = P.Do<
  [
    P.Let<"first", JSON>,
    P.Optional<P.WhiteSpaces>,
    P.Let<
      "rest",
      P.Optional<
        P.Do<[P.Literal<",">, P.Optional<P.WhiteSpaces>, P.Return<CSV>]>,
        []
      >
    >,
    P.Ap<Tuples.Prepend, ["first", "rest"]>
  ]
>;

type Arr = P.Do<
  [
    P.Literal<"[">,
    P.Optional<P.WhiteSpaces>,
    P.Let<"values", CSV>,
    P.Optional<P.WhiteSpaces>,
    P.Literal<"]">,
    P.Ap<Identity, ["values"]>
  ]
>;

And here is how to use it:
type res1 = Call<JSON, '[{ "key": "hello" }]'>;
//    ^? ["", P.Ok<[{ key: "hello" }]>]

type res2 = Call<JSON, '["a", "b"]'>;
//    ^? ["", P.Ok<["a", "b"]>]

type res3 = Call<JSON, '["a", null]'>;
//    ^? ["", P.Ok<["a", null]>]

type res4 = Call<JSON, "null">;
//    ^? ["", P.Ok<null>]

type res5 = Call<
  //  ^? ["", P.Ok<[null]>]
  Arr,
  "[null]"
>;

type res6 = Call<
  //  ^? ["", P.Ok<[1, 2, 3]>]
  CSV,
  "1 ,2  ,  3"
>;

type res7 = Call<
  //  ^? ["", P.Ok<[1, null, 3]>]
  CSV,
  "1 ,null  ,  3"
>;

type res8 = Call<
  //  ^? ["", P.Ok<[1, null, "str"]>]
  CSV,
  '1 ,null  ,  "str"'
>;

type res9 = Call<
  //  ^? ["", P.Ok<[1, null, "str"]>]
  JSON,
  '[1 ,null  ,  "str"]'
>;

type res10 = Call<
  //  ^? ["", P.Ok<[1, { a: { b: "hello" } }, "str"]>]
  JSON,
  '[1, { "a": { "b": "hello" } },  "str"]'
>;

Owesome! 🎉
I love the Do Let Ap utils. makes everything simple to grasp, i think that nothing would prevent my implementation to also have it since it's just adding the ability to Sequence parsing to take a Fn that returns a Parser.
Our implementations are almost the same. The difference is only :

your parser is a naked function that returns a tuple
my parser is a :

augmented function that returns an object
add a name to basic parsers
remember the parameters
the last two above add automatic introspection, while your implementation needs to wrap every combinator with a mapError to create meaningful messages.

I think we debate wether introspection is a nice or not. i personnally think it allows easier error handling out of the box. But maybe adding mapError to customize errors is fine.
And about object vs tuple as the returned type of the parser, i prefer readbility for the user. I think object is also faster. But maybe i'm wrong.
Here are some minor things that are easily fixable :

When using literals to create numbers and word matching should be wrapped in a mapError to create a meaninfull message. Indeed, current implementation yields cryptic union of all possible literals that is not user friendly.
In general, we should wrap all common combinators with a mapError to make better error messages
Words should match common definition : [a-zA-Z_][0-9a-za-Z]*
implement more combinators (see my list)

But instead of diverging more. I think we should try to merge our implementations. I don't like spliting the effort here since both are almost identical.


Chore: Format tools

Motivation
This addresses #1139 which was created to complete the first task in the tool scrubbers guild. The main changes are:

formatted all folders within the /ol folder.
tidy up some code errors when issues occured from formating or were previously in the code.

Test Plan
Run cargo build to verify that the programs could compile after changes were made
Related PRs
N/A

I think very useful/needed PR. I have been also personally formatting wherever I see a problem (e.g. long lines, unnecessary empty lines etc..). Especially the long lines are not good for readability and for the code reviews for people who use GIT tools which vertical views.
Is this a manual effort or done by a tool?

Thanks @simsekgokhan. It was done via the rustfmt cmd.


Bump mysql connector to 6.x

Bump mysql java connector. All tests green. Fixes #18

Thanks for the pull request. I would like to avoid implicitly upgrading all dependents on this package to 6.x especially since it's a development release. I would be OK publishing a separate artifact that works with the 6.x release, would that work for you? Let me know if you'd like to do it or want me to take care of it.

Yes, I actually was not aware of the fact that 6.x is a development release until we had an internal discussion at my company after doing dependency bumping. We actually decided to revert back to 5.1.39 which is the latest stable release.
As it is a pretty small job, I anyway think it would be cool to have this available for 6 if you can take care of it. Speaking against it is the fact that the mysql team can make incompatible changes that don't follow semantic versioning, but again, it would be cool to test it. I know it is working, as I got it working by just copying the code from this repo in its entirety and creating the SocketFactory that is in this pull request.

New artifact for Connector/J 6.x is available:
http://search.maven.org/#artifactdetails|com.google.cloud.sql|mysql-socket-factory-connector-j-6|1.0.2|jar


Different fonts on dark/light theme

I see that when I pass the setThemeDark(true); , then the Timer Text has a different font than the Timer text in the light theme..

The issue happens on all the versions of Emulators: API 21 to API 27
Screenshot recorded on: API 28 (Real Device)

I can't find a reason why this is happening. Nothing in that view is conditional on isThemeDark (it is effectively the same in either mode).
Based on the screenshots it looks you have been tinkering with the default fonts, maybe something in your code is not properly applying that font override in the dark theme path. The hour and minute views are framework TextViews, rather than AppCompatTextViews. I intend to rectify that shortly, but it might help in your search.

I ended up tweaking the Time Label TextView's style..
Also, thanks for the great library.. :D


use default style when type is none

fixes #9143
There is a problem. This fixed the default style applied but does not the fact that "none" style is applied when you remove all the analysis in a layer
cc @viddo

Probably solved #9116 too

:checkered_flag:


Build Crashlytics in Swift Package Manager

Primarily done by converting to repo-relative headers.
Also changed GDT and GDTCCT to be one SPM target since I couldn't find a way to keep them separate as a dependency in SPM.
Open question is SPM may not have a good way to set the linker option - -sectcreate __TEXT __info_plist
Got a successful upload from the Crashlytics quickstart.

I'll redo this in more incremental PRs


syntax

here's an idea for the syntax of the new Kronometer
Kronometer
{
	useHomeDay = False
	useHomeYear = False
	useLeapYears = False
	resetMonthsAtYear = 0    // Forces the calendar to start from Month 1 after n years
                                 // This will allow to not worry about fitting months perfectly into years
                                 // (0 = don't reset months)

	CustomTime
	{
		Second
		{
			singular = Second
			plural = Seconds
			symbol = s
			value = 1
		}
		Minute
		{
			singular = Minute
			plural = Minutes
			symbol = m
			value = 60
		}
		Hour
		{
			singular = Hour
			plural = Hours
			symbol = h
			value = 3600
		}
		Day
		{
			singular = Day
			plural = Days
			symbol = d
			value = // default  ==>  3600 * (GameSettings.KERBIN_TIME ? 6 : 24)
		}
		Year
		{
			singular = Year
			plural = Years
			symbol = y
			value = // default  ==>  3600 * (GameSettings.KERBIN_TIME ? 6 * 426 : 24 * 365)
		}
	}

	Months
	{
		Month
		{
			name = string
			symbol = string
			days = int
		}
	}

	DisplayDate
	{
		// Syntax (Year):
		// {Y0} = Year.symbol
		// {Y1} = Year.singular
		// {Y2} = Year.plural
		// {Y}  = Year.current

		// Syntax (Month):
		// {Mo0} = Month.symbol
		// {Mo1} = Month.name
		// {MoN} = Month.number
		// {MoC} = Month.cardinal (1st, 2nd, 3rd, ...)

		// Syntax (Day):
		// {D0} = Day.symbol
		// {D1} = Day.singular
		// {D2} = Day.plural
		// {D}  = Day.current

		PrintData
		{
			offsetYear = 1
			offsetDay = 1
			display = {Y1} {Y}, {D1} {D}
		}
		PrintDateNew
		{
			offsetYear = 1
			offsetDay = 1
			display = {Y1} {Y}, {D1} {D}
		}
		PrintDateCompact
		{
			offsetYear = 1
			offsetDay = 1
			display = {Y0}{Y},{D0}{D:00}
		}
	}
}

Since this mod will be usable without Kopernicus, but it still requires the Kopernicus.Parser
would it be possible to bundle the Kopernicus.Parser dll with it?
could this create issues if it was loaded alongside Kopernicus?

done, look at the readme!


load can lead to program exiting

Description: Answering "yes" to a prompt that comes up while attempting the load command when one's buffer has been modified can lead to the program exiting.
Expectation: The requested file be loaded and displayed and the editor remain running :)
Steps to reproduce follow.

At shell:
$ cd joule
$ janet src/joule.janet project.janet

In editor:

Make some change -- e.g. type space followed by backspace
Ctrl-L to load another file
Type a path, say, src/joule.janet
Answer "yes" to Are you sure? Unsaved changes will be lost. (yes/No/save)

Observe being back at shell prompt, then:
$ echo $?
1

Scroll back up to see:
$ janet src/joule.janet project.janet
error: could not find method :+ for nil
  in inc [boot.janet] on line 148, column 32
  in update [boot.janet] on line 1520, column 15
  in edup [src/joule.janet] (tailcall) on line 77, column 11
  in editor-scroll [src/joule.janet] on line 202, column 11
  in editor-refresh-screen [src/joule.janet] on line 451, column 34
  in <anonymous> [src/joule.janet] on line 1148, column 17
  in <anonymous> [src/joule.janet] on line 1151, column 16
  in main [src/joule.janet] (tailcall) on line 1145, column 3
  in run-main [boot.janet] on line 3783, column 16
  in cli-main [boot.janet] on line 3928, column 17

The environment here was:

Linux box
joule-editor 60d258c
janet 1.26.0-dev-a8a78d45

Replicated! I'll probably address this as part of v0.0.5.
Thanks for reporting!


Add Cyber Armor to Security & Compliance

fixes #1711
Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape ready!
Built with commit fd13d1efcc9d185edd55d4361c16bfd26ab6135e
https://deploy-preview-1712--landscape.netlify.app

Looks good to me.  Requested by Cyber Armor in #1711


Missing value handling

Speaking to @cboettig today, he mentioned that a lot of datasets he's dealing with have missing values.
What is our approach to handling these?  Using numpy as underlying storage, we'll run into problems with any data-type other than floats.  Alternatives include: using Pandas DataFrames as the backend, or using NumPy MaskedArrays.

Certainly a tricky issue -- pandas does appear to have pretty rich support for this, but with that flexibility also comes an increased complexity / more gotchas where intuition from base pandas and numpy is harder to apply to pandas objects (which helps me further appreciate why the foundations team chose to create datascience instead of teaching pandas).  Perhaps there are clever ways around this, but maybe this is just out of scope for datascience and connectors can just focus on the kinds of things that can be done with the tools of the foundation course, or tackle the burden of introducing a new tool to address issues not covered there (e.g. introducing a bit of pandas, maybe more as a preview than a skill to master, just to handle missing data).  At least we already have the coercion methods between Tables and DataFrames.

my 2 cents is that Pandas is super powerful for these kinds of situations, but as you say it's a PITA to really teach. I'd try to stray away from having lots of NaNs in data during the class, and (maybe at the end?) spend a little bit of time showing what "real" data looks like and mentioning some of the tools we have for dealing with it (e.g., pandas, string parsing, etc). To me that stuff is more like "advanced data munging" and is maybe a step beyond an intro course.

and FWIW it's also probably something that most students won't appreciate until they've run into some problems like these on their own.
I remember being totally confused as to why it was such a big deal that dataframes can easily contain different types of data, along with missing values, etc. After doing enough data analysis it made me appreciate it later on, but that took some time.

Carl,
My 2c on the missing data issue:  The course design emphasized that every
idea should have an inferential side, a computational
side and a real world side.  Missing data certainly has the latter.
Computationally indeed one could get into the representational
issues you're mentioning.  Inferentially, the problem is that of imputing
values for the missing variables, not merely representing
them, and doing that right requires something like the multiple regression
perspective that we mostly punted on in this course,
at least in its first instantiation.  I do think that missing values (and
multiple regression) should be a major focus of a followup
course.
Mike
On Fri, Dec 18, 2015 at 2:59 PM, Carl Boettiger<EMAIL_ADDRESS>wrote:

Certainly a tricky issue -- pandas does appear to have pretty rich support
for this, but with that flexibility also comes an increased complexity /
more gotchas where intuition from base pandas and numpy is harder to apply
to pandas objects (which helps me further appreciate why the foundations
team chose to create datascience instead of teaching pandas). Perhaps
there are clever ways around this, but maybe this is just out of scope for
datascience and connectors can just focus on the kinds of things that can
be done with the tools of the foundation course, or tackle the burden of
introducing a new tool to address issues not covered there (e.g.
introducing a bit of pandas, maybe more as a preview than a skill to
master, just to handle missing data). At least we already have the coercion
methods between Tables and DataFrames.
—
Reply to this email directly or view it on GitHub
https://github.com/dsten/datascience/issues/162#issuecomment-165916146.

John and I went over this issue quite a bit last summer.  There is no
question that handling missing values is a strength of pandas.  And it is
extremely subtle. Addressing that while these students are in  their very
very first exposure is too complicated and distracting.  Recognizing where
it arises and dealing with it as part of the cleaning and or  curating is
advised.  You will find that you need to do some of that regardless to
bring it down to the freshman level.  A follow-on course, probably at the
upper division can reasonably tackle it.  I did a huge number of example
analyses with all sorts of messy public data - in dsten/demos to verify
that this was reasonable.  Every area has their "professional strength"
aspects - whether it be arcgis, or pandas, or transys or any of a number of
things.  But you cannot have everything first.  You have to start
somewhere, build a foundation, and then build structure upon that.
I'd be happy to look with you at the data set and analysis where this is
arising.  Lunch that there is a natural work around that makes it better
suited to freshman.
On Dec 17, 2015 1:51 PM, "Stefan van der Walt"<EMAIL_ADDRESS>wrote:

Speaking to @cboettig https://github.com/cboettig today, he mentioned
that a lot of datasets he's dealing with have missing values.
What is our approach to handling these? Using numpy as underlying storage,
we'll run into problems with any data-type other than floats. Alternatives
include: using Pandas DataFrames as the backend, or using NumPy
MaskedArrays.
—
Reply to this email directly or view it on GitHub
https://github.com/dsten/datascience/issues/162.

Thanks @mijordan3 @deculler @papajohn , your perspective and experience in this is really helpful, (and particularly for someone like me with limited experience teaching at this level).  I can relate somewhat as a new user to python, learning both Tables and pandas, that Tables avoids quite a few barriers and issues that pandas creates by how it handles indexing, etc.
David, I'd love to look at any of your demos that show a bit of handling of missing data.  I don't think dsten/demos exists (??), but I have found the examples you have in deculler/TableDemos very helpful.  In particular, the Mapping_Wrangling_... notebook seems pretty relevant to this discussion, though my quick skim didn't show where any of the columns that contain NaNs are actually used.  Nevertheless it was interesting & surprising to realize that columns in Tables could have mixed type, including NaN, float, and str in the same column! (such as the "Units" column of the Parcels.csv data shown there).  It's columns like that which I have found tricky to use apply functions with that need to handle the missing data; if only at the most trivial level for this course (e.g. just omitting missing data).
For instance, here is one example that @stefanv originally helped me with that required a bit of explicit NaN handling: https://github.com/dsten/ecology-connector/blob/master/fish/fish.ipynb , where we just need to take care that float < missing returns missing instead of False.  Notably the strategy we take there fails on columns with mixed type since we can't call np.isnan on data that is sometimes a float (np.nan) and sometimes a string.  (Sometimes the lack of things like pd.nafill() or pd.nadrop() make the steps required look more complicated).
Anyway, thanks again for your thoughts.  I will be doing my best to avoid pulling freshmen into needless missing-value complexity, but just trying to see myself clear to the best way of doing so.

A pragmatic first step may be to have a few different ways to handle missing values when loading a Table.  This should fill the gap described in Carl's second last paragraph?

I think a reasonable solution for missing value handling is to load data as a pandas dataframe, process the missing values there, then convert to a Table.

@papajohn Thanks, that's good to hear! I've recently been doing just that in my notes so far and it works rather nicely.  Sometimes the change back to Table feels pedantic, e.g. when I'm just going to call df.plot() anyway since both pandas and Table have plot methods.

I wonder if it would make sense, in the long run, to turn Table into a view
of a Pandas dataframe. It has the advantage that advanced features could be
available with an easy table.df.api_call.

Yep — nice idea. I'm trying out such an implementation now. I should have
it working soon.
On Wed, Jan 13, 2016 at 3:01 PM, Stefan van der Walt <
<EMAIL_ADDRESS>wrote:

I wonder if it would make sense, in the long run, to turn Table into a view
of a Pandas dataframe. It has the advantage that advanced features could be
available with an easy table.df.api_call.
—
Reply to this email directly or view it on GitHub
https://github.com/dsten/datascience/issues/162#issuecomment-171465325.


feat/pinecone_namespace

Can you please add a parameter to your pinecone writer to allow a pinecone namespace? I tested it locally and the version of the pinecone library you are using already supports this parameter so it would be a very simple fix. You basically just need to add it as a parameter to the method upsert_batch inside the class PineconeDestinationConnector in the file connector/pinecone.py (and obviously inside the Configs and whatever other methods need to be updated for this parameter to get passed down to this method).
Just as a side note, that same upsert_batch method is using an outdated exception from the pinecone.core.client.exceptions package. The exception you are using is called ApiException however this no longer exists and was replaced by PineconeApiException.

Hello, Can someone please address this, we saw the commit in unstrctured and it was removed, we need namespace for multi-tenancy. This should be a quick fix

Would be a great and seemingly easy to implement feature

Can you add a reference to the namespace support you're referring to in the pinecone SDK? Or the change that was originally made in the unstructured repo?

I've opened this PR to support passing in an optional namespace: https://github.com/Unstructured-IO/unstructured-ingest/pull/159

this is an important feature, please address this ASAP


FIX: Correction the call_query_collection.filter after changing the API

@jamalex can we get this merged?

This should be addressed in https://github.com/jamalex/notion-py/pull/95 (which has now been merged to master, and will soon be released as 0.0.26). Thanks!


404 Not Found

I'm did the following step from https://github.com/mozilla/pdf.js/wiki/Setup-pdf.js-in-a-website
When i click on the link to view pdf is display 404 not found i have no idea what wrong with it.

A web server does not point for the PDF.js root or not properly handles PDF.js files as static resource. Not information provided about which application server is used and how it's configured. Closing as incomplete.
Provide link to the repo with complete code and steps to reproduce the issue on different computer to reopen the issue.


GitHub Workflows security hardening

This PR adds explicit permissions section to workflows. This is a security best practice because by default workflows run with extended set of permissions (except from on: pull_request from external forks). By specifying any permission explicitly all others are set to none. By using the principle of least privilege the damage a compromised workflow can do (because of an injection or compromised third party tool or action) is restricted.
It is recommended to have most strict permissions on the top level and grant write permissions on job level case by case.

I think this makes sense, but I am wary of merging this because I don't know that it'll work when a release is made and I don't have the time/energy to debug this and fix it for the next release.
Are you able to simulate a release in your local fork to test out of this permissions are all that's required?

Take a look at https://github.com/sashashura/wasm-bindgen/actions/runs/3651731741

Ok thanks!


Missing declaration for use of non-medcom defined extensions

Given that a care communication message is sent in FHIR format and that it downstream is converted using the VANS services, what can then be expected for non-medcom declared FHIR extensions in the converted format?

Only the elements in a CareCommunication marked with MustSupport should be expected to be converted, due to limitations in the (X)DIS91. So non-MedCom FHIR extensions will not be converted.
We can add that to the documentation for the conversion service.

But the XDIS91 allows an xs:any(element) IIRC. How does that fit together?

Can you please point it out? https://svn.medcom.dk/svn/releases/Standarder/Den gode korrespondance/ I haven't seen it in the existing XDIS91 standard and I'm not sure vendors supports the element.

https://svn.medcom.dk/svn/drafts/Konverter/trunk/Konverter/xml/Schemas/2006/07/01/EMessage.xsd - line 19.

If I read the schema correct - they would have to support it.

I just consulted my colleagues. The xml tag names "LocalElements" is only an element in the XDS for OIOXML (XDIS91), not the EDIFACT (DIS91) - so you will run into problems mapping between the two, and the LocalElements is not a part of the standard. Further, the LocalElements are only used if there is an agreement between the sending and receiving part, if there isn't an agreement systems must be able to read the messages without failing if LocalElements is included, but are not required to load in the information in LocalElements (see page 12 http://svn.medcom.dk/svn/releases/Standarder/Syntaks og kommunikationsregler/XML/Dokumentation/).

how does that apply to the services that are to convert between EDI/XML and FHIR?

Self-defined FHIR extension will not be converted to OIOXML, and LocalElements in an OIOXML will not be converted to FHIR.

ok


Unreachable code in function with an empty return

Hello,
I have found this strange if .. else statement in the code.
https://github.com/modcluster/mod_proxy_cluster/blob/11f6f73f06c8d1ff62c16b6b8c67b47fb20fd73f/native/mod_proxy_cluster/mod_proxy_cluster.c#L1268-L1271
The function has some additional statements that are therefore unreachable. Is this a bug, or is it intended? If the latter, wouldn't it be better to omit the function completely as it does not do what it says? Or am I missing something?
It was introduced here  8576d58d82ba667ac703030f3b1852eab40f9547.
Thanks.

Assigning @jfclere to clean this up. Thanks!

that is obviously a bug!

the:
+++
else
return;
+++
Should be removed.

The else statement was removed.


dbWriteTable error with empty char column

I have a R table with an empty column of type character. When I try to write this table to Exasol with dbWriteTable (to a non-existing table), an invalid CREATE TABLE statement is generated:
[EXASOL][EXASolution driver]syntax error, unexpected '-', expecting UNSIGNED_INTEGER_ [line 1, column 815]

In addition: Warning message:
In max(nchar(as.character(x)), na.rm = TRUE) :
  no non-missing arguments to max; returning -Inf

In these cases, the function max(nchar(as.character(x)), na.rm = TRUE) returns -Inf and the generated CREATE TABLE statement tries to create a column of type VARCHAR(-Inf).
Workaround:
backup your R table, identify empty char columns, temporary fill them with something, execute dbWriteTable, create and save DDL statement on exasol, drop table on exasol, create new table on exasol with saved DDL and fix varchar lenghts, write backup table to exasol.

Hello @tomazweiss,
Thank you for reporting this bug.
I think, the problem happens here. The function dbDataType can't determine the database type for the empty column in the data.frame. I am not sure, if this is fixable, I need to look deeper into this problem.
However, you can use field_types parameter to specify the database types for the columns directly to work around the problem. https://github.com/exasol/r-exasol/blob/506ba2d3a3dd22eb80fe127087bb3635236118ba/R/EXADBI.R#L1345

Hello @tomazweiss,
Thank you for reporting this bug.
I think, the problem happens here. The function dbDataType can't determine the database type for the empty column in the data.frame. I am not sure, if this is fixable, I need to look deeper into this problem.
However, you can use field_types parameter to specify the database types for the columns directly to work around the problem. https://github.com/exasol/r-exasol/blob/506ba2d3a3dd22eb80fe127087bb3635236118ba/R/EXADBI.R#L1345


Print the workqueue doc id when no possible site

Print the workflow and the workqueue docid when the agent has no resources to split an acquired element.
Second commit corrects a bunch of pylint complaints

test this please

It's going to be useful for debugging legit cases of elements stuck in Available status in the agent, due to lack of input location for instance.
@ticoann you may merge it (or merge it to your PR #6856 ...)


Errata in exponentiation formula

@csoroz thanks, good catch!
Could you please point the PR to the change commit and not the merge commit into your master?
Maybe you'd have to reopen a new PR, AFAIK you can change the PR target but not the origin.
Or you do a git reset HEAD~1 on your master and force push it, that should work, too.


Container resources cannot use reference expressions

Aspire Container resources end up being generated in the provision phases of azd deployments. This causes problems when environment variables refer to project urls. We end up with go templates in the bicep definition which fail to be evaluated. We should move container resources to yaml the docker files and projects
var builder = DistributedApplication.CreateBuilder(args);

var c = builder.AddRedis("cache");

c.WithEnvironment(context =>
{
    context.EnvironmentVariables["REDIS_EP"] = c.GetEndpoint("tcp");
});

builder.Build().Run();

resource cache 'Microsoft.App/containerApps@2023-05-02-preview' = {
  name: 'cache'
  location: location
  properties: {
    environmentId: containerAppEnvironment.id
    configuration: {
      activeRevisionsMode: 'Single'
      ingress: {
        external: false
        targetPort: 6379
        transport: 'tcp'
      }
    }
    template: {
      containers: [
        {
          image: 'docker.io/library/redis:7.2.4'
          name: 'cache'
          env: [
            {
              name: 'REDIS_EP'
              value: 'tcp://cache.internal.{{ .Env.AZURE_CONTAINER_APPS_ENVIRONMENT_DEFAULT_DOMAIN }}:6379'
            }
          ]
        }
      ]
      scale: {
        minReplicas: 1
      }
    }
  }
  tags: union(tags, {'aspire-resource-name': 'cache'})
}

We should move container resources to yaml the docker files and projects

We'll need to place these in the same directory as the app host when we infra synth them.
We'll also need to update the project importer logic so that these are pushed up during azd deploy, by creating synthetic projects for them.

https://github.com/Azure/azure-dev/issues/3409 😄 ??????

Will be fixed by #3820.  The generated cache.tmpl.yaml file looks like this now:
location: {{ .Env.AZURE_LOCATION }}
identity:
  type: UserAssigned
  userAssignedIdentities:
    ? "{{ .Env.AZURE_CONTAINER_REGISTRY_MANAGED_IDENTITY_ID }}"
    : {}
properties:
  environmentId: {{ .Env.AZURE_CONTAINER_APPS_ENVIRONMENT_ID }}
  configuration:
    activeRevisionsMode: single
    ingress:
      external: false
      targetPort: {{ targetPortOrDefault 6379 }}
      transport: tcp
      allowInsecure: false
    registries:
      - server: {{ .Env.AZURE_CONTAINER_REGISTRY_ENDPOINT }}
        identity: {{ .Env.AZURE_CONTAINER_REGISTRY_MANAGED_IDENTITY_ID }}
  template:
    containers:
      - image: {{ .Image }}
        name: cache
        env:
          - name: AZURE_CLIENT_ID
            value: {{ .Env.MANAGED_IDENTITY_CLIENT_ID }}
          - name: REDIS_EP
            value: tcp://cache.internal.{{ .Env.AZURE_CONTAINER_APPS_ENVIRONMENT_DEFAULT_DOMAIN }}:6379
    scale:
      minReplicas: 1
tags:
  azd-service-name: cache
  aspire-resource-name: cache

@ellismg I'd like to only emit the managed identity good if we need to. That might be a longer-term fix though as it requires knowing a if any bicep based resources are referenced.


Fix typos in README.md

Found a couple of typos while reading so sending a pull request.

Many thanks for your corrections, @sharjeelaziz!
/accept
/lgtm

/ok-to-test


Added Django 1.7, Py3 support

@wesleykendall
I had to rename the .check() to method on BaseAssertion and ModelAssertion to .check_assertion() because Django 1.7 reserves the model method name check. I've added a note in the release notes and changed updated the major version number to 1.0.0
As far as I know, this isn't used in Ambition projects, everything else looks like its using django-narrative. It doesn't look like narrative is using .check() on any models, so I think that app should be ok without any breaking changes.
https://docs.djangoproject.com/en/1.7/ref/checks/#models
@joshmarlow FYI

@micahhausler I'll let @joshmarlow merge this one when he gets back if you are ok with that

Fine by me!

@micahhausler looks like there are a few failing tests and a flak8 issue.  Over all looks good to me.  Thanks for the general cleanup.  Once the build passes, I'm good to merge.

Coverage remained the same at 100.0% when pulling eff3e6639d6ab3c2afd88fd9723553706b4315a7 on micahhausler:develop into 0631ce00153a9413b07cc5cd500807e01239133a on ambitioninc:develop.

Coverage remained the same at 100.0% when pulling eff3e6639d6ab3c2afd88fd9723553706b4315a7 on micahhausler:develop into 0631ce00153a9413b07cc5cd500807e01239133a on ambitioninc:develop.

Coverage remained the same at 100.0% when pulling eff3e6639d6ab3c2afd88fd9723553706b4315a7 on micahhausler:develop into 0631ce00153a9413b07cc5cd500807e01239133a on ambitioninc:develop.


Generate Zod types to allow enabling slow types check

JSR doesn't like Zod because it wants exported declarations to be manually typed. I've disabled the slow types check for now, but it'd be nice to re-enabled it. The only way I know to do so is to explicitly generate the type. It'd be nice to do that anyway because the inference of Zod types is a bit heavy anyway.

This was fixed by #41


patch_sklearn() is not working but command line python3 -m sklearnex <file.py> is working

Please fix the issue cause this package may have many potentials

Can you please share details on versions used and code for reproduction.

as referred by documentation
python 3.11
sklearn 1.3.0
sklearnex 2023

sklearnex 2023.* is outdated, 2024.0.1 is available in all channels and 2024.1.0 was just released in PyPI (pip) yesterday.

sklearnex 2023.* is outdated, 2024.0.1 is available in all channels and 2024.1.0 was just released in PyPI (pip) yesterday.

But still 2024.1.0 does not support python 3.12.0

sklearnex 2023.* is outdated, 2024.0.1 is available in all channels and 2024.1.0 was just released in PyPI (pip) yesterday.

But still 2024.1.0 does not support python 3.12.0

python 3.12 support will be added in the next releases.

sklearnex 2023.* is outdated, 2024.0.1 is available in all channels and 2024.1.0 was just released in PyPI (pip) yesterday.

But still 2024.1.0 does not support python 3.12.0

already supported in pypi and conda intel channel


custom filename for massConvert command

Adds option to provide custom filename for generated file:
hana-cli massConvert SFLIGHT -n sflight.cds -a
With this option is possible to generate several schemas in one folder
Resolves #25

 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.


[PHPStanRules] Skip BinaryOp on NoMaskWithoutSprintfRule

Ref https://github.com/rectorphp/rector-src/pull/207/checks?check_run_id=2807810751#step:5:45

All checks have passed 🎉 @TomasVotruba it is ready for review.

:+1:

:+1:


Adds command aad user license remove, Closes #4510

This PR adds command aad user license remove.
Closes #4510

Thank you @nicodecleyre! We'll try to review it ASAP!


ROX-12276: Update vuln refs to new GraphQL identifiers

Description
A little more cleanup
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
FIXED: Old endpoint referenced on postgres Namespace -> Fixable CVEs

FIXED: Old endpoint referenced Deployment -> fixable CVEs

FIXED: Most common vulnerabilities widget on deployment page references the old endpoint on postgres

Secret bonus fix!
Operating system on Image Components page

/test all

/retest-required


All "Quit" Menu Options Should Quit Workers

@lamixer opened #109 because using the Quit function in the Gnome panel menu for Nylas did not exit all of the Nylas workers. The Quit in the "File" menu in the hidden dev menu quits the works, so this isn't intuitive. The menu is shown below:

This issue is to make sure that all references to "Quit" should quit the workers. @dweremeichik what are your thoughts on this proposal? Should we have two options (one just closing the app and one for quitting the workers)? It could be said that the X on the window will quit the app and any menu Quits will quit the workers.

Ohh, this is a fun one. I remember seeing something about this type of thing in a HIG a while ago. So, for gnome, not specifically the panel, see https://developer.gnome.org/hig-book/unstable/menus-standard.html.en <IP_ADDRESS>
Basically we could do a close, and a quit. Don't know if that is too confusing though. At a minimum we should change the wording. If you "close" the app, it should still show in the tray, and the workers should keep running. If you "quit" the app, it and all of it's workers should close (gracefully).

"Close Window" and "Quit Nylas" should work

This menu is a gnome 3 menu. the "Quit" shown is a default entry that is on all applications that are open. After quite a bit of looking on the webs, I can't find a way to change this behavior (either in code or the .desktop file)

That quit has to call something, or at least send a signal? Perhaps we are handling a signal incorrectly?

Ya that's what I was thinking. However, my best guess is that the signal is the same signal that is sent by pressing the x on the window.
I think either way we have to figure out how to rename these default actions. This would either be in the . desktop file at /usr/share/applications or write some code to interface with gnome

So this brings up another interesting question, should the system tray icon/menu be a thing?
Do users like the way it works, or should the application always sit in the taskbar?
If the application were to always sit in the taskbar,  with no system tray, this would be a no-brainer.


Port 1.12.2 models

[ ] minecraft:

[ ] normal rail:

[x] flat
[ ] raised
[x] curved

[ ] Speed/Powered rail:

[ ] flat
[ ] raised

[ ] Activator rail:

[ ] flat
[ ] raised

[ ] Detector rail:

[ ] flat
[ ] raised

[ ] Better RailRoads
[ ] Charset Transport Carts and Rails
[ ] Extra Rails
[ ] Steve Carts Reborn
[ ] Thaumcraft
[ ] Rails Plus
[ ] RailCraft

i hate raised rails :angry::angry::angry:


[BUG] Resize changes the orientation of some images automatically which is undesirable

Description
We're using SkiaSharp to resize our images. For most images, it works just fine. But for some images it rotates them in addition to resizing them, which is undesirable. The rotation is automatic.
Code
Code is long. I have attached an image that has this problem.
Expected Behavior
It should resize images, without automatically rotating them.
Actual Behavior
It resizes images, and for some images it automatically rotates them too.
Since it happens for all version on all operating systems (we have tested Windows and Linux), thus we just provide one image that has this problem. Please test this image.

This image is actually a landscape image with some EXIF metadata saying that it should be rotated. This is normal for 'portrait' format images from camera phones

Without seeing your code it's difficult to say but you need to either handle this metadata (SKCodec.EncodedOrigin) or preserve it in your output jpeg.
Related:
https://github.com/mono/SkiaSharp/issues/836
https://stackoverflow.com/questions/44181914/iphone-image-orientation-wrong-when-resizing-with-skiasharp

@ojb500 thank you so much for clarification. Now I understand that some photos have this intrinsic metadata.
Can I clean their metadata altogheter?


great stuff but..

please can u add the stairs and ladder just like in the jeramy tutorial:
https://www.youtube.com/watch?v=0BSlIpJ0wl8
https://www.youtube.com/watch?v=AvUYR2lzhEQ
thank you.

Dear aztecsensei,
Thank you very much for your valuable comment. We'll add this in later version


make build script generated code Windows friendly

Windows uses \ as path delimiter. Build script should escape them or replace with *nix friendly / when generating literals for include_bytes.

i just realized that issue is fixed in latest source, but somehow i'm getting older build that still does \ wrong.


Cannot install goapp command

When doing go get -u -v github.com/murlokswarm/app/..., the installing of the goapp command fails with the following error
# github.com/murlokswarm/app/cmd/goapp
./exec.go:46:4: undefined: app.Error

Oh dammit i forgot to check the cmd in the last update, will fix this asap! Thanks for reporting this. So sorry for this.

Thanks for being so responsive!


Create a cargo-linera tool

Motivation
Linera applications written in Rust are becoming non-trivial, because they require two WebAssembly binaries (the contract and the service), and because unit tests need special configuration to run correctly while integration tests must be compiled for the host target. These details might be hard for newcomers to grasp, so anything that helps with the mental load should ease adoption.
Proposed Solution
In Rust, it's not uncommon to write Cargo CLI extensions to automate part of that. Therefore, we could write a cargo-linera extension. Initially, it could have the following features:

cargo linera new: sets up a new application project structure, with a common state module, and two WebAssembly binaries.
cargo linera test: runs the unit tests in WebAssembly using a custom test runner, and runs the integration tests on the host target.

Closed by #640 .


[2.0] Handle all types of callable

As reported in the review of #682, we have multiple points where we accept a callback (breadcrumbs, before_send...).
We need to test for all the kinds of callable, and use call_user_func to do it right.

I think we got all the spots now where this was necessary, right?
So we can close this.

In that case we would have 2 issues:

we have to add tests to be sure that no-one writes a regression in this respect
I'm not sure if that relying on the exception alone is a good practice: since this is an error reporting tool, blowing up with an exception while reporting an error is very bad! We should check that those cases  are stopped when setting the callable, not when using it!

I'm going to close this as the issue is pretty old. Also adding tests for each type of callable for each place where we accept them would be so much work for no real benefit, since the typehint should be enough and if someone tries to pass something wrong then it's correct that an exception is thrown and if possible we will capture it anyway


Topydo

This PR add the topydo program (a cli todo application).
It also updates the python arrow package, because the version in nixpkgs was too old for the program (the only pkg depending on arrow builds fine with the new version).
Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
[x] Built on platform(s): NixOS
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip" : Got an (apparently unrelated) problem with the dwarf-fortress pkg
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Could you also add meta information?
I don't think there are any other packages depending on arrow so updating it shouldn't be an issue.


Backport OpenShift CI to GitHub Action changes

Description
This backport PR does the following:

Enables GitHub Action container builds and pushes for releases branches and tags (in this case, for release-2.33) provided by:
https://github.com/stackrox/scanner/pull/1376
Enables multiarch builds and multiarch container images provided by:
https://github.com/stackrox/scanner/pull/1490

Associated OSCI PR: https://github.com/openshift/release/pull/53192

hmm not sure why we have these failures, though I believe it is ok to only support amd64 for this release, as we had done before. Future releases will support multiple

O.B.E - we're only supporting this change for 2.34+


Respect status_background in message-style

When using default for @catppuccin_status_background, it still keeps background color for message-style. You can see it when you type a command in tmux.
Current implementation (with @catppuccin_status_background "default"):

Respecting @catppuccin_status_background (with @catppuccin_status_background "default"):

@vdbe done!

@vdbe done!

@vdbe could you, please, approve this PR


Error: Enzyme Internal Error: unknown composite type undefined

I'm getting this error when try to apply enzyme and I couldn't find any relative issue about this.
Here's the test.js;
import React from 'react';
import AccountLoginForm from './LoginPage';
import sinon from 'sinon';
import { mount, shallow, configure } from 'enzyme';
import { expect } from 'chai';
import Adapter from 'enzyme-adapter-react-15';
import configureStore from 'redux-mock-store';

configure({ adapter: new Adapter() });

const mockStore = configureStore();

sinon.spy(AccountLoginForm.prototype, 'componentDidMount');

describe('<AccountLoginForm />', () => {
  it('calls componentDidMount', () => {
    const wrapper = shallow(<AccountLoginForm />,  { context: { store: mockStore() } });
    expect(Foo.prototype.componentDidMount.calledOnce).to.equal(true);
  });
});

Here's the LoginPage;
import React, { Component } from 'react';
import { Form, Icon, Input, Button, Checkbox, Row, Col, Alert, Card } from 'antd';
import {connect} from 'react-redux';
import {bindActionCreators} from 'redux';
import * as api from '../../apiCalls/axiosCalls';
import * as authenticationActions from '../../actions/authenticationActions';
import './LoginPage.css';

const FormItem = Form.Item;

class AccountLoginForm extends Component {

  constructor(props) {
    super(props);
    this.state = {
      errorValidation: true
    }
  }

  componentDidMount(){
    console.log("DENEME")
  }

  handleSubmit = (e) => {
    e.preventDefault();
    this.props.form.validateFields((err, values) => {
      if (!err) {
        const userData = {
          username: values.userName,
          password: values.password
        };
        api.postLogin(userData)
        .then((response) => {
          this.props.actions.userLoginCompleted(response);
          this.props.router.push('/');
          this.setState({ errorValidation: true });
        })
        .catch((err) => {
          // TODO update field warnings based on login failure
          this.setState({ errorValidation: false });
        });
      }
    });
  }

  render() {
    const { getFieldDecorator } = this.props.form;

    return (

      <Row align="center">

        <Col >

          <Card style={{ width: 800, margin: "auto", marginTop:"200px"}}>
            <Row align="middle" className="page-title-bar" gutter={36}>

              <Col span={14} className="login-box">

Netas IOT Platform

                {/*

                  Nulla sit amet est. Aenean viverra rhoncus pede. Fusce vel dui. Nunc nec neque.

Sed hendrerit. Suspendisse eu ligula.

 */}
              </Col>
              <Col span={10}  >

                <Form onSubmit={this.handleSubmit} className="login-form">
                  <FormItem>
                    {getFieldDecorator('userName', {
                      rules: [{ required: true, message: 'Please input your username!' }],
                    })(
                      <Input prefix={<Icon type="user" style={{ fontSize: 13 }} />} placeholder="Username" />
                      )}
                  </FormItem>
                  <FormItem>
                    {getFieldDecorator('password', {
                      rules: [{ required: true, message: 'Please input your Password!' }],
                    })(
                      <Input prefix={<Icon type="lock" style={{ fontSize: 13 }} />} type="password" placeholder="Password" />
                      )}
                  </FormItem>
                  <FormItem>
                    {/* {getFieldDecorator('remember', {
                valuePropName: 'checked',
                initialValue: true,
              })(
                <Checkbox>Remember me</Checkbox>
                                )}
              Forgot password */}
                    {!this.state.errorValidation &&
                      <Alert
                        message='Error'
                        description="Login is failed."
                        type="error"
                        className="login-form-error"
                        showIcon />
                    }

                    <Button type="primary" htmlType="submit" className="login-form-button">
                      Log in
                    </Button>
                  </FormItem>
                </Form>
              </Col>
            </Row>
          </Card>

        </Col>
      </Row>

    );
  }
}

const LoginPage = Form.create()(AccountLoginForm);

function mapStateToProps(state, ownProps){
  return {
      authentication: state.authentication
  };
}

function mapDispatchtoProps(dispatch) {
  return {
    actions: bindActionCreators(authenticationActions, dispatch)
  };
}

export default connect(mapStateToProps, mapDispatchtoProps)(LoginPage);

When I run;
mocha --require ignore-styles --compilers js:babel-core/register frontend/src/antd/Login/test
I get this error;

< AccountLoginForm  />
1) calls componentDidMount
0 passing (41ms)
1 failing

< AccountLoginForm  />
calls componentDidMount:
Error: Enzyme Internal Error: unknown composite type undefined
at compositeTypeToNodeType (node_modules\enzyme-adapter-react-15\build\ReactFifteenAdapter.js:61:13)
at Object.getNode (node_modules\enzyme-adapter-react-15\build\ReactFifteenAdapter.js:254:27)
at new ShallowWrapper (node_modules\enzyme\build\ShallowWrapper.js:120:37)
at shallow (node_modules\enzyme\build\shallow.js:19:10)
at Context. (D:/Users/buraku/Desktop/ReactProjects/feature_buraku_v3/iot_core_client/frontend/src/antd/Login/LoginPageTest.js:17:21)

npm ERR! Test failed.  See above for more details.

Do I make mistake or is there a bug ?

It looks like your component has two HOCs (so for your test to pass, you'd need two .dive() calls) - what happens if you temporarily remove one or both? It'd be helpful to narrow down the problem to one of your HOCs or to your wrapped component.

Which two HOCs? Could you please identify them for me to remove one of them?

connect is one, and Form.create() appears to be another one.

I removed them one by one and tried but it throws the same error.

You're using the react 15 adapter; precisely what version of React are you using?

"react": "^15.5.4"
Maybe I configured Jest wrong, I don't know.
I just add moduleNameWrapper for CSS files.
"jest": {
    "moduleNameMapper": {
      "^.+\\.css$": "identity-obj-proxy"
    }
  }

I'm hitting the same issue using React 15.6.1 after upgrading from Enzyme 2.9.1 to 3.1.0.

If either of you could post minimal code for a repro (a repo would be ideal) then I think we could fix it quickly.

I’ve haven’t narrowed it down to minimal version. I may not as upgrading to React 16 fixes the problem for me and that was my eventual goal anyway.

Getting this error too

I had the same error. And I found that in my package.json, react-test-renderer version was 16.0, while by react and react-dom was 15.6. So I changed the react-test-renderer to version 15.6.1, and the error message was gone.

Certainly all react- packages need to have the same major and minor versions; please check your dependencies.

I've experienced the same issue. react and react-dom was 15.6.2 while react-test-renderer was 16.0. The error is gone after fixing versions.

@burakuluu do all your react- packages match versions?

I'm getting this with all react packages as ^16.0.0 with enzyme ^3.1.0

Seeing this as well with all react 16

I think react-test-renderer should add to dependencies  in enzyme-adapter-react-15 https://github.com/airbnb/enzyme/blob/master/packages/enzyme-adapter-react-15/package.json#L47  which enzyme-adapter-react-16 has already did.

@yesmeck it's a peer dep, which means that you need to manually install it at the top level.
It has to be a peer dep, because prior to React 16, the minor version had to match exactly - so since the v15 adapter works for 15.5 and 15.6, you'd have to manually specify which minor you're using.

You have to use the same versions. If you're using React 16, also use v16 for the Adapter. I had this issue and that fixed the problem.

@sungthecoder Thanks, It works.

I'm going to close this; happy to reopen tho if it's not resolved.

Ok, I've tried a buncha things.
I'm using "react": "15.1.0"
I'm also using: "jest-environment-enzyme": "^5.0.0" and configured it as:
  "jest": {
    "testEnvironment": "enzyme",
    "testEnvironmentOptions": {
      "enzymeAdapter": "react15.4"
    },
    "setupTestFrameworkScriptFile": "<rootDir>/test/setupTests.js"
  }

Now before trying this, desperately, I had just installed enzyme itself and configured the correct adapter, receiving the same error:
Enzyme Internal Error: unknown composite type undefined
What I'm testing is:
import React from 'react';
import ReactDOM from 'react-dom';
import { shallow, mount, render } from 'enzyme';
import CheckBox from '../checkbox';

it('renders without crashing', () => {
  shallow(<CheckBox />);
});

And to simplify things, I made CheckBox be the simplest thing:
import React from 'react';

export default class CheckBox extends React.Component{
  render() {
    return

;
  }
}

I'm not familiar with jest-environment-enzyme. Can you file a new issue, and include your full package.json?

@ljharb I'll just not use it so I'm extra relevant to this issue, if that's fine. I'm not using it at the moment, and my current setup is (same error):
"react": "15.1.0",
"enzyme": "^3.2.0",
"enzyme-adapter-react-15.4": "^1.0.5",

I still need your full package.json; does npm ls exit zero?

Nah @ljharb it has problems for sure. I guess I'll go about resolving those before I bother you/anyone else. Thanks

Not resolved. This issue observed only when using shallow, mount works fine.
"enzyme": "^3.3.0",
"enzyme-adapter-react-15": "^1.0.5",
"react": "^15.6.1",
"react-scripts": "1.0.10"

Running through react-scripts test --env=jsdom:
import React from 'react';
import Adapter from 'enzyme-adapter-react-15';
import { configure, shallow } from 'enzyme';
import MyComponent from './MyComponent';

configure({ adapter: new Adapter() });

describe('MyComponent component', () => {
  it('should do smth', () => {
    const wrapper = shallow(<MyComponent />);
    expect(1).toEqual(1);
  });
});

import React, { Component } from 'react';

export default class MyComponent extends Component {
  render() {
    return

  }
}

Result:

The same works with mount. @ljharb

Same issue for me, i've got a small repo here to reproduce: https://github.com/ActuallyACat/enzyme-test/blob/master/package.json
I'm using a much older version of React (v15.2.1). Mount is fine, shallow throws " Enzyme Internal Error: unknown composite type undefined".

I am facing the same issue with the project as well
TESTS
describe('Shallow rendering composite nodes', () => {
  it('should shallow render the test', () => {
    let wrapper = shallow(<Hello />)

    expect(true).toBe(true)
  })

  it('should shallow render a class component', () => {
    let wrapper = shallow(<HelloClass />)

    expect(true).toBe(true)
  })
})

describe('Shallow render known node types', () => {
  it('should render a paragraph tag', () => {
    let wrapper = shallow(
Hello
)

    expect(true).toBe(true)
  })
})

I have tried to check the issue in the package by adding a few logs to get a  better understanding of the issue.
RAMBLING
The shallow helper calls createShallowRenderer which uses react-addons-test-utils and somewhere near Line 175
you call getRenderOutput and proceed to return an element which wants to figure out if you have a class component or a stateless component if you have a user created(or composite) component.
WOULD SOMETHING LIKE THIS WORK?
The issue is that renderer._instance._compositeType is not available on the object. I have looked at bill and teaspoon where they do something similar to:
const teaSpoonElementType = renderer._instance._instance._reactInternalInstance._currentElement.type
console.log('Teaspoon Element Type\n', teaSpoonElementType)
if (isClass(teaSpoonElementType)) {
  console.log('Class component, might have state')
} else if (typeof teaSpoonElementType === 'function') {
  console.log('Stateless function')
} else {
  console.log('Could be a text node')
}

But I could not find any code to create _compositeType, I suspect the version of react-addons-test-utils might be wrong or might not be creating the _compositeType property.
To alleviate the issue without making any destructive changes,  I have tried to fix of the version of react-addons-test-utils to v15.2.1 as expected by React.

I was able to fix it by installing<EMAIL_ADDRESS>which is matching my version of react:
"react": "^15.6.1",
"react-test-renderer": "^15.6.1",
"enzyme": "^3.3.0",
"enzyme-adapter-react-15": "^1.0.5",
"react-scripts": "1.0.10"

It looks like your component has two HOCs (so for your test to pass, you'd need two .dive() calls) - what happens if you temporarily remove one or both? It'd be helpful to narrow down the problem to one of your HOCs or to your wrapped component.

i have two  HOCs, so the command error Method “simulate” is only meant to be run on a single node.
please tell me how can i solve this problem ,thanks very much

@chenglin0006 as in the quoted comment, two .dive() calls.


⚠ Bofrost.gr has degraded performance

In e88bc4e, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 4170 ms

Resolved: Bofrost.gr performance has improved in 8449f42.


PTFE-821 Add actions package ecosystem to dependabot updates

Waiting for approval
The following approvals are needed before I can proceed with the merge:

the author

one peer


2.0.0

Fixes #9
Fixes #16
Fixes #20

Current coverage is 60.86% (diff: 61.37%)

No coverage report found for master at 74d6ff6.

Powered by Codecov. Last update 74d6ff6...53b762c


Make QueryFragment generic over the backend, return to monomorphisation

This serves two primary purposes. The first is that we are generic over
the backend, which we were always going to need to do eventually -- if
for no other reason than to prevent certain expressions from compiling
when used with the wrong type of backend.
The second benefit is that we are no longer using trait objects for the
query builder, as we can now know a single concrete type that we have to
work with. This means that QueryBuilder no longer needs to be object
safe in order for Expression to be object safe. This also means that
we can remove the object safety constraints from NativeSqlType and
inline things like IntoNullable.
For the most part, the only other piece of code that had to change was
Changeset, as it also needed to become generic over the backend. I
think I can likely remove this in the future. Most of this would have
been easier with HKTs.

Ok the tests should be passing on this now. Other than potentially fixing this so Changeset doesn't need to be generic over the backend, this is good to go. I can always do that change later, as well.
I've broken out most of the code that was pre-requisite for this and committed it separately, so this change should actually be pretty boring for the most part. It's worth noting that the main benefit here is that we're no longer using trait objects for the SQL type, though being able to fail to compile if you try to do an array comparison on sqlite is pretty nice, too.
I think this is ready for a final review. @mcasper @mfpiccolo?

I went through briefly and everything looks good aside from the travis builds for stable and beta.  I can take a better look at it later tonight.

Yeah, travis is just wigging out in general. It's not related to these changes.


istioctl workload sets discovery address for revision

If the revision is set, we try to use the same discovery address that we setup in /etc/hosts
fixes https://github.com/istio/istio/issues/34058

In response to a cherrypick label: new pull request created: #35194

In response to a cherrypick label: new pull request created: #35195

In response to a cherrypick label: new pull request could not be created: failed to create pull request against istio/istio#release-1.10 from head istio-testing:cherry-pick-35048-to-release-1.10: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for istio-testing:cherry-pick-35048-to-release-1.10."}],"documentation_url":"https://docs.github.com/rest/reference/pulls#create-a-pull-request"}

In response to a cherrypick label: new pull request could not be created: failed to create pull request against istio/istio#release-1.11 from head istio-testing:cherry-pick-35048-to-release-1.11: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for istio-testing:cherry-pick-35048-to-release-1.11."}],"documentation_url":"https://docs.github.com/rest/reference/pulls#create-a-pull-request"}


Update DeveloperGuide.md

Hi
On Sun, 25 Sept 2022 at 11:36, Zhang Bozheng @.***>
wrote:

Merged #47 https://github.com/AY2223S1-CS2103T-T10-4/tp/pull/47 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/AY2223S1-CS2103T-T10-4/tp/pull/47#event-7452888953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMFNZLXIGYXVC6PMKWAAIN3V77CFHANCNFSM6AAAAAAQU47AZQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


173 OpenCL device support and implementation

This pull request merges in OpenCL device support.

OpenclDevice (subclass of Device) is implemented.
OpenCL kernels and unit tests written. Note that kernels for random value generation is in distribution.cl.
A workaround to make Travis-CI tests pass has also been implemented. Travis-CI currently does not support OpenCL well (not without a lot of strange workarounds) so instead I added a switch in CMake to skip OpenCL unit tests on Travis.

Some shortcomings, notes, and future work:

Unit tests for OpenCL will randomly segfault if the unit tests are run individually (with gtest_filter). I suspect this is due to the shared_ptr being used in the constructor. They run fine if you run all of them at once.
Current implementation of OpenclDevice only chooses the first platform and first device encountered. This is expected to be fixed in SINGA-192 Platform Singleton, where a new class will handle multiple OpenCL platforms and devices in the same computer and choose one according to user configuration.
When the CUDA device and the OpenCL device are the same, I haven't tested that.

If you want to test OpenCL on Intel CPUs or GPUs, you can download the required files here. Intel and Nvidia's support for OpenCL is only up to version 1.2, so that is what we'll work with.
I believe AMD has support for 2.0 and beyond, but I don't have any AMD C/GPUs to test this with so I don't know.

(py2)wangwei@slave2:~/singav1/incubator-singa/test/python$ python test_tensor.py
Traceback (most recent call last):
  File "test_tensor.py", line 27, in <module>
    from singa import tensor
  File "../../build/python/singa/tensor.py", line 25, in <module>
    from . import singa_wrap as singa
  File "../../build/python/singa/singa_wrap.py", line 28, in <module>
    _singa_wrap = swig_import_helper()
  File "../../build/python/singa/singa_wrap.py", line 24, in swig_import_helper
    _mod = imp.load_module('_singa_wrap', fp, pathname, description)
ImportError: ../../build/python/singa/_singa_wrap.so: undefined symbol: _ZN5singa8PlatformC1Ev


Use the exception handler with Spring Data Rest

Seems like Spring Data Rest has it's own exception handling set up, but it would be nice to get this framework to play nicely with that one.

Is there an example how to use this framework? it seems the instruction does not tell how to use.
Dave


Feedback

Please provide list of screens that your app will have

Added the list of screens to the ppt.


Fix Tesseract for big endian machines

@AlexanderP, @jbreiden, the 2nd commit fixes Tesseract for big endian ppc64 (and I expect also for other big endian machines).

Thank you to the GCC Compile Farm Project who helps by giving me access to a big endian ppc64 test machine.


BrowserConsoleHandler no longer gives any output with monolog 1.18.0

Background:
composer require monolog/monolog installs v1.18. Therefore lots of people will use this version.
Steps to re-produce:

Install monolog 1.18
Try using the BrowserConsoleHandler
Notice that it no longer adds any HTML or javascript to the page
Try 1.17.2
Notice this works as expected.

Test code:

pushHandler(new BrowserConsoleHandler());
$logger->addWarning('Hello world');

I guess https://github.com/Seldaek/monolog/commit/1048aa4a5932db4c6471fecbb22eb19e56e47e74 broke it then, but.. why? I think maybe because before we defaulted to html unless you send a content-type header with something else than html. Does your app send content-type for html pages?

Hi Seldaek, I've not spent a lot of time investigating but I can tell you that setting the content type to html does cause the

haha, fixed it.. PR coming through :)

Yeah I just saw the commas as well :/ Silly echo statement from before.

Please PR against 1.x branch btw

It's now available in 1.x if you composer require monolog/monolog:^1.18@dev you'll get the fixed version.

Thanks, sorry I realised I'd requested it to be merged into the wrong thing after I'd left the office - thanks for merging it anyway, this makes a big difference to our team :+1:


Does sarama-cluster  supports kafka 2.0?

Error: kafka: error while consuming applog/2: kafka: error decoding packet: CRC didn't match expected 0x0 got 0xe38a6876

@langzi-zmg you need to set the correct sarama.Config.Version when connecting to Kafka, nothing to do with sarama-cluster. Also, since https://github.com/Shopify/sarama/pull/1099 was merged and released, this library is officially deprecated.


Quick start for monitoring.

Here is the mapping JIRA task: https://truera.atlassian.net/browse/MUX-1513

@saketbairoliya2 can you PR this into release/rc-1.39 instead? That branch is now available and will become release/prod later today

@saketbairoliya2 can you PR this into release/rc-1.39 instead? That branch is now available and will become release/prod later today

Updated the base branch. @joshreini1


Fix #1149 - Handle  shift+space key

Issue: The shift+space key is being sent to neovim as  , not <s-space> as would be expected.
Defect: The keyboard layout/mapping we leverage shows   as being the result character from shift+space key, and in that case we ignore the shift and just send a space to neovim. This works fine for alphabetical characters - for example, if we see a P which is shift+p, we know to ignore the shift key, and just send P - but that doesn't make sense in the case of space.
Fix: Special case the ' ' key here.

Fixes #1149


package.json: change "filename" into "main"

"main" is the official key to refer to the package entrypoint, see https://docs.npmjs.com/files/package.json
In particular, without this modification the package does not load in node (latest verisons) nor browserify.
> (adv-ui)[vperron@M]<~/dev/adv-ui> iopm install floatthead --save
> (adv-ui)[vperron@M]<~/dev/adv-ui> node
> require('floatthead')
Error: Cannot find module 'floatthead'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:278:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at repl:1:1
    at REPLServer.defaultEval (repl.js:132:27)

It works very well after this change.

sure, but arent you going to have a problem using this since it doesnt declare module.exports ?
want to PR that in too :)

also, since i havent touched package json in ages, can you change it to "main": "dist/jquery.floatThead.js",

Hmmm I don't think we'll run into an issue with module.exports since you attach yourself to jquery.
There will be no use case like:  var thead = require('floatthead'),  it wouldn't make much sense :)
Requiring the module, without exports, makes it attach the code to the global jQuery symbol, which is what's wanted.
Still, you're right, I'll change the PR to point to dist/jquery.floatThead.js instead.

merged, thanks.
You caught me a day after a release, so this wont be pushed out for a bit. you can just point your stuff to my latest git commit though

damned ;) no worries. Thanks for the merge !

just released this as 1.2.13


How to export results to scvelo

Dear developer,
I have calculated kinetic rates and pseudo times using cellDancer. Now I would like to do further analysis, i.e. find the top developmental genes and plot them in a heat map as function of pseudo time. I didn't see any function in cellDancer to do this (is it possible?).
The other alternative is to feed the kinetic parameters into scvelo annData, which can estimate likelihoods on a gene basis to identify the top developmental genes. I was wondering, how to export the output from cellDancer into scvelo annData object. I already have a scvelo annData of the same system and can replace the relevant data slots. I just need to know which slots to update with cellDancer values. Thank you so much.

Hello, it's been a while since we heard back from you regarding this issue. We will be closing this issue. However, please don't hesitate to reopen it or create a new issue if you have further questions or concerns. Thank you for your understanding.

So sorry that I forgot to update here. Your instructions worked perfectly. Yes, this can be closed now. Thank you very much for your prompt response.


Bump vendored jquery version to 3.4.1.

The jquery version vendored with yard was very out of date
and had security vulnerabilities this updates it to version 3.4.1
Description
This was setting off security scanners. I rebuilt docs with the new jquery version and clicked around and things seem to work without errors in the console.
Completed Tasks

[x] I have read the Contributing Guide.
[x] The pull request is complete (implemented / written).
[x] Git commits have been cleaned up (squash WIP / revert commits).
[x] I wrote tests and ran bundle exec rake locally (if code is attached to PR).

Coverage increased (+0.2%) to 93.704% when pulling 9ed7586a390155039710ccf011f127f4ba5ce04a on adreyer:bump_jquery into ebd6269df6d9d12d4dce0fae53fbf80dfe7d5c4f on lsegal:master.


lxd/storage/dir: Don't fail when quota are set

Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@brauner should be good to go


[DNM] Test out new docutils beta

This installs the new docutils beta so we can see how it changes the end result

@aaturner how do you test this out with Sphinx? I am seeing Read the Docs errors related to incompatible versions:
Exception occurred:
  File "/home/docs/checkouts/readthedocs.org/user_builds/pydata-sphinx-theme/envs/773/lib/python3.9/site-packages/pkg_resources/__init__.py", line 800, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (docutils 0.19b1 (/home/docs/checkouts/readthedocs.org/user_builds/pydata-sphinx-theme/envs/773/lib/python3.9/site-packages), Requirement.parse('docutils<0.19,>=0.14'), {'sphinx'})
The full traceback has been saved in /tmp/sphinx-err-48j4jolk.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.

Can you try with not read the docs? They add a bunch of stuff to vanilla Sphinx, I wasn't aware that it included version constraints too.
Are you able to get the full traceback log mentioned in the error message?
At worst you could build the documentation locally -- we're really looking here for that nothing changed visually between 0.18.1 and 0.19b1; and that nothing breaks code wise.
A

I believe docutils is now released, so we can close this: https://pypi.org/project/docutils/


Use backticks around CSS selectors in lint messages.

Fixes #340.
/CC @cvrebert

LGTM.

Erm, put "fixes #340" in the commit message.

Oops, pressed the merge button :/


Multiple Domain Module Control

I make project that have multiple domain, sample: pemda.localhost.loc and disdik.localhost.loc
now I have a module with name kiosk
my question is:
how to make kiosk is enable for pemda.localhost.loc and disable for disdik.localhost.loc
if I do Module::disable('kiosk') the module will disable for both domain.
or
it is possible to add ability to doing like this
{
    "name": "Kiosk",
    "slug": "kiosk",
    "version": "1.0",
    "description": "This is the description for the Kiosk module.",
    "enabled": [
    	"pemda.localhost.loc",
    	"other.localhost.loc",
    ],
    "disabled": [
    	"disdik.localhost.loc",
    ],
}

Module::disable('kiosk', 'disdik.localhost.loc');
Module::enable('kiosk', 'pemda.localhost.loc');
Module::enable('kiosk', 'other.localhost.loc');

At this time I have no plans to implement such functionality and I probably never will. It isn't the package's responsibility to manage or even be aware of the domain that it's on. This is something you should build within your application to manage.

I do fix this on myproject.
may I clone your repo to add functionality like that.. maybe someone have issue like me.

Of course you may~ :+1:


Remove unused method in native epoll.

Motivation:
Unused methods create warnings on some C compilers.  It may not be feasible to selectively turn them off.
Modifications:
Remove createInetSocketAddress as it is unused.
Result:
Less noisy compilation

cc: @Scottmitch

@carl-mastrangelo please update the commit message to also have a one line sentence on the start... Now it starts with Motivation:

@normanmaurer Done.

@carl-mastrangelo I will pull in once the ci job finished.

Thanks @carl-mastrangelo!
4.1 (5e86325) 4.0 (ad341ea)

Thanks @Scottmitch @carl-mastrangelo !


RLP ctl script kills RLP before starting

When starting RLP we want to ensure there is not already a running instance of the RLP.
Also fixed indentation of CTL script.
Fixes #285.

This still needs to be verified with a bosh deploy and a monit stop/start while a RLP is running.

Updated to also include fix for #284.

Also need to verify this is working for TC with bosh deploy + monit stop/start.

I deployed this to a bosh-lite and did:
bosh -e lite -d loggregator restart log-api
bosh -e lite -d loggregator restart reverse_log_proxy

I also went onto both VMs and messed with the pid file and restarted the process. It stopped the old process and started the new one.


No documentation on how to specify policies in a config file

Rasa Core version: 0.13.1
Python version: 3.6.6
Operating system (windows, osx, ...): OS X 10.14.2
Issue: There is no documentation on how to provide policies via a configuration file or how to feed this file to an Agent.
I can't figure a way to achieve this code with a configuration file:
agent = Agent('chat_domain.yml', policies = [MemoizationPolicy(), KerasPolicy()])

When I run this I get:
Exception: Passing policy configuration parameters to `agent.train(...)` is not supported anymore.

When I add a line to my chat domain, it has no effect. What is the correct thing here? I will add it to the docs if only someone will tell me. A google search hits nothing and fifteen minutes with the code yields nothing either.
Content of domain file (if used & relevant):
intents:
    - order_pizza
    - greet

entities:
    - size
    - toppings

slots:
    size:
         type: text
    toppings:
        type: text

templates:
    utter_greet:
        - 'Hello, how can I help you?'
        - 'Hi, I am here to help.'
    utter_get_pizza_size:
        - 'What pizza size?'
    utter_get_pizza_toppings:
        - 'What toppings would you like on your pizza?'
actions:
    - utter_greet
    - utter_get_pizza_size
    - utter_get_pizza_toppings
    - actions.ActionOrderPizza

policies:
    - 'KerasPolicy'
    - 'MemoizationPolicy'

I'm sorry but I'm confused. The pages that @akelad have linked in the documentation say to do exactly what @rjurney was doing.

*　Configuring polices in code
You can pass a list of policies when you create an agent:
from rasa_core.policies.memoization import MemoizationPolicy
from rasa_core.policies.keras_policy import KerasPolicy
from rasa_core.agent import Agent

agent = Agent("domain.yml",
               policies=[MemoizationPolicy(), KerasPolicy()])

This does not work.
Trying the rasa_core.train script you mentioned also doesn't work:
python -m rasa_core.train -d domain.yml -s data/stories.md -o models/current/dialogue -c config.yml
That just directs me to use rasa train core, which does not have the same options, and I've got no reference to see if I'm doing it right or not.

Links reference in the comments don't have any information?
Can anyone direct me where can i get info?

try this way:
file train_initialialize.py
from __future__ import absolute_import
from __future__ import division
from __future__ import print_function
from __future__ import unicode_literals

from rasa_core import utils
from rasa_core.agent import Agent
from rasa_core.policies.keras_policy import KerasPolicy
from rasa_core.policies.memoization import MemoizationPolicy
from rasa_core.policies.sklearn_policy import SklearnPolicy

if __name__ == '__main__':

    utils.configure_colored_logging(loglevel="DEBUG")

    training_data_file = './data/stories.md'
    model_path = './models/dialogue'
    agent = Agent("clio_domain.yml",policies=[MemoizationPolicy(max_history = 2), KerasPolicy()])

    training_data = agent.load_data(training_data_file)

    agent.train(training_data)

    agent.persist(model_path)

file config.yml
policies:
  - name: "KerasPolicy"
    featurizer:
    - name: MaxHistoryTrackerFeaturizer
      max_history: 5
      state_featurizer:
        - name: BinarySingleStateFeaturizer
  - name: "MemoizationPolicy"
    max_history: 5

run (in jupyter notebook, for example):
!python train_initialize.py -c config.yml


Actual Context from Multiple Messages like Bing Chat

I did a bunch of tests, and it made it very clear that the chat bot has a lot of trouble remembering previous messages, and it will even tell me at times that it can't know more than anything but the current message, like this for example:
"GitHub Copilot: As an AI language model, I can only access the current message or prompt that I receive. I do not have the ability to keep track of previous messages or context beyond the current message. However, I can provide assistance based on the current message or prompt to the best of my ability. Please let me know how I can assist you further."
This is a huge step down from Bing Chat and is very, very disappointing. It also can't handle near the amount of characters that the current version of Bing chat lets you paste (but not type out) in a single message, which compounds the problem. I feel like certain code I can't paste in Bing chat because I'd be afraid of it being used as training data, but with Copilot Chat as long as you have the right settings turned off it's supposed to keep your code confidential. So it's upsetting that the Copilot Chat doesn't have the same ability as the Bing Chat for understanding context and for character limit.
Please add this.

Copilot Chat does have conversation history, even if it says it doesn't 😁
We include as much as will fit in the context window. If you have an example of a conversation where this still isn't working as expected, please share a specific example

Copilot Chat does have conversation history, even if it says it doesn't 😁
We include as much as will fit in the context window. If you have an example of a conversation where this still isn't working as expected, please share a specific example

As far as I can see, the issue isn't relevant anymore since the recent updates with GPT-4 .
It would be nice though if there was documentation or a video or something explaining exactly how it handles the conversation history along with the current context in the editor.
I sometimes find myself creating a chat in the editor rather than the sidebar, in order to make sure it's not using any context of the open file and is instead only using the conversation history as context. I feel like when I'm in the sidebar it's replies could get weird or off topic from what I'm talking about in the moment, because it's referencing the current context it can see in the open file or what's selected in the open file, when that's not what I'm concerned with during that message. If that's not the case I would love to hear an explanation for how it handles that.
But anyway, it seems like resources on how all this works are lacking, and I've only discovered what I have by doing a lot of testing and messing around. I love the new updates and it has worked fantastic once I got the hang of it, but it wasn't the most straightforward process.

I agree 100% that it's sometimes not helpful when random context from your open editor gets included automatically. This will actually be a focus in January, making sure that the user is in control of the chat context. Follow https://github.com/microsoft/vscode-copilot-release/issues/675 for details

I agree 100% that it's sometimes not helpful when random context from your open editor gets included automatically. This will actually be a focus in January, making sure that the user is in control of the chat context. Follow #675 for details

Glad to hear this is being worked on, thanks for the update!


delta and spark master

we prefer to build against spark master and do our own internal releases from spark master. we have had no problems doing this while also leveraging many other open source projects that relate to spark (datasources, machine learning libraries, etc.), we build these other projects ourselves for spark master and our internal spark releases.
then we added support for delta and things changed. delta's source code lives mostly in org.apache.spark.sql and ties deeply into spark using unstable apis. it is as if there is another piece of spark now that is out of sync with spark master. compiling delta for spark master seems non-trivial and this project does not maintain an active branch for spark master/snapshot. suddenly it became difficult for us to update spark to latest snapshot or a a new internal release based off spark master.
one might think our case is somewhat unusual. but there are a lot of companies that like us build against spark master. this is important for the spark community as we help find (performance/correctness) issues in spark.
please have an active branch that is maintained against spark master. or refactor delta so its not so deeply tied into spark itself. or merge delta into spark so its maintained together.
thanks!

in current situation a version of delta against spark's branch-3.0 (instead of master) would also be very helpful

We will have it soon. As soon as we have made the 0.6.0 release (last release on spark 2.4, end of march), we will migrate delta to depend on spark 3.0 nightly snapshots. So we will have delta master compatible with spark 3.0 sometime in April.

thats good news. thanks for update.


AmplifyConfig folder is not generated again after deleting it

Describe the bug
Use amplify-tools to generate AmplifyConfig folder, delete the folder and try to generate it again with Cmd+B, the folder is not generated again.
To Reproduce
Steps to reproduce the behavior:

Create a new Xcode project
Add a Build Phase to your app target.
Click on the project in the project workspace, then click the app target, and then click on Build Phases.
Tap the + button to add another phase, and select New Run Script Phase
Drag the new Run Script phase to move the phase so that it runs prior to the Compile Sources phase
Update the Run Script build phase title to Run Amplify Tools, and then update the shell script to have a single line with: ${PODS_ROOT}/AmplifyTools/amplify-tools.sh
Now that Amplify tools is added to the build process, it will run when you build you project. Build your project in Xcode Cmd+B
After step 7 is done, delete the AmplifyConfig by "Moving to trash" and try Cmd+B again.

Expected behavior
The AmplifyConfig folder and the containing four files should be generated again.
Environment(please complete the following information):

Amplify Framework Version: 1.0.1
Dependency Manager: Cocoapods
Swift Version : 5.1

Device Information (please complete the following information):

Device: Simulators
iOS Version: iOS 13.5

This is still a relevant issue and needs to be triaged and prioritized.

Seems like we don't want to support Amplify Tools anymore


Make "invalid data type" fatal

"Invalid data type" comes from the all-digits format check, which now has a default severity of :fatal.
Pivotal story: 127281899

LGTM (take 99 (cycle [🎈])

@sduckett (repeat 99 :balloon:)


NIFI-10749 Correct Windows NPM Cache for GitHub Workflow

Summary
NIFI-10749 Corrects the Windows NPM cache configuration for GitHub workflow actions.
The changes remove the custom step to get the NPM cache directory and instead rely on the default NPM cache location for Windows, which is the current user's home directory, followed by \AppData\npm-cache.
The corrected behavior can be observed in the Post Cache Node Modules step indicating that the cache contains saved files.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 8
[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

See the following GitHub Action run for an example of the NPM cache success for Windows:
https://github.com/exceptionfactory/nifi/actions/runs/3381071846/jobs/5614633106

+1 LGTM, thanks for the fix! Merging to main


Missing proto cache invalidation on repeated setting of new prototype

Writing to proto can leave the TypeHandler::protoCachesWereInvalidated set, and writing to proto again with the same type on the RHS object can skip proto cache invalidation.

I believe this is fixed by #5695?


DataAnnotationLocalizerProvider not called when configured via DI-registered MvcDataAnnotationsLocalizationOptions

Title
A DataAnnotationLocalizerProvider lambda is not called for any localizable annotation when configured via dependency injection-registered MvcDataAnnotationsLocalizationOptions.
Functional impact
For our project there should be one central set of language resource files, and not one per class where localization is used. Thus we have a dummy class that is used as the typeparam for any injected IStringLocalizer. For the data annotations we want the same and thus configured the DataAnnotationLocalizerProvider to have the factory always create a localizer for this dummy class.
For structuring the code, we so far used the pattern of configuring options for middlewares and services separately, that is not passing it directly to the extension method, but registering them with the DI container.
This does not fully work with the MvcDataAnnotationsLocalizationOptions, in the sense that a set DataAnnotationLocalizerProvider is not invoked when requests happen that load a model having appropriate annotations. The lambda for configuring the options is however executed and the property also set correctly.
The pattern of passing the options directly to the AddDataAnnotationsLocalization method works though.
Minimal repro steps
services.Configure<MvcDataAnnotationsLocalizationOptions>(options =>
{
    options.DataAnnotationLocalizerProvider =
        (type, localizerFactory) => localizerFactory.Create(typeof(Localization.SharedStrings));
}

services.AddMvc()
    .AddViewLocalization()
    .AddDataAnnotationsLocalization();

public class MyViewModel
{
    [Required(ErrorMessage = "test1")]
    [EmailAddress(ErrorMessage = "test2")]
    public string Email { get; set; }
}

Expected result
DataAnnotationLocalizerProvider being called and causing strings to be loaded from Localization.SharedStrings.<culture>.resx.
Actual result
DataAnnotationLocalizerProvider not called and causing strings to be loaded from default convention-based <namespace>.MyViewModel.<culture>.resx.
Notes
This works as expected when configured like this:
services.AddMvc()
    .AddViewLocalization()
    .AddDataAnnotationsLocalization(options =>
    {
        options.DataAnnotationLocalizerProvider =
            (type, localizerFactory) => localizerFactory.Create(typeof(Localization.SharedStrings));
    });

Further technical details

Visual Studio 2015 with IIS Express
Windows 10
.NET Framework 4.5.2 target
ASP.NET Core 1.0.0

@rynowak What's the expected behavior here?

The reason the bottom code works and the top code doesn't work is that the various "option setup" actions are all called, in sequence. So your code correctly sets up the options, but then a moment later MVC's default implementation gets called and overwrites the delegate you set with its own delegate that has different behavior.
So you can either:

Use MVC's option builder to set the delegate (this is the recommended pattern)
Set your options after MVC, in which case both option setup actions will run, but yours will run last, and will "win"


Email not received upon sponsor application

Ewa reports when a sponsor applied, no email was received by<EMAIL_ADDRESS>(https://basecamp.com/1849796/projects/9507827/messages/47182677#comment_324424248).
We have code to send this, but apparently somewhere along the chain something is busted.

Fixed by infrastructure team.


fetchEntityWithEntityID returns null user always

User is present in the database but fetchEntityWithEntityID returns null
  User user = StorageManager.shared().fetchEntityWithEntityID(parentId, User.class);
parentId is udid of the user.
I am eagerly waiting for the pending issues to be resolved(To release app on play store).When all the pending issues will be resolved?

@bensmiley  As per docs https://github.com/chat-sdk/docs it is
User user = StorageManager.shared().fetchEntityWithEntityID(userEntityID, User.class);
Is it changed in recent SDK version


First CVC for alterphp/easyadmin-extension-bundle

duplicate of #322

I did not know @BackEndTea pushed it !


refactor: remove unused test helpers

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
97

Relevant Lines:
98

💛  - Coveralls


Decentralized ML framework

Is your feature request related to a problem? Please describe.
We are a team of developers building a decentralized blockchain based ML framework. In this work, multiple machines accross internet should connect and send files to each other in each round of training. The head node responsible for aggregating the files that workers has passed to it. Head node should be switched every iteration to make it decentralized.
Describe the solution you'd like
We need a message passing mechanism that provide the IP and status of each node in the network at each iteration. Depending on those information the computers decide which computer to communicate with and send their training files to.
Describe alternatives you've considered
We have used http protocol and json files successfully for this communication.
Additional context
We are looking into any message passing library that can work with many computers and has the the feature to update the head node and workers IP to make it decentralized.

This is a very interesting project. While I am not anything close to specialist on libp2p, I am working on a somewhat similarly structured project (supporting decentralized backup instead of ML in my case).
For what it's worth :
I use the Libp2pStreamMounting feature from the kubo system that permits direct connection to any known node. The same thing is certainly possible using an implementation of libp2p directly, but I am using Python and py-lip2p does not offer that feature yet.

Thanks, we are also using Pyhon and would like to find a solution in Python. I appreciate if you can explain your Libp2pStreadMounting solution and also if someone else from Libp2p can clarify if there is a native solution for python in their package.

Thanks, we are also using Pyhon and would like to find a solution in Python.

py-libp2p is not maintained. Your best bet is to bind to one of the other implementations. I would expect this to be reasonably easy with rust-libp2p and go-libp2p.

py-libp2p is not maintained.

Yes it is. @pacrob and myself are maintaining py-libp2p. It is behind on a lot of features but we could use the help getting it caught up.

The best place for this discussion is to post it into Ideas in the py-libp2p discussion forum