Add FREE attribute to some inherently free encounters

As title.

Thanks!
You've not quite got the syntax right -- it's a space, not a tab, after free -- but I'll fix it up and add a couple more omitted free monsters.

Okay, so I can't fix it because I don't have permissions to push to your repo. But if you could make the following monsters free:

That's bizarre, there's no permission changes since the last PR.

lynyrd
%monster%
the Tunnel of L.O.V.E. enemies
the Cargo Cultist Shorts bell fights
Sssshhsssblllrrggghsssssggggrrgglsssshhssslblgl

That would be nice.

Can do!

Autoscend also says that Travoltron, the Peripatetic Pasta and the wandering accordionists are free, so maybe those too?

I think it's wrong about Travoltron but I'll double check in the game itself after rollover when the account I use to test autoscend is done with its current run. The others are ancient revamp wanderers which I don't think there's any way to fight now anyway.

Confirming Travoltron costs an adventure to kill. autoscend is definitely wrong about that one. At 15 advs before the fight I cleavered it to death in the relay browser and had 14 advs afterwards and no message in the GCLI of "This combat did not cost a turn".

It might be because this one is from your main branch?

The CBB consumables one was on my main branch too. I haven't created any branches in my fork ever. I'm blaming GitHub for being weird.


Update framework builders to use Windows-Server bot.

This is to use the same images for all the windows bots and make
efficient use of capacity.
Bug: https://github.com/flutter/flutter/issues/68056

Landing as TBR to minimize the impact during working hours.


marge calculator having similar operation and same input (vector addition and subtraction)

Proposal
we can merge these in one by providing multiple button for different operation  as they have same input this will reduce the height of page and it became easy to navigate as name of site makes math easy

i like to work on it under gssoc21


chore: release 0.30.1

:robot: I have created a release *beep* *boop*
0.30.1 (2019-06-21)
Bug Fixes

deps: include missing @grpc/grpc-js dependency (#665) (5f42f60)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-pubsub/releases/tag/v0.30.1

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Error	CS4012	Parameters or locals of type 'SepReader.Row' cannot be declared in async methods or async lambda expressions.

@nietras
I'm sure this is a general dotnet issue but I haven't been able to do a good search on the problem. Maybe you can help me work through this problem.
My main looks like this:  static async Task Main(string[] args) since I am calling async methods elsewhere.
When I try to use either:
while (reader.MoveNext())
{
   var row = reader.Current;
}

or
foreach (var row in reader)
 {
 }

The error occurs at the definition of the row, CS4012	Parameters or locals of type 'SepReader.Row' cannot be declared in async methods or async lambda expressions.
Suggestions are welcome.

See #114 and README

THANKS.  Closing


smartSummarize align to week/month

I was wondering if there's a way to get smartSummarize to consistently align to week or month?

Hi @shakefu I would like to test your code if you can do a PR . And perhaps I could help you. I need also this feature and I think could be really interesting.
Thanks a lot !!

Right on. I'll try to get a PR put together in the near future.


Migrate to latest 1.0.0-SNAPSHOT

This PR performs the required changes for users to work with the latest Spine snapshots.

@mdrachuk can you please also do this? https://github.com/SpineEventEngine/config/issues/33


Do you want to have the enhancement of existing game ? 😀 Describe yourself..
The mid line of the court is going off the court

Describe the solution you'd like
I will make it such that it is confined to the court limit
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @VijaySamant4368 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Snowflake: Context Deadline Exceeded

Hi,
We are trying to query the Snowflake data source and we are facing the Context Deadline Exceeded Error, on exploring we found out that it timeouts after 2 min. How can we increase this timeout period?

I see that we can pass intervalMs: 600000 and maxDataPoints: 1000, but it doesn't seem to change the query

Thanks!

any updates on this?

Can you check whether the same query performed directly in Grafana also results in this error? I would assume the problem is may be with the query itself that it's taking too long.


[Event Request] table 5901  "Service Item Line" - OnBeforeCheckResponseTime

Hello, would it be possible to add OnBeforeCheckResponseTime event to table 5901  "Service Item Line", like this:
        field(13; "Response Time"; Time)            trigger OnValidate()
...
                  SkipResponseTimeHrsUpdate := true;
                 if "Response Time" <> xRec."Response Time" then begin
                     GetServHeader();
                     IsHandled := false;
>                     OnBeforeCheckResponseTime(Rec, IsHandled); //New Event
                     if not IsHandled then
                         if ("Response Date" = ServHeader."Order Date") and
                            ("Response Time" < ServHeader."Order Time")
                         then

     [IntegrationEvent(false, false)]
     local procedure OnBeforeCheckResponseTime(var ServItemLine: Record "Service Item Line"; var IsHandled: Boolean)
     begin
     end;

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: 10531.


strike_lookup won't propagate onchange event

I need to detect an onchange event when using the strike_lookup component so I've wrapped it in a <div>, but it looks like the component does an event.stopPropagation so the event won't bubble up to the parent component.
<div id="mfrInput" onchange="{!c.refreshSGIList}">
	<c:strike_lookup label="Manufacturer"
			object="Account"
			searchField="Name"
			placeholder="Select a manufacturer"
			iconName="standard:account"
			order="Name"
			loadingMessage="Loading..."
			errorMessage="Invalid input"
			showRecentRecords="true"
			value="{!v.newTask.Manufacturer__c}" />
</div>
<aura:if isTrue="{!v.newTask.Manufacturer__c != null}">
	<c:getSGIList mfrId="{!v.newTask.Manufacturer__c}" year="2018" aura:id="SGIList" />
</aura:if>

Is there any workaround for this, or any plans to enable onchange for strike_lookup?

@davidchengIC  I'm also looking for the ability to capture the onchange event. Were you able to find a workaround? @kevinberonilla is there a workaround for this, or should an enhancement be submitted?

@mrichmon12 Unfortunately  I couldn't find a workaround.

@davidchengIC  I was able to work around this by creating a change event handler in my component for the value bound to the strike lookup.  Hopefully this works in your situation as well.
Component
<aura:handler name="change" value="{!v.myAccount}" action="{!c.myActionHandler}"/>

<c:strike_lookup
	aura:id="myAccount"
	label="Account"
	object="Account"
	searchField="Name"
	placeholder="Select an Account"
	iconName="standard:account"
	subtitleField="Industry"
	order="Name"
	limit="5"
	loadingMessage="Loading..."
	errorMessage="Invalid input"
	showRecentRecords="true"
	value="{!v.myAccount}"/>

Controller
myActionHandler : function(component, event, helper) {
    //do something
}

@mrichmon12 Thanks that works for me too!


Feature request: add a slow movement key

There are needs to move cursor more precisely, such as data point extraction on a digital image.
slow_motion_speed: 5
slow_motion_key: t

The issue is“tap the movement key”moves several pixels currently, my current speed setting is 50. I think a slow motion speed like 5 will give pixel precise movements.

This was implemented and merged in 3683ce3. d (by default) can be used to reduce cursor speed.


ui#1941 - Decommison old poms

CardResponseV2Summary is not being used

this is incorrect and should be closed
            "card": {
                "cardNumber": "439576******3213",
                "createdBy": "esky-system"
            }


Feature Request: Add tag support to archivebox schedule

Type

[ ] General question or discussion
[X] Propose a brand new feature
[ ] Request modification of existing behavior or design

What is the problem that your feature request solves
Adding tag support to scheduled imports (such as an rss feed) offers an easy way to search snapshots for bulk scheduled archives or filter out noise.
What hacks or alternative solutions have you tried to solve the problem?
archivebox add CLI supports tags, but not the archivebox schedule command
How badly do you want this new feature?

[ ] It's an urgent deal-breaker, I can't live without it
[X] It's important to add it in the near-mid term future
[ ] It would be nice to have eventually

[X] I'm willing to contribute money to fix this issue
[ ] I like ArchiveBox so far / would recommend it to a friend
[ ] I've had a lot of difficulty getting ArchiveBox set up

Done! 8ab17f0
Pull the latest :dev build to get this early, or wait for the 0.7.2 release.
https://github.com/ArchiveBox/ArchiveBox#install-and-run-a-specific-github-branch


Fixed #21238 -- Fixed restoring attributes when pickling FileField and ImageField.

https://code.djangoproject.com/ticket/21238

@felixxm Your solution was fine. I applied it to the patch and changed the tests to handle the cases that you mentioned.

@hramezani Thanks for updates :+1: I'm still not convinced that it's a right solution, because with custom __getstate__() and __setstate__() we're not hitting Field.__reduce__() so we can miss sth. I'll try to analyze this again in the next few days. (\cc @akaariai)

@hramezani Thanks :+1: I split commits to make it clearer what we're actually fixing and pushed minor edits.

Excuse my ignorance, but is this change included in the latest django version 3.0.4?It was hard for me to follow.
We are still experiencing this issue under 3.0.4, is that possible?

Patch will in included in Django 3.1.


How to append external directory?

I would like to access the standard go library from my own repo.
If I comment gutentags from my .vimrc and run ctags -R /user/local/go/src . it generate the correct tags file. Is there a way to tell gutentags about my external directory? I read in the docs about .gutctags but I am not sure what should be it's content. I tried this:
ctags
-R
/usr/local/go/src
.

and other variations but nothing worked.

The .gutctags file is supposed to have the same syntax as the .ctags file and other Ctags options files, so you can check the Ctags documentation about that. Typically, you wouldn't add "ctags" on the first line, since the file only contains options... I'm not quite sure if it supports adding more files and directories, though.
You can run ctags manually by passing --options=/path/to/project/.gutctags in order to test that, too, since that will be a lot easier to test than with Gutentags.

it looks like I can achieve the same functionality using vim-go plugin. thanks!


⚠ Processors has degraded performance

In a806292, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 50 ms

Resolved: Processors performance has improved in a287387.


Does vllm support the Mac/Metal/MPS?

I ran into the error when pip install vllm in Mac:
RuntimeError: Cannot find CUDA_HOME. CUDA must be available to build the package.

Hi @Phil-U-U, vLLM does not support MPS backend at the moment as its main target scenario is to be deployed as a high-throughput server running on powerful accelerators like NVIDIA A100/H100.

Hi @Phil-U-U, vLLM does not support MPS backend at the moment as its main target scenario is to be deployed as a high-throughput server running on powerful accelerators like NVIDIA A100/H100.

That's understandable. Although, with Mac Studio, many people and companies are starting to use the Mac as LLM servers. I get very good t/s on Mac Studio M2 Ultra using llama.cpp, but something like vLLM on the Mac would be a game changer. I hope the dev team considers this.

I'm with @ibehnam , Mac studio would be great for vllm

+1

+1

+1

+1

+1

+1

Right, many developers are on Macs while testing out architectures. Yes we'll commonly deploy to linux, but if we can't quickly test and experiment locally on our daily use machines... we'll mostly like look elsewhere for an inference engine. Would be great if it could be supported, even if performance suffers on 'Apple Metal'.

+1

Does this issue still exist?

@evolu8 Other solutions such as llama.cpp, LLM-MLC, Apple MLX, etc.?

Does anyone know about any alternatives to VLLM until this is supported?

same here please +1

+1
pip install vllm failed on my mac m1 pro

+1On Feb 20, 2024, at 3:13 AM, Ather Shu @.***> wrote:
+1
pip install vllm failed on my mac m1 pro
image.png (view on web)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

+1

+1

+1

+1 I use a macbook and a mac studio. With the M3 architecture's unified memory these are very capable and cost effective systems for development and testing.

I'm willing to sponsor that effort because what we pay will be in savings in simply hardware on a mac.

+10000

+1 It would be very useful

+1 mac support will be very useful for experimentation

I was about to try it with Snowflake arctic AI on my mac M2 Max hosted in AWS and I couldn't :/

More reason to support Mac. Best in class $/token: https://x.com/awnihannun/status/1786069640948719956

it is not possible for vllm to support mac when triton https://github.com/openai/triton does not support mac.

It already supports it if you install it from the source code.
(vllm) ~/Projects/triton/python/dist git:[main]
ll
-rw-r--r--  1 <user>:<group>    97M May 29 12:03 triton-3.0.0-cp311-cp311-macosx_14_0_arm64.whl

I am also on darwin, when I build from source
git clone https://github.com/vllm-project/vllm.git
cd vllm
pip install -e .

I still get
 AssertionError: vLLM only supports Linux platform (including WSL).

can we support mac?

can we support mac?

+1

+1

+1

+1

+1

+1 While high throughput servers will be the main target on deployments the need for these in the debug experiment stage is highly needed

+1

+1

+1

+1

+1

Support for Mac M1, M2 ➕

+1

+1

Guys, please add a "like" in the main comment.

okay

+1

+1

+1

+1

Support for Mac M1, M2 ➕

I wish install vllm in m1 mac pro
how to install ?

If on Apple silicon, then have a look at Exo Platform (OSS), which supports MLX (using python) and allows inference with LLM sharding using multiple Macs @ https://github.com/exoplatform

Yeah, this is common, in my experience of running so many models locally, cuda and GPUs is way worse than a simple M1 pro.

+1
this is needed to test out basic models locally before deploying

+1

+1

+1

+1

+1
On Sat, Oct 5, 2024, 9:19 PM ExtraTon618 @.***> wrote:

+1
—
Reply to this email directly, view it on GitHub
https://github.com/vllm-project/vllm/issues/1441#issuecomment-2395287408,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALWGZH4M5ZOPLQLP4RIBZITZ2C24DAVCNFSM6AAAAAA6JX4GAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGI4DONBQHA
.
You are receiving this because you commented.Message ID:
@.***>

+1 would love to have vllm support for mac

+1

+1

+1
On Wed, Oct 30, 2024, 1:25 PM Nikita Bragin @.***>
wrote:

+1
—
Reply to this email directly, view it on GitHub
https://github.com/vllm-project/vllm/issues/1441#issuecomment-2448300692,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALWGZH4U2XBFQROZRWIKWJLZ6E6BZAVCNFSM6AAAAAA6JX4GAKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBYGMYDANRZGI
.
You are receiving this because you commented.Message ID:
@.***>

+10086

+10087

+10088

+1111111

I think https://github.com/vllm-project/vllm/pull/9228 should solve it now. You can build and run using macos.
That's not for performance though, mainly for developing.

+1

I think #9228 should solve it now. You can build and run using macos.
That's not for performance though, mainly for developing.

Compile failed on mac m3. There are a few problem I found:

hard limit to linux sysytem: https://github.com/vllm-project/vllm/blob/main/setup.py#L36
hard limit to use linux command "/proc/cpuinfo" https://github.com/vllm-project/vllm/blob/main/cmake/cpu_extension.cmake#L29

Let's make vLLM avaliable on mac CPU first.


PageStorage: Fix empty page cause TiFlash failed to start

What problem does this PR solve?
Issue Number: close https://github.com/pingcap/tiflash/issues/9282
Problem Summary:
By dumping the entries from PageStorage WAL, we can locate that two entries cause this issue:
[PageDirectoryFactory.cpp:225] ["{type:VAR_ENT, page_id:0x020102000000009CA4430A08, ori_id:0x.0, version:108969363.0, entry:PageEntry{file: 551, offset: 0x15376, size: 0, checksum: 0x0, tag: 0, field_offsets: [], checkpoint_info: invalid}, being_ref_count:1}"] [thread_id=1]
[PageDirectoryFactory.cpp:225] ["{type:PUT    , page_id:0x020102000000018204A17C03, ori_id:0x.0, version:127726467.0, entry:PageEntry{file: 551, offset: 0x15373, size: 15, checksum: 0x12D46C1C0D5B51FE, tag: 0, field_offsets: [], checkpoint_info: invalid}, being_ref_count:1}"] [thread_id=1]

The first entry is a page with data.size == 0, placed at blob_id=551, offset=0x15376. And later another page is placed at blob_id=551, offset=0x15373 with data.size == 15.
The first page split the free block into smaller pieces free blocks.
The second page will cover the place of the first page. And our code can not handle this situation with "empty page data".
What is changed and how it works?

For handling the existing empty pages, we won't let it split the free block into small pieces:

when STDMapSpaceMap::markUsedImpl accept a block of length == 0, it will directly return true instead of splitting the free block into smaller pieces.

For newly created empty pages, we store them at the front of BlobFile to avoid further issues:

BlobStats::BlobStat::getPosFromStat and STDMapSpaceMap::searchInsertOffset accepts a block of length == 0, it will return insert_offset == 0, which we can "store the empty pages" at the front of the BlobFile for newly created empty pages.

In BlobStore::read(...), we ensure that reading empty pages won't cause NPE or other unexpected exceptions

PageStorage: Fix empty page cause TiFlash failed to start

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Fix an issue that TiFlash write node may fail to restart under disaggregated arch

/retest

/retest


Audio files won't make MP4s

Hi Jalagar
I have a few issues with the Audio...I left detailed notes in my other issue that you closed, but not sure if you get alerts from closed issues. Wanted to make you aware...

Closing as duplicate, feel free to re-open the old one https://github.com/jalagar/animated-art-engine/issues/146


Some suggestions

I added a few more lines to check off.
Also reworded some of the sentences.

good suggestions!

thanks!


docs: :building_construction: function diagram for create_properties_template()

Description
These changes add the diagram for creating the properties template of the official Data Package Spec. I think it's possible to download the full spec from one URL. But maybe this will need two functions, one for the package and one for the resources. Thoughts? Alternatively, I could add a URL or options arg to select which spec to download, the package or the resource.
Reviewer Focus

This PR needs an in-depth review.

If this function will be used to update a locally saved / cached sample datapackage.json file (as opposed to being run every time that file viewed, for example), then I also think that it works as a single function. Because it prepares a properties file that includes package and resource properties, so any view... function can expose the part of the object it needs (without redownloading anything). But yeah, I'm also not sure if I understand the context 100%.

@martonvago related to the conversation during the meeting about frictionless and getting the official schema, I can't find any function within the package that does that:
https://framework.frictionlessdata.io/docs/framework/actions.html
The closest seems to be the Package class: https://framework.frictionlessdata.io/docs/framework/package.html but even then, it doesn't look like it provides the full template, only that it takes a data file and makes a package from it. Which isn't exactly what we want.

@martonvago related to the conversation during the meeting about frictionless and getting the official schema, I can't find any function within the package that does that: https://framework.frictionlessdata.io/docs/framework/actions.html
The closest seems to be the Package class: https://framework.frictionlessdata.io/docs/framework/package.html but even then, it doesn't look like it provides the full template, only that it takes a data file and makes a package from it. Which isn't exactly what we want.

Ah okay! Yeah, I was thinking of Package maybe being able to do it somehow, but then not really


Add Release Cheatsheet for Results

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[x] Tested your changes locally (if this is a code change)
[ ] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user-facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contain the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

@enarha

Generally the documentation is good. Following it, I was able to do the v0.7.0 release. There was an issue with storing the release.yaml and version folders in the correct paths, so I had to fix those manually. I'll look into this later to determine if it's something we need to fix in the script or in the documentation.

Does this release cheatsheet work fine?

Does this release cheatsheet work fine? @enarha

The Cheatsheet is good and we can merge that PR. There is an issue which is a bug in some of the release scripts used. I've created https://github.com/tektoncd/results/issues/595 to handle it.

/lgtm

@avinal Let's approve this. It might fix the go coverage test that's failing if this gets merged.

Yeah last release was successful, for now its good. We can iterate on this.
/approve
/lgtm


fix(nuxt): return types of useAsyncData when using with getCachedData

🔗 Linked issue

Fixes #25903

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Currently, the data type returned by useAsyncData is not inferred correctly when using it with the getCachedData option. This PR fixes the issue.

📝 Checklist

[x] I have linked an issue or discussion.
[x] I have added tests (if possible).
[ ] I have updated the documentation accordingly.

Looks like it breaks the types when using with transform option, I'm looking into it.

Looks like it breaks the types when using with transform option, I'm looking into it.

It should be fixed now.


fix: DOMException in scroll restoration

Issue
When trying to scroll to a hash, if the provided hash is not a valid selector it currently throws a DOMException.
I noticed this when viewing the blog https://remix.run/blog/remix-vs-next and clicking on a header with an emoji: "Remix ❤ the Web"  as it has a href of #remix-%EF%B8%8F-the-web
https://user-images.githubusercontent.com/1018141/150040187-5ada8897-e37b-4b84-bc18-476ec3cc5e0d.mov
Fix
I've attempted to fix this by wrapping the call to document.querySelector with a try/catch block to avoid the exception and log the error. If there is a different, preferred approach to resolve this let me know.

Has pr already #936

fixed in #936 thanks!


Add version check for CLI update

What does this PR do? It adds a version check feature to the "agenta" CLI, notifying users when a new version is available on PyPI.
This PR will close #148

@CodiumAI-Agent /describe

Preparing pr description...

@CodiumAI-Agent /review

Preparing review...

PR Analysis

🎯 Main theme: Adding a version check feature to the agenta CLI
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
✨ Focused PR: Yes, the PR has a clear and coherent title and description, and all PR code diff changes are properly derived from the title and description.
🔒 Security concerns: No, the PR does not introduce any obvious security concerns. It fetches data from a public API and does not involve any user input or sensitive data.

PR Feedback

💡 General PR suggestions: The PR is well-structured and the code changes are clear. However, it would be beneficial to add tests to ensure the new feature works as expected. Also, consider handling the scenario where the 'requests' or 'pkg_resources' imports fail due to these packages not being installed.

How to use

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option.
/describe: Modify the PR title and description based on the contents of the PR.
/improve: Suggest improvements to the code in the PR. These will be provided as pull request comments, ready to commit.
/ask <QUESTION>: Pose a question about the PR.

@CodiumAI-Agent /improve

Preparing review...


Convergence of the Grammar + Logic + Model

A Cicero template is composed of three things:

a template grammar, i.e. natural language text with placeholders
logic, written in Ergo
a data model, written in CML.

This triple is very powerful, it allows for

separation of concerns when designing templates, i.e. a lawyer can build a grammar without pollution by the implementation of the logic
guaranteed error-free execution for well-typed logic, https://docs.accordproject.org/docs/ergo.html#design-choices
complex user-defined types, see https://models.accordproject.org
re-use of individual template components across templates

However, for simple use cases and for those learning the protocol this increases the barrier to entry.
I propose that the model and logic for a template should be optional components. This gives the natural language primacy.  This will also allow the progressive automation of clauses through the following phases.

Text only (but identifiable, versioned and hashed as with all templates).
Text + model. Placeholders in the grammar have inline type annotations. Custom types are defined in a model file if necessary.
Text + model + logic. As in the current protocol specification. Notice that for execution, a model file is required to define the runtime models (e.g. Request / Response / Emit types). The runtime model does not natually fit within the grammar as the template model does.

In step 2 above, the TemplateModel class can be derived from the annotations in the grammar. There is no execution model so there may be no need for a separate model file.
I also propose that a subset of the Ergo language can be defined inline in the grammar.  For example, this would allow arithmetic expressions to be captured alongside the corresponding text.

Annex B. Acceptance Criteria
...
Shipments that have a temperature or humidity reading outside the agreed range have a price penalty applied calculated using the Formula for Breach Penalty Calculation below. The breach penalty factor to be used is [{breachPenaltyFactor: Double}].
Formula for Breach Penalty Calculation:
penalty = number of shipment units x difference between sensor reading and agreed range x breach penalty factor

[{ergo:
define function calculatePenalty(units: Double, breachDelta: Double}: Double {
  return units * breachDelta * clause.breachPenaltyFactor
}
}]

For contract templates  that include references to an external clause template in the grammar. The logic and model can be inferred from that relationship. This means that the model and logic do not also need to reference externally defined files.
Collectively, these proposals allow for a spectrum of template complexity. From simple templates defined only by a grammar file with inline type annotations for the template model, through to complex templates with re-useable modules of logic and custom-types.

I support this but it should be noted that the easier we make (2) the less likely people are to reuse models across clauses/contracts, which can create the "tower of Babel" problem. Is only suitable for primitive types?

I should make my comment clearer.

Custom types are defined in a model file if necessary

By "if necessary", I mean if custom types are necessary. Complex types should not be defined in-line. However an attribute of the template model may be annotated with the name of a complex type that is defined in the model-file.

Doesn’t this mean that type names are now spread across tem and cto files? Eg rename a type in a cto file and you have to potentially check all tem files for refs? What about meta properties like optional, range, arity and decorators for localisation?
It also means that the tem parser now creates a dynamic template model class in the model manager, so you will need a namespace from somewhere.

Yes, that's correct. Regarding renaming, we already have that problem between the model and logic files.
I guess we need to decide if the benefit of inline definitions in the grammar outweighs these compromises.

I would go on a limb and suggest that the following distinction might make some sense:

(Text + Clause model). Placeholders in the grammar have inline type annotations. Custom types are defined in a model file if necessary. Text only is an example of this where the Text has no parameters and the inferred model is trivially asset TemplateModel {}.
(Text + Clause model) + (Logic + Runtime Model). As in the current protocol specification. Notice that for execution, a model file is required to define the runtime models (e.g. Request / Response / Emit types). The runtime model does not natually fit within the grammar as the template model does.

@jeromesimeon note that it is not desirable to have the request/response types defined in Ergo as they have to be shared with the integration clients that are generating the JSON payloads.


"Something went wrong. Please try again" after IPS update

After updating to v4.1.14 users are unable to use this sign in method and are met with the "Something went wrong. Please try again" error.
No errors in system and error logs on ACP.

Can confirm this problem since the v4.1.14 update. Was working fine prior to update. Not seeing any errors aside from what the OP mentioned. Trying to connect the steam account on the Account Settings page just redirects back to the forums as if nothing happened.

I think it has something to do with the recent change in the API with \IPS\Http\Url (https://invisionpower.com/release-notes/4114-r49/). It only shows what was changed to contributors, however.
Doing some debugging, I believe the error is within the validate function, probably with the above mentioned  API change:
$response = (string)\IPS\Http\Url::external('https://steamcommunity.com/openid/login')->request()->post($params);

@johngalt good guess, but in the post referenced in that release note (in the contributor chat), the functions utilized in this plugin, are unaffected.

Rather than a single \IPS\Http\Url class, there are now 3 classes, each of which extends the previous:
\IPS\Http\Url, objects of which are external URLs
\IPS\Http\Url\Internal, objects of which are internal URLs
\IPS\Http\Url\Friendly, objects of which are front-end friendly URLs
You still create objects in the same way (so backwards compatibility is maintained) using ::internal(), ::external(), etc. or there is a new ::createFromString() method to just take a string.

so it's something else, but looking into it.

so far determined its authentication issue.   Soon™  (lol shouldn't take as long as valve though)

Appears that IPS released an update (<IP_ADDRESS>) and it was supposed to fix most 3rd party addons.

Version <IP_ADDRESS>: This is a small maintenance update to address issues some clients had with certain URLs. We have also seen clients having issues with many third-party plugins and this update will attempt to better capture those errors.

But on my site, I got no change with the plugin. everything in the plugin /should/ work  as nothing is depreciated/removed.

Updating to the latest version of IPS didn't change anything. Still no error or system logs.

Look's like OpenID does not validate... Causing the error.

Just uploaded the old Url class

oh you're talking about reverting the URL Class back to pre 4.1.14.x   ;;; hmm. i guess that's a hacky workaround, but still is a IPS issue that needs to be addressed.

@NightlyDev
Mind sending me the old Url class so I can do the same thing?

They released another update, but the issue seems to persist.

We can confirm the issue is still happening on the latest update of <IP_ADDRESS>

For anyone who wants to use NightlyDev's workaround until the dev is able to resolve this issue, here is an older version of Url.php.  Follow his instructions and you should be fine.  Make sure you don't overwrite the Url.php that is there, make a new file called Urlold.php
http://pastebin.com/YQ4ZTpLQ

Hello, I am trying to do this workaround. How exactly do i do this?

Hi All,
In the update IPS pushed out, it contained an updated query parser which broke some of the values in being sent by Steam. This change meant that the plugin was sending wrong values back over to steam's service and steam was unable to verify a valid request was made to login to the system, so it returned false.
A quick fix for this will be to update system\Http\Url.php find line 1064 and replace
$keyAndValue = explode( '=', $part );

with
$keyAndValue = explode( '=', $part, 2 );

---- Diff
-$keyAndValue = explode( '=', $part );
+$keyAndValue = explode( '=', $part, 2 );

Here's a link to the bug I've posted on the IPS Tracker.
https://invisionpower.com/4bugtrack/active-reports/url-query-string-parsing-is-too-aggressive-strips-extra-from-urls-breaks-openid-r12395/

Thanks for the update, Lavoaster.
I've made the changes you've suggested on line 1064 and I'm still experiencing the same result.
Do you have any other suggestions? Thank you.

Can confirm what @jimjim135 said.

Doesn't seem to fix the issue on PHP7

Penguin. Im on php7 and the edit lavo posted works fine.

Anyone who needs it can find the old url class here: https://gist.github.com/aStonedPenguin/1e454ee0b930e5f0cf34977dd47518a4

Upon doing lavo's fix, and reverting any changes made by previous "workarounds". it should work to login to the suite on the frontend.
also just a heads up distributing the oldclass (or any part of the suite) is against IPS's SOS anyway.

@jimjim135 same problem here. I can't sign in on the cp through steam :(

Can confirm that lavos fix in the URL class solves it. GG IPS Team :(

Updating to the newest beta version of 4.1.15 fixes the issue.

The fix was done one the other ticket. It was deemed ips issue. And set to
fix on 4.1.15

I'm mobile so I can't edit my last post lol. But the other ticket is referring to this one.
I'll add a thing to this ticket saying it was fixed by IPS.

Hey @madman32395 currently we're unable to upgrade our IPS. We've applied the hotfix that lavo mentioned which worked perfect for the website front-end.
Our issue now is the admincp. Still getting "Something went wrong. Please try again." when attempting to login with Steam via the AdminCP. Any ideas?
Happy to provide a phpinfo(); dump if needed.

I am still receiving this same error. I have reinstalled the plugin multiple times.
I am running IPB v4.1.15 on nginx 7.0.

Even with Friendly URLs and rewrites disabled, the issue is still occurring. I am running the latest versions of both IPB w/ unmodified Url.php and the plugin.

It works correctly on 4.1.15 on PHP7 and nginx for me, tunneling the request directly to PHP without rewriting the (friendly) url.

I have Nginx and PHP 5.6, with IPS 4.1.15, I have the same issue @Flumadiddle
The same error, when steam redirect to my forum, I see error message, and the account not linked.
"Something went wrong. Please try again"
What configuration have you @stormdr1ve ?

Using the old URL class fixes the issues you guys are having now.

are you guys using the version available here or the one from the IPS Marketplace? Try uploading the one from the marketplace and see if that helps.

I download and using the plugin from Marketplace  @madman32395

Anyone trying to use ACP authentication, can you download this and replace the file at system\Login\Steam.php

@madman32395
Looks like you were right! The nginx rewrites I had configured were not taking care of query strings correctly. I fixed them and the plugin is now working properly.
Old rewrite rules:
        location / {
            try_files  $uri $uri/ /index.php;
        }
        location ~^(/page/).*(.php)$ {
            try_files  $uri $uri/ /index.php;
        }

New rewrite rules:
        location / {
            try_files $uri $uri/ /index.php?$args;
        }

The updated Steam.php file worked. AdminCP is accessible now.
You're wonderful.

Thank you @Flumadiddle, I apply new rewrite rules, u posted here, and now works for me!


ContextMenu: hidden text

The text content of the context menu is not readable in the mobile version with the light theme.

Simple solution could be:
adding color: var(--text1); to https://github.com/ldbv-by/bav4/blob/709cc787ef61007e00981f2935673ace9c423d8d/src/modules/map/components/contextMenu/mapContextMenuContent.css#L1-L6


Service really down ?

Error right deployment to Bluemix:
The service may be down at the moment; please check here for the current status.
If the service is OK, the app may not be configured correctly, please check workspace id and credentials for typos.
If the service is running and the app is configured correctly, try refreshing the page and/or trying a different request.
Status of conversation service is green, though.

@ColbyJohnIBM Could you take a look at integrating better error handling.. Basically when we get an error from the service (https://github.com/watson-developer-cloud/car-dashboard/blob/master/app.js#L90) perhaps send a message back that the client can use.. 401 = The credentials you are using don't appear to be correct etc..

I didn't recognize that another conversation service was created, so I changed the workspace id. thx


refactor: add time zone and remove unused code

What does this pull request change?
Why is this pull request needed?
Issues related to this change

@soofstad I had to add a dependency to be able to convert the ISO to a local date object without the time zone. I know you have feelings about bundle-size, so please have a look :)


Add Flan-T5 notebook

What does this PR do?
Adds a notebook example for Flan-T5
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


fix: broken links in docs

@adierkens Can you please review?

@KetanReddy @hborawski Tried adding the new file with required documentation. Can you please check if it looks ok?

@hborawski Thanks for adding the hacktoberfest label. For the contribution to be counted, "hacktoberfest-accepted" label is needed instead. Can you please add that if the changes look ok to be merged? Thanks again!

@KetanReddy @hborawski Tried adding the new file with required documentation. Can you please check if it looks ok?

You're going to want to remove the references to the non-opensourced components that are referenced. Also it looks like there is a header without any text underneath it? Lastly, and this is something we should probably formalize in our docs, is to follow the language conventions that the rest of the docs use. Things like:

Not using "things" as its too generic
Refer to the project as "Player" (as a proper noun) not as a reference "the player"

@KetanReddy @hborawski Tried adding the new file with required documentation. Can you please check if it looks ok?

You're going to want to remove the references to the non-opensourced components that are referenced. Also it looks like there is a header without any text underneath it? This is something we should probably formalize in our docs, but ideally doc pages should follow the language conventions that the rest of the docs use. Things like:

Not using "things" as its too generic
Refer to the project as "Player" (as a proper noun) not as a reference "the player"

@KetanReddy I didn't write this content but took it from the link given in the issue: https://ibp-cicd-core.prod1-ibp.a.intuit.com/player/player/latest/index.html#/core/architecture/cross-platform-players as it is accessible now. I am not too sure what the non-opensourced components are, can you help me understand wha changes need to be made? I can change "the player" to "Player" if you prefer that, but unclear about other things as I copy pasted it from the original source and also used the same header format as in other doc files in this repo.

@KetanReddy Tried changing the content as per your suggestions. Please let me know if any additional changes are required as I am not too familiar with the functionality of the plugin and required content for this file.

@KetanReddy Done the suggested changes. Can you please check?

@KetanReddy @hborawski Can you please review and merge if this looks ok?


Web GUI not working on Internet Explorer

The GUI doesn't work on Windows 10, Internet Explorer

It does work on Microsoft Edge.
It loads the initial state, but never proceeds to the "start a cluster" dialogue. If the cluster was started with a different browser, the "Browse" button has no effect.
Ideally, the GUI should work on any modern widely used browser. If support for Internet Explorer is not possible, it should at least display a warning to the user.

IE is broken, who would have thought? :grin: Sadly, It's still supported for quite some time, IIRC IE10 support runs out at the beginning of 2020...
In all seriousness, I guess I'm missing some polyfill, or just hitting some edge case... should be fixable in any case.

I investigated this a bit more in detail. There were indeed some missing polyfills etc. - unfortunately, this is not all that is required. SVG support in IE11 doesn't appear to be sufficient; focusing an SVG element using javascript is a pain, and only possible using an ugly hack; also, proper sizing of inline SVGs is not possible without some more hacks. Currently, after fixing the missing polyfill issues etc. you get something like this:

Then, having the wrong size messes up the coordinate calculations. A possible fix is the "padding hack", which unfortunately comes with its own problems.
I fixed the missing polyfill stuff, but I'll postpone further work on this for now; maybe there is a better solution for this I haven't thought about yet. Sadly, feature detecting proper inline SVG sizing could also be quite complex.
As a note; for any old Windows XP machines, I'd propose installing Firefox 52 ESR, if they need to run the LiberTEM GUI.


Please add devices we need

[x] mtk: @freedomtan
[x] samsung: @Mostelk
[ ] Qualcomm: @mohitmundhragithub

Mediatek

device name
SoC
notes

Xiaomi Redmi 10X 5G
Dimensity 820
0.7

Vivo S9 5G
Dimensity 1100
1.0

Oppo Find X5 Pro
Dimensity 9000
2.0

Vivo X90 Pro
Dimensity 9200
3.0

Samsung from @Mostelk

device name
SoC
notes

Samsung Galaxy Note 20 ULTRA; SM-N985F
Exynos 990
(v0.7)

Samsung Galaxy S21+ 5G SM-G996N
Exynos 2100
(v1.0)

Samsung Galaxy S22+ 5G SM-S906B
Exynos 2200
(v2.0)

Samsung Galaxy S24+ 5G
Exynos 2400
V4.0?

device name | SoC | notes
-- | -- | --
Xiaomi 13 pro | SD 8Gen 2 | V3.0
Redmi Note 12 turbo | SD 7+ Gen 2 | V3.1
Asus ROG Phone 6 | SD 8+ Gen 1 | V2.1
Oppo Reno 8 pro (china variant) | SD 7 Gen 1 | V2.1
Xiaomi Mi 12 | SD 8 Gen 1 | V2.0
Xiaomi Mix 4 | SD 888+ 5G | V1.1

@mohitmundhragithub decision was to limit to 6 devices per org at any time, please chose accordingly, taking into account V4.0 submission.

@mohitmundhragithub decision was to limit to 6 devices per org at any time, please chose accordingly, taking into account V4.0 submission.

sure. Removed the last 1.
We don't know the device to be used for V4.0 submission yet.

Samsung developer site has almost all the Samsung phones, https://developer.samsung.com/remotetestlab/docs/1/about-remote-test-lab/, https://developer.samsung.com/remote-test-lab
Can we do it automatically? I mean without integrated with our current github based CI/CD.
@anhappdev please help check it.

After some testing, here is what I found out about Samsung Remote Test Lab:

We can either use the web interface or Android Studio to interact manually with a physical device (similar to how we interact with a simulator)
I did not find a CLI program or an automated way to launch a device and run tests on it. Either using the web interface or Android Studio, we always need to open their website to start a session / launch a device.
They have devices with the Exynos SoC. I can manually run the app with Samsung backend on the Samsung Galaxy S24 Ultra (Korea location).


Empty switch: extraneous blank line

Input:
switch(1){}

Output:
switch (1) {

}

Expected:
switch (1) {
}

(just like empty if)
Do we care? ¯\_(ツ)_/¯
https://jlongster.github.io/prettier/#{"content"%3A"switch(1){}"%2C"options"%3A{"printWidth"%3A80%2C"tabWidth"%3A2%2C"singleQuote"%3Afalse%2C"trailingComma"%3Afalse%2C"bracketSpacing"%3Atrue%2C"doc"%3Afalse}}

This was found using my fuzzer in #371

Yeah, we should fix that! Thanks


Viewportgrid changes

explicitly describe axes hiding logic option

since there's 3 states - no axes, 1 hidden, 0 hidden - I used an enum which avoids allocations

extract view matrix inversion (which is a bit costly) out of the loop
remove gridIndex to simplify indexing in the loop (used axisIndex instead and the result is indentical)
moved axes color change out of loop
removed axes rotation - maybe there's something I'm not seeing, but AFAICT the initial rotation should be enough

I removed it because the logic wasn't working with more than 2 axes - one of them wasn't being rotated

Gentle ping on this - what do you think?

@garychia

Sorry. I did not notice this. It looks great.


[Bug]: Bad alignment in discussion page

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
There is bad alignment of discussion page send btn lie in footer.
Expected behavior
When we render discussion page then it will have some padding from bottom
Add screenshots (if applicable)

On which device or platform are you experiencing this bug?
No response
Confirmation

[X] I have read the Bitbox Contributing Guidelines
[X] I am a GSSOC-EXT contributor
[X] I am a HACKTOBERFEST contributor
[X] I have starred the Bitbox repository

@Anuj3553 Please assign the label:)

Hyy @harshbhar0629, I'm assigning this issue to you.

Can you give the star to the repo?

Already done:)

Already done:)

Thankyou bro


How can I build a sql as 'select xxx for update'

How can I build a sql as 'select xxx for update'

The library doesn't support this currently, but you could implement it yourself using a custom model. There's an example of doing something similar here: https://github.com/mybatis/mybatis-dynamic-sql/tree/master/src/test/java/examples/paging
I'd like to make this easier in the future, but for now this is the best way.

thanks for yours help!
------------------ 原始邮件 ------------------
发件人: "Jeff<EMAIL_ADDRESS>发送时间: 2020年4月27日(星期一) 凌晨5:37
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [mybatis/mybatis-dynamic-sql] How can I build a sql as 'select xxx for update' (#194)
The library doesn't support this currently, but you could implement it yourself using a custom model. There's an example of doing something similar here: https://github.com/mybatis/mybatis-dynamic-sql/tree/master/src/test/java/examples/paging
I'd like to make this easier in the future, but for now this is the best way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


[MRESOLVER-570] Remove excessive strictness of OSGi dependency metadata

Mark optional Maven dependencies as optional in OSGi metadata
It is an explicit decision from bndtools to require explicit configuration of a user to make a dependency optional, even if the corresponding maven dependency is optional: https://github.com/bndtools/bnd/issues/2713

Widen version range for org.slf4j.spi package to [1.7,3)

Move bnd configuration section from execution to plugin configuration in order to simplify extensions in sub-modules.

Update bnd-maven-plugin from version 6.3.1 to 6.4.0.
I didn't went up to 7.0.0 since that requires Java-17 to run and I saw that you already updated on the master for maven-resolver 2.x.

Fixes MRESOLVER-570

please mvn spotless:apply and commit changes (formatting is enforced at build)

@HannesWell I wonder if the intra-project imports should require a lower range, i.e. [1.9,1.10).  The reason is that there's no real guarantee on the project's internals, so if someone starts mixing minor versions, it will probably fail.

@HannesWell I wonder if the intra-project imports should require a lower range, i.e. [1.9,1.10). The reason is that there's no real guarantee on the project's internals, so if someone starts mixing minor versions, it will probably fail.

Ah, just spotted that you came up to the exact same conclusion with #507

@HannesWell I wonder if the intra-project imports should require a lower range, i.e. [1.9,1.10). The reason is that there's no real guarantee on the project's internals, so if someone starts mixing minor versions, it will probably fail.

Ah, just spotted that you came up to the exact same conclusion with #507

Yes.
Thanks for your review. I created a separate PR to just unify the build configuration regarding the bnd-maven-plugin and maven-jar-plugin via https://github.com/apache/maven-resolver/pull/509. If that is submitted before this one the noise due to moving things around could be reduced.

This is now based on the changes made in https://github.com/apache/maven-resolver/pull/518 since it allows to implement this a bit more elegant.

@gnodet can we proceed with this one? Thanks in advance.


inconsistent input preprocessing in PyTorch demo

Dear author,
Thank you for the contribution of this work. I'm trying to use the pre-trained network as a feature extractor. To best utilize the pre-trained weights, I must figure out how input images are pre-processed during pre-training and follow it exactly. However, I find two different pre-processing approaches in this repository.
The first one is found in the Tensorflow training code. The data will firstly be rescaled to [0, 1]. Then in preprocess_input function, since the default mode is caffe, it will reorder the channels from "RGB" to "BGR", and subtract the mean value of ImageNet (doc).
https://github.com/BMEII-AI/RadImageNet/blob/0ce16f7375db4236e646829d1eca61cdb4282133/breast/breast_train.py#L144-L153
The second one (in the PyTorch Demo) rescales the input to [-1, 1], which is different from the first one and will result in different input distribution and output of the pre-trained network.
class createDataset(Dataset):
    def __init__(self, dataframe, transform=None):
        self.dataframe = dataframe
        self.transform = transforms.Compose([transforms.ToTensor()])

    def __len__(self):
        return self.dataframe.shape[0]

    def __getitem__(self, index):
        image = self.dataframe.iloc[index]["img_dir"]
        image = cv2.imread(image)
        image = (image-127.5)*2 / 255
        image = cv2.resize(image,(224,224))
        #image = np.transpose(image,(2,0,1))
        if self.transform is not None:
            image = self.transform(image)
        label = self.dataframe.iloc[index]["label"]
        return {"image": image , "label": torch.tensor(label, dtype=torch.long)}

It will be much appreciated if there can be clarification on this issue. Thanks!

I'm also confused by it. I checked ImageDataGenerator code,  https://github.com/keras-team/keras/blob/601488fd4c1468ae7872e132e0f1c9843df54182/keras/preprocessing/image.py#L1849-L1852. What it did inside is to first use preprocess_input to zero-center the images, then use rescale=1./255 to rescale. So during training, the images will be normalized to around [-0.5,0.5]???

Are there any updates regarding this matter?


Bug 2077974: Prevent reconcile error when policy status is missing

Handles the error when the policy status is missing and reconcile in 30 seconds
Signed-off-by: Saeid Askari<EMAIL_ADDRESS>

@jc-rh Can you take a look at this?

/lgtm

/hold


[Enhancement] Hide unused accounts and future expences

I just added a transaction 1 week in the future, and in the transaction is shows as a "normal" transaction. Think it would be better to either hide it or dim it down (like 50% opacity or something?) for more clarity that it's a transaction now yet made. Maybe add this as an option for those who want to see or hide it?
Also I noticed a couple of old, closed, accounts is showing in the "Assets Accounts". I think these should be hidden by default, because they are not in use anymore. Maybe another option for those who still wanna see them? :)
Is there a way to contribute with translation? I'm Swedish, so I could translate the app to my language.

@dreautall They are indeed greyed out, but the are not in use anymore so a better option would be to hide them altogether IMO :)


Chad 8532 port additional fingerprints of the zigbee thermostat dth for the d 5 x 84 yu to lua driver

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

Original DTH: https://github.com/SmartThingsCommunity/SmartThingsPublic/blob/master/devicetypes/smartthings/zigbee-thermostat.src/zigbee-thermostat.groovy


Use a list of oemols for PointMutationExecutor input instead of files

This PR adapts the input of PointMutationExecutor to allow for a list of oemol objects to be used instead. Both @schallerdavid and I were having difficulty parsing ligand .sdf files containing a nitrogen that was perceived as needing to be tetrahedral at this line in perses.utils.openeye. This PR allows the user to bypass internal oemol preparation.
Changed:

PointMutationExecutor now takes inputs: .sdf, .pdb, [oemol].
ligand_file renamed to ligand_input to be more general.

An example script of how to prepare the [oemol] object is below, input names changed from my use case:
from perses.app.relative_point_mutation_setup import PointMutationExecutor
from perses.utils.openeye import generate_unique_atom_names
import oechem
from simtk import unit

ifs = oechem.oemolistream()
ifs.open('./ligand.sdf')
mol_list = [oechem.OEMol(mol) for mol in ifs.GetOEMols()]
mol = mol_list[0]

if len([atom.GetName() for atom in mol.GetAtoms()]) > len(set([atom.GetName() for atom in mol.GetAtoms()])):
        mol = generate_unique_atom_names(mol)
oechem.OEAssignAromaticFlags(mol, oechem.OEAroModelOpenEye)
oechem.OEAssignHybridization(mol)
oechem.OEAddExplicitHydrogens(mol)
oechem.OEPerceiveChiral(mol)

mol_list = [mol]
protein_file = './protein.pdb'

x = PointMutationExecutor(
protein_file,
'1',
'510', # an example residue number
'VAL', # an example mutation
ligand_input=mol_list,
ionic_strength=0.15 * unit.molar,
small_molecule_forcefields='openff-1.2.0'
)

@glass-w I noticed your script above that generates the oemol does not include the functionality covered by these lines in createOEMolFromSDF() ... Did you get any errors when running the above script? if not, it may be fine to not include the lines I linked (as long as when you inspect the old and new pdbs from the htf, the small molecule geometries look ok). but it is probably best practice to incorporate the lines i linked above in your script for generating the oemol (you'll just have to find a workaround that checks that carbons have tetrahedral stereochemistry and nitrogens are pyramidal)

This PR adapts the input of PointMutationExecutor to allow for a list of oemol objects to be used instead.

Instead of OEMols, can we use openforcefield.topology.Molecule objects?
These will be much more portable in the future.
Internally, we can use molecule.to_oemol() whenever we need an OEMol().

This PR adapts the input of PointMutationExecutor to allow for a list of oemol objects to be used instead.

Instead of OEMols, can we use openforcefield.topology.Molecule objects?
These will be much more portable in the future.
Internally, we can use molecule.to_oemol() whenever we need an OEMol().

is there anything lossy between oemols and OFF molecule objects? if not, then this is perfectly fine and can be added here.
there probably should be a test written somewhere here.

is there anything lossy between oemols and OFF molecule objects?

I believe some non-chemical information, such as SD tags, would be lost. But atom ordering and geometry should be preserved.
Is there any particular information you're worried about being lost, @dominicrufa? If so, we can check (or you can just ask on the OFF slack).

@zhang-ivy

Did you get any errors when running the above script?

I didn't get any errors when running the above script and the PointMutationExecutor successfully generated a series of htfs.

as long as when you inspect the old and new pdbs from the htf, the small molecule geometries look ok

Yes the geometries looked fine when visualising.

you'll just have to find a workaround that checks that carbons have tetrahedral stereochemistry and nitrogens are pyramidal

Yes this is a key issue when using OE3DToInternalStereo. It seems that for a subset of these ligands running OE3DToInternalStereo considers a nitrogen to be a stereocenter so then running has_undefined_stereocenters returns True. At the moment there doesn't seem to be a workaround.
I agree that if a user were to prepare their own OEMol object for parsing to PointMutationExecutor then they should apply both checks.

@jchodera

Instead of OEMols, can we use openforcefield.topology.Molecule objects?

Yes I think this would be an idea for the future. Perhaps we can raise this as an issue and generate a separate PR? Right now this change is needed for functionality with this set of ligands.

@glass-w have you re-run your example script above after making the changes today? just want to make sure that changing elif isinstance(ligand_input, oechem.OEMol):  doesn't break anything

@zhang-ivy yes I've re-run in a batch job using this functionality:
from perses.app.relative_point_mutation_setup import PointMutationExecutor
from perses.utils.openeye import generate_unique_atom_names
import oechem
from simtk import unit

ifs = oechem.oemolistream()
ifs.open('../ligand.sdf')
mol_list = [oechem.OEMol(mol) for mol in ifs.GetOEMols()]
mol = mol_list[0]

if len([atom.GetName() for atom in mol.GetAtoms()]) > len(set([atom.GetName() for atom in mol.GetAtoms()])):
        mol = generate_unique_atom_names(mol)
oechem.OEAssignAromaticFlags(mol, oechem.OEAroModelOpenEye)
oechem.OEAssignHybridization(mol)
oechem.OEAddExplicitHydrogens(mol)
oechem.OEPerceiveChiral(mol)

protein_file = '../protein.pdb'

x = PointMutationExecutor(
protein_file,
'1',
'510', # an example residue number
'VAL', # an example mutation
ligand_input=mol,
ionic_strength=0.15 * unit.molar,
small_molecule_forcefields='openff-1.2.0'
)

it runs and produces all of the relvant output files. Again the above script is for my use case but for general OEMol preparation the user should use the lines you mentioned in your earlier comment.

is there anything lossy between oemols and OFF molecule objects?

I believe some non-chemical information, such as SD tags, would be lost. But atom ordering and geometry should be preserved.
Is there any particular information you're worried about being lost, @dominicrufa? If so, we can check (or you can just ask on the OFF slack).

i think this is sufficient


Fix occupation computation of non-singlet states

Prior to this fix, the ParticleNumber property would incorrectly
compute the alpha and beta occupations:
    prop = ParticleNumber(4, (2, 1), [2.0, 1.0])
    # >>> [1.0, 0.5]
    print(prop.occupation_alpha)
    # >>> [1.0, 0.5]
    print(prop.occupation_beta)

After this fix, the computation is corrected to return:
    prop = ParticleNumber(4, (2, 1), [2.0, 1.0])
    # >>> [1.0, 1.0]
    print(prop.occupation_alpha)
    # >>> [1.0, 0.0]
    print(prop.occupation_beta)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.008%) to 85.368%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
10105

Relevant Lines:
11837

💛  - Coveralls


Add lock around graphql requests.

Similar to existing requester. Prevents double pagination.

          1 Warning

:warning:
Please provide a linked tracker story for this PR in the description

  Generated by :no_entry_sign: Danger


Sys version fix

Python sys.version may return
"3.8.10 (default, Jun  4 2021, 15:09:15)
[GCC 7.5.0]" (with /n instead of one line)
so it breaks generation script to work. At least for me.
Also I added Poetry project for better dependency control and gitignore for Runs.

Thank a lot for your contribution!


Check running on linux from the beginning

since we do not build cnt for other platform, this is not needed anymore


[asgardeo saml sdk] Assign system property javax.xml.parsers.DocumentBuilderFactory from the code affect other applications in same JVM

Describe the issue:

According to the current implementation of asgardeo-java-saml-sdk, 'javax.xml.parsers.DocumentBuilderFactory' system property assign from the code base. Therefore, other application which has different implementation other than   'org.apache.xerces.jaxp.DocumentBuilderFactoryImpl' will fail because of this
How to reproduce:

Check with some other implementation which has xml parser
Expected behavior:

All application should work independently in same JVM.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version:  IS 5.10.0
OS: Windows, Linux

Optional Fields
Related issues:

Suggested labels:
WSO2 IS, asgardeo

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!


Script opens, displays some text, then immediately closes

The app opens, and appears to be able to log into my accounts, but then it immediately closes without doing anything. I have double checked my chrome drivers and have installed all requirements.

The image shows how far it gets to, then it closes. I can't figure out why it's closing.
I haven't tested python version because I don't know what version I'd have to install, so maybe that's the issue?

Update: it appears to flash some line of text as well, but it's far too fast to be able to read.

Issue resolved: I had an outdated version of my chrome driver. I used this script to try and update the chrome driver, but I didn't update it properly. I had to manually update the driver.


[Feature] Implement S3/HDFS Integrations for CHYT in Ytsaurus

Clickhouse offers valuable integrations like S3, HDFS, and others. Adding support for these integrations in CHYT within Ytsaurus would greatly enhance convenience and functionality.

Any updates here?

Yes, you are correct. We need the implementation of S3 and HDFS table functions in CHYT within Ytsaurus, similar to how it's executed in ClickHouse. Specifically, the feature should allow us to import and export data from and to S3/HDFS, using a syntax akin to this ClickHouse example:
INSERT INTO FUNCTION s3('https://clickhouse-public-datasets.s3.amazonaws.com/my-test-bucket-768/test-data.csv.gz', 'CSV', 'name String, value UInt32', 'gzip')
SELECT name, value FROM existing_table;

I am not certain if exporting data to HDFS using ClickHouse's HDFS table function is possible.
However, using the HDFS table engine, it can be achieved. Therefore, it might be more preferable to use the HDFS table engine instead of the table functions for exporting data to HDFS.

Status update: S3 functions have been supported in CHYT for a few months now. HDFS functions are in the backlog and we do not have plans to support them in the near future.


Support multiline string in Swift

Adds support for multiline strings and refactors a bit of the shared lexing rules with single-line string.

Hi @Anteru, curious if there's a timeframe we might be able to expect this PR to reviewed/merged? We might be interested in bumping pygments to a pre-release version if it's merged prior to 2.18.0 release. 🙏

We're already past due the planned release time. I hope to get to it this month, but stuff is coming up :(

Sounds good, thank you for taking the time! Whenever this is merged I think we'll just look to point pygments at HEAD and not wait for the official release :)


Default get_cmd_states.py range now includes end of available data

If --stop is not defined, the code now puts no end limit on the
cmd_states fetch.  Previously the default --stop was Now, which was
not idea for actual use cases.

I presume you've tested the four possibilities for (start, stop) x (explicit, implicit) and gotten the expected behavior?

I did, though I forgot to check to see if there were any tests already in
in the project that use get_cmd_states.  I also don't know if there is any
code "out there" that I should check for compatibility.


Migrate to null safety

ITS-PRO: MA-95
This is done when:

[x] Explore options for achieving null safety on package [liquid_progress_indicator] (https://pub.dev/packages/liquid_progress_indicator/versions) which is not actively maintained

https://github.com/UCSD/liquid_progress_indicator

[ ] Test and upgrade to background_fetch: 1.0.0-nullsafety.2 (@lukedutta9)
[ ] Test and upgrade to beacon_broadcast: 0.3.0-nullsafety.0 (@lukedutta9)
[ ] Flutter 2 migrated to null safety (@lukedutta9)

Tested https://github.com/UCSD/liquid_progress_indicator with Campus Mobile Speed Test, continues to work as usual. 'dart migrate' verifies compatible with null-safety.


Saver- Chrome Extension to load all tabs of a sessions.

Describe the project you want to add with tech stack
Saver allows you to save Chrome tabs as workspaces so that you can declutter your Chrome, restore your tabs at any time and save valuable memory in just 2 clicks.
I have used those tech stacks to create this...

HTML
CSS
JS (mainly)

@pranjay-poddar please assign me in this task


Upgrade TypeScript and Babel deps

The CI failure is weird:

it doesn't happen to me on my machine
optional chaining should be enabled by default (see here)

Looks like the dependencies have been upgraded to higher versions now on main. Should we close this?


Change margin split from % to ‰.

The protocol only supports 0-100.

done: https://github.com/Loopring/looper/pull/75


Add lookup function to Map

It would be convenient to have a method on Map that makes a lookup and runs a callback function if the value doesn't exist yet (and returns it):
var map = { "foo": 499 };
map["foo"]; // => 499
map["bar"];  // => null
map.lookup("bar", () => 32);  // => 32

This would potentially also be useful during the migration to non-nullable types. We could then make the index operator throw if the entry is not in the map.

Note that this is also competing with the existing idioms
var value = map[key] ??= createDefaultValue();  // inserted
var value = map[key] ?? createDefaultValue();   // not inserted

which, unlike putIfAbsent and lookup, is allocation-free in the not-absent case (no closure).

Good point, Stephen. Unfortunately, the existing idiom doesn't handle the case where null is desired and stored in the map. Often, it is useful to avoid null, but not always.

Closing for now.
We haven't made a final decision yet, but this method would/will be revisited with non-nullable types (and their changes to the core libraries). No need to keep this bug open.

Should this issue be reopened now that nnbd work is further along? Or is it being tracked internally/elsewhere?
I'm also thinking of a use-case where null is a valid value in the map, but if it isn't present I'd like to default it to something different (similar to the notion of undefined in JavaScript) when assigning a nullable variable.


Add workspace checksum to avoid unnecessary terraform init execution

Description of your changes
Maintain a checksum of the workspace directory content and skip running terraform init if the checksum value has not changed since the last reconciliation.
The workspace content is regenerated on every reconciliation so that any changes to the inline or remote workspace, anything in the workspace.spec, the ProviderConfig credentials or configuration will be reflected in the new workspace content and the checksum will be changed accordingly.  If the checksum changes then terraform init is re-executed to pick up the new changes.  But if nothing has changed then terraform init is redundant and just wastes (a lot of) CPU, and so it can be skipped.
The checksum is stored in the Workspace Status, which does not get updated until after the SECOND reconciliation cycle after creation (status updates on Create are overwritten) so terraform init will be executed for the first TWO reconciliations but after that it will only be executed on changes to the content of the workspace/configuration/credentials/etc.
Fixes #81
I have:

[X] Run make reviewable to ensure this PR is ready for review.

How has this code been tested
Tested in a development environment running a variety of Workspaces, and verified through logging and timestamp analysis that terraform init is only called when expected.

Also note that reducing the number of terraform init calls is especially important now that we are using the shared provider cache, since terraform only allows one terraform init instance to access the cache at a time and we enforce this behavior with a semaphore to prevent crashes in the CLI.  Reconciliation throughput can be increased significantly if we don't require all reconciliations to go through this bottleneck on every cycle.

Thanks @ytsarev - I think we can start with this implementation and refine as needed.  Thanks!


Server not starting

robby@robby-U56E:~/TwitchPlaysX$ npm start

><EMAIL_ADDRESS>start /home/robby/TwitchPlaysX
> node ./app/server

sh: 1: node: not found

npm ERR! Linux 4.10.0-28-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>start: `node ./app/server`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node ./app/server'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the TwitchPlaysX package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./app/server
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs TwitchPlaysX
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls TwitchPlaysX
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/robby/TwitchPlaysX/npm-debug.log
robby@robby-U56E:~/TwitchPlaysX$

Updating node does not help.

I know about issue #6

If you are getting node not found then it's not on your PATH for some reason? Make sure you can run node in the console/command line?


Sanitize the rendering of an unescaped parameter

Pretty paranoid but just in case there is something potentially dangerous in the user's filename.

Doesn't this happen regardless?


Add light mode to docs

Describe the feature
Hi,
Dark theme burns my eyes so I can't read the documentation (unless reader mode which isn't really practical in this case - smaller code blocks, no syntax coloration...), it's a small detail but it makes it really hard to learn houdini (which look awesome by the way).
Usually projects docs are either light or light+dark but almost never just dark (I suppose because of this).
Thank you!
Criticality
No response

Oh no! Sorry to hear that dark mode is so painful for you.
It might take a bit but we'll work on putting a light mode together 👍

Hi I will try to send an implementation in the next few weeks!


Update ClipboardEventInit to match W3C spec

Update ClipboardEventInit interface to match the W3C spec.
See Microsoft/TypeScript#28998 for additional information.

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

TS now prefers directly fetching W3C standard rather than manually maintaining types.
I recommend:

Add the following item to the inputfiles/idlSources.json.
{
  "url": "https://www.w3.org/TR/clipboard-apis/",
  "title": "Clipboard"
}

Remove ClipboardEventInit from addedType.json and also ClipboardEvent from overridingTypes.json.

Run npm run fetch -- Clipboard

Commit the result

@saschanaz Thank you for the guidance. That approach does sound better. I have applied those changes.


Navbar and join us button in about page is not working

I am gssoc contributor, and I want to work on this issue.

sorry we don't need this section.


Added an Error type

This makes it easier for people who are using the try!() macro to convert errors into their own Error type. It isn't possible to diffrentiate between errors raised from RocksDB and other string errors at the moment.
This adds a simple Error type that wraps String. People using RocksDB can now implement impl From<rocksdb::Error> for MyError and add custom behaviour for handling RocksDB errors.
I've implemented From<rocksdb:Error> for String and added a to_string method to the Error type itself to make it easier for those relying on strings to upgrade.

:+1:  Thanks!


pip install "bocoel[all]" issue

WARNING: bocoel 0.0.0 does not provide the extra 'all'

Hmm.... The newest version is 0.1.0. Let me check what the problem is.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
ibsaaqa11, @mattheworris, @drbh, @durango, @EdwinGuo, @ErKiran, @imranismail, @hhstore, @jamshidi799, @maximebrugel, @lirc571, @reconstructive, @ziogaschr, @Aren49, @jiangplus, @ejmahler, @6, @craigjson, @LindenWang01, @bakkdoor

@pavana21
Scammer

Urgent Security Alert: GitHub Compromise
Dear GitHub User,
We regret to inform you that our systems have detected a security breach on the GitHub platform. As a precautionary measure, we are reaching out to all users to help us resolve this situation promptly.
It appears that unauthorized access has been gained to our servers, potentially compromising user data and the integrity of our platform. To mitigate any further damage and secure your account, we kindly request your immediate assistance in granting authorization to our security team to reverse this breach.
Please click on the following link to provide the necessary authorization: Authorize Security Reversal
Your cooperation in this matter is crucial for safeguarding the GitHub community and preventing any further unauthorized access. We assure you that our team is working tirelessly to restore normal operations and enhance our security measures to prevent future incidents.
Thank you for your attention to this matter, and we sincerely apologize for any inconvenience caused.
Best regards,
GitHub Security Team
skellet0r, @maazullah96, @udit-gulati, @Dylan-Burns, @Agu-Ribo, @m4n3z40, @developerfred, @vroomzel, @ethull, @MojtabaMonfared, @alexjorgef, @nullbutt, @GastonGuan, @JordyKingz, @imoric, @fernandoe321, @Pebaz, @vvonchain, @ytfei, @wancaibida

Exciting Job Opportunity at Microsoft!
Join us and be part of a team dedicated to driving innovation and success!
How to Apply:
To apply for this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.nining, @MacroChip, @anthonyoliai, @raho1, @dankostiuk, @fungilation, @mgladson, @3mg, @Otherbright, @ravachol70, @petermiller310, @ahmet, @andreisilviudragnea, @BitBaseBit, @DPS0340, @gat786, @BlankerL, @shunkakinoki, @Revolution1, @goodlucks7788


Finish 0-30 Tutorial Content

There is now a skeleton of the 0-30 tutorial in the files z230-pt1.md and z230-pt2.md.
There is still missing content in here, though, and someone needs to fill in the blanks.
That is currently missing includes:

[x] Numpy coverage (Should this be here?)
[x] Pandas coverage (Should this be here?)
[x] Matplotlib + Cartopy  (Should this be here?)
[ ] GitHub (last steps should be push up to GitHub?)
[ ] Git conflicts, reverting git commits, etc.
[x] Jupyter Notebooks & Lab
[x] netCDF4-python?

Also need to generate (maybe) slides for "static" material presented during the tutorial.

Decision:  Using Jupyter Notebooks, we will cover Numpy, netcdf4, Pandas, Matplotlib+Cartopy on Day 2 (not as part of the 0-30 content)

Git conflicts:
Have them create branches of their work - usually copy paste file, but best practice for entertain 2 different train of thoughts is a separate branch - do old US way and new US way as 2 different wind chill formulae to motivate this.  Trying to emulate this can make it seem simpler than it is, maybe this is too advanced for day 1? Just show simple example, or link them to a better resource.
Community models are a great use case for branches and forks. Need to understand branches to contribute to CESM or other community models.

Let people follow curiculae https://github.com/NCAR/ncar-python-tutorial/blob/master/curricula/z230-pt1.md so that they can reference if if they get stumbled on one step. Maybe make an image  of code blocks so that people can't copy paste but can look at their leisure (and KEEP UP), maybe these images also go on powerpoint slides.
Reinforce idea of not mixing tabs and spaces!

Maybe name directory, spring2020tutorial instead of mysci

Maybe we want to show open and close file before using 'with' context manager.

Have them look at the header at some point

step 39, windchill = formula, return windchill. Separation of concerns (compute and return on separate lines to reduce confusion)

Suggestion for red, green, yellow post it notes to indicate needing help, being ready to move on, still working.

Make sure computation functions are written in a way that is easy to read (assign constants to variable before doing arithmetic on them, for ex)

Are pycache and gitignore high priority information?

Need examples on usefulness of init.py


no work to IOS simulator

Launching lib/main_dev.dart on iPhone 12 in debug mode...
lib/main_dev.dart:1
Xcode build done.                                            1,3s
Failed to build iOS app
Error output from Xcode build:
↳
objc[99216]: Class AMSupportURLConnectionDelegate is implemented in both /usr/lib/libauthinstall.dylib (0x1f3f09160) and /System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x114b142b8). One of the two will be used. Which one is undefined.
objc[99216]: Class AMSupportURLSession is implemented in both /usr/lib/libauthinstall.dylib (0x1f3f091b0) and /System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x114b14308). One of the two will be used. Which one is undefined.
** BUILD FAILED **
Xcode's output:
↳
ld: in /Users/idontsudo/.pub-cache/hosted/pub.dartlang.org/flutter_openim_sdk-0.0.9+1/ios/Framework/OpenIMCore.framework/OpenIMCore(000005.o), building for iOS Simulator, but linking in object file built for iOS, for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
note: Using new build system
note: Building targets in parallel
note: Planning build
note: Analyzing workspace
note: Constructing build description
note: Build preparation complete
/Users/idontsudo/flutter-im-project/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.5.99. (in target 'Toast' from project 'Pods')

Please use the latest version

thx, but now not build to release mode

Please update the sdk to : 1.0.2

Please use the latest version

thx, but now not build to release mode

Please update the sdk to : 1.0.2

flutter_openim_sdk:
       git:
          url: https://github.com/OpenIMSDK/Open-IM-SDK-Flutter.git
          ref: fix-ios-simulator

Hello, still not work, what I miss?
flutter_openim_sdk: 3.4.0+2
Parse Issue (Xcode): Module 'flutter_openim_sdk' not found


Fix object anim parent inverse

Parent inverse wasn't taking into account for object animation.
This PR adds handling of parent inverse for object animations.
NOTE: It doesn't attempt to handle cubic spline tangents

Thanks @pjoe! Would you mind adding a simple .blend file reproducing the issue?

That is part of the PR in scenes/06_parent-inverse-anim.blend :)

Oops, I see 😅  — that's perfect thanks. My only concern then is that the top-center cube is rotated to match its parent in the glTF asset, but not in the Blender scene:

blender
three.js

babylonjs also matches the three.js screenshot.

Hmm yeah that doesn't look quite right :S
I had debug prints in the exporter while testing that definitely showed the rotation quats being corrected, but I will take a look tomorrow (have a crazy deadline today).

This is another issue, if you turn off convert Z up to Y up it works correctly.
So apparently rotation animation doesn't handle the Z up to Y up correctly :S

After a bit more investigation, it's the applying of the parent-inverse for rotations that doesn't correctly work with the z-up to y-up conversion.
Looking into fixing ... if I can just make those damn quaternions 'do the right thing' (tm)

Finally got this working :)
I also added quaternion and axis-angle rotation anims to the test file.
It now shows correctly in https://gltf-viewer.donmccurdy.com/

Thanks! 🎊


YT connection error

It seems to me, that I am connected well,

but I do not have any feedback data

"TEST" stream no longer exists. It was removed.
Current front of future streams:

LOG after STOP/START module:

Do you have an idea, what is wrong, please?

It doesnt look like connection error at all. You are most likely not sending data from your pc/atem (streamovaci software) to youtube servers.

It doesnt look like connection error at all. You are most likely not sending data from your pc/atem (streamovaci software) to youtube servers.

EN: I thought that data was available NOT ONLY when sending data from streaming software to YT. This means that I have an overview of planned broadcasts and possibly ongoing broadcasts (if I am sending data from OBS/ATEM)
CZ: Měl jsem za to, že ta data jsou k dispozici, NEJEN když se posílají data ze streamovacího software do YT. Tedy že mám přehled o plánovanžých vysíláních a případně o probíhajícím vysílání (pokud odesílám data z OBS/ATEM)

Created test private straming:

You can see, there is 1 user watching the livestream, the name of broadcast is "Natural Loop Test" and ID of YT-file "LSALvVzA_cU"

When I check the variables in Companion:

You see, there is something. I do not receive data from YT.
When I check preset buttons:

Concurrent viewrs is 0, but should be 1.

I created button to refresh YT connection. It does not help me:

Restart of module:

Still NO DATA and old content:

The issue is still there. Any idea how to fix it?


Bug Report: Null Object Reference at startup.

As soon as the game compiles,exported it both as release and debug
Used beta6 and windows x64

Nvm guys! I got it fixed
Turns out the recent polymod version is broken so i just downloaded an old one
to anyone experiencing the same issue:
haxelib remove polymod
haxelib git polymod https://github.com/larsiusprime/polymod bb5f0a1


Be more precise on flatpak types

Command::saveAsScript and exists in utils may also should use PathLike for consistency


Some traversey type thing

So I came up with this thing:
what
  :: forall f g t a b
   . Functor f
  => Functor g
  => Newtype t b
  => (b -> t)
  -> ((g b -> g t) -> a -> f t)
  -> a
  -> f b
what _ f = map unwrap <<< f (map wrap)

I was thinking about calling it traverseF, but that's not really right at all aside from superficially having the same shape for the value being operated on and result.
The use case is motivated by @matthewleon's semigroups, for example, the First semigroup can replace the current First monoid provided by Maybe, but it works in a different way, you'd use it something like this:
map (un First) $ foldMap1 (map First) =<< NEA.fromArray [Nothing, Just "foo", Just "bar", Nothing]

Or with the combinator:
what First foldMap1 =<< NEA.fromArray [Nothing, Just "foo", Just "bar", Nothing]

So it's an ala kind of thing, but kinda traverse-y too :smile: It's part way between ala (map unwrap (f wrap) and alaF (map unwrap <<< f <<< map wrap). alaT? :wink:

Or maybe it shouldn't exist at all?

I was thinking, when I wrote the example code for using the First semigroup, that it would benefit from something like this. Will have a closer look later.

I think the code shown here can be written differently now that we have coerce.


F 703 support multi recipient

related to Giveth/giveth-dapps-v2#703

@MoeNick  @CarlosQ96 @MohammadPCh
After merging this PR ( we are waiting for current release after that we would review and merge this)
The frontend should change two some web services:
createProject and updateProject
Instead of sending walletAddress you should send this in the body of request
 addresses: [
        {
          address: walletAddress,
          networkId: 1 (for mainnet) or 3( for ropsten) or 100 (for xDai),
        },
      ]

Getting projects
In every service that frontend gets project list or project detail , it would get address infoes as well with adding this to graphql query
 addresses {
          address
          isRecipient
          networkId
        }

And then based on each network that user selected on metamask frontend should use the address of that network, and make sure that the isRecipient is true for that address

@CarlosQ96 Please review it, but I keep it draft to not merge it until the frontend guys @MohammadPCh  or @MoeNick  tell us to merge it, because this PR changes are not backward compatible and staging may break if we merge it without implementing frontend side

Quick question, is the previous purple address list lost? gotta reintroduce them in the adminbro?

Yeah we would lose them, but now there isn't any purple addresses in our DB and I told Ashley to not add any addresses until we merge this, so it would not make a problem

I think we can skip deleting the other mutations for now. The sql injection is fixed. You can merge.


Issue with Token Renewal for LeetPush App

I’ve been using the LeetPush app to push my LeetCode solutions to my GitHub account, and it has been incredibly helpful. However, I recently encountered an issue: the GitHub token for the app expired, and I couldn't find any way to renew or update the token within the app.
Despite generating a new token on GitHub, there seems to be no option to link it back to the LeetPush app. This has made it impossible for me to continue using the app as intended.
I would greatly appreciate it if you could address this issue or provide guidance on how to resolve it.

I think this will be helpful to generate a new GitHub access token here, after that, you can click the Edit button next to the Push button for the LeetPush extension and renew it, and you are ready to go.


Logo and importance of OVN to be more prominent on home page, above the fold

I moved OVN Vision to header section, but I'm not sure the logo can be increased in size. It would not work to incorporate the logo into the wide header image (currently city scape) because of the sizing of different browsers.
@pmotch @jcstine

Agree that the logo would be difficult to change for the moment.


View.Dispose doen't call UngrabMouse if MouseGrabView is the view itself.

This unit test will fail. The MouseGrabView must unset, otherwise the dispose view will be accessible if the mouse is hover it.
    [Fact]
    [AutoInitShutdown]
    public void MouseGrabView_Is_Set_To_Null_If_A_View_Was_Disposed ()
    {
        var count = 0;
        var view = new View { Width = 1, Height = 1 };
        view.MouseEvent += (s, e) => count++;
        var top = new Toplevel ();
        top.Add (view);
        Application.Begin (top);

        Assert.Null (Application.MouseGrabView);
        Application.GrabMouse (view);
        Assert.Equal (view, Application.MouseGrabView);
        top.Remove (view);
        view.Dispose ();
#if DEBUG_IDISPOSABLE
        Assert.True (view.WasDisposed);
#endif

        Application.OnMouseEvent (new () { X = 0, Y = 0, Flags = MouseFlags.Button1Pressed });
        Assert.Null (Application.MouseGrabView);
        Assert.Equal (0, count);
    }

I think the bug is in the test (and whatever code it emulates in the library).
If someone calls Application.GrabMouse it is a bug if they don't call Application.UnGrabMouse.

I think the bug is in the test (and whatever code it emulates in the library).
If someone calls Application.GrabMouse it is a bug if they don't call Application.UnGrabMouse.

If AutoInitShutdown was used then it will be set to null in ResetState. This problem was happening to me when I removed and disposed a view that previously called GrabMouse. By moving the mouse, the MouseGrabView associated with this view was processed.

I think the bug is in the test (and whatever code it emulates in the library).
If someone calls Application.GrabMouse it is a bug if they don't call Application.UnGrabMouse.

If AutoInitShutdown was used then it will be set to null in ResetState. This problem was happening to me when I removed and disposed a view that previously called GrabMouse. By moving the mouse, the MouseGrabView associated with this view was processed.

I think you need to figure out how to make Menu correctly call Application.UnGrabMouse since it was who called Application.GrabMouse.

I think you need to figure out how to make Menu correctly call Application.UnGrabMouse since it was who called Application.GrabMouse.

By chance it wasn't the Menu that initially caused me this problem but rather the scroll bar, after it was removed and disposed. During the unit tests, the Mouse also threw the error and I corrected that error. If you want me to remove the code in the Dispose method, please say so.

I do. By putting it there you are hiding another bug further up the chain.

I do. By putting it there you are hiding another bug further up the chain.

Done,


Make qdrant collection name a constant

QDRANT_COLLECTION: str = environ["QDRANT_COLLECTION"]
and
 shared_vectorestore_name: str = QDRANT_COLLECTION
This makes an instance of qdrant shareable

I'll add to the env to set the name of the shared collection.

thanks