Fixes for InfluxDB queries

Fix a typo to allow building
Add support for backtick-quoted strings, because double-quotes are really a necessity for InfluxDB queries

Refs #618

Hrm, any idea how to fix travis to install deps on this branch?

the influx files need to be vendored to the _third_party directory with github.com/mjibson/party

The current tests appear to require a real connection to a DB and some arbitrary data, so that's not going to fly.  Other sources don't include any tests, are we better to just delete the integration tests?

Backticks won't work because the conf file already uses backticks. However I added single quotes, so you can use those:
alert a {
  crit = influx('select * from "db"."retention"."measurement"', ...)
}

Could you un-party the code for now (remove the new _third_party stuff and point the imports back to their normal place)? We'll do that in another PR so we can actually see this and post comments on github. It's currently too big to render it all. Don't worry about the travis stuff.
The tests were just there so I could type go test and get something to run. They will not be committed as is since they require a real database. They'll either be removed or mocked.

Backticks won't work because the conf file already uses backticks.

I don't follow, do backticks have some special conflicting use in the configs?

However I added single quotes, so you can use those

Single quotes don't work from the expression UI for me - I get expr: invalid syntax, I haven't traced this yet.  I see strconv.Unquote() used in a couple of places - that expects a character literal if it encounters single-quoted input.

So apparently (influxdb/influxdb#1757) identifiers need to be double-quoted, and string-literals need to be single-quoted, so it looks like we'd need escaping if backticks are out of the question.

It also looks like we can't currently do GROUP BY time() due to splitting out the time range as parameters.  The InfluxDB parser barfs when the group is included but there're no time conditions, and the Bosun code needs to parse the query before it can append the time conditions.  This is another blocker.  Thoughts on the best approach?

I don't follow, do backticks have some special conflicting use in the configs?

Yes. Multi-line strings are created using backticks, and there's no escaping, so you can't have a backtick in a backtick-delimited string.

identifiers need to be double-quoted, and string-literals need to be single-quoted

Ugh. I didn't know that. I thought double quotes were all it used. This means Bosun will have to add in escape characters to either its conf or expr parsing. This needs a bit more discussion before a decision is made about where to do the escaping.

InfluxDB parser barfs when the group is included

Hmm. I'll look into this more. No obvious solutions come to mind.

@mjibson Any thoughts on the direction bosun should take with this?
Looking to get InfluxDB support across the line. I have been starting here https://github.com/bosun-monitor/bosun/compare/influxdb-query

I'm no longer involved in Bosun, but I assume it's where it was a few months ago. If someone is willing to update the existing influx PR and fix the issues listed here and will maintain it going forward, the team may merge it.

@nathanielc I think it would be best just to go with triple quoted strings as another format of literal strings since we already use backticks. The parser will need to be updated for that though.

@kylebrandt ok thanks, I'll take a crack at it.

I'm no longer involved in Bosun

miss you already @mjibson ! hope you're having a good time with the new gig.

@kylebrandt See https://github.com/bosun-monitor/bosun/pull/1289 This should address the issues that have been discussed.

Closing this issue as InfluxDB support was added in https://github.com/bosun-monitor/bosun/pull/1291


Update answer option values to match PCORNET CDM specifications.

This applies specifically to the my-quality-of-life.yml survey. However, a consistent system for values should be followed: 1 (low, negative, bad) - 5 (high, positive, good)
The one exception in the CDM:
Social Roles
& Activities I have trouble doing
all of my regular
leisure activities with
others

5=Never
4=Rarely
3=Sometimes
2=Usually
1=Always

@remomueller @krand7 Do you see any functionality problems with such an update?

@remomueller @krand7 Value updates should also be done for ESS questions and PROMIS questions, to match the defined scoring criteria. This would get rid of the need to do complex mappings in the survey report functionality.


Update README.md

Overview
Update readme
Testing/Steps taken to ensure quality
No changes were made t ocode
Notes
s3_regional_buckets key is not documented. Its significance is important, because of the recommended practice when it comes to using nested stacks inside the organization repos. If the project is not uploaded to different buckets in each region, the multi-region tests fail, as the QSS3Bucket is not found in QSS3Region
Testing Instructions
N/A

/do-e2e-tests


Inconsistency in empty placeholders through the Console App

Is your suggestion related to an experience ? Please describe.

There are inconsistencies in empty placeholders through the Console App. Refer following Screenshots.
Edit Role > Assign Users

Edit Group > Assign Users

Edit Group > Assign Roles

Describe the improvement

These UIs need to be consistent.
Additional context

N/A

Edit user > Active Sessions

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Generate the consolidated release notes for 1.3.4

Signed-off-by: Zelin Hao<EMAIL_ADDRESS>Description
The consolidated release notes for 1.3.4.
Issues Resolved
Part of #2205
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2336 (23001f8) into main (13174f5) will increase coverage by 5.38%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2336      +/-   ##
=============================================
+ Coverage     94.61%   100.00%   +5.38%
=============================================
  Files           209         6     -203
  Lines          4309       105    -4204
  Branches         29        19      -10
=============================================
- Hits           4077       105    -3972
+ Misses          226         0     -226
+ Partials          6         0       -6

Impacted Files
Coverage Δ

...workflow/opensearch/build_artifact_check_plugin.py

src/manifests_workflow/input_manifests.py

src/test_workflow/perf_test/perf_test_cluster.py

src/system/temporary_directory.py

tests/jenkins/jobs/RunIntegTestScript_Jenkinsfile

...sts/jenkins/jobs/DetectTestDockerAgent_Jenkinsfile

...teg_test_start_properties_opensearch_dashboards.py

tests/jenkins/jobs/UploadToS3_Jenkinsfile

...test_workflow/perf_test/perf_multi_node_cluster.py

src/assemble_workflow/bundle_opensearch.py

... and 193 more

Continue to review full report at Codecov.

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


Make signups go directly into NationBuilder

Use the API, but to do that we need an API key. http://nationbuilder.com/developers_apply

I want to make sure I understand the task. Nationbuilder apparently makes developers create 3 examples that use their API before they will issue a key. So: we need someone to write the 3 examples so we can get our own API key? If that's the case, I can do this I'm familiar with APIs... Thanks!

Great!  Yeah we requested an API key a couple days ago, hopefully we'll get
it soon.  Once we do I'll ping you.
On Tue, May 24, 2016 at 10:25 PM frankimbur<EMAIL_ADDRESS>wrote:

I want to make sure I understand the task. Nationbuilder apparently makes
developers create 3 examples that use their API before they will issue a
key. So: we need someone to write the 3 examples so we can get our own API
key? If that's the case, I can do this I'm familiar with APIs... Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/BrandNewCongress/website/issues/19#issuecomment-221456915

OK standing by

@frankimbur Want to take a shot at getting the API key?  I think the person who was attempting to get it for us last time didn't end up finishing.

@saikat:  Sorry I didn't get back to you to confirm your last message - I can see in another thread that @evanhl has been tasked on this, so I'll give him a chance to make some progress and stand by to pitch in if his efforts stall....


Homepage Help Component

Added OCC Community Help Desk with link as a rich text component to OCC Home page.
Critical Changes
Changes
Issues Closed

@sdbolt - I just approved your updates. Would you like the honor of actually merging this pull request?

Success! Thank you for the learning opportunity @mkolodner and @Nickers3

Yay! Well done!
Now, would you like to take the Developer role on this Task: https://metecho.herokuapp.com/projects/ombudsmancloudcare/release-13/occ-help-and-training-page-change-links ?


Dev geo

I just did the first merge request.
I added 2 basic data processors.
EPSG Enrichter
LatLngToGeo

I also added a description and icon in the depending resource folder for detail information
For testing I use the http://api.open-notify.org/iss-now.json stream.
Following issues occurs during developing:
To get it run on my dev environment I had to modify the pom file and change the resources for StreamPipes. I also added the JTS core maven package. This major change leads to import package changes from
import org.apache.streampipes.messaging.kafka.SpKafkaProtocolFactory;  to import org.streampipes.messaging.kafka.SpKafkaProtocolFactory;
Therefore I stopped tracking the geo processors already implemented in this section. I also modified the GeoJvmInit.java and GeoJvmConfig.java.
What is the best way to synchronize the pom file resources to use the same package
For some reason, git thinks the LatLngToGeo PE is part of another PE:
e.g. GoogleMapsGeocodingParameters.java → ...ors/latLngToGeo/LatLngToGeoParameter.java
I guess this has be fixed on my side. But if I watch the commit history, I just see mine and have no clue why this happens.
After setup the pipeline I was checking dashboard for the event stream. The event is forwarded but nothing is coming up.
During Adapter Setup I added an Icon for the Adapter, but this is not showing up in the pipeline editor.
I added some resource descriptions. The icons for the PE's  are shown in the pipeline but if I activate "show documentation" during PE configuration, it is referencing to the Static Google Maps Geocoding. This happens for both PE's.

incompatible package settings due wrong IDE import


Shader functions dfdx/dfdy/fwidth don't work on browsers

https://developer.mozilla.org/en-US/docs/Web/API/OES_standard_derivatives
Reported by @jakecoffman

The fix is not so easy for WebGL2.

This requires to change the GLSL output to the version 300 (especially for WebGL 2.0)

@jakecoffman OK, now dfdx should work on browsers!
Next, I'll confirm this on mobiles. Probably these don't work now.


feat(jsii): reflect stabilities in doc comment summaries

In order to make the stability of certain API elements more clear, note
any non-standard API stabilities in the APIs summary docstring.
This makes it clearly visible in the IDEs autocomplete.
IMPLEMENTATION NOTES
We put the onus of rendering of the (experimental)/(deprecated) prefixes on the code generators for
every language. We played with the idea of sticking it directly into the manifest but then the API reference
would also have had it, and the API reference already has color coding and graphical symbols to indicate
these statuses.
To modify the .js/.d.ts generation:

Reversed the order of program.emit() and assembler.emit(). assembler.emit() now comes first,
so that we know the correct stability before we start emitting code.
For every node we encounter, we regenerate the doc comment block to include stability in the summary
line if appropriate. We remember the desired doc comment block for later.
The annotation is being done by TypeScript transforms. The original comments are stripped out of the
SourceFile (in-memory) and replaced with synthetic comments containing the generated doc comments.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 94f67290ca37c7cd4e794cf29cfeac8119a3726d
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

I thought out goal was to have "experimental" be the FIRST THING in the documentation? Not the last one?
Does it also render in the TypeScript documentation? (In the .d.ts files)

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 72cb24b103cbeb0b4fe45768fcba8e660e372038
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 414d36888367469c16cd97b240cf950529ff44a9
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: ac7252891a564277f445b90a452d49470912f695
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 7baf9878d69415afa01a61d4d017a43e8a36ad1b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 10ce89aa4c539102a7d667eacbb4a41c1de4be9d
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 5646ee30d5e532f33926fbb86bbb362d4106a7eb
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

BTW - I ❤ how the PR description represents the decisions made and why. That is inspiring 😁

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: beb2857f3a45817107cec7f68345fe8a933c437e
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 523ae27a5c958a4adc274861c02b3ed3096b9a3f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 4968cc39a175ae15f22e3733e36839971bb015a7
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: f4cc913fc338bb7ecf52d8c1b5ccde3b48f2b273
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Pagination Should Provide {collection_name}/index.html as Equivalent of Page1 {collection_name}/1/index.html

implemented https://github.com/josnin/pyjan26/commit/c3781d584452cb777f34ce3f5992bc60efe84ac5


AllOperations 2

Yup that it! Last part, run the gradle task check locally to see and fix static analyzer violations. I think a variable has wrong case.

Got it! Looking into it. I am also writing test cases for deleteAll.
Thanks,
Sanchay
On Tue, May 16, 2017 at 11:07 PM Mike Nakhimovich<EMAIL_ADDRESS>wrote:

Yup that it! Last part, run the gradle task check locally to see and fix
static analyzer violations. I think a variable has wrong case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NYTimes/Store/pull/207#issuecomment-301972931, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVAy5g_LQjFWQ_uDeDkUElu06Dw-LSKkks5r6mSHgaJpZM4NcweN
.

@ramonaharrison Please take a look. Do we need BarCodePathAllResolver anymore now?

@ramonaharrison  I have made changes. Should I write SourceAllPersisterTest just like SourcePersisterTest? I am working on PMD..unfortunately gradlew check is not working in my local.

This looks great @sanchay-nyt. Adding tests for SourceAllPersister would be much appreciated. Otherwise, once one more reviewer gets a chance to take a look, I think it's good to go.
Thank you!

Example on Readme or wiki on how to use it would be optimal as well.
On May 26, 2017 10:23 AM, "Ramona Harrison"<EMAIL_ADDRESS>wrote:

This looks great @sanchay-nyt https://github.com/sanchay-nyt. Adding
tests for SourceAllPersister would be much appreciated. Otherwise, once one
more reviewer gets a chance to take a look, I think it's good to go.
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NYTimes/Store/pull/207#issuecomment-304296200, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEUX3Mar_sdqJiCGlx8SgUaDYlpHksO2ks5r9uB5gaJpZM4NcweN
.

@ramonaharrison Sure. Will write test cases for all persister .
@digitalbuddha  Sure. Will update Readme.

@digitalbuddha Could this PR be used already or do you think there will be breaking changes?

there will be no breaking changes.  You will need to use the new Resolver that is in this pr.  Additionally you will need to use the new Persister. By doing so you will now be able to call persister.readAll(type) or persister.deleteAll(type) directly to clear all barcodes for a particular type.

this is last feature we are merging into rx1 branch prior to deprecating it.


drivers: sensor: ms5837: Use devicetree property to identify MS5837 type and correct MS5837-30BA conversion

The current device type determination using the factory ID is unable to differentiate MS5837-02BA01 and MS5837-30BA01 devices resulting in the 02BA compensation being used for a 30BA device. In addition, the 30BA conversion units don't match 02BA and the resulting pressure value is off by a factor of 10.
A submodel enum property is added to devicetree that is used to specify which specific device type is in use, "02ba" (default) or "30ba".
In addition, ms5837_compensate_30's pressure result now matches the units of ms5837_compensate_02 so the read pressure is as expected.
Refer to #60950 for more context.

After reviewing the guidelines from the bot, I realized I need to revise my commit messages. I will need to open a new PR.


Allow get when prop does not exist.

Should behave just like any other object and return undefined.

:+1:


Obj loader refactor

yeah the move is important.
See the definition of make_smart_refctd_ptr its basically a new wrapper, you don't want to new here


How can I change the "Zoom in/Out" with mouse scroll

Hey!
How can I change that the mouse scroll zoom in/out will be affected even without pressing CTRL?
Thanks!

Hello,
The keys cannot be configured in the library but you can fix it like this:
    const editor = new Drawflow(id);
    editor.zoom_enter = function(event, delta) {
      event.preventDefault()
      if(event.deltaY > 0) {
        // Zoom Out
        this.zoom_out();
      } else {
        // Zoom In
        this.zoom_in();
      }
    }
    editor.start();


[CI]add a workflow to cancel previous running workflow

add a cancel workflow so when there is a new push
previous workflow will be cancelled which will help to reduce
concurrent github runner usage and reduce unnecessary waiting time.
refer to https://github.com/styfle/cancel-workflow-action#advanced-pull-requests-from-forks
according to github page here
this file has to be in default branch first to take affect.
this is a part of issue #2014
Signed-off-by: Lan Luo<EMAIL_ADDRESS>

/skip-all

@luolanzone we may have to look into https://github.com/styfle/cancel-workflow-action#advanced-all-but-latest. I'm not sure how effective this will be in its current form, given that the cancel workflow may not be able to run if the pipeline is saturated (we do have a lot of different workflows, and only a fixed number of them can run at a given time).

@antoninbas  you are right, I think current solution will only cancel running workflows, is OK to use "all but latest"?  if yes, I will verify the usage and submit a new PR later.

@antoninbas you are right, I think current solution will only cancel running workflows, is OK to use "all but latest"? if yes, I will verify the usage and submit a new PR later.

I don't see an issue with using "all but latest" but I'm just going by the documentation. I didn't test it or researched it.


Improve highlightments and popups in HCM (bug 1830850)

Modify the text and background colors in popup to fit a11y requirements
Add a backdrop filter on clickable areas in using a svg filter mapping canvas colors to Highlight and HighlightText ones.

/botio test

/botio-linux preview


Re-factor can only find instances of function in opened files

This might be a limitation of vscode rather than this extension but I thought I'd raise it anyway just encase (feel free to close if its a vscode issue)
Describe the bug
Using vscodes refactor features in an R package seems to only detect occurances in files that have been opened at least once during the session. If a file has not been opened then the occurance is not detected and it will be excluded from the renaming.

In the above screenshot instances of the "draws" function are only shown from files that I have previously opened. If I haven't opened the file then the occourance of "draws" is not shown
To Reproduce
Steps to reproduce the behavior:

Create a function
Use it in several different files
Reset your vscode instance
only open 1 file using the function
right click -> "go to references"
only the instances in files that have been opened at least once will be shown

Do you want to fix by self? (We hope your help!)
No
Expected behavior
that all instances would be shown even if the file hasn't yet been opened
Environment (please complete the following information):

OS: macOS
VSCode Version: 1.63.2
R Version: 4.1.2
vscode-R version: v2.3.6

Ok this is weird it seems to work on some functions as exected but not in others, I'm not sure yet what causes some to fail and not others....
For example on the draws function as show in the above screenshot it doesn't work at all in a fresh vscode session but then for another function validate it shows everything as expected ... :S


Downgrade<EMAIL_ADDRESS>

Hello all,
I've been working a little with this awesome plugin and I found an issue with nodemailer, when I try to register a new user with sendConfirmEmail: true I receive:
{ cause: [Error: Unsupported configuration, downgrade Nodemailer to v0.7.1 to use it],
  isOperational: true }

I managed to solve it by doing so... But I still have the feeling that I'm missing something since by default Superlogin installs<EMAIL_ADDRESS>Could you guys give me some clues about it?
Thanks a lot!

see https://github.com/colinskow/superlogin/pull/31 for a fix

Thanks @cgestes .
Should have checked PR before ask


Tutorial examples yield TypeError: 'function' object is not subscriptable

Tutorial examples do not work
On upperwall.add_section :
TypeError: 'function' object is not subscriptable

Thanks for your report. I update the correct code in examples/moc/moc.ipynb, you can try if the problem is solved.


Integrate benchmarking bot for cumulus

resolves https://github.com/paritytech/cumulus/issues/468
@JesseAbram please review.

Needs a little more than this. Some modifications to support the output folder format I think.


Penny-Punching Princess [PCSE01143]

Game summary

Game name: Penny-Punching Princess
Game serial: PCSE01143

Vita3K summary

Version: v0.1
Build number: 687
Commit hash: https://github.com/vita3k/vita3k/commit/ebee18e

Test environment summary

Tested by: waterflame
OS: Windows 7
CPU: Intel i7-2600
GPU: AMD Radeon HD 6700 Series
RAM: 8 GB

Issues
Crash on load. 0xDEADBEEF
Screenshots
NA
Log
NOTICE: Due to being above 3.65(my vita is on 3.65, why this matters), I can't actually use maidump to ramdump the eboot(and modules), so this is a decrypted file+comp pack test. It seems to work.
vita3k.log
Recommended labels

Crash
Module Crash(?)

Vita3K summary

Version: v0.1
Build number: 805
Commit hash: https://github.com/vita3k/vita3k/commit/4279f967

Issue
So we ReF00d, you can now dump 3.65+ games via maidump. It loads further now... Wooo
Log
vita3k.log

Vita3K summary

Version: v0.1
Build number: 984
Commit hash: fb0798e of pull/456 (I don't want to append every issue to the PR)

Penny-Punching Princess - [PCSE01143].log
Issue
Crash on shader gen

Vita3K summary

Version: v0.1
Build number: 1315
Commit hash: https://github.com/vita3k/vita3k/commit/5b070aac

PennyPrincess.log

Vita3K v0.1.5 2601-f803e615
goes ingame
only problem is the music is a horrible buzzing sound
vita3k.log

as of https://github.com/Vita3K/Vita3K/commit/22f307df3869c78aaed3bd890a3b30c9f92bdc7a, the audio is fine now. Initially when I was testing on Vulkan, it crashed initially with a out of memory error twice during the tutorial, and now it's gone.
Screenshots

Logs
vita3k.log
Recommeded Labels

Ingame +


Token Method bug ?

Using str(token) should return the equivalent of token.orth_.encode('utf8') . However when I use it I get back the equivalent of token.text_with_ws . My work around is unicode(token.orth_) .
Thanks for a great package !

Thanks, fixed.
This sort of fix is nasty because it involves a change in semantics that might silently break users code. To help people ease the transition, I've added a toggle spacy.tokens.token.keep_Token__str__semantics. Passing True to this function sets a global switch that preserves the old semantics.
This function will disappear in a future versions --- it's just there to help people migrate their code.

Thanks for the quick fix.


Loader implementations for ORMLite

I usually use Loaders when populating lists with data, so I've made a couple of Loader implementations for executing queries. There are more ways to query the database in ORMLite, so I'm planning to add more implementations, or maybe find a more generic way to cover all use-cases.
Please let me know if there's anything I can add like unit tests or additional documentation. Thank you.

Hello guys, any update on this ?

@Egorand  I tried your loader implementation, it's very slow. Can you show me best practice of using this loader to really fast and much good result than call from UI thread ?


[Text Analytics] Rename the WellKnownFhirVersion enum to FhirVersion

Resolves: https://github.com/Azure/azure-sdk-for-net/issues/34487
Renames the WellKnownFhirVersion enum to FhirVersion based on feedback from the Architecture Board. Additionally updates the changelog to reference this change, but also a handful of previous changes that were missing.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.AI.TextAnalytics


direccion de contratos de token

Describe the feature request
I don't see the search for token contracts as intuitive.
Maybe a section where all the contracts come. When I went to the network, I have not been able to find the horse token to add it to my metamask wallet in the latest version
Describe alternatives you've considered
a search tab by name of token addresses to add to metamask
Additional context
Additional context here.

@llombetu thanks for the feedback. you are referring to the search on the documentation site right?

that is

I don't understand this feature request, is it about our documentation or the bridge UI?

Let me close this issue but feel free to reopen it with additional info.


feat: vitest use client and server side testing for kit

Closes #270

The small problem I see here, is that we now have more or less completely different code for vite and kit. But since there is nearly no overlap, i don't think there is a good way around this.

requires an update to .gitignore for vitest.workspace temporary bundle files, similar to vite.config and vitest.config
depending on which tsconfig is used, vitest.config.ts needs to be added to that too

requires an update to .gitignore for vitest.workspace temporary bundle files, similar to vite.config and vitest.config

Does this file get's generated for you? For me i only see vite.config.ts.timestamp-* being generated, even while running pnpm test. Additionally it seems like vite recently merged a PR that's going to generate those files outside of the project root either way. Looks like this will be a change that will be introduced in vite@6

depending on which tsconfig is used, vitest.workspace.ts needs to be added to that too

Why should this be necessary?

Requires further modifications as pointed out by @dominikg here https://github.com/dominikg/vitest-example-svelte5/pull/1

Requires further modifications as pointed out by @dominikg here dominikg/vitest-example-svelte5#1

those changes are required for vite-6, however given vitest only works with an override (no-go for our cli) or as a beta with that, i suggest we roll back the vite version in our templates to vite-5 until vitest@3 was released and we were able to ensure the config we create actually works.
By the time vitest@3 is out, i hope that users that created projects from now till then can just keep using the setup and update versions. If a breaking change in vitest@3 requires a config change, i wonder if we have the ability to run an upgrade command as well?

If a breaking change in vitest@3 requires a config change, i wonder if we have the ability to run an upgrade command as well?

will settle for a closed issue documenting the error and new config
may have to share the link a few times but we'll catch up

Needs the following things to be done

use vite@6 in create templates again
wait for vitest@3 release


Tooltip error in IE/Edge

Bug, feature request, or proposal:
Hovering fast over multiple (3+) tooltips throws the following error in IE11 and Edge:
Unhandled Promise rejection: Unable to get property 'play' of undefined or null reference ; Zone: <root> ; Task: Promise.then ; Value: TypeError: Unable to get property 'play' of undefined or null reference TypeError: Unable to get property 'play' of undefined or null reference

What is the expected behavior?
I would like to be able to hover fast over as many tooltip items as i'd like without getting any error.
What is the current behavior?
Hovering over one or two tooltip elements fast works, but 3+ (dragging your mouse pointer fast over them) throws the error. Everhything works as expected in Chrome/FF but not IE/Edge.
What are the steps to reproduce?
Have a look at this Plunkr:
http://plnkr.co/edit/cQ0mGZM41iaccV5ZOIdt?p=preview
What is the use-case or motivation for changing an existing behavior?
Better support for IE11 and Edge.
Which versions of Angular, Material, OS, browsers are affected?

Angular v4.0.2
Material v2.0.0-beta.3
Windows 10
Internet Explorer 11
Edge

Is there anything else we should know?
My real app is built using Angular CLI and I have enabled the following polyfills:
/** IE9, IE10 and IE11 requires all of the following polyfills. **/
import 'core-js/es6/symbol';
import 'core-js/es6/object';
import 'core-js/es6/function';
import 'core-js/es6/parse-int';
import 'core-js/es6/parse-float';
import 'core-js/es6/number';
import 'core-js/es6/math';
import 'core-js/es6/string';
import 'core-js/es6/date';
import 'core-js/es6/array';
import 'core-js/es6/regexp';
import 'core-js/es6/map';
import 'core-js/es6/set';

/** IE10 and IE11 requires the following for NgClass support on SVG elements */
import 'classlist.js';  // Run `npm install --save classlist.js`.

/** IE10 and IE11 requires the following to support `@angular/animation`. */
import 'web-animations-js';  // Run `npm install --save web-animations-js`.

/** Evergreen browsers require these. **/
import 'core-js/es6/reflect';
import 'core-js/es7/reflect';

I get the same error message randomly on IE11 and Edge maybe 1 out of 3 times when I open a dialog component.

Dialogs works fine for me in both IE and Edge.

My issue was solved by updating web-animations-js to v2.2.4.


Live

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

msteams-platform/bots/how-to/conversations/subscribe-to-conversation-events.md
95
6
link
:white_check_mark:

msteams-platform/get-started/glossary.md
94
20
link
:white_check_mark:

msteams-platform/toolkit/AAD-manifest-customization.md
94
11
link
:white_check_mark:

msteams-platform/toolkit/Integrate-with-developer-portal.md
95
2
link
:white_check_mark:

msteams-platform/toolkit/TeamsFx-SDK.md
91
13
link
:white_check_mark:

msteams-platform/toolkit/TeamsFx-collaboration.md
93
10
link
:white_check_mark:

msteams-platform/toolkit/TeamsFx-multi-env.md
98
19
link
:white_check_mark:

msteams-platform/toolkit/TeamsFx-preview-and-customize-app-manifest.md
90
13
link
:white_check_mark:

msteams-platform/toolkit/add-API-connection.md
87
7
link
:white_check_mark:

msteams-platform/toolkit/add-capability.md
94
6
link
:white_check_mark:

msteams-platform/toolkit/add-resource.md
96
5
link
:white_check_mark:

msteams-platform/toolkit/build-environments.md
93
3
link
:white_check_mark:

msteams-platform/toolkit/create-new-project.md
99
2
link
:white_check_mark:

msteams-platform/toolkit/debug-background-process.md
91
12
link
:white_check_mark:

msteams-platform/toolkit/debug-local.md
93
7
link
:white_check_mark:

msteams-platform/toolkit/debug-overview.md
91
7
link
:white_check_mark:

msteams-platform/toolkit/deploy.md
95
1
link
:white_check_mark:

msteams-platform/toolkit/explore-Teams-Toolkit.md
94
3
link
:white_check_mark:

msteams-platform/toolkit/faq.md
92
11
link
:white_check_mark:

msteams-platform/toolkit/install-Teams-Toolkit.md
100
1
link
:white_check_mark:

msteams-platform/toolkit/provision.md
92
39
link
:white_check_mark:

msteams-platform/toolkit/publish.md
95
6
link
:white_check_mark:

msteams-platform/toolkit/sideload-and-test-app.md
95
7
link
:white_check_mark:

msteams-platform/toolkit/teams-toolkit-fundamentals.md
93
5
link
:white_check_mark:

msteams-platform/toolkit/test-app-behavior.md
96
2
link
:white_check_mark:

msteams-platform/toolkit/tools-prerequisites.md
98
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit ed74884:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/buildapps-TTK.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/add-bot.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/advanced-section.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/config-section.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/developer-portal.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/Dev-portal.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/dev-portal-first-page.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/enter-app-name.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/overview.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/publish-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/select-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/select-import-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/select-new-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/select-new-bot.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tdp/select-tools.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/3.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/5.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/accounts.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/add-API/myAPI connection.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/azure-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/build-environment-developer-portal.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/build-environment-developer-portal-1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/create-new-teams-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/debug/bot-endpoint.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/debug/debug-run.png
:white_check_mark:Succeeded
View

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


[PLT-9543] [PLT-9557] [E2E Test] Wallet connect user flow + Testing campaign workflow

https://input-output.atlassian.net/browse/PLT-9543
https://input-output.atlassian.net/browse/PLT-9557

Summary by CodeRabbit

New Features

Introduced GitHub Actions workflow "Playwright Tests" for automated testing.
Added new functionality for communication between webpages and the "nami-wallet" extension.
Introduced web pages within the Nami extension for wallet creation, hardware connection, internal popup, main popup, and signing Trezor transactions.
Added test cases and configurations for Playwright, including wallet connection, login functionality, and test campaign monitoring.
Implemented logic to use mock session credentials and wallet connection data for development and testing purposes.

Bug Fixes

Updated .gitignore to exclude more directories from tracking.

Documentation

Minor updates in the README.md file regarding Certification Service integration.

Tests

Enhanced testing capabilities with the addition of specific data-testid attributes across various components for easier test interaction.

I've added @KJES4 and @EehMauro to have a look at it.

Keeping PR closed.


Missing dot in the output file name

PR #1463 accidentally removed . (dot) in the output file name when --EmitMLIR.  This patch adds dot back.
Signed-off-by: Tung D. Le<EMAIL_ADDRESS>

Jenkins Linux s390x Build #6364 [push] Missing dot in the outpu... started at 14:12

Jenkins Linux amd64 Build #6348 [push] Missing dot in the outpu... started at 13:12

Jenkins Linux amd64 Build #6348 [push] Missing dot in the outpu... passed after 1 hr 6 min

Jenkins Linux ppc64le Build #5486 [push] Missing dot in the outpu... passed after 1 hr 26 min


setup.py fails to install with error "TypeError: init() takes exactly 2 arguments (4 given)"

Platform/operating system (i.e. Raspberry Pi with Raspbian operating system,
Windows 32-bit, Windows 64-bit, Mac OSX 64-bit, etc.):  Raspberry Pi 2 running Raspbian Stretch (specifically Hassbian)

Python version (run python -version or python3 -version):  python --version: 2.7.13; python3 --version: 3.5.3

Error message you are receiving, including any Python exception traces:
Output from "sudo python setup.py install":
running install
running bdist_egg
running egg_info
writing Adafruit_DHT.egg-info/PKG-INFO
writing top-level names to Adafruit_DHT.egg-info/top_level.txt
writing dependency_links to Adafruit_DHT.egg-info/dependency_links.txt
reading manifest file 'Adafruit_DHT.egg-info/SOURCES.txt'
writing manifest file 'Adafruit_DHT.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-armv7l/egg
running install_lib
running build_py
running build_ext
creating build/bdist.linux-armv7l/egg
creating build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/Beaglebone_Black.py -> build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/common.py -> build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/__init__.py -> build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/platform_detect.py -> build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/Test.py -> build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/Raspberry_Pi.py -> build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/Raspberry_Pi_2.py -> build/bdist.linux-armv7l/egg/Adafruit_DHT
copying build/lib.linux-armv7l-2.7/Adafruit_DHT/Raspberry_Pi_2_Driver.so -> build/bdist.linux-armv7l/egg/Adafruit_DHT
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/Beaglebone_Black.py to Beaglebone_Black.pyc
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/common.py to common.pyc
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/__init__.py to __init__.pyc
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/platform_detect.py to platform_detect.pyc
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/Test.py to Test.pyc
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/Raspberry_Pi.py to Raspberry_Pi.pyc
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/Raspberry_Pi_2.py to Raspberry_Pi_2.pyc
creating stub loader for Adafruit_DHT/Raspberry_Pi_2_Driver.so
byte-compiling build/bdist.linux-armv7l/egg/Adafruit_DHT/Raspberry_Pi_2_Driver.py to Raspberry_Pi_2_Driver.pyc
creating build/bdist.linux-armv7l/egg/EGG-INFO
copying Adafruit_DHT.egg-info/PKG-INFO -> build/bdist.linux-armv7l/egg/EGG-INFO
copying Adafruit_DHT.egg-info/SOURCES.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
copying Adafruit_DHT.egg-info/dependency_links.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
copying Adafruit_DHT.egg-info/top_level.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
writing build/bdist.linux-armv7l/egg/EGG-INFO/native_libs.txt
zip_safe flag not set; analyzing archive contents...
creating 'dist/Adafruit_DHT-1.3.2-py2.7-linux-armv7l.egg' and adding 'build/bdist.linux-armv7l/egg' to it
removing 'build/bdist.linux-armv7l/egg' (and everything under it)
Processing Adafruit_DHT-1.3.2-py2.7-linux-armv7l.egg
Removing /usr/local/lib/python2.7/dist-packages/Adafruit_DHT-1.3.2-py2.7-linux-armv7l.egg
Copying Adafruit_DHT-1.3.2-py2.7-linux-armv7l.egg to /usr/local/lib/python2.7/dist-packages
Adafruit-DHT 1.3.2 is already the active version in easy-install.pth

Installed /usr/local/lib/python2.7/dist-packages/Adafruit_DHT-1.3.2-py2.7-linux-armv7l.egg
Traceback (most recent call last):
File "setup.py", line 94, in <module>
ext_modules       = extensions)
File "/usr/lib/python2.7/distutils/core.py", line 151, in setup
dist.run_commands()
File "/usr/lib/python2.7/distutils/dist.py", line 953, in run_commands
self.run_command(cmd)
File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command
cmd_obj.run()
File "build/bdist.linux-armv7l/egg/setuptools/command/install.py", line 65, in run
File "build/bdist.linux-armv7l/egg/setuptools/command/install.py", line 115, in do_egg_install
File "build/bdist.linux-armv7l/egg/setuptools/command/easy_install.py", line 360, in run
File "build/bdist.linux-armv7l/egg/setuptools/command/easy_install.py", line 576, in easy_install
File "build/bdist.linux-armv7l/egg/setuptools/command/easy_install.py", line 627, in install_item
File "build/bdist.linux-armv7l/egg/setuptools/command/easy_install.py", line 677, in process_distribution
TypeError: __init__() takes exactly 2 arguments (4 given)

List the steps to reproduce the problem below (if possible attach code or commands
to run):
git clone https://github.com/adafruit/adafruit_python_dht.git
cd adafruit_python_dht
sudo apt install build-essential python-dev
sudo python setup.py install

Any thoughts on what might be causing this? I previously had it working but had to reinstall.

This is probably not a specific issue with this library as I've seen this issue pop up elsewhere but you might want to try to update your distutils and setuptools. I don't know if that will fix it but it might be a start.

Hey, I was never able to get it working and the distutils/setuptools were up-to-date at the time. Not tried it since as my HomeAssistant instance was able to pick up and query the sensor with no issues. Will try again tonight.

@DoctorOctagonapus,
I couldn’t find a better way to contact you so I’m writing here. This comment will be deleted in one week to keep the issue discussion clean.
My name is Marat, I am a PhD student at Carnegie Mellon University. As a part of my research, I’m trying to better understand the sources of issue reports in open source.
Can you please help me by answering few questions about this issue? It shouldn’t take more than a couple of minutes. Here’s the google form: link
I will share everything I learn from answers to this survey with the community afterwards.
Thank you very much!


fixed validation in self-exclusion

@nazaninreihani Do we need anything in translation?

@arnabk I will check translation, BTW we have many missing strings from before in translations

@nazaninreihani Can you point to missing translations? Will debug if there is anything wrong


New TLS driver functionality breaks recipe on RHEL7 hosts

:ghost: Brief Description
The new functionality that allows you to select the rsyslog TLS driver via the default['rsyslog']['tls_driver'] = 'ossl' attribute broke runs on RHEL7. This is because the rsyslog-openssl package is not available on that platform and the default is set to use openssl; RHEL/CentOS uses only the gnutls rsyslog implementation.
:pancakes: Cookbook version
v7.4.0
:woman_cook:  Chef-Infra Version
Still on 14.13.11 :(
:tophat: Platform details
RHEL7. Also present on RHEL8
Steps To Reproduce
Steps to reproduce the behavior:
Run the recipe on RHEL7/Centos7 with tls enabled and all other attributes default.
:police_car: Expected behavior
The recipe run completes
:heavy_plus_sign: Additional context
It may be useful to have a test case for TLS on/off in the kitchen suite for this cookbook. Would have caught this bug.

@petracvv If you can make a PR which addresses this, we can certainly review and merge it!


Bump prometheus/client_golang to v1.11.1

Changes
Similar to https://github.com/knative/serving/pull/12653 , bumping prometheus/client_golang to address https://github.com/advisories/GHSA-cg3q-j54f-5p7p. The rational here is that for tooling such as Whitesource(OS security tool), a vulnerability will be raised when using this project as a dependency.
At the moment this is the case due to Tekton. This means that anyone using Tekton/Pipelines as a go dependency, might get flagged, depending on their OS security tool.
Release Note
Bump prometheus/client_golang to v1.11.1 in order to address CVE-2022-21698

🤔
/retest

/retest

@pierDipi any ideas on the failing codecov test check?

@pierDipi any ideas on the failing codecov test check?

it's unrelated to your changes.


Combined upgrade

Closes #10947
Closes #10946
Closes #10945
Closes #10944
Closes #10943
Closes #10942
Closes #10941
Closes #10940
Closes #10938
Closes #10936
Closes #10934
Closes #10933
Closes #10932
Closes #10930
Closes #10929
Closes #10928
Closes #10923
Closes #10921
Closes #10919

pyramid-jinja2 2.9.1 was yanked from pypi, and now produces installation warnings. I don't know if this matters to the project yet. https://github.com/Pylons/pyramid_jinja2/commit/eed807a93f610503d9cab814c4c70d0621e708cb

Looks like it's due to https://github.com/Pylons/pyramid_jinja2/issues/163. Since we're using jinja2>=3.0, I think this doesn't affect us.


Typescript + Babel Plugin

Hello!
I've been trying to create a minimal example using UI5 + Typescript + babel-plugin-ui5-next + https://github.com/ui5-next/types, but I'm having some problems that I'd like to know if you have inputs.
1 - Component Metadata
In your example in https://github.com/ui5-next/ui5g (at least the Walkthrough one), you define your component metadata as a property, like this:

This is being a big problem for me as I haven't been able to create a typescript class variable syntax that outputs this code. I've tried:
private metadata = { manifest };
// transpiled code has metadata as a property being initialized in constructor. When going through your plugin, Component is not able to read the manifest

static metadata = { manifest };
// transpiled code has metadata being set after class definition with Component.metadata = { manifest}. Same error happens.

const metadata = { manifest }; // before class definition
// transpiled code has metadata out of class scope and the same error happens;

I imagine you know much better than me the inner workings of this metadata property, do you have any suggestions?
2 - Can I use just this plugin?
As I said, I was trying to create a repo with the minimal infrastructure possible, so I didn't want to use all gulp tasks and wanted to focus just on TS -> JS transpile & JS -> UI5JS babel transformation, but I imagine several other things happen your gulp 'pipeline', because I got many of my javascript files to be really close to the ones on your sample project, but when I ran babel using ONLY the babel-plugin-ui5-next, I got errors like this:

When removing my export default from classes, it worked, but the JS files on your sample are export default, so I don't really understand if I need to go through other transformations before. A lot of the code I was testing for the Typescript transpiling I ended up copying and pasting to the ui5wt generated app so I could test if the transpiled JS ended up being a valid app.
If you've ever created a typescript sample with these tools you've created, I'd love to take a look. Just so you understand why I'm really interested in all your projects: currently we use UI5 and Typescript in my team, but we had to create a custom Typescript fork and this hinders us a bit, as we don't get the newest features. Besides that, using JSX and ES6 seems like the future for me, so I'm really hyped about all these things you've already made. I don't have a lot of knowledge in the inner workings of the frameworks, but I'm happy to help whenever possible.
For the code I'm making, it's available here: https://github.com/lucasheim/base-ts-ui5-jsx-project
One thing to notice, one package I'm using, custdev-sapui5-infra-util is only available on SAP npm registry.
Thanks for the help!

Got it. I'm on vacation today so that here is a quick reply.
In fact, now, babel plugin only support es6 syntax with flow type, and I think the only important thing is that ide integration (type doc, auto important), so that I created this module.
But I guess you want to create an strict-type ui5 project, it's an interesting idea, cause I don't think they're necessary to do strict type check for frontend, and will reduce developers code flexibility.
But all in all, for interesting, there maybe have two solutions to introduce type check into ui5.

Enhance babel-plugin-ui5-next (with babel-transform-tyoescript), directly compile ts to js. And just use ts tool to check type.
Use flow type in js to check type.

From my mobile device, have a nice day or good night. :)

I will do some feasibility check, maybe take 2 or 3 days. See you later.

Thanks @Soontao, you're a hero. I think strict typing is a tradeoff, losing flexibility to gain quality and being less error-prone when developing. Besides that, flexibility ends up being illusional a lot of times if you take into account possible runtime errors that could be type-checked.

This plugin is compatible with Babel 7 now, so that this plugin could process typescript syntax now.

[ ] Create a minimal UI5 typescript project with a typescript subset (without non-necessary feature like namespace)
[ ] Enhance babel-plugin-ui5-next, add typescript transform logic.
[ ] Rewrite gulpfile
[ ] Test & fix plugin & make typescirpt project running.

Hello @lucasheim,
I've released a pre-alpha version for ui5 in typescript, and you just create a new ui5 typescript project with the skeleton Walk Through (Typescript) of generator-ui5g.
Attention, current generator only generate a project which COULD transform typescript to ui5 source, but NOT do the type check, user should to add script like tsc --noEmit before build  to do the typescript type check.
Here are Transform Tests for Tyepscript
Here is A Guide Video
And, addtional, i've checked the custdev-sapui5-infra-util module in SAP internal github, and it's a grunt based build system, AND, if you want to integrate this babel plugin to grunt tasks, just reference my gulpfile tasks and re-define the tasks in the custdev-sapui5-infra-util, even you don't need to change the babelrc file.
If you have any other ideas, just add comments here.
Thanks

closed by almost finished


(Closes #35) Changes for jax-metal

Three changes to get jax working with metal:

Prefer METAL device if available.
Compute RoPE with float instead of complex.
Reimplement top_k to avoid broken jax.lax.top_k on jax-metal.

i think the 'frog' branch needs to be merged into main to see if this implementing works with the new structure or not.

I still don't know how to push updates to this branch so I am continuing with the "frog" integration in my version (pr-57).

With all the jit annotations commented out, this jax-metal version is about 10x slower than the native MLX version: https://github.com/samefarrar/entropix_mlx
On an M2 Air I get about 1.5 tps with this jax-metal implementation (no jit), 4.5 tps with pytorch metal, and 15tps with entropix_mlx.


fix site-packages parameter for focal fossa

Pull Request (PR) description
On Ubuntu 20.04 there is no parameter --no-site-packages for virtualenv binary anymore. The --system-site-packages stays the same.
This Pull Request (PR) fixes the following issues
Fixes '--no-site-packages' parameter on Ubuntu Focal Fossa
root@6e6956e915f1:/# grep -i focal /etc/os-release
VERSION="20.04 LTS (Focal Fossa)"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

root@6e6956e915f1:/# virtualenv --version
virtualenv 20.0.17 from /usr/lib/python3/dist-packages/virtualenv/__init__.py

root@6e6956e915f1:/# virtualenv --help | grep 'no-site-packages'
root@6e6956e915f1:/#

root@6e6956e915f1:/# virtualenv --help | grep '^[\ -]*system-site-packages'
  --system-site-packages        give the virtual environment access to the system site-packages dir (default: False)

and this doesn't affect any other Debian type OS?

and this doesn't affect any other Debian type OS?

Ah you are right. Just did check for our needs. I will check also for other OSes later this day. Thanks for reminding!

@igalic I checked other supported OSes (using latest docker container). At Debian the virtualenv binary is not used and on latest CentOS virtualenv does still have the --no-site-packages option.

I've missed this PR and made a new one: https://github.com/voxpupuli/puppet-python/pull/552
I think it doesn't make sense to check for any OS version, as --no-site-packages is deprecated since a long time.
(see https://virtualenv.pypa.io/en/16.7.9/reference.html)
The default has changed to not use site-packages anyways, so we don't need to be explicit here.

I've missed this PR and made a new one: #552
I think it doesn't make sense to check for any OS version, as --no-site-packages is deprecated since a long time.
(see https://virtualenv.pypa.io/en/16.7.9/reference.html)
The default has changed to not use site-packages anyways, so we don't need to be explicit here.

So i can close this one?

@crazymind1337 Good question.
@igalic What do you think?

i think we've accumulated a lot more feedback on what the best approach to solving this issue is in @saz's pr, so let's keep going there


Add 'surprise me' to !quote

Remove unused imports & constants
Typo in !quote brief (errata: !quotes is not an alias)
Add 'surprise me' to !quote

refactor extraction of quote from dict to a static function of it's own

Add more random quotes to the bot

Migrate coupling from commands.py to units.py.


[Tests-Only]Bumped commit id of web in ocis

Bumped web commit id in ocis

Created a new issue to complete the bump. https://github.com/owncloud/QA/issues/665 it is much complicated than regular bumping.

I expect this to fail because of https://github.com/owncloud/web/pull/4627 not being in ocis yet


Change Query with the name of the return type table

The good if the library of this resource to create the following classes below
Entity
@entity
class MaquinaTipoEntry {
  MaquinaTipoEntry({int? id, this.empresaId, this.syncId, this.nome, this.ativo = true});

  int? id;

  int? empresaId;

  int? syncId;

  String? nome;

  bool ativo;

  int? pessoaInclusaoId;

  int? pessoaEdicaoId;

  DateTime? dataHoraInclusao;

  DateTime? dataHoraUltimaEdicao;
}

Dao base
abstract class BaseDao<T extends EntryLogBase> {
  @floor.Query(r'SELECT * FROM $table_name_of_return_type WHERE id = :id')
  Future<T?> findById(int id);

  @floor.Query(r'SELECT * FROM $table_name_of_return_type')
  Future<List<T>> getAll();

  @floor.insert
  Future<void> insert(T entry);

  @floor.update
  Future<void> update(T entry);

  @floor.delete
  Future<void> delete(T entry);

  Future<void> save(T entry) async {
    if (entry.deleted) {
      await delete(entry);
    } else if (entry.isSaved()) {
      await update(entry);
    } else {
      await insert(entry);
    }
  }
}

Dao
abstract class MaquinaTipoDao extends BaseDao<MaquinaTipoEntry> {

}

Changes in library
Change the variable $table_name_of_return_type in query by name table of return type
I make the change in a fork of the project for use in a project of mine, but I would like to use a library of yours if it is implemented, if it helps the commit with the changes is this https://github.com/ErliSoares/floor/commit/94a3871b6a4cdb8a7f7dc264fea6b5df922d888a
Expected example
@override
Future<MaquinaTipoEntry?> findById(int id) async {
  return _queryAdapter.query('SELECT * FROM maquina_tipo WHERE id = ?1',
      mapper: (Map<String, Object?> row) => MaquinaTipoEntry(
          id: row['id'] as int?,
          empresaId: row['empresaId'] as int?,
          syncId: row['syncId'] as int?,
          nome: row['nome'] as String?,
          ativo: (row['ativo'] as int) != 0),
      arguments: [id]);
}

Thank you for your effort! you might want to open a PR if you would like to see this in floor. Implementation-wise, I personally think it would be better to follow #460 or #134 with a @readAll annotation or similar rather than an "arbitrary" keyword string in the query (which is btw much longer than most table names). Those arbitrary strings aren't valid sql and will therefore make adding an sql parser for deriving types or checking queries much more difficult. I see that the special annotations are much more limiting but I can't identify much of a usecase for more versatility here.
If you do want to make the effort to create a PR, make sure to add some tests covering the feature :) .


Horizontal scrolling bar

Corpus de connaissances en management de projet (PMBoK).md.zip

Ho last update juste remove the bug! As a second issue is it normal that the "X" to close the setting page is offset of center?

Ho last update juste remove the bug! As a second issue is it normal that the "X" to close the setting page is offset of center?

Hi again Guillaume, no - the "X" should be all OK. From what i see you do not use Discordian's fonts (Catamaran) which as - from my description in the repo:
IMPORTANT: For best experience possible, please download the fonts and install them in your system before using the theme. Provided fonts closely resemble those in Discord app and are sort of required here
Download fonts and all should be A-OK

Hi again Guillaume, no - the "X" should be all OK. From what i see you do not use Discordian's fonts (Catamaran) which as - from my description in the repo:
IMPORTANT: For best experience possible, please download the fonts and install them in your system before using the theme. Provided fonts closely resemble those in Discord app and are sort of required here
Download fonts and all should be A-OK

As to Horizontal ScrollBar - do you still have the issue ? I designed the theme so there is no horizontal scrollbar because it is visually cluttering whole note but i can see i missed the tables so while checking your provided note in zip file i see the bar and can scroll.
To be honest i would love to remove it in next release, because - same as with long codeblocks - you can easily scroll such parts with **Shift" button. I will have to think about it though

As to Horizontal ScrollBar - do you still have the issue ? I designed the theme so there is no horizontal scrollbar because it is visually cluttering whole note but i can see i missed the tables so while checking your provided note in zip file i see the bar and can scroll.
To be honest i would love to remove it in next release, because - same as with long codeblocks - you can easily scroll such parts with **Shift" button. I will have to think about it though

For the issue it’s gone. I did not knew we can side scroll with shift. Thank you
Téléchargez Outlook pour iOShttps://aka.ms/o0ukef

De : Radek Kozak<EMAIL_ADDRESS>Envoyé : Wednesday, January 27, 2021 4:39:44 AM
À : radekkozak/discordian<EMAIL_ADDRESS>Cc : Gadwood<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [radekkozak/discordian] Horizontal scrolling bar (#20)
As to Horizontal ScrollBar - do you still have the issue ? I designed the theme so there is no horizontal scrollbar because it is visually cluttering whole note but i can see i missed the tables so while checking your provided note in zip file i see the bar and can scroll.
To be honest i would love to remove it in next release, because - same as with long codeblocks - you can easily scroll such parts with **Shift" button. I will have to think about it though
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/radekkozak/discordian/issues/20#issuecomment-768161419, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQRUZ3QUQ44VJSMQWMJCFJDS37NOBANCNFSM4WSTITHQ.

For the issue it’s gone. I did not knew we can side scroll with shift. Thank you
Téléchargez Outlook pour iOShttps://aka.ms/o0ukef

De : Radek Kozak<EMAIL_ADDRESS>Envoyé : Wednesday, January 27, 2021 4:39:44 AM
À : radekkozak/discordian<EMAIL_ADDRESS>Cc : Gadwood<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [radekkozak/discordian] Horizontal scrolling bar (#20)
As to Horizontal ScrollBar - do you still have the issue ? I designed the theme so there is no horizontal scrollbar because it is visually cluttering whole note but i can see i missed the tables so while checking your provided note in zip file i see the bar and can scroll.
To be honest i would love to remove it in next release, because - same as with long codeblocks - you can easily scroll such parts with **Shift" button. I will have to think about it though
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/radekkozak/discordian/issues/20#issuecomment-768161419, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQRUZ3QUQ44VJSMQWMJCFJDS37NOBANCNFSM4WSTITHQ.

Hello. I have the same bug. No horizontal bar on page with large table. Last theme and plugin.

Hi @derwish-pro , as i already explained answering @Gadwood - there is no horizontal bar in note "by design" because it only clutters the view but you should easily scroll everything that requires horizontal scrollbar by scrolling with Shift pressed. Have you tried that ?
Is the horizontal scrollbar necessary for you - i could provide a quick snippet fix for you in such case if you can't do with the solution above.
Also it would be very helpful if you could attach the problematic note so i could debug the problem in depth. Thanks !

It works with Shift only in the edit mode. In view mode it doesn`t.

Hi @derwish-pro , sorry for delay. Yes i was able to reproduce the problem - indeed you were right, something was off with Preview mode. I already have working solution - so thank you for spotting this. Will be out in next version with other fixes.
Still i'm for no horizontal bars if there is Shift+scroll working - it looks so much better on overall UI. But i'm wondering what do you think - is the lack of visual h-scrollbar disturbing ? Or maybe i should just make it an option in the plugin for the user to decide - would that be ok ? Let me know
Thanks once again for reporting the issue, appreciated

It would be great to have the option


Some question about benckmark

Hi there :
In the benchmark of proteinGym substitutions ,i see three data splitting methods are evaluated separately. Is this substitutions  benchmark trained on the single mutation scanning dataset (~690k)? Or is it combined with the single- and multi-point mutation data for training (~2.7M)? If it is combined and trained together, I don’t seem to see the multi-mutation cv data file cv_folds_multiples_substitutions contains the fold_random_5, fold_modulo_5 and fold_contiguous_5 fold columns like the single cv mutation dataset owned.
In addition, I also want to know when building the benchmark, is a separate model trained for each task data? If so, how to choose a suitable model for the prediction of proteins outside the dataset?
Thanks

I'm not on the protein gym team, however, in the paper it mentions that  fold_random_5, fold_modulo_5 and fold_contiguous_5 are only for singles the substitutions_singles folder. This would be Random, Contiguous, and Modulo evaluations in the paper.
Regarding doubles, I pretty sure that cv_folds_multiples_substitutions only contains the full DMS datasets that have multiple mutants, but if they have single mutants their single mutants are included in substitutions_singles. Multiple mutants were evaluated int the Random split only, as Contiguous and Modulo are position based and would likely be substantially more complicated in this setting.
Hope that helps!
However, someone on the protein gym team may correct me :)

Thank you @brycejoh16  for your reply, it was really helpful for me to understand single mutant substitutions benchmark dataset. However, for multiple substitutions  dataset, is there any benchmark score like single mutant benchmark presented in proteinGym website?

I personally haven't seen a benchmark score for this data type (multi mutants), maybe protein gym team is keeping that internal for now?

Hi @Meron-TANG,
Thank you for the question! To clarify the construction of the various CV schemes:

CV folds - substitutions - singles (https://marks.hms.harvard.edu/proteingym/cv_folds_singles_substitutions.zip): these files provide the 3 CV schemes (random, modulo, contiguous) for all 217 assays in the ProteinGym substitution benchmark. As @brycejoh16 pointed out already, these only include single amino acid substitutions (ie., if an assay contains multiple mutants, these mutants are removed) given the practical challenges to assign multiple mutants to folds in the modulo and contiguous schemes.

CV folds - substitutions - multiples (https://marks.hms.harvard.edu/proteingym/cv_folds_multiples_substitutions.zip): these files provide the random CV scheme for all assays in the ProteinGym substitution benchmark with at least one multiple mutant. Note that we keep all mutants in these files (singles and multiples), with the mutational depth being provided in the mutation_depth field. While there is no performance benchmark yet on these files yet (the ProteinGym paper focused on comparing performance across the 3 CV schemes on single mutants across all 217 assays), we had one analysis on a subset of these assays in the ProteinNPT paper (see section 4.4 and Figure 2) -- this was run on ProteinGym v0.1 though / did not have as many assays with multiple mutants as we have now.

Thanks a lot to @brycejoh16 for his (much quicker) response to the above!
Kind regards,
Pascal

Hi @Meron-TANG - closing this issue as I believe it is fully addressed by the above, but feel free to re-open if needed.
Best,
Pascal


Add a put method to Matrix

change get to return a numpy friendly format so we can do numpy.array(Matix().get())
add a put method to insert data back into the matrix also in a numpy format add in method to get and set individual values by index

thanks for adding the typing on the selfs, you can type the one in init as well.
Can you also pull self.mat into the local namespace for get and set so that we aren't performing the . lookup 16 times.

you will have to explain how to pull in to the local namespace.
mat = self.mat does not work results in Cannot convert 'matrix_t' to Python object
matrix_t mat = self.mat also does not work with Syntax error in simple statement list
this is where i get lost because i am not familiar with cython.

@Kovak: FYI, typing isn't needed on the self parameter. Cython already knows what type it should be, and I can confirm that this works at least on 0.21.2. The generated C code will be identical whether self is typed or not. In fact, you can put str self and Cython will just ignore you. :)
Also, pulling a cdef attribute into the local namespace is wasteful. Accessing a cdef attribute is just an offset from a pointer and is far quicker than Python attribute access.

Just relised this would not merge cleanly so i have rebased so it will now :)
based on the comments above i dont think i need to change anything else at the moment ?

Sorry about that @olymk2 ignore my comments, I was basing the typing off of outdated Cython documentation it seems.


Split screen bug in Numbers, Keyshot, Pages apps

In the case of Apple applications - Pages, Numbers, Keyshot - opening new documents in the tabs (not new windows) is treated as opening new windows. For example - on an empty desktop, opening a sheet in Numbers will correctly fill the entire screen. But opening a new document in a new tab will cause the application to take up only half of the screen.
Here is screen recording with the bug:
https://github.com/user-attachments/assets/152cc003-86d0-461e-88ca-8bf1d2a42fc3

See https://github.com/nikitabobko/AeroSpace/issues/68


[CLOSED] API/Javadoc jar has inconsistent naming

 Issue by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT
Originally opened as https://github.com/twogee/missing-link/issues/19

API/Javadoc jar has inconsistent naming

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 23 Jul 2011 at 4:03

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

- Fixed inconsistent naming for api zip

Original comment by<EMAIL_ADDRESS>on 23 Jul 2011 at 4:04

Changed state: Fixed

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Original comment by<EMAIL_ADDRESS>on 19 Aug 2011 at 12:51

Added labels: Project-ant-http


Not compatible with new voyager version.

the-control-group/voyager#3836
broke the code during installation here:
 Symfony\Component\Debug\Exception\FatalErrorException  : Trait 'TCG\Voyager\Traits\HasRelationships' not found at  vendor\pvtl\voyager-pages\src\Page.php:12

I have same issue.

HasRelationships trait has been removed from 1.1.11 the-control-group/voyager/issues/3861
need to update the Page model

This has been resolved in the latest release

@mcnamee
Are you sure?
I have a broken staging site right now after a composer update. We had v2.2 installed.
I rolled back to 1.1.11, same error. Then I saw that it still existed in 1.1.10, and now the exception disappeared - but the data isn't coming through as expected
The fault may lie in some of Voyager's controllers and views, but some clarification on this issue would be nice.


Remove quotes around restriction note

These quotes are unneccessary. The restriction note is a statement that should be included "verbatim", not quoted.

The official restriction note does contain quotes. The given page does look like something you would sign and print without further changes.


please use prop-types instead of React.Proptypes

We're working on this. See https://github.com/Root-App/react-native-mock-render/pull/16


[SPARK-48226][BUILD] Add spark-ganglia-lgpl to lint-java & spark-ganglia-lgpl and jvm-profiler to sbt-checkstyle

What changes were proposed in this pull request?
The pr aims to add

spark-ganglia-lgpl to lint-java
spark-ganglia-lgpl and jvm-profiler to sbt-checkstyle

Why are the changes needed?
1.Because the module spark-ganglia-lgpl has java code
2.Because the module spark-ganglia-lgpl & jvm-profiler has scala code
3.Although these module codes currently comply with the specification, in order to avoid problems like https://github.com/apache/spark/pull/46376, they will occur again in future modifications.
Does this PR introduce any user-facing change?
No.
How was this patch tested?

Manually test.
Pass GA.

Was this patch authored or co-authored using generative AI tooling?
No.

Thank you. Could you file a JIRA issue, @panbingkun ? 😄

Done.

late LGTM


Serializing Joda time

Is there any way to serialize-deserialize org.joda.time.DateTime properly with google-http-client-jackson2?
Many thanks to anyone who knows the answer!
Ilia

This issue may no longer be relevant due to its age. Feel free to re-open.


Expand group about page for farms (and farm users)

When a farmer clicks on a farm group, they can see a different, more comprehensive, set of data & additional widgets.
Currently we just see an "About us" section, a privacy section and maybe a "request to join" section

I am assuming that the about page will need configurable widgets, since different group types will probably require different widgets.
To introduce the concept of widgets to the "about" page for a farm group, we have several options:

hardcode a conditional rendering for farm groups and add a fixed bunch of widgets when its a farm group
introduce new widget records in the db for each 'about' widget, and add those for every group (with farm and regular groups having different sets of about widgets)
introduce a new type or page column to the group_widgets, add the appropriate data to existing group_widgets records and add the appropriate additional group_widgets for farms.

Option 1 is the quickest and occurs the most technical debt.
Option 2 requires no changes to the db schema but would get out of hand if we started adding more and more contexts where widgets are used in the app
Option 3 requires some database updates and would probably be the cleanest if we add another widget set.
So, how likely are we to add other areas of the application that have configurable widgets?

im a little confused, wouldn't we need to do #2 and #3 together? we will need the new widgets, and need to know where to display them in the group. I generally like the idea of making our widget code more flexible so we can start to use widgets in more places and eventually have custom views with custom widgets on them!

Both 2 and 3 options will require adding completely novel widgets. The main difference is in how we handle existing widgets (one already in the explore/home page) that will be placed in a new context.
Take a group that needs a active_members widget on the home and their about pages. The existing home context is covered by a group_widgets record related to a widgets record (of active_member).
Option 2 would cover the new context of the about page by creating a whole new widgets record, say active_member_about and then also creating a record in group_widgets related to active_member_about. The front-end would know that both active_member and active_member_about map to the same widget but in different places.
Option 3 would instead skip creating a new widgets record. Instead, it would detail the context of the widget in the new column on group_widgets, in a new record specific to that group. So the group would have two group_widgets records related to the same active_members widget, differentiating them because they have different values in their page column. And the front-end would then know where to put the different widgets.
I'm not sure how much we rely on the order column in group_widgets to be correct but option two would make that a bit weird....
Does that explanation make it clearer what the distinction between the two options is?

Yes thanks! And I definitely like option 3 better. Again that will make it much easier to have custom views in the future with different sets of widgets. I wonder if we might want to rename groups_widgets to be views_widgets or page_widgets… For now fine to leave as groups_widgets.

On Nov 30, 2021, at 3:04 PM, Tom Watson @.***> wrote:
Both 2 and 3 options will require adding completely novel widgets. The main difference is in how we handle existing widgets (one already in the explore/home page) that will be placed in a new context.
Take a group that needs a active_members widget on the home and their about pages. The existing home context is covered by a group_widgets record related to a widgets record (of active_member).
Option 2 would cover the new context of the about page by creating a whole new widgets record, say active_member_about and then also creating a record in group_widgets related to active_member_about. The front-end would know that both active_member and active_member_about map to the same widget but in different places.
Option 3 would instead skip creating a new widgets record. Instead, it would detail the context of the widget in the new column on group_widgets, in a new record specific to that group. So the group would have two group_widgets records related to the same active_members widget, differentiating them because they have different values in their page column. And the front-end would then know where to put the different widgets.
I'm not sure how much we rely on the order column in group_widgets to be correct but option two would make that a bit weird....
Does that explanation make it clearer what the distinction between the two options is?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/Hylozoic/hylo-evo/issues/966#issuecomment-983107269, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAA5HIYWBUK5VXKRGRTUP3UOVKAVANCNFSM5GWFJOUQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I’m not actually sure that the components are the names we want to align around. We call the landing page Explore right, so maybe we should use that? Would be good to align the public facing names with the component names with the names in the database. Shall we do that? GroupDetail makes sense to me though.

On Dec 1, 2021, at 11:50 AM, Tom Watson @.***> wrote:
Ok, so in the above discussions we were referencing the group about page as one context and the group home page. In the front-end, these are the GroupDetail component and the LandingPage component, so I will try to use those names instead
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/Hylozoic/hylo-evo/issues/966#issuecomment-984000115, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAA5HM524XAAB2XH2Z6LC3UOZ4AHANCNFSM5GWFJOUQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

about this farm (mission, etc)
opportunities to collaborate
farm details (comparison)
Requests & offers
projects
events
Farm (farm landscape) map
Farm managers
Privacy settings

Adding these widgets to an existing group, for testing purposes
INSERT INTO group_widgets(group_id, widget_id, "order", created_at, context)
VALUES
(<GroupId>, 1, 1, current_timestamp, 'group_detail'),
(<GroupId>, 15, 2, current_timestamp, 'group_detail'),
(<GroupId>, 14, 3, current_timestamp, 'group_detail'),
(<GroupId>, 4, 4, current_timestamp, 'group_detail'),
(<GroupId>, 8, 5, current_timestamp, 'group_detail'),
(<GroupId>, 7, 6, current_timestamp, 'group_detail'),
(<GroupId>, 16, 7, current_timestamp, 'group_detail'),
(<GroupId>, 17, 8, current_timestamp, 'group_detail'),
(<GroupId>, 18, 9, current_timestamp, 'group_detail');

Ok, looks like the GroupDetail component is doing double time: used in two places. One is internally facing and one externally facing.
How it is used when someone is a part of a group:

And what it looks like when you are external to a group and looking at its page:

We want all these extra widgets to be available for the second instance (external user of a group) but I think it would rapidly overload and jankify the first usecase of the GroupDetail. So I think the better strategy is to create a whole new component (maybe we could name this one About) and route to that in the appropriate circumstances.
@tibetsprague Does that make sense?

Yes I think that makes sense. We will just want to re-use components on the pages as much as we can

On Dec 2, 2021, at 4:37 PM, Tom Watson @.***> wrote:
Ok, looks like the GroupDetail component is doing double time: used in two places. One is internally facing and one externally facing.
How it is used when someone is a part of a group:
https://user-images.githubusercontent.com/5193218/144524247-ba5f4aac-b003-413c-a0f3-c079eff576f9.png
And what it looks like when you are external to a group and looking at its page:
https://user-images.githubusercontent.com/5193218/144524603-7f6b0c8d-3e24-4962-89e4-bfcc04691f9f.png
We want all these extra widgets to be available for the second instance (external user of a group) but I think it would rapidly overload and jankify the first usecase of the GroupDetail. So I think the better strategy is to create a whole new component (maybe we could name this one About) and route to that in the appropriate circumstances.
@tibetsprague https://github.com/tibetsprague Does that make sense?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Hylozoic/hylo-evo/issues/966#issuecomment-985115313, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAA5HPIWRFT6I3SFJ6KO33UPAGM5ANCNFSM5GWFJOUQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


fix example template_midi_prammgr

Hi! I needed to do some changes to get this example running. I just wanted to have a running example for a WAM instrument. Maybe some one can tell me if this is correct?
I'm aware that the change for hostGroupKey is not good but I couldn't find a way to pass the key down to the processor. 😐

Hi @KnappSas
Thank you for the fix.
In fact the groupKey is reserved for a host (group of WAMs) to get WAM instances in the audio thread. So, for the WAM itself, it should not have this key unless it is making a sub-group.
I'll try to fix these examples ASAP by adding a group inside these WAMs.

@owengc The processors getter is not exposed in the WamGroup for now. It is hard to interact between different processors in a group (i.e. attach event listeners). Do you think we can make the getter available?

@owengc The processors getter is not exposed in the WamGroup for now. It is hard to interact between different processors in a group (i.e. attach event listeners). Do you think we can make the getter available?

Never mind, I figured out how to communicate between processors in the module, which is actually via ModuleScope, not a sub-group.
Fixed in PR #88

fixed in #88


test: from execution result

Resolves #485
Checklist

[x] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

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

Project coverage is 89.81%. Comparing base (9663d29) to head (168e96b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #536      +/-   ##
==========================================
+ Coverage   89.68%   89.81%   +0.12%
==========================================
  Files         107      107
  Lines       34759    34765       +6
==========================================
+ Hits        31175    31223      +48
+ Misses       3584     3542      -42

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

not too sure why CI is failing, is this coming from me?

I thought about doing the same yes, will update!

@juanbono is this good for you?


Undercurl for diagnostics

This is a great theme, but I was wondering if it would be possible to use undercurl for diagnostics. Currently, the solid line makes it somewhat harder to read.

Thanks, I'm glad you like it! 😄
Sure, that is doable. I was basing it off of what is done in the IntelliJ New Dark theme but I can see how it can be a bit hard to read since the line is bolded:

This does seem quite a bit nicer:

Thanks a lot for a quick fix! Is it possible to release to Jetbrains marketplace?

Already done, it should be up now, it was just awaiting approval!

Thank you!
On Mon, Feb 27, 2023 at 4:50 PM Ryan Mehri @.***> wrote:

Already done, it should be up now, it was just awaiting approval!
—
Reply to this email directly, view it on GitHub
https://github.com/rmehri01/onenord-jetbrains/issues/5#issuecomment-1446575037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC3PER23KPU5NTC5L3PRC3WZTELZANCNFSM6AAAAAAVGXOFYU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add Electric-sql

I've added the electric-sql project with a preliminary implementation of the LogicInterface using the electric-sql client.
The tests are not passing for electric-sql. It fails at the first attempt to interact with the DOM, saying it can't find the #own-name-submit button. When I run npm run start:electric-sql I do see the #own-name-submit button and can submit my name, so I'm not sure why the test can't find it.
When I try to run the app I can enter my name, but then it gets stuck on the loading spinner and won't load the example data (every other project besides Firebase also does this. Firebase loads the users but not messges). When I run the project in dev mode and go to http://localhost:3000/?replication=false&add-example-data=true it does seem to try to load the example data, but then the wa-sqlite library throws an error in the web assembly code: Error: cannot rollback - no transaction is active when this.logic.addMessage() is called. I'm not sure how to debug that. I've asked about it in the Electric-sql discord channel.
I've added all the scripts that I believe are necessary to build, serve, and test the electric-sql project. Electric-sql normally expects a backend with which to sync to be a part of the build process (to create the typescript client), but it also has a local-first-only mode that does not require an actual backend, but does require docker to be running during the build process so it can stand up an ephemeral postgres instance to generate the client and then tear it down. I used that strategy to keep things simple, and the docker requirement is mentioned in the README.

I found the source of the error that was thrown from the wa-sqlite library when running the dev server. The project was loading the messages before the users, but the messages reference the users, so it did not want to add messages when the foreign keys referred no non-existent entitites. Reversing the order of the addMessage() and addUser() loops solved that.

I've narrowed down the current problem in the tests. There is a JS error being thrown when the app is first trying to render. It comes from the wa-sqlite library. In the electric-sql example react vite app, the wasm files are bundled in a way that the app can read them without any special treatment, but this app threw an error saying that it failed to fetch the wasm code:

So I added a postinstall script that moves the wasm files to the /assets folder. This seems to work when I run the dev serve. The browser can fetch the code and the app renders. But testCafe doesn't seem to like it.

I'm still looking into potential solutions for this issue. I've never used Angular or testCafe before, so if you have any prior experience that might help in solving this, please let me know :)


[Ruby] Slow to save files in CSV format

Hi developers
Saving Arrow::Table in CSV format may be slow.
Ad hoc benchmarks...
require 'arrow'
require 'csv'
require 'gr/plot'

t = Arrow::Table.load('some_nice.tsv', format: :csv, delimiter: "\t".ord)

n = 1.step(1000, 100).to_a
arrow_save_times = []
csv_save_times = []

n.each do |i|
  t2 = t.slice(0, i)

  start = Time.now
  t2.save('test.csv')
  arrow_save_times << p(Time.now - start)

  t2 = t.raw_records

  start = Time.now
  CSV.open('test2.csv', 'w') do |csv|
    t2.each do |r|
      csv << r
    end
  end
  csv_save_times << p(Time.now - start)
end

GR.stem([n, arrow_save_times], [n, csv_save_times],
        labels: ["arrow", "CSV"], xlabel: "lines", ylabel: "time", location: 2)
GR.savefig("arrow.png")
gets

Could you open an issue on JIRA? https://issues.apache.org/jira/projects/ARROW/issues/

Thank!
I'll give it a try.


add ci/cd

discussed earlier with @HelgeGehring the possibility of adding CI/CD based on https://cloud.google.com/architecture/managing-infrastructure-as-code

prototyped https://github.com/proppy/rad-lab-deploy based on https://github.com/GoogleCloudPlatform/solutions-terraform-cloudbuild-gitops

@mgielda @kgugala, @mithro was mentioned we might want to use a similar approach to allow custom runner deployment https://github.com/antmicro/runner by just forking a repo and authorizing the Google Cloud Build GitHub App, what do you think?


[Event Request] report 840 "Suggest Worksheet Lines" - Create new event OnInsertCFLineForJobPlanningLineOnBeforeInsertTempCFWorksheetLine

Describe the request
Could we please have a new event OnInsertCFLineForJobPlanningLineOnBeforeInsertTempCFWorksheetLine
in the function, InsertCFLineForJobPlanningLine from report 840 "Suggest Worksheet Lines"
this is similar to an existing event named "OnInsertCFLineForCustLedgerEntryOnBeforeInsertTempCFWorksheetLine" in the procedure InsertCFLineForCustLedgerEntry
Code:
local procedure InsertCFLineForJobPlanningLine()
   var
       Job: Record Job;
       InsertConditionHasBeenMetAlready: Boolean;
   begin
       if (TempCFWorksheetLine."Source Type" = TempCFWorksheetLine."Source Type"::Job) and
          ("Job Planning Line"."Job No." = TempCFWorksheetLine."Source No.") and
          ("Job Planning Line"."Planning Date" = TempCFWorksheetLine."Document Date") and
          ("Job Planning Line"."Document No." = TempCFWorksheetLine."Document No.")
       then begin
           InsertConditionHasBeenMetAlready := InsertConditionMet();
           TempCFWorksheetLine."Amount (LCY)" += GetJobPlanningAmountForCFLine("Job Planning Line");
           InsertOrModifyCFLine(InsertConditionHasBeenMetAlready);
       end else
           with CFWorksheetLine2 do begin
               Init();
               "Source Type" := "Source Type"::Job;
               "Source No." := "Job Planning Line"."Job No.";
               "Document Type" := "Document Type"::Invoice;
               "Document Date" := "Job Planning Line"."Planning Date";

               Job.Get("Job Planning Line"."Job No.");
               "Shortcut Dimension 1 Code" := Job."Global Dimension 1 Code";
               "Shortcut Dimension 2 Code" := Job."Global Dimension 2 Code";
               "Cash Flow Account No." := CFSetup."Job CF Account No.";
               Description :=
                 CopyStr(
                   StrSubstNo(
                     Text025,
                     Job.TableCaption(),
                     Job.Description,
                     Format("Job Planning Line"."Document Date")),
                   1, MaxStrLen(Description));

               if "Job Planning Line"."Planning Due Date" = 0D then
                   if "Job Planning Line".UpdatePlannedDueDate() then
                       "Job Planning Line".Modify();
               SetCashFlowDate(CFWorksheetLine2, "Job Planning Line"."Planning Due Date");
               "Document No." := "Job Planning Line"."Document No.";
               "Amount (LCY)" := GetJobPlanningAmountForCFLine("Job Planning Line");

   			//>>>>
               OnInsertCFLineForJobPlanningLineOnBeforeInsertTempCFWorksheetLine(CFWorksheetLine2, Job, "Cash Flow Forecast", "Job Planning Line");
   			//<<<<

               InsertTempCFWorksheetLine(CFWorksheetLine2, 0);
           end;
   end;

Event signature:
    [IntegrationEvent(false, false)]
    local procedure OnInsertCFLineForJobPlanningLineOnBeforeInsertTempCFWorksheetLine(var CashFlowWorksheetLine: Record "Cash Flow Worksheet Line"; Job: Record Job; "Cash Flow Forecast": Record "Cash Flow Forecast"; "Job Planning Line": Record "Job Planning Line");
    begin
    end;

Thanks in advance,
Sameer
Additional context
We require updating some fields in CFWorksheetLine before performing the insertion.
Internal work item: AB#502634

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