Enable support for Resumable index builds in conjunction with @TimeLimit

Strict maintenance windows (such as those imposed by Azure Automation -- 3 hours) can cause maintenance on large tables to fail.
@TimeLimit can be used to limit the overall execution time, however it may only get partially through an index before the time limit is reached.
Resumable Online Index Rebuilds along with the "retry" nature of Azure Automation could complete a long running job over several iterations.

Modernize index maintenance with Resumable Online Index Rebuild

Thank you. I have this high on the list of features to add.

I have released a new version with support for online resumable index rebuilds.
https://ola.hallengren.com/versions.html
It setting the MAX_DURATION for the index rebuild, if you are using the @TimeLimit parameter.
EXECUTE dbo.IndexOptimize
@Databases = 'USER_DATABASES',
@Resumable = 'Y',
@TimeLimit = 3600

Hi olahallengren,
thank you very much for your scripts, they are very useful.
I'm having a problem with the index rebuild operations when using the resumable option = 'Y'. It's a very cool new feature on SQL 2017 and the script uses it very well but, when its used in combination with the TimeLimit it breaks when reaches the max_duration (this is a expected behavior but when the script is inside a job it will report failure and then retry).
In my case I want the job to run for 3 hours and then resume the operations on another day. Because the job fails it will be retried and the actual execution is more than 3 hours...
Is it possible to not break the execution of the script when the index rebuild reaches the max_duration?
I hope you understand my question.
Thank you once again.

@fernandojncarvalho I run the IndexOptimise in two SQL Agent jobs.
In the first job that runs weekly, I pull out all the indexes that need optimisation using the following,
EXEC [dbo].[IndexOptimize] 	@Databases = 'USER_DATABASES', 	@FragmentationLow = NULL, 	@FragmentationMedium = 'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE', 	@FragmentationHigh = 'INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE', 	@FragmentationLevel1 = 5, 	@FragmentationLevel2 = 30, 	@LogToTable = 'Y', 	@Execute = 'N';
This captures the Index rebuild/defrag commands in the CommandLog table. After running the IndexOptimise command, the "EndTime" column is set to null for all the records just inserted.

 		UPDATE [dbo].[CommandLog] SET [StartTime] = GETDATE() WHERE [ID] = @logID; 		RAISERROR('Excuting IndexOptimize command for ID:%i (%i of %i) - %s',10,1,@logID, @currCount, @totalCmds,@cmdSample) WITH NOWAIT; 		EXEC sp_executeSql @command = @sqlCmd; 		UPDATE [dbo].[CommandLog] SET [EndTime] = GETDATE() WHERE [ID] = @logID; 		RAISERROR('Command complete for ID:%i (%i of %i) - %s',10,1,@logID, @currCount, @totalCmds,@cmdSample) WITH NOWAIT; 		FETCH NEXT FROM cmds INTO @logID, @sqlCmd 	END 	IF (@currCount < @totalCmds) 	BEGIN 		RAISERROR('IndexOptimize finishing due to elapsed time, executed %i commands out of %i',10,1,@currCount, @totalCmds) WITH NOWAIT; 	END 	ELSE 	BEGIN 		RAISERROR('All commands executed within allowed time window',10,1) WITH NOWAIT; 	END 	CLOSE cmds 	DEALLOCATE cmds END -- we have work to do ELSE BEGIN -- we have work to do 	RAISERROR('IndexOptimize has nothing to execute',10,1) WITH NOWAIT; END -- we have work to do

Hi,
thank you for your suggestion.
I have to tweak it a little because i'm rebuilding very large indexes, that
sometimes don't fit the duration window of the job, that's why i'm using
the @Resumable = 'Y'. In this cases i have to change the command on the
commandLog table to do a resume and not to try a new rebuild on the same
index on the next execution. But its a really nice start.
Thank you very much.
philcart<EMAIL_ADDRESS>escreveu no dia terça, 27/11/2018 à(s)
21:00:

@fernandojncarvalho https://github.com/fernandojncarvalho I run the
IndexOptimise in two SQL Agent jobs.
In the first job that runs weekly, I pull out all the indexes that need
optimisation using the following,
EXEC [dbo].[IndexOptimize] @Databases = 'USER_DATABASES',
@FragmentationLow = NULL, @FragmentationMedium =
'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE',
@FragmentationHigh = 'INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE',
@FragmentationLevel1 = 5, @FragmentationLevel2 = 30, @LogToTable = 'Y',
@Execute = 'N';
This captures the Index rebuild/defrag commands in the CommandLog table.
After running the IndexOptimise command, the "EndTime" column is set to
null for all the records just inserted.

 UPDATE [dbo].[CommandLog] SET
[StartTime] = GETDATE() WHERE [ID] = @logID; RAISERROR('Excuting
IndexOptimize command for ID:%i (%i of %i) - %s',10,1,@logID, @currCount,
@totalCmds,@cmdSample) WITH NOWAIT; EXEC sp_executeSql @command = @sqlCmd;
UPDATE [dbo].[CommandLog] SET [EndTime] = GETDATE() WHERE [ID] = @logID;
RAISERROR('Command complete for ID:%i (%i of %i) - %s',10,1,@logID,
@currCount, @totalCmds,@cmdSample) WITH NOWAIT; FETCH NEXT FROM cmds INTO
@logID, @sqlCmd END IF (@currCount < @totalCmds) BEGIN
RAISERROR('IndexOptimize finishing due to elapsed time, executed %i
commands out of %i',10,1,@currCount, @totalCmds) WITH NOWAIT; END ELSE
BEGIN RAISERROR('All commands executed within allowed time window',10,1)
WITH NOWAIT; END CLOSE cmds DEALLOCATE cmds END -- we have work to do ELSE
BEGIN -- we have work to do RAISERROR('IndexOptimize has nothing to
execute',10,1) WITH NOWAIT; END -- we have work to do
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/olahallengren/sql-server-maintenance-solution/issues/32#issuecomment-442215135,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Am36vhrArYAeHyVV2uhv_j_Yu5Slrzblks5uzafvgaJpZM4SxEQe
.


transformation in other coordinate systems

Hi konmast3r,
thank you for this great plugin!
By any chance, have you throught about adding tranformation of the coordiante system? Perhaps to match GPS data? I would image this should work with PROJ ( https://proj.org/usage/transformation.html ).
Best regards,
Jens Henrik

That's a good idea. That could potentially work with BlenderGIS, as well. It's definitely a future improvement worth implementing.


Pinned notes

What if you could pin notes or tags to the command menu?

FYI I have already explored this a while ago.
My initial thought on this was as simple as that the pinned notes have a #pinned tag. And the notes are automatically sorted with the #pinned tags first (to make it appear on top).
But, been busy lately so probably working on this in the near future.


Is this addon able to retrieve Scene from eWeLink?

I would like to have home assistant entity for the scene in eWeLink app as well
Is it possible? Not building automation of home assistant from scratch seperately

#336


Identify keycode in event for select2-selecting

Hey guys just a quick question, how do we capture the keycode in event for select2-selecting callback?

Sorry, the 3.x version is not maintained anymore. But feel free to fork the repo and fixes what you need.


TabItemsChanged doesn't seem to get raised anymore in WinUI 2.8

WinUI version: 2.8.2
downstream: https://github.com/microsoft/terminal/issues/15121
After updating to 2.8, Terminal's TabView no longer seems to raise TabItemsChanged events anymore. Breakpoints in the Terminal get hit for TabDragStarting and TabDragCompleted, but we don't get a TabItemsChanged.
Looking at the diff between 2.7 and 2.8, it looks like there was some code added around m_isDragging which might end up filtering out the TabItemsChanged events. #6632 introduced the one I'm thinking of, and that was first available in a 2.8.0 prerelease.
I think I can work around it with some finagling on my side, but this was a serious unexpected regression others might want to know about.

https://github.com/microsoft/microsoft-ui-xaml/pull/8420

@zadjii-msft would you mind closing this one out now that we've verified it? thx!


Improve manage application page

Implement "Review Applications v2" with an updated more functional design that requires less clicks to go through applications

I will address the different limit on DocumentEditor component in a later PR. Right now the problem is that I want the character limit to be based on text but the form only has limit checks for the html output which is longer than the pure text


Adding install with Docker tutorial

Reviewed by @Princesso, relevant Asana cards: https://app.asana.com/0/1199987786100094/1200391957003289/f and https://app.asana.com/0/1199987786100094/1200540011612714/f


tests: adjust security tests for systemd v232

systemd v232 gained some protection features both at nspawn and exec level, changing some behavior used while testing rkt insecure mode.
This PR adjusts the NoNewPrivileges check and temporarily disable the sysrq-trigger one.
Closes https://github.com/coreos/rkt/issues/3367

lgtm


Are there any methods to change KeepCell, priority or method on an existing podtable instance?

Hello again:
I have been doing some work with podtable today. One thing I would like a user to be able to do
is choose which columns are kept and the order that columns are moved to the child pane.
Assuming I have:
  table1  = new Podtable ( '#mytable',  KeepCell = [4,5,6], priority = [1,2,3,27,9,8,7,6,14,18,10,11,12])

is there a:
   table1.configure(KeepCell = [1,2,3], priority = "10,11,12,27,9,8,7,6,14,18")
   table1.reconfigure()

that I can use to change the settings and force podtable to restructure the table?
Alternatively is there a:
table1.release()

method to remove all of podtable's changes. Reverting back to the original table
so I can call:
  table1  = new Podtable ( '#mytable', KeepCell = [1,2,3], priority = "10,11,12,27,9,8,7,6,14,18")

with the user's new settings?
Thanks.

So if i get you right
you want to be able reconfigure the podtable initial config setup that's sounds interesting i will add that to the to do list
But the part i don't get is why do you need a release method and revert table to it original state and make a new instance again

Now i get it thanks for your suggestions i will also add this to the todo list

Work is currently going on concerning this i will close this issue for now

There has been a new release with a destroy method to revert any alteration made to the target table in the DOM


Add coq-autosubst to the platform

It could be useful to have Autosubst integrated in the platform.
CC the maintainers: @RalfJung @co-dan
Autosubst is quite stable, and is used in various Iris developments. Ensuring that there are releases of the library (not "dev") that are compatible with current versions of Coq should be easy enough, and would help keeping these Iris developments updated.
Would this make sense?

For reference, Autosubst is a pure Coq library that lives in Coq-community, where it has been maintained for the last few years.

There is a board which takes such decisions, but I don't see a good reason for not adding it.
Indeed adding it to Coq Platform should help to avoid the issue that it is not updated to latest Coq in due time discussed on Zulip today.

Out of curiosity: might this be useful to prove the context handling lemmas for reflexive tactics? Essentially one has already introduced context variables, properties of these, not yet introduced forall variables and properties of these and needs to reason about introduction and generalisation of variables, swapping the order of variables, swapping the order of premises and the like. I find proving such things quite annoying ...

No objections from my side, but I can't promise to have the time to do new autosubst releases after each Coq reease.

Since autosubst is a Coq-community, I consider it by definition community maintained. Owners and other members of Coq-community can help out with autosubst compatibility and releases as necessary (cc: @Zimmi48).

The main question is how complicated this development is and how likely it is that someone else can help out in due time. Since afaik from Zulip @haselwarter is working on this right now, we should wait until he is finished with the new release. He can then share his few on this.

There is a board which takes such decisions, but I don't see a good reason for not adding it.

@MSoegtropIMC Can you say more about this? I was not aware that such a board had been created, and it doesn't look like it was documented.

FTR: the raw PR diff (https://patch-diff.githubusercontent.com/raw/coq-community/autosubst/pull/32.diff) has 521 lines. I would say it is quite feasible to have this community maintained, but we will discuss this in the board.


xSQLServerSetup: xSQLServerMaxDop fails on machines reporting 2+ CPUs

Scenario
Use a multi-CPU (e.g.: dual socket server) machine (you can try this with an Azure DS15_v2 VM in Azure), create a DSC config that uses xSqlServer and use xSQLServerMaxDop in Auto mode:
    xSQLServerMaxDop Set_SQLServerMaxDop_ToAuto
    {
        Ensure = 'Present'
        DynamicAlloc = $true
        SQLInstanceName = $SQLInstanceName
    }

Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
Using SQL Server 2016 SP1 Enterprise, Windows Server 2016 Datacenter, xSQLServer 7.1 (8+ has some issues when deploying it through an Azure Automation Account due to too long path names).
Issue
On machines reporting multiple NUMA nodes/processors - the code fails as it only expects one processor.
E.g.:
On a single CPU machine:
$cimInstanceProc = Get-CimInstance -ClassName Win32_Processor
Returns a single object that has a property called NumberOfLogicalProcessors
On a multiple CPU machine:
$cimInstanceProc = Get-CimInstance -ClassName Win32_Processor
Returns an array of two objects, thus, the following line:
$numProcs = (Measure-Object -InputObject $cimInstanceProc -Property NumberOfLogicalProcessors -Sum).Sum
Fails with:

Measure-Object : The property "NumberOfLogicalProcessors" cannot be found in the input for any objects.
At line:1 char:14

... numProcs = (Measure-Object -InputObject $cimInstanceProc -Property Nu ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Measure-Object], PSArgumentException
FullyQualifiedErrorId : GenericMeasurePropertyNotFound,Microsoft.PowerShell.Commands.MeasureObjectCommand
As the array does not have any property called "NumberOfLogicalProcessors".

In situations where there are multiple CPUs:
if($cimInstanceProc -is [array]) ...
Either the first CPU (if the CPUs are identical) should be used in that instance (which one would assume is the most frequently encountered scenario) or a combination of data from all the CPUs.

Similar to issue 801, I have submitted a fix, it's in progress.

@GArionMSFT When PR #802 is merged, then please test again (with the Dev version). I will add this issue to the PR so it will auto-close on merge. If the PR does not solve your problem, then feel free to reopen this issue.


Missing table value causes Karate to hang

I have seen this a couple of times and recently a team member got stuck, so I thought it would be worth filing an issue.
When using a Scenario Outline, if the value for one of the columns/fields is missing, Karate locks up, shortly after starting.  It does not exit and no error message is provided.
When we start the tests, we just see the version printed out (and the comment about backing up the old target/karate-reports directory) and then it just stops.
➜  ~ java -jar karate.jar demo.feature
15:26:54.027 [main]  INFO  com.intuit.karate - Karate version: 1.1.0

Eventually, I will kill it using Ctrl+C.
This issue can be reproduced using the following Scenario Outline, where the third/final row of the table is missing a value for the food column.
Feature: Empty table value problem

  Scenario Outline: Karate can add numbers
    * def sum = ~~start + ~~add_this
    * match sum == ~~expected
    * karate.log(`I like ${food}`)
    Examples:
      | start | add_this | expected | food  |
      | 1     | 1        | 2        | fish  |
      | 2     | 2        | 4        | pizza |
      | 3     | 3        | 6        |

We will get the same behavior if there is a pipe symbol at the end of the problematic third row, without a space, like so:
      | 3     | 3        | 6        ||

However, it will work if there is even one space between the pipe symbols, like this:
      | 3     | 3        | 6        | |

Now that I know what to look for, I can resolve this issue in our tests, but it can be very confusing when you attempt to start your tests and Karate neither runs them, nor exits with an error message.
Ideally, I would like to see Karate print an error message, identifying the problematic test/line, and then exit.
Notes:

This issue is so easy to demonstrate, that I hope the .zip file will not be necessary.  I can build one, if necessary, but this Scenario Outline is all that is needed.
We use the standalone .jar file.
I have tried the latest release (v1.1.0) and I also built karate.jar from the source today to make sure this issue had not already been fixed in the develop branch.

@nathanchilton thanks ! should be fixed now, do confirm

I'm sorry it took me so long to see your comment that you had fixed it -- the same day that I reported it!
I just built from the develop branch and verified that the issue is fixed.
Thank you, @ptrthomas !
This is perfect!


The Open Closed Principle

There ought to be a way to have zero a config React setup that meets 90% of people's needs out of the box (like you do) and still provide for some extensibility. The eject solution is like jumping off of a bridge. There ought to be some in-between such that easy things are easy and more custom things are possible.
There is even a software development principle describing this exact goal: The Open Closed Principle.
Why is it in the year 2017 we can't make create-react-app extensible? Maybe instead of zero-configuration the goal should be zero configuration for the 90% use-case.
I know I am identifying a problem and propose no solution. But if anyone can solve this problem its you guys (super genius Dan A).
My particular use-case is that I would like to:

Use typescript because my ide has lousy support for Flow. And IDE support is one of the main reasons for using types.
Still receive all of the other wonderful stuff provided by create-react-app including your updates.

@StokeMasterJack I'm using create-react-app-typescript which is just create-react-app with some slightly different scripts for TypeScript.
Thanks to that and some tweaks to config files, I've got TypeScript working with:

Jest
Kadira Storybook
Custom tslint.json rules
Custom tsconfig.json options
Visual Studio Code (with source-mapped debugging)

And I haven't ejected yet.
Having said that, I agree that being able to customize certain things in create-react-app without ejecting would be a very desirable goal.
Happy to share my package.json and other config files if you are interested.

It is impossible to provide a good setup without curation and integration work. Take a look at all the merged PRs: most of the work is not in adding features, but in carefully wiring the tools up to work well together, and balancing their strong and weak sides against each other. Some things (e.g. syntax overlay and now runtime crash overlay) are just not possible to do unless you can make strong assumptions about the rest of the system.
We do open up some things to be configurable over time if people ask for it (e.g. we let you configure the server port, the opened browser, the CI coverage reporter). However, doing the same for another language (TypeScript) is not possible without doing twice the work. The toolchains just aren’t the same unless you’re ready to settle for a mediocre experience where tools don’t really understand each other.
We are not TypeScript experts, and can’t provide a cohesive experience like this. It would require making different choices in different parts of CRA, and would be equivalent to forking the project. But we neither have people who know TS toolchain well, nor the resources to maintain it.
So this is exactly why we propose forking as a solution. We don’t have the resources to create a hundred unique setups tailored to different people’s workflows but we can try our best with one setup that we think works for most React users. You can do the same with TypeScript or any other version.
I hope this gives some insight into how we think about this. See also core ideas.

Having said that, I agree that being able to customize certain things in create-react-app without ejecting would be a very desirable goal.

As always, our answer is that you should file issues with specific requests and we’re happy to consider them. It happened in the past many times.


Be able to test the Bug Tracker Auto-create ticket to Azure Devops Services

Describe the solution you'd like
A test button on the configure screen that fires based on your setup, the create ticket behaviour on the linked bug tracker.
Describe alternatives you've considered
None.
Additional context
This is so you can test you have configured it correctly, creating where you want it and it is working.

Still an issue which would be good for review?


automate releases

[ ] Build and release assets on tags
[ ] Build and release containers on tags

@jhernandezb this is done no?

yes its done


Docker image 'latest' tag is develop, not master

Describe the bug
The stellargraph/stellargraph docker image (https://hub.docker.com/repository/docker/stellargraph/stellargraph) is published from the develop branch to latest, meaning there's no releases for our stable master releases, and docker run stellargraph/stellargraph:latest uses the develop branch, not one of our releases.
If we're publishing docker images, we should make sure we emphasise the stable ones more.
This is coordinated by:
https://github.com/stellargraph/stellargraph/blob/492c4e30d962a9e4ce73d37924ef6e4b1bb597ea/.buildkite/hooks/pre-command#L4-L16
https://github.com/stellargraph/stellargraph/blob/492c4e30d962a9e4ce73d37924ef6e4b1bb597ea/.buildkite/pipeline.yml#L241-L248
Environment
StellarGraph 492c4e30d962a9e4ce73d37924ef6e4b1bb597ea
Additional context
N/A

Does anyone even use our docker images?

I think our CI will do at most two pulls per build:

one for the cache when building
one for the push step, to get the image and repush it

https://github.com/stellargraph/stellargraph/blob/1cca6b6c6f4cd1fea7c6932918cba18d15eef6d4/.buildkite/pipeline.yml#L232-L260

pipeline
builds

stellargraph-public
3545

stellargraph (now unused)
1866

Total
5411

This implies that at most 10822 of the pulls are from our CI. I don't think any of our other pipelines use this docker image. I don't know if 15k pulls is a lot.
Removing the image would certainly be one way to resolve this issue.
That said, now that we have sg.datasets, it could be an easy way for a docker-familiar user to get a Jupyter Lab environment started and running the demos.

If target market is data scientists, not sure if anyone is using our Docker image.
Metrics from docker hub are not useful to see what's happening.
Could discuss whether we stop publishing Docker image and then do it properly.


 Removing these photos would make this theme a lot lighter. Thanks a lot!

True, you can specify which folder? If you mean the pictures of the devices you can only keep the photos that you use and eliminate everything else but it is a job that you should do on your project because depending on the type of project everyone can use a mock-up of the device that he prefers


[Feature Request] jtidy

Checklist

[X] I searched exising issues and no one else requests similar feature.
[X] I think that 25%+ users are positive towards this feature.

Describe the feature
https://github.com/jtidy/jtidy
Additional context
No response

https://central.sonatype.com/artifact/com.github.jtidy/jtidy/1.0.5


How to know version number

I'd like to know the version number of the zsh-autosuggestions package that is installed on my machine
I can't remember installing this manually, it must be via oh-my-zsh
locate zsh-autosuggestions prints nothing even after updatedb
find ~/.oh-my-zsh | grep zsh-autosuggestions prints nothing
Not sure where this gets installed
Describe the solution you'd like
Either an advice on the installation location, or a command like zsh-autosuggestions --version
Describe alternatives you've considered
n/a
Additional context
Sorry for the silly question

I install zsh-autosuggestions by cloning the repo inside custom plugins folder using the command below.
git clone https://github.com/zsh-users/zsh-autosuggestions $HOME/.oh-my-zsh/custom/plugins/zsh-autosuggestions

So I can see the version by running the following command:
cat $HOME/.oh-my-zsh/custom/plugins/zsh-autosuggestions/VERSION

Output:

REF: https://github.com/erdaltsksn/dotfiles/blob/03619bbaccc71b0282beb0cdec6acd74481b18ec/scripts/preinstall#L21-L24


"feat: Nicholas Nadeau Honored as a Finalist for Montreal's Startup Community Awards 2023 Mentor of the Year Award"

""

:tada: This PR is included in version 2.13.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


bugfix: NavigatoriOS doesn't update any scenes when 0th scene is replaced

Calling navigator.replace(0, scene) has no effect.
This is because 0 is false in Javascript so when
this.state.updatingAllIndicesAtOrBeyond == 0

(meaning update all indices starting with 0)
The whole expression evaluates to 0, i.e. false ->  therefore no update
happens.
Explicitly checking for not-equal to null (!= will convert undefined to null automatically) fixes the issue.

Implicit falseyness strikes again!
@facebook-github-bot shipit

This was fast, guys!
On the side note, I only learned that 0 coerces to false through this bug. I always thought that if (variable) is a the* way to check for null/undefined. Not for numeric values, it seems.

@ognen It's worse than that, null, false, undefined, 0, "", [] all evaluate to false.

[] evaluates to true in js (but false in php)

@vjeux hmm, I was confused by the fact that if ([] == false) {...} passes, but you're right,  if ([]) {...} also passes. I have no idea why that would be.

The == behavior is really misleading and we're using === so that it is no longer an issue.
The table you want learn is reduced to the if() one: http://dorey.github.io/JavaScript-Equality-Table/


TypeError: SQL.Database is not a constructor

description
I'm writing a plugin for https://github.com/tinytacoteam/zazu
And I'm writing es5.
source
This works fine.
const db = new Trilogy(historyDbPath)

While this not:
const Trilogy = require('trilogy');

const db = new Trilogy(historyDbPath, {
    client: 'sql.js',
  });

error
Cannot use 'new' with an expression whose type lacks a call or construct signature.ts(2351)

TypeError: SQL.Database is not a constructor
    at readDatabase (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/trilogy/dist/trilogy.js:125:14)
    at Object.create (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/trilogy/dist/trilogy.js:153:30)
    at Pool._createResource (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/generic-pool/lib/Pool.js:308:42)
    at Pool._ensureMinimum (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/generic-pool/lib/Pool.js:340:12)
    at Pool.start (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/generic-pool/lib/Pool.js:414:10)
    at new Pool (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/generic-pool/lib/Pool.js:128:12)
    at Object.createPool (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/generic-pool/index.js:11:12)
    at connect (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/trilogy/dist/trilogy.js:151:15)
    at new Trilogy (/Users/linonetwo/Desktop/repo/zazu-firefox-bookmarks/node_modules/trilogy/dist/trilogy.js:1009:19)

expected
Works as the same.
actual
Maybe this works fine if I use import, but now I have to write require, and it doesn't work.

Interesting, glad to see sql.js has reached a stable version. I should have put a version restriction on the peer dep to account for this, but we'll do that now and make it ^1.0.0.

FYI I had a hard drive fail on me recently, and I'm having a hell of a time getting sqlite3 to build when cloning here.. Guess we've come full circle! If anyone would like to tackle this in a PR it would be very appreciated.
Make backups people 🤕

Actually, sql.js don't work properly in electron https://github.com/kripken/sql.js/issues/271 so I switch to exec(sqlitebinary xxx) to finish my plugin, So...

I finally got back a somewhat workable environment to look at this a bit - node-gyp is an awful experience..
And now I'm wondering how this came up at all. How did<EMAIL_ADDRESS>end up installed for you to cause this issue? trilogy 1.x's dependency on sql.js is ^0.4.0 which would only install up to 0.5.0. I was able to install and pass all tests.
I'm also finding a separate issue in kikobeats/osom where the author changed the node engine requirement from >= 4 to >= 8 in a minor version which breaks installation of trilogy, so we'll have to pin it to 2.2.2 (this library is removed in trilogy 2.x).
In any case I'm hoping to do some work on this over the weekend, to pin some dependencies at the very least.

I can't reproduce this but published 1.4.6 that updates a couple dependencies and fixes the problem with osom on newer Node versions.

Thanks for the update, if someone else tries to use my plugin and failed to use it, I will try trilogy again to fix my plugin.
Thanks for your packages, your param.macro is very useful to me!

@linonetwo glad to hear it, thanks :smile:


Add edit group button

Users should have the ability to edit group names.

https://github.com/bradwj/friend-advisor/commit/6d3f33a98a5c86db0ef2c06d7f1962eec72b046a
Just pushed some changes to develop that adds support on the backend to edit group names and descriptions


dont fix sexadecimal numbers in CommentedMaps

fix_sexadecimals_numbers should not be called on CommentedMaps like "- subnet: <IP_ADDRESS>/23"

thanks phil


Cython version?

Thanks for open-sourcing the implementation!
While installing the requirements with pip install -r requirements.txt, I am getting many build errors related to Cython, using a Windows 10 system.
Can you please share further instructions to setup?

I have the same issue too, on a Mac arm system. However, only specifying the Django version and removing the sklearn==0.0, it seemed to work for me. I've attached the requirements file that I've used.
requirements.txt

Thanks!
Btw I tried on WSL 2 with virtual environment and the same series of steps worked.


[Databricks] Link Airflow task timeout with Databricks Notebook task

Based on the analysis and discussion comments on the issue https://github.com/astronomer/astronomer-cosmos/issues/145,
we pass the Airflow task's execution_timeout parameter value available with the DatabricksNotebookOperator as the timeout_seconds parameter for the Databricks Jobs Run API.
closes: #145

Example DAGs task showing the status as Timed Out.

LGTM. Seems like this solution is simple and easy to maintain 👍 . Have you manually run the tests to make sure this doesn't break any unit tests? We'll need to do that until the CI gets merged.

Thanks. Yes did run the unit tests and made sure that they don't break.

LGTM. Seems like this solution is simple and easy to maintain 👍 . Have you manually run the tests to make sure this doesn't break any unit tests? We'll need to do that until the CI gets merged.

Thanks Daniel. I felt I had run unit tests on this branch, but I had missed running them here, had run them on the other two branches. We did need to alter the tests as they were breaking. Time for the CI to run theses tests sooner :)


Don't allow making a wallet the default for a currency that it doesn't have an address for

Does this one actually make sense to do?


Errors on pages login-idp-link-confirm and login-idp-link-email

Hi!
It's me again. I have a problem with templates login-idp-link-confirm and login-idp-link-email. She's linked with keys again.
How to reproduce the problem:

register mail<EMAIL_ADDRESS>(via template register.ftl)
add to keycloak GitHub identityProvider
have a GitHub account with email<EMAIL_ADDRESS>try logging in by template login.ftl via GitHub

Problems:

login-idp-link-confirm.ftl takes a long time to load. As a result , it gives 500.
login-idp-link-email.ftl (you can see it after fixing the template login-idp-link-confirm.ftl). It takes a long time to load, but template works

How to fix:
The problem is in .data_model.identityProviderBrokerCtx  and .data_model.brokerContext. But .data_model.brokerContext.username is used in login-idp-link-email.ftl. I found all the keys in brokerContext and calculated the keys that break the template: ["contextData", "idpConfig", "idp", "authenticationSession"].
New exceptions for .data_model object:
<#if
    ["contextData", "idpConfig", "idp", "authenticationSession"]?seq_contains(key) &&
    are_same_path(path, ["brokerContext"])
>
        <#continue>
</#if>
<#if key == "identityProviderBrokerCtx" && are_same_path(path, []) ><#continue></#if>

Hi @Ann2827,
Thank you for the detailed issue, I didn't implement this page, it's from the community.
Unfortunately I have no time to deal with keycloaky issues right now.
I will happily merge your PR though
You deserve to appear in the contributor list!


cmake : darwin patch

Only apply the darwin patch in the darwin environment
This patch appears to be for the darwin environment only and fails on my linux system

What kind of failure is it? It's possible that the patch doesn't even apply because cmake got updated to a version that contains it upstream.
/cc @aszlig who introduced it in 5eb3dd8.


LCP between two arbitrary suffixes

Review the current algorithm for getting the length of the longest common prefix between arbitrary pairs of suffixes. It seems that there are simpler methods than the current one.

The simpler method consists in a matrix generated by certain suffix array implementation so it has been preferred maintain the current method. This is what the world uses.


[Feature Request] Ability to drag native files from the system into a droppable area

Hello there 👋,
This looks like an amazing library, but unfortunately it doesn't seem to be possible to drag-and-drop native files from the system (Finder, Explorer, etc.) into a droppable area on a page. It would be nice if this feature could be added, something like: https://react-dnd.github.io/react-dnd/examples/other/native-files
Keep up the good work 👍

Not the maintainer but I think this won't be possible since the library does not use HTML5 dnd api.
For more details see the library architecture

@jnavb Thank you for pointing that out.


Toggable maintenance mode for frontend

@creed-victor I can work on this one, do you have an idea in mind for styling? Maybe just the Sovryn logo and text below it with "Platform is under maintenance, please try again later" - both in center of the page.
We could also get the design team to create some art of a sleeping badger that we can add to the background, but I'm not sure whether that's "professional" enough for the app.

This one will also require some backend work, you worked with node.js right?

We could also get the design team to create some art of a sleeping badger that we can add to the background, but I'm not sure whether that's "professional" enough for the app.

Pretty good idea, maybe @t-z-u team can make something?

For task itself, there should be multiple type of maintenance:

[ ] Full maintenance - change entire dapp with that maintenance "badger"
[ ] Partial maintenance - only show note in each screen with a text

Maintenance mode should be toggle-able from our admin panel, admin signs message, sends it to backend, backend verifies and saves to database.
When user visits dapp, dapp sends request to backend, retrieves maintenance status and if maintenance set then we showing that full maintenance page or only note/ribbon.
I think that when admin set's maintenance he should be able to add optional maintenance message and choose if maintenance is full (disables entire website) or partial.
Currently we have partial maintenance set by .env and which requires redeployment, it prevents users from making new trades and swaps only so I think it would be perfect if when setting partial maintenance admin would be able to choose which services he wants to disable by checking checkboxes:

Opening Trades & Swaps
Opening Loans & Borrows
Closing Trades
Closing Loans & Borrows
Adding & Removing AMM liquidity
ETH bridge and etc.

If all checkboxes are selected it should be still considered as partial maintenance, so users would be able to see all blockchain data there but they will not be able to make new txes.

you worked with node.js right?

Yep 👍
Your solution sounds great, but I don't see an admin panel/db in any of the repos, can you give me permissions?

@soulBit @creed-victor
can we concentrate on getting the dapp and design system sorted first?
i count lend, borrow, liquidity, new portfolio section, landing page before we have a full product

@creed-victor this is implemented on feat/maintenance branches in db & admin repos, and on feat/maintenance-toggle branch in this repo.
Three things to note:

I've allowed separate messages to be set for each "feature" that can have maintenance toggled, since wording a single generic message could get tricky otherwise.
Db has a new table to store the features that can be toggled, and includes a sample message for each that I used while testing, but these can be reworded as needed. With the exception of full site maintenance, when maintenance is active for a feature, these messages will be applied to relevant buttons in the app via tooltips, in the same way as the current maintenance mode. One potential issue is not having translations for these messages, but I can't see a solution other than having a predefined list to choose from.

Message that is signed by user to authorize setting maintenance is generic (similar to login auth message). Should this include the maintenance message entered by the user?

When full site maintenance is enabled and a route other than / is accessed, there are a couple of unmounted component state errors that I'm not sure how to debug, could you take a look?

Thanks @soulBit, can you make it as draft PR? It should deploy preview of the feature branch


added templating to the build.js.test task

When you run tests the config throws a parsing error.
The reason is that the templating wasn't activated for this flow yet.
Related to: https://github.com/mgechev/angular2-seed/issues/1048

Would you fix the build in order to merge?

I was looking at this with regards to the coverage being broken. I think perhaps the build.js.test task should only be building the tests and the build.tests task should also run build.js.dev
seems bad to be having two seperate paths for building the same dev js.
this would also allow the watchers to only rebuild js or tests depending on what had actually changed.


How can performance be improved ?

Hi,
Thank you for sharing the repository.
It's great there's a demo ready to run already.
Unfortunately I ran into some (memory) issues with TensorFlow 1.6 with (a 2GB) GPU.
I could run the demo using TensorFlow 1.4 CPU, but I get about 1fps.
I've tried reducing the webcam resolution as much as possible, but got no visible improvements.
How did you manage to get ~40fps ?
Thank you,
George

Hi George,
First you should first install the GPU version of the TensorFlow and run it. For me with GPU version the FPS is around 45 :)
Thanks!
Asif

Hi @asifzhcet11
Thank you very much for the feedback.
I'll try to compile TensorFlow 1.4 with GPU support on OSX and see how I fare.
By any chance have you tried exporting the model for Android ?
Thank you,
George

Hi George,
The macbook normally comes with an AMD GPU which is not supported by Tensorflow yet. So you have to test this on machines with Nvidia GPU.
And I haven't tried to work with this model for Android
Thanks!
Asif

Hi Asif,
Thanks again for the clarifications.
I'm using an older macbook with a 2GB nVidia GPU.
It's a shame I got OOM errors with Tensorflow 1.6 GPU version.
(Will have to build Tensorflow 1.6 GPU from source)
What GPU are you using ?
Many thanks,
George

Hi George,
Did you install the correct CUDA and cuDNN version for your GPU. You can give it a try by using conda and the installing below package for the complete installation of TensorFlow 1.4 with CUDA and cuDNN
conda install -c cjj3779 tensorflow-gpu
https://anaconda.org/cjj3779/tensorflow-gpu
I am using GeForce GTX 1080
Thanks!
Asif

Hi Asif,
I'm using CUDA 8.0.61 and cuDNN 5.1.10.
I can't use pip nor conda to install tensorflow-gpu as I'm using OSX and these options work only on Windows and Linux.
At the moment I can only think of compiling tensorflow 1.4 with GPU support from source just to test if I can get close to 40fps.
Thank you,
George

Hi George,
Please install conda as described below in the link and I hope you will be good to go :)
https://conda.io/docs/user-guide/install/macos.html
Thanks!
Asif

Hello everyone
I am using a GTX 960 card but I still get about 4FPS
I have correctly set up CUDA and CUDnn
even nvidia-smi shows 3.7Gigs usage in GPU
can i do something to get better frame rates?

Hi, I am running the code on ubuntu 16.04, I can run the code with tensorflow. I download tensorflow-gpu by using conda install -c cjj3779 tensorflow-gpu but, I have 'ImportError: libcublas.so.8.0: cannot open shared object file: No such file or directory' error. How can I run the code with tensorflow gpu?

Hi @DilaraSina
It could be because of the path and library path is not set to the cuda 8.0 specific directories. If not present just add the path and give it a try.
Thanks,
Asif

Thanks, I run on gpu and reached ~35 fps

Hi, @asifzhcet11
I'm sorry to trouble you. Do you know where I can get the training data ,I want to retraining cpm_hand model.

Hi @liang0724s,
Below is the dataset I believe is used. I am not certainly sure but the guys who developed the convolution pose machines are from the same university and used datasets from the below resource.
http://domedb.perception.cs.cmu.edu/handdb.html
Thanks,
Asif

Hi @liang0724s,
Below is the dataset I believe is used. I am not certainly sure but the guys who developed the convolution pose machines are from the same university and used datasets from the below resource.
http://domedb.perception.cs.cmu.edu/handdb.html
Thanks,
Asif

Hi @asifzhcet11
Thank you very much for the feedback.
Leo

Hello everyone
I am using a GTX 960 card but I still get about 4FPS
I have correctly set up CUDA and CUDnn
even nvidia-smi shows 3.7Gigs usage in GPU
can i do something to get better frame rates?

Hello everyone
I am using a GTX 960 card but I still get about 4FPS
I have correctly set up CUDA and CUDnn
even nvidia-smi shows 3.7Gigs usage in GPU
can i do something to get better frame rates?
It is impossible, I am using GTX960M with about 12FPS, maybe you should Set FLAG to the correct parameter

Hi @kautukkundan ,
Did get a better frame rates by setting FLAG to the correct parameter ?
Hi @jgjxzqw ,
The compute capability of GTX 960 and GTX960M respectively are 5.2 and 5.0, the performance is not very bad,  why the frame rate is so low?
I am using GT720M with about 2FPS, and its compute capability is 2.1.


Frontend dev

I have added a company list component that will be used by admins to view company lists and add new companies to the system.
routing and authentication policies are still under development so to access some of this refer to the routing modules file.
always remember to have user logged in when accessing these views because of the authentication setup in the django backend.
updates and improvements on the way

awesome stuff, tomorrow i should also start adding to the python  codebase 🥳


Feature to Login into the application

Content-Type': 'application/json. There was a typo error, and we failed to use the right parameter name in the router.


Use windows-2019 image for CI build

The 2022 image is currently causing issues, presumably we need to upgrade to Cake 2.x
See https://github.com/dnnsoftware/Dnn.Platform/pull/5004#issuecomment-1030281326 for details

This is a release management task, as per our policy, we are self-approving it.


[Query] Scale on Edit photo page does not reduce less than 32% on trying to reduce while setting profile photo.

Frequency: 100%
Repro on build version: Andoird Alphajores Build V1.18.3
Repro on devices: Google pixel 2XL (11), google pixel XL (10), Moto g5s Plus (8.1)
Pre-condition:

User must have freshly installed application.
User must have complete education flow & user is on “Create account Step 1 of 5” screen

Repro Steps:

Launch the app.
Tap on “Add Photo” icon from “Create account Step 1 of 5” screen.
Tap on “Choose from library” option & allow permission.
Select any photo and user is redirected to “Edit photo” screen.
Tap on scale option at button, try to adjust scale manually & observe.

Current Behavior: Scale on Edit photo page does not reduce less than 32% on trying to reduce while setting profile photo.
Expected Behavior: Scale on Edit photo page should reduce below 32% on trying to reduce while setting profile photo.
Impact: Bad user impact
Attachment:
Scale issue.mp4

This is fine for now, thanks!


Checking for isComponent vs isApp fails in super dev mode

https://github.com/Axellience/vue-gwt/blob/master/src/main/resources/com/axellience/vuegwt/client/jsnative/VueGwtTools.js#L24
is the culprit.
Checking for property '$$el' explicitly fails as super-dev appends the build increment (for example '$$el_0_g$')

Suggestion would be to pass isComponent in from GWT.

Thanks for the report 👍
This error occurs probably because you haven't put the JsInterop flag in your superdevmode:
-generateJsInteropExports
I'm working on an update to make the API easier (by dropping VueApp and keeping only VueComponent). I'll also improve the documentation for this kind of issues.
Should be out today.


AreaType as enum

What type of PR is this?

correction

What this PR does / why we need it:
Fix desalignment regarding areaType values
Which issue(s) this PR fixes:
Fixes #131
Special notes for reviewers:
To be incorporated to RC
Changelog input
* areaType is now an enum in UPPER_CASE for all 3 APIs

@akoshunyadi please review also

The "content" looks good to me. But I think we should change the versions to 0.1.0-wip/0.2.0-wip again, otherwise we will have multiple versions with -rc.

I don't know how to proceed here with the versioning. The fix has to be applied to main branch for sure and also to the RC. Now both branches were sync'd.
In case of renaming the versions in main, they should be "0.3.0-wip" and "0.2.0-wip" from now on, to reflect that they are the WIP versions for the following release, while the stable releases are in their dedicated branches. We have the option to fix the current -rc, which was very recent and not still announced officially, or create a new -rc2. Taking into account that we are just releasing this RC and the additional effort to create a new release with branch, changelog, etc, I'm in favour of fixing 0.10.0-rc and communicate that the RC is ready (with this fix). If we have more fixes in the coming weeks we may consider creating a new -rc2.
Wdyt @akoshunyadi , @bigludo7 ?

Hi @jlurien @akoshunyadi
we don't have yet an official way to handle these situation, therefore I can only share my thinking and how we have handled it  in QoD previously:
a) a release candidate is not a release, so there is no need to have a permanent branch for it. But it's a tagged commit, there is the changelog update for it and it has the version number
b) while the 0.n.0-rc is out but not yet released, my recommendation would be to merge to main only fixes which must go also into the release 0.n.0, and don't merge PRs which are already meant for the next release
c) the version within the YAMLs to be changed back to "v0.2.0-wip" ... which actually is not a version but just a hint that this commit is part of the work in progress towards the release, and it's "unreleased".
d) if after some fixes you decide that it would make sense to create another release candidate before the release then indeed v0.2.0-rc2 would be the right version for that (including changes between rc1 and rc2 in the changelog)
A different approach would be to continue in main already with 0.n+1.0-wip and use the rc branch to cherry-pick the needed fixes. My impression is that this is not supported directly in github but can done with GitHub desktop (see here) or git.

Isn't there an agreed recommendation for releasing already? + @hdamker

Isn't there an agreed recommendation for releasing already? + @hdamker

There are Camara_Versioning_Guidelines.md but they don't say anything about release candidates yet and leaves freedom to sub projects:

The commonalities WG does not impose any restrictions on the sub projects for creating other branches, tags which might be needed/useful to address their specific requirements. The above said guidelines are only applicable for the subproject releases.

There are several issues open regarding release management which need to be addressed, see for an overview https://github.com/camaraproject/ReleaseManagement/issues/9

Thanks @hdamker for your input. some comments inline:

Hi @jlurien @akoshunyadi we don't have yet an official way to handle these situation, therefore I can only share my thinking and how we have handled it in QoD previously:
a) a release candidate is not a release, so there is no need to have a permanent branch for it. But it's a tagged commit, there is the changelog update for it and it has the version number

I agree with that, but here I was copying the model I saw in other subprojects, particualrly QoD, where we indeed have a dedicated branch for 0.10.0-rc.

b) while the 0.n.0-rc is out but not yet released, my recommendation would be to merge to main only fixes which must go also into the release 0.n.0, and don't merge PRs which are already meant for the next release

Totally agree here. In this case, it is a fix which should be taken forward to future releases, so it makes sense to fix main as well.

c) the version within the YAMLs to be changed back to "v0.2.0-wip" ... which actually is not a version but just a hint that this commit is part of the work in progress towards the release, and it's "unreleased".

That would work as long as we are not adding any new content intended for future release.

d) if after some fixes you decide that it would make sense to create another release candidate before the release then indeed v0.2.0-rc2 would be the right version for that (with changes between rc1 and rc2 documented in the changelog)
e) the actual release v0.n.0 would then be done always based on main, with documenting the complete changes compared to the previous release again in the changelog

We may hold this fix just in main branch for a while, and create a new RC2 is some weeks, with this a other fixes to come. However, I find that having the current RC without the fix may confuse people as well. It's like saying, forget about that 0.2.0-rc and test the one in main. So exceptionally, and due to this fix being reported very fast I would cherrypick this fix into the current RC as well.

A different approach would be to continue in main already with 0.n+1.0-wip and use the rc branch to cherry-pick the needed fixes. My impression is that this is not supported directly in github but can done with GitHub desktop (see here) or git. The advantage would be that the work on the next release could already be startet ... but than can be confusing I suppose.

That would be necessary if RC period is so long that work on next release is suspended, but so far RC period are usually few weeks. We can avoid that by now, I think

@jlurien also inline, as I have two comments (hope we don't need to continue that):

Thanks @hdamker for your input. some comments inline:

Hi @jlurien @akoshunyadi we don't have yet an official way to handle these situation, therefore I can only share my thinking and how we have handled it in QoD previously:
a) a release candidate is not a release, so there is no need to have a permanent branch for it. But it's a tagged commit, there is the changelog update for it and it has the version number

I agree with that, but here I was copying the model I saw in other subprojects, particualrly QoD, where we indeed have a dedicated branch for 0.10.0-rc.

My bad ... yes, while the release-0.10.0-rc branch would not be necessary, I had created it to have consistent web links for the rc and the later actual release in the README.md.

b) while the 0.n.0-rc is out but not yet released, my recommendation would be to merge to main only fixes which must go also into the release 0.n.0, and don't merge PRs which are already meant for the next release

Totally agree here. In this case, it is a fix which should be taken forward to future releases, so it makes sense to fix main as well.

c) the version within the YAMLs to be changed back to "v0.2.0-wip" ... which actually is not a version but just a hint that this commit is part of the work in progress towards the release, and it's "unreleased".

That would work as long as we are not adding any new content intended for future release.

d) if after some fixes you decide that it would make sense to create another release candidate before the release then indeed v0.2.0-rc2 would be the right version for that (with changes between rc1 and rc2 documented in the changelog)
e) the actual release v0.n.0 would then be done always based on main, with documenting the complete changes compared to the previous release again in the changelog

We may hold this fix just in main branch for a while, and create a new RC2 is some weeks, with this a other fixes to come. However, I find that having the current RC without the fix may confuse people as well. It's like saying, forget about that 0.2.0-rc and test the one in main. So exceptionally, and due to this fix being reported very fast I would cherrypick this fix into the current RC as well.

You can't change the 0.2.0-rc as it already published and probably used by some people (see also https://semver.org/#spec-item-3). But nothing is hindering you to immediately create a 0.2.0-rc2 with this fix. Or even create it within the fix PR if there is no other bug issue to be addressed before. Then also the question about the version within the YAML is solved (as it will be 0.2.0-rc2). Just add the new release candidate to the CHANGELOG.md and point within the README to the rc2 within the same PR.

I have just versioned the fixed version as -rc2, so we can create a new release with the fix.
Please review again @bigludo7, @akoshunyadi, @alpaycetin74

Merging, as this is an important fix


Adding ability to configure base url from config and use a different json library

@leiyangyou Thanks a lot! I'll do my best to review this in next couple of days.

Any news on this pull request? The last commit fixes Testla for Elixir 1.2

@Wijnand Fix for 1.2 has been added to master (https://github.com/monterail/tesla/releases/tag/v0.2.0)
@leiyangyou Thanks a lot for all the changes. I will cherry pick the commits and put all of that together.

All merged and v0.2.1 released.


Updating euro mean parity.

Corrected official HRK to EUR exchange rate in the process of croatian adoption of the euro.
https://euro.hnb.hr/-/hi11072020_vujcic_n1
https://www.tportal.hr/biznis/clanak/vujcic-otkrio-po-kojem-tecaju-cemo-mijenjati-kune-u-eure-ali-i-porucio-nije-dobro-drzati-gotovinu-kod-kuce-rizik-je-od-pljacke-foto-20210913/print

@jtomic-croz did anyone have time to take a look ?

@darko55555 took a look at it now, I'merge it and release a new version today :)

@jtomic-croz Awesome 🥳

@darko55555 @LeonaLj version 1.0.4 which includes new exchange rate is released
<EMAIL_ADDRESS>https://unpkg.com/<EMAIL_ADDRESS>

@jtomic-croz @LeonaLj 🚀

@jtomic-croz 🥳


Widget Not Compatible with Google Now Launcher

Cannot add widget using Google Now Launcher

Very odd. Reboot resolved issue. When opening other launcher widget showed up before reboot.  Took reboot to show up in Now Launcher.

I have had a complaint of this issue in the past (although not specifically with Google Now Launcher, to my knowledge), and I'm not sure what causes it. It does seem to be a problem that a number of people have, although I don't think I've experienced it myself. Also, I don't think it's a fault with GNL in particular, because I use GNL on my main development device.
http://forums.androidcentral.com/droid-razr-maxx/187678-widgets-list-not-updating-immediately.html
http://androidforums.com/threads/widget-does-not-appear-after-installation.575877/
By some sources it appears that this is by design, because Android does not remove an app from the "stopped" state until the app is manually started by the user:
http://stackoverflow.com/a/8700787/1124748
https://groups.google.com/forum/#!topic/android-developers/bvlk3EOV6Xk


Mypy error cannot subclass "ModelFactory" (has type "Any")

Hi, when using the ModelFactory I get the next mypy error:
Class cannot subclass "ModelFactory" (has type "Any")  [misc]

For example with the code in the readme:
from datetime import date, datetime
from typing import List, Union

from pydantic import BaseModel, UUID4

from pydantic_factories import ModelFactory

class Person(BaseModel):
    id: UUID4
    name: str
    hobbies: List[str]
    age: Union[float, int]
    birthday: Union[datetime, date]

class PersonFactory(ModelFactory):
    __model__ = Person

I think this might be related with this pydantic issue.
Do you have any idea on how to solve it?

this is probably the same issue as the other issue - please test v0.4.3 and let me know if this persists for you. Now - the test suite of this package has a lot of use of such code and all of it passes type checking.

The issue was that in my mypy.ini I had the next configuration set:
[mypy]
disallow_subclassing_any = True

The default for mypy is False, so I guess that it's something that's ok not to check, so I'm going to close the issue


adding shopping_recipes pr

If your PR is related to a contribution to the taxonomy, please, fill
out the following questionnaire. If not, replace this whole text and the
following questionnaire with whatever information is applicable to your PR.
Describe the contribution to the taxonomy

...
...
...

Input given at the prompt

   ...

Response from the original model

  ...

Response from the fine-tuned model

  ...

Contribution checklist

[ ] The contribution was tested with ilab generate
[ ] No errors or warnings were produced by ilab generate
[ ] All commits are signed off (DCO)
[ ] The qna.yaml file contains at least 5 seed_examples
[ ] The qna.yaml file was linted and prettified (yaml-validator can do both)
[ ] An attribution.txt file in the same folder as the qna.yaml file.

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck

@instructlab-bot-Gregory-Pereira precheck
(running both with granite)


Minor tweaks

Small changes that came up while I was working on the Kotlin version:

Add unicode escape instead of non-printing character;
Use computeIfAbsent for initializing map entries. (This could probably be done in other places as well, but it was near where I was fixing the non-printing character so I put it in.

Thanks.
By the way, speaking of computeIfAbsent, the MemoTable.memoTable field used to be a Map<Clause, NavigableMap<Integer, Match>>, which mapped from memoKey.clause to memoKey.startPos to match. I used computeIfAbsent to create a new ConcurrentSkipList<Integer, Match> for the inner maps if they didn't exist. With this data structure, it's very fast to find all neighboring matches of a given rule of interest. This might possibly be useful to you with your whitespace-sensitive parser.

Thanks!


Missing options for some api endpoints

There are some options that are not passed on, e.g. the user activity feed has a since_id option that would generate less data over the wire when polling the feed. Only limit and max_id is supported.
It'd be nice if you could just pass the params in an object to the function instead. Like this:
untappd.userFeed(cb, user_name, {
  since_id: last_id,
  limit: 5
});

That way you won't need to maintain the params if new ones show up in the API and this lib can instead just stringify the params instead of what is currently done:
that.userFeed = function(callback, lookupUser, params) {
    // snip-snip guard statements
    return get("/v4/user/checkins/"+lookupUser, QS.stringify(params), callback);
}

Agreed.  A PR would be welcome.

See the two PRs #9 and #10 for solving this. I know the diff is quite large but I think the refactoring prevents this same type of issue from being reopened in the future by allowing an object of key's and values instead of explicitly declaring them.

Alright, despite screwing up the merge and having to do it by hand, I think I got this merged in.  Please take a look and let me know if I missed something else.  Also published new version to NPM.


Make the map pool tiles on the website homepage clickable, instead of being a static image

A little UX suggestion.
Right now, there is an image of some map pools on the website homepage which looks like this: https://imgur.com/kq4m2wo
As this is a static image, the user can't click on anything here. There are no hyperlinks.
I think it would be better if these tiles could be clicked on to take the user directly to the corresponding map pool page.
Currently, if the user sees a map pool that they find interesting in this image, they need to search for it on a different page, which takes longer and could be a hassle.

That's a preview of the functionality of Hitbloq. It's not supposed to be an actual element.


.ftpignore

How to setup the .ftpignore file????????????

From what it looks like, it works like a .gitignore file. See: https://git-scm.com/docs/gitignore

currently, it is not yet documented from anywhere in the repo, tried using the .gitignore patterns but it seems my .git and .ftpignore is still being uploaded to my server even if I have the .ftpignore setuped

Would be really nice to have some doc here. For me it seems to work for single files but not for folders or patterns and also only for local -> remote sync, not for local <- remote.

The new version is capable of handling this.


Add option to generate ES Maps for protobuf map fields

Greetings!
Protobuf-ES currently generates plain JS objects for protobuf map fields, instead of Map instances. The manual mentions that this was decided for compatibility with "popular libraries":

ECMAScript Map objects have great support for key types, but many popular libraries don't support them correctly yet. For this reason, we use an object to represent map fields.

Maps have been supported by browsers and other JS runtimes for a while now (roughly since 2015), and from personal experience, seem to be widely supported across the ecosystem.
Are there any plans to revisit this decision? Alternatively, could an option be added that can be set to generate maps instead of objects? Something similar to what is available for 64-bit ints, where strings can be generated instead of bigints.
Thanks!

Hey Josephus, I can't give you an exhaustive list of popular libraries, but state management packages like Redux and frameworks like Next.js (SSR with getServerSideProps) don't support Maps yet.
An option to generate maps as Map might be a good solution.

Yes, an option would be great to have.

I took a quick-ish look at this. An option sounds easy on the surface, but this would require a new kind of option that propagates from protoplugin to protobuf. This is because descriptor/reflection based code in protobuf (e.g. create) will need to know what kind of code was generated by protoc-gen-es in order to behave correctly. One example is setting defaults when creating a new message from a schema (i.e. createZeroField).
I'm relatively sure no mechanism for communication from protoplugin to protobuf exists just yet, and in fact the dependency DAG is the other way around. One option I see would be to extend descFieldMapCommon with a hint about the generated map type for the reflection code and for protoplugin to set this during codegen based on an argument. I understand that this breaks the encapsulation of protoplugin, but I'm not sure I see another option if reflection code is meant to behave differently depending on codegen options. Using descFieldMapCommon is merely an example. Extending DescMessage would work almost the same way.
type descFieldMapCommon<T extends ScalarType = ScalarType> = T extends Exclude<ScalarType, ScalarType.FLOAT | ScalarType.DOUBLE | ScalarType.BYTES> ? {
  readonly fieldKind: "map";
  // other fields omitted...

  /** Whether the generated code expects an ES6 map or a plain object. Hint for reflection code. */
  es6Map: boolean;
} : never;

Wiring this will mean that protoplugin will need to pass something to the Registry it uses during code generation, since the registry initializes the Desc* types. This would need to be a public option on registry creation code. Client code could generate code with this option set and instantiate an arbitrary registry later without it, which would be a mismatch. This seems not great but I don't see a different way to do this. It's probably an edge case, but it seemed worth mentioning.
Even though this solution is not ideal (imho), the alternatives seem even more convoluted. It would also be possible to create options for relevant utilities (e.g. CreateOptions) to specify whether code was generated with ES6 maps. But this requires consumers to always set this option or risk runtime reflection errors.
Maybe someone else has a better idea or I may have overlooked something obvious.


How to add multiple backend servers

sir .
How to add multiple backend servers?

see this report: https://github.com/jc21/nginx-proxy-manager/issues/375


Generic Sub Field Packager

Hi. Thanks for a great library.
How would I create a Sub Field Packager?
The JPOS library has the ability to create sub fields. Here is the XML from JPOS:
 <isofieldpackager
            id="127"
            length="999999"
            name="RESERVED PRIVATE USE"
            class="org.jpos.iso.IFA_LLLLLLBINARY"
            packager="org.jpos.iso.packager.GenericSubFieldPackager">
        <isofield
                id="0"
                length="0"
                name="PLACEHOLDER"
                class="org.jpos.iso.IF_CHAR"/>
        <isofield
                id="1"
                length="8"
                name="BITMAP"
                class="org.jpos.iso.IFB_BITMAP"/>
</isofieldpackager>

I am setting up the fields so:
var packager = new Packager();
packager.fld = [ new IFA_NUMERIC(4, "MESSAGE TYPE INDICATOR"), ...

Is there a simple way to add a new sub field packager to the array with fields defined inside that?
Many thanks!

I basically want to recreate this PostPackager.
I created this ISOBasePackager:
var p127 = new jspos.ISOBasePackager();
p127.fld = [
	new IF_CHAR(0, "PLACEHOLDER"),
	new IFB_BITMAP(8, "BITMAP"),
	new IFA_LLCHAR(32, "SWITCH KEY"),
	new IF_CHAR(48, "ROUTING INFORMATION"),
	new IF_CHAR(22, "POS DATA"),
	new IF_CHAR(73, "SERVICE STATION DATA"),
	new IFA_NUMERIC(2, "AUTHORIZATION PROFILE"),
	new IFA_LLCHAR(50, "CHECK DATA"),
	new IFA_LLLCHAR(128, "RETENTION DATA"),
	new IFA_LLLCHAR(255, "ADDITIONAL NODE DATA"),
	new IFA_NUMERIC(3, "CVV2"),
	new IFA_LLCHAR(32, "ORIGINAL KEY"),
	new IFA_LLCHAR(25, "TERMINAL OWNDER"),
	new IF_CHAR(17, "POS GEOGRAPHIC DATA"),
	new IF_CHAR(8, "SPONSOR BANK"),
	new IFA_LLCHAR(29, "AVS REQUEST"),
	new IF_CHAR(1, "AVS RESPONSE"),
	new IFA_LLCHAR(50, "CARDHOLDER INFORMATION"),
	new IFA_LLCHAR(50, "VALIDATION DATA"),
	new IF_CHAR(45, "BANK DETAILS"),
	new IFA_NUMERIC(8, "AUTHORIZER DATE SETTLEMENT"),
	new IFA_LLCHAR(12, "RECORD IDENTIFICATION"),
	new IFA_LLLLLCHAR(99999, "STRUCTURED DATA"),
	new IF_CHAR(253, "PAYEE NAME AND ADDRESS"),
	new IFA_LLCHAR(28, "PAYER ACCOUNT INFORMATION"),
	new IFA_LLLLCHAR(8000, "ICC DATA")
];

And then added it as Field 127:
/*127*/ new ISOMsgFieldPackager(new IFA_LLLLLLBINARY(99999, "RESERVED PRIVATE USE"), p127),

When I use the official Java JPOS library I am able to unpack field 127.
When I use jsPOS field 127 is null.
Any pointers to solving this will be greatly appreciated.


I can't find a way to tell the component to usea a locale.

I don't know of its an issue per se but I don't see it documented so I wonder what I should do to tell the component to use a locale, I want to use the spanish locale for moment.js for example.

There's nothing built into react-moment for changing the locale. At least not yet. In the meantime you should be able to set the locale globally by calling moment.locale(). In theory that will also change the locale of the times/dates displayed by react-moment.
http://momentjs.com/docs/#/i18n/

I tried this
import moment from 'moment'
moment().locale('es')

nothing happens, its weird, your component won't use moment globally I think.

I wonder if the problem is from using moment().locale('es') vs moment.locale('es'). Calling moment as a function using moment() may be returning an instance, and your locale setting only affects that instance.


Bedrock example not working

Describe the bug
The json scrape graph example is not working, both as-is and with the boto3 client as the "client" key. v1.20.1 & 1.20.0-beta.2
To Reproduce
``python
client = boto3.client("bedrock-runtime", region_name="us-west-2")
graph_config = {
"llm": {
"client": client,
"model": "bedrock/anthropic.claude-3-sonnet-20240229-v1:0",
"temperature": 0.0
}
}
************************************************
Create the JSONScraperGraph instance and run it
************************************************
json_scraper_graph = JSONScraperGraph(
prompt="List me all questions and options in the math section, no answers.",
source=json.dumps(items[:3]),  # Pass the content of the file, not the file object
config=graph_config
)
result = json_scraper_graph.run()
print(json.dumps(result, indent=4))

Error instancing model: 1 validation error for ChatBedrock
temperature
extra fields not permitted (type=value_error.extra)
TypeError                                 Traceback (most recent call last)
Cell In[98], line 19
9 # ************************************************
10 # Create the JSONScraperGraph instance and run it
11 # ************************************************
13 json_scraper_graph = JSONScraperGraph(
14     prompt="List me all questions and options in the math section, no answers.",
15     source=json.dumps(items[:3]),  # Pass the content of the file, not the file object
16     config=graph_config
17 )
---> 19 result = json_scraper_graph.run()
20 print(json.dumps(result, indent=4))
File /opt/anaconda3/envs/med_device/lib/python3.12/site-packages/scrapegraphai/graphs/json_scraper_graph.py:93, in JSONScraperGraph.run(self)
85 """
86 Executes the web scraping process and returns the answer to the prompt.
87
88 Returns:
89     str: The answer to the prompt.
90 """
92 inputs = {"user_prompt": self.prompt, self.input_key: self.source}
---> 93 self.final_state, self.execution_info = self.graph.execute(inputs)
95 return self.final_state.get("answer", "No answer found.")
File /opt/anaconda3/envs/med_device/lib/python3.12/site-packages/scrapegraphai/graphs/base_graph.py:263, in BaseGraph.execute(self, initial_state)
261     return (result["_state"], [])
262 else:
--> 263     return self._execute_standard(initial_state)
File /opt/anaconda3/envs/med_device/lib/python3.12/site-packages/scrapegraphai/graphs/base_graph.py:184, in BaseGraph._execute_standard(self, initial_state)
171     graph_execution_time = time.time() - start_time
172     log_graph_execution(
173         graph_name=self.graph_name,
174         source=source,
(...)
182         exception=str(e)
183     )
--> 184     raise e
185 node_exec_time = time.time() - curr_time
186 total_exec_time += node_exec_time
File /opt/anaconda3/envs/med_device/lib/python3.12/site-packages/scrapegraphai/graphs/base_graph.py:168, in BaseGraph._execute_standard(self, initial_state)
166 with get_openai_callback() as cb:
167     try:
--> 168         result = current_node.execute(state)
169     except Exception as e:
170         error_node = current_node.node_name
File /opt/anaconda3/envs/med_device/lib/python3.12/site-packages/scrapegraphai/nodes/generate_answer_node.py:133, in GenerateAnswerNode.execute(self, state)
127 if len(doc) == 1:
128     prompt = PromptTemplate(
129         template=template_no_chunks_prompt ,
130         input_variables=["question"],
131         partial_variables={"context": doc,
132                             "format_instructions": format_instructions})
--> 133     chain =  prompt | self.llm_model | output_parser
134     answer = chain.invoke({"question": user_prompt})
136     state.update({self.output[0]: answer})
File /opt/anaconda3/envs/med_device/lib/python3.12/site-packages/langchain_core/runnables/base.py:436, in Runnable.or(self, other)
426 def or(
427     self,
428     other: Union[
(...)
433     ],
434 ) -> RunnableSerializable[Input, Other]:
435     """Compose this Runnable with another object to create a RunnableSequence."""
--> 436     return RunnableSequence(self, coerce_to_runnable(other))
File /opt/anaconda3/envs/med_device/lib/python3.12/site-packages/langchain_core/runnables/base.py:5553, in coerce_to_runnable(thing)
5551     return cast(Runnable[Input, Output], RunnableParallel(thing))
5552 else:
-> 5553     raise TypeError(
5554         f"Expected a Runnable, callable or dict."
5555         f"Instead got an unsupported type: {type(thing)}"
5556     )
TypeError: Expected a Runnable, callable or dict.Instead got an unsupported type: <class 'NoneType'>

**Expected behavior**
A clear and concise description of what you expected to happen.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Desktop (please complete the following information):**
 - OS: [e.g. iOS] macos
 - Browser [e.g. chrome, safari]
 - Version [e.g. 22]

**Smartphone (please complete the following information):**
 - Device: [e.g. iPhone6]
 - OS: [e.g. iOS8.1]
 - Browser [e.g. stock browser, safari]
 - Version [e.g. 22]

**Additional context**
Add any other context about the problem here.

This problem is solved in 1.20.0-beta.1(https://github.com/ScrapeGraphAI/Scrapegraph-ai/issues/635)
please pip install scrapegraphai==1.20.0-beta.1


Instagram Stories - Android - Update in progress...

Hi
I've been working on this library a bit as I want to extend it's functionality in various areas on both iOS and Android.
The main one I'm looking at right now is Instagram Stories for Android. The goal: get IG to open straight away with the media on stories (no Android selector for Direct, Feed, Stories).
I've managed to make some changes which I believe should work but the share seemingly tries to open IG (I see a white screen appear) then almost instantly closes and returns to my app.
Note, I am currently only dealing with a background image but once that works I will get videos and stickers working.
The main thing I am doing is making the intent that is fired be an com.instagram.share.ADD_TO_STORY intent. The rest of the code in singleShareIntent.java fires as normal:
...
this.getIntent().setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
this.reactContext.startActivity(this.getIntent());
TargetChosenReceiver.sendCallback(true, true, this.getIntent().getPackage());
...

I've used FB's docs as a reference: https://developers.facebook.com/docs/instagram/sharing-to-stories/
Can anyone see anything obvious I might be missing? I'm quite stuck at the moment!
File Changes
shareIntent.java (new "instagram_stories" else if...)
public void open(ReadableMap options) throws ActivityNotFoundException {
        this.options = options;

        ...

        String socialType  = "";
        if (ShareIntent.hasValidKey("social", options)) {
            socialType = options.getString("social");
        }
        if (socialType.equals("whatsapp")) {
            String whatsAppNumber = options.getString("whatsAppNumber");
            if (!whatsAppNumber.isEmpty()) {
                String chatAddress = whatsAppNumber +<EMAIL_ADDRESS>                this.getIntent().putExtra("jid", chatAddress);
            }
        }
        else if(socialType.equals("instagram_stories")) {
            //ig stories keys
            this.setIntent(new Intent("com.instagram.share.ADD_TO_STORY"));
            this.getIntent().setPackage("com.instagram.android");

            ShareFile fileShare = getFileShare(options);
            if(fileShare.isFile()) {
                intent.setDataAndType(fileShare.getURI(), "image/*");

//                if (options.hasKey("stickerImage") && !options.getString("stickerImage").isEmpty()) {
//                    Uri stickerAssetUri = Uri.parse(options.getString("stickerImage"));
//                    this.getIntent().putExtra("interactive_asset_uri", stickerAssetUri);
//                    this.reactContext.getCurrentActivity().grantUriPermission(
//                            "com.instagram.android", stickerAssetUri, Intent.FLAG_GRANT_READ_URI_PERMISSION);
//                }

                this.getIntent().addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION);

                if (options.hasKey("backgroundBottomColor") && !options.getString("backgroundBottomColor").isEmpty()) {
                    this.getIntent().putExtra("bottom_background_color", options.getString("backgroundBottomColor"));
                }

                if (options.hasKey("backgroundTopColor") && !options.getString("backgroundTopColor").isEmpty()) {
                    this.getIntent().putExtra("top_background_color", options.getString("backgroundTopColor"));
                }

                if (options.hasKey("attributionLink") && !options.getString("attributionLink").isEmpty()) {
                    this.getIntent().putExtra("content_url", options.getString("attributionLink"));
                }
            }
        }

        ...

        } else if (!TextUtils.isEmpty(message)) {
            this.getIntent().putExtra(Intent.EXTRA_TEXT, message);
        }
    }

RNShareModule.java

added instagram_stories to SHARES

index.js (fix - issue for android, it passes using an underscore)
INSTAGRAM_STORIES: NativeModules.RNShare.INSTAGRAM_STORIES || (Platform.OS === 'android' ? 'instagram_stories' : 'instagram-stories'),

Are you watching unfiltered logcat at the same time? Usually there is a good reason thrown into there in the form of a stack trace or similar

Are you watching unfiltered logcat at the same time? Usually there is a good reason thrown into there in the form of a stack trace or similar

Yea I did think that but nothing. It seems to hand over then it’s just thrown back with no reason :/

OK - this seems to be working now...false alarm. I must have had some caching going on somewhere along the line..
Question 2:
The sticker URL needs to be a local file.
Do you think that if (for example) and external http(s) URL is provided for this sticker it should be (a) downloaded and processed within this component on the android side OR (b) dealt with in the source app's JS code which passes (for example) a Base64 string as the URL to the component.
I've started working on "(a)" but it got me wondering if this is the wrong direction.


Using CardFilter.All causes huge hit in performance

I'm primarily using the Board.Cards property, since I'm interested in all cards on the board, regardless of which list they're in. When I run the following code, it executes relatively quickly (5-10 seconds for the 15-20 boards I care about):
this.trello = new TrelloFactory().Me().Result;
foreach (var board in this.trello.Boards)
{
    // board.Cards.Filter(CardFilter.All);  // <-- This causes huge perf hit
    await board.Cards.Refresh();
}

But if I uncomment the filter line, I hit a massive performance hit, possibly even more than 100x (waiting several minutes and it just finished the first board).
Details:

Used in a .NET Framework 4.6.2 Console Application on a Windows 10 machine (Version 1809, OS Build 17763.379). I see the same behavior on a Windows Server 2016 machine (Version 1607, OS Build 14393.2906)
The first board I'm trying this on has 8 lists, about 40 visible cards, and 15 archived cards
Looking at the Fiddler trace, it looks like the API is returning all of the cards relatively quickly (within a second or two of the default query), but it appears that the parsing/processing of the response is what is getting hung up.
I'm using a paid Manatee.Trello license
I downloaded the git repo and referenced it in my project instead of using the nuget package. I'm not really familiar with the library code, so I haven't been able to debug fully myself. But it looks like there is a lot of time being spent on line 42 of ReadOnlyActionCollection.cs, which is:

internal ReadOnlyActionCollection(Type type, Func<string> getOwnerId, TrelloAuthorization auth)
    : base(getOwnerId, auth)
{
    _updateRequestType = RequestTypes[type];

    EventAggregator.Subscribe(this);
}

Maybe there is a concurrency performance issue with the lock used in EventAggregator.cs on line 58?:
public static void Subscribe(IHandle subscriber)
{
    if (subscriber == null) throw new ArgumentNullException(nameof(subscriber));

    lock (Handlers)
    {
        if (Handlers.Any(x => x.Matches(subscriber))) return;

        Handlers.Add(new Handler(subscriber));
    }
}

Thanks for the report. I'll check it out.

Do you need the actions on the cards?  Turning off the actions download (via Card.DownloadedFields) might be a workaround for now.

Also, I think you meant to put this in Manatee.Trello, not Manatee.Json.  I'm going to close this and open one over there (I'll tag you).

Ha, whoops...yes, I meant Manatee.Trello...sorry about that.


Add Kharoshthi support

I installed all Google Noto Fonts and Firefox still does not support the display of Kharoshthi. See for example https://en.wikipedia.org/wiki/Kharoshthi_(Unicode_block)

We currently plan to support Kharoshthi script between next year and 2019. Please stay tuned.

Humm.  We have NotoSansKharoshti in https://github.com/googlei18n/noto-fonts/tree/master/unhinted
Sure, it's not in noto-source yet.


FP: Large profit FP on a safe execution, which is just a simple transfer

https://explorer.forta.network/alert/0x498b5b3fa1151f2602536f773bdd6c360026ecd28b6e873c7c35fb17d2c3fd99

solved in https://github.com/NethermindEth/Forta-Agents/pull/419


FAF5 Demand: County to County Processing

Trello: https://trello.com/c/TM5vwOeG/42-create-either-an-asset-factory-or-some-reusable-code-to-project-grouped-faf5-data-by-origin-destination-to-county-centroids

Probably some refinements to make this explanation really useful...but looks good!

I've updated the documentation. Please take a look, and let me know what you think. If everything looks good, feel free to merge in the PR.


Named interface detection accidentally picks up nested declarations in nested interfaces scenario

I get this error:
org.springframework.modulith.core.Violations: - Module 'A' depends on named interface(s) 'B :: NamedInterface' via ABC -> XYZ. Allowed targets: common, B, B :: NamedInterface.

So A is a module. Also B is a module. A needs to access NamedInterface which has a package-info.java inside the package with a NamedInterface of 'NamedInterface'.
ApplicationModule 'A' has the following allowedDependencies:

common
A
B
B :: NamedInterface

Still I get the violation error.
When I change the allowedDependencies as follows, it works without an error:

common
A
B
B :: *

I'm also using named interfaces in the exact same way in other places and it works perfectly when explicitly declaring it as dependency. I could not find any reason why it does not work, especially as the violoation error message lists exactly that as allowed.
It's really puzzling me.
I'm using 1.2.0 of modulith.

Thanks for bringing this up. Any chance you can provide a reproducer?

@odrotbohm
Took me a bit, was on vacation :)
https://github.com/jwedel/spring-modulith-dependency-error-reproducer
So, maybe this is not 100% the intended use (we are currently in the process of pulling public api to module root level) but this error message is still kind of weird.
So if you run ModulithApplicationTests then you'll get:
org.springframework.modulith.core.Violations: - Module 'domain1' depends on named interface(s) 'domain2 :: persistence.dto' via com.reproducer.modulith.domain1.Domain1Service -> com.reproducer.modulith.domain2.persistence.Domain2Repository. Allowed targets: domain2, domain2 :: persistence.dto.

If you change the dependencies from module 1 as follows it works:
@org.springframework.modulith.ApplicationModule(
      allowedDependencies = {
            "domain2",
            "domain2 :: *",
      }
)
package com.reproducer.modulith.domain1;

It looks like there's a bug in our named interface detection that currently invalidly assigns Domain2Repository to the persistence.dto interface. That said, once that's corrected, your example still fails as module1 depends on both the presistence interface (via the repository) and the persistence.dto interface (via the DTO). It feels like you're overdoing the technical separation a bit.

On the one hand, as I mentioned before, we are currently on the way of factoring our application. To do this, It would actually help to see some more complex examples either in the documentation or maybe in repository.
Ok the other hand, the separation of public APIs on root level generally sounds like a great idea, but in our case, we have a quite complex application and in this example, the DTO package has around 15 files as our base DTO is quite complex as well. And by the rule of seven, we start introducing sub packages if things grow too much.


Page not found issue when visiting a link in a svelte project

I had a problem with the navigation between the sites of the project. As a beginner in web development that's used to work with Django, it was completely foreign for me that I have to implement a routing mechanism by myself. After a research on the internet, I found some routing packages that either did not work or were deprecated. Fortunately, I found a good router for the project, which you find at the end of the README file.

This is the router I used to solve the routing issue:
https://github.com/ItalyPaleAle/svelte-spa-router


Error installing sqlite3 manual

Hello, I'm new to node, npm so forgive me. I try to install openvpn-status on Debian 10 server and have error Error: Please install sqlite3 package manually. I open that thread.
~/openvpn-status$ node server.js
/home/przemek/openvpn-status/node_modules/sequelize/lib/dialects/abstract/connection-manager.js:88
        throw new Error(`Please install ${moduleName} package manually`);
        ^
Error: Please install sqlite3 package manually
    at ConnectionManager._loadDialectModule (/home/przemek/openvpn-status/node_modules/sequelize/lib/dialects/abstract/connection-manager.js:88:15)
    at new ConnectionManager (/home/przemek/openvpn-status/node_modules/sequelize/lib/dialects/sqlite/connection-manager.js:22:21)
    at new SqliteDialect (/home/przemek/openvpn-status/node_modules/sequelize/lib/dialects/sqlite/index.js:14:30)
    at new Sequelize (/home/przemek/openvpn-status/node_modules/sequelize/lib/sequelize.js:324:20)
    at Object.<anonymous> (/home/przemek/openvpn-status/database.js:3:19)
    at Module._compile (node:internal/modules/cjs/loader:1108:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1137:10)
    at Module.load (node:internal/modules/cjs/loader:973:32)
    at Function.Module._load (node:internal/modules/cjs/loader:813:14)
    at Module.require (node:internal/modules/cjs/loader:997:19)
    at require (node:internal/modules/cjs/helpers:92:18)
    at Object.<anonymous> (/home/przemek/openvpn-status/server.js:11:12)
    at Module._compile (node:internal/modules/cjs/loader:1108:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1137:10)
    at Module.load (node:internal/modules/cjs/loader:973:32)
    at Function.Module._load (node:internal/modules/cjs/loader:813:14)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:12)
    at node:internal/main/run_main_module:17:47

I try also apt-get install sqlite libsqlite3-dev,
~/openvpn-status$ sudo  npm install nw-gyp -g
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142

changed 96 packages, and audited 97 packages in 6s

2 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

~/openvpn-status$ npm install sqlite3 --build-from-source

up to date, audited 1581 packages in 4s

1 package is looking for funding
  run `npm fund` for details

found 0 vulnerabilities

~/openvpn-status$ sudo npm install -g node-gyp
[sudo] password for przemek:
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142

added 97 packages, and audited 98 packages in 4s

3 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

~/openvpn-status$ npm fund
<EMAIL_ADDRESS>└── https://github.com/sponsors/RubenVerborgh
    └──<EMAIL_ADDRESS>
Please help because I don't understand any of this.
Regards,
Przemek

Hi,
Have you attempted to npm install sqlite3?.

Hi,
Have you attempted to npm install sqlite3?.

Hi, yes I try that also.

Hi,
I'm closing the issue, I believe this in an issue in the "sequalize" module. Please open up an issue there.
You can still comment here and seek help but I do have to close it given that it's not an issue in our library.


JWTStatelessUserAuthentication class doesn't work

Defining default authentication classes to JWTStatelessAuthentication produces the following error
ImportError: Could not import 'rest_framework_simplejwt.authentication.JWTStatelessUserAuthentication' for API setting 'DEFAULT_AUTHENTICATION_CLASSES'. ImportError: Module "rest_framework_simplejwt.authentication" does not define a "JWTStatelessUserAuthentication" attribute/class.

package including JWTStatelessUserAuthentication has not been released yet. Use JWTTokenUserAuthentication instead.

package including JWTStatelessUserAuthentication has not been released yet. Use JWTTokenUserAuthentication instead.

Then this page should be updated

thanks @parthokr


531 documentation uplift for contingency manager classes

Enhancements to the docstrings for some of the classes and methods.
Checked locally in a sphinx build.

Discussions here centre on relative pathing and some grammatical points. It's important to get this merged promptly, so I will proceed with this PR and take the issues noted under advisement for the future.


Unknown type name 'ppp_pcb'

Compiling PyCopy 1.11 with the suggested ESP-IDF branch I get this error: unknown type name 'ppp_pcb'
Seems to be happening in Loboris fork too: https://github.com/loboris/ESP32-PPPOS-EXAMPLE/issues/15
So far, I can skip this error by adding this line to Makefile:

INC += -I$(TOP)/lib/lwip/src/include

But now I've another error:

In file included from network_ppp.c:39:0:
../../lib/lwip/src/include/lwip/sockets.h:428:2: error: #error LWIP_SOCKET_OFFSET does not work with external FD_SET!
#error LWIP_SOCKET_OFFSET does not work with external FD_SET!

Fo the last error, I commented the line 408 in $(TOP)/lib/lwip/src/include/lwip/sockets.h

/* FD_SET used for lwip_select */ // LINE 407
//#ifndef FD_SET  // LINE 408

...and added these in the next line (409):

#ifdef LWIP_SOCKET_OFFSET
#undef FD_SET
#undef FD_CLR
#undef FD_ISSET
#undef FD_ZERO
#undef _types_fd_set
#undef fd_set

Source: https://github.com/espressif/esp-idf/issues/1141
But again, I got another error:

../../lib/lwip/src/include/lwip/arch.h:125:19: error: conflicting types for 'mem_ptr_t'
typedef uintptr_t mem_ptr_t;

All of this seems to be related to esp-idf since it changes constantly. I'm working in the suggested branch for PyCopy 1.11: 5c88c5996
Any help will be appreciated :)
👋

Well well, I opted for delete my entire cloned repo and started over again. No issues this time!!
My best guess is maybe I missed something, or maybe the fact I was using GNUmakefile instead makefile. I'm running it in Microsoft WSL and in the rush I chose wrong my custome makefile.
Sorry @pfalcon for I've opened such useless thread 😞

Right, in my branch, I don't make any changes related to esp32. So, the situation would be exactly the same as upstream. My plan was to update to the latest upstream and tell you so ;-). Glad you didn't wait and tried to figure it on your side.
The cause also might be that you didn't update git submodules, you'd need to run git submodule sync / git submodule update regularly.


Revert "[Unity][Parser] Check well-formedness in the parser"

Reverts apache/tvm#16569

Could we perhaps just revert the driver changes if that's what's causing the MLC failure? They are unrelated to the main functionality of the PR and I would rather have the checking for future tests

@slyubomirsky reverting the driver change also works, just wonder if that will break some of the existing tests

The test should have already been broken, oddly! The error was not triggered by anything I added.

agree that we should have a carved out testcases here, will followup in comment soon, as well as in the original thread
I believe it was along the original lines of comment in the changed order, which i assume we say that  we cannot merge dyn smem beyond the kernel boundary,
  // MergeSharedMemoryAllocations must be applied after SplitHostDevice
  // because the merged allocation site is at the beginning of each device function
  mixed_pass_list.push_back(tir::transform::MergeSharedMemoryAllocations());

tests/python/tir-transform/test_tir_transform_inject_ptx_async_copy.py::test_vectorize_cp_async_in_if_then_else was the test that prompted me to change the ordering and it had been failing for several weeks before my PR, oddly. For some reason it was not triggering in CI though.

OK, given that case we should perhaps first revert the driver order only and disable(then look into the failed tests).
We will still get a UT for the ordering case, which I assume would might relates to getting that dlight function compiled, and then reduce to a smaller case

https://github.com/apache/tvm/pull/16770 Smaller-scale reversion

Thank you, @tqchen ,  for the follow-up, and for helping to make the more targeted reversion!

closing in favor of https://github.com/apache/tvm/pull/16770


scheduler: support reserving pods resources

Ⅰ. Describe what this PR does

Reservation supports reserving "pods" resources. It helps when we want to limit the allocatable "pods" of the reservation owners.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in make test

/lgtm
/approve


"Close All Terminals" X icon should not be red

At least when no terminals are open, it should match the color of the other icons in the status bar.
Red icons should only indicate an error, and my eye is continuously drawn to that X and then I realize there's no errors.
I realize you may have made it red since it will destroy all running terminals, but to have it red even with no terminals running makes no sense from a UI perspective.

Yeah, this is annoying and confusing UX.


Added a word

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
Added info
For resources
Description
Why is this valuable (or not)?
It gives more understanding
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

breaks the link


pre-install: pre-install: Remove nydus-snapshotter config

We've been leaving the nydus-snapshotter config behind because we were
trying to remove it from a path that was changed from the first
iteration of the nydus-snapshotter addition, but ended up not being
noticed during review.
While here, let's also make sure to entirely remove
/opt/confidential-containers/share, as nydus-snapshotter is the only
bit of code using that.
Fixes: #278
While here let's also pre-remove ctr cleanup of nydus snapshotters
This has proven to be useless as it is, as ctr doesn't have access to
the /var/run/{containerd,containerd-nydus} to actually be able to
perform any operation.
We could, obviously, expose those to the operator, but I'm very much
against adding even more host mounts to the operator, unless this is
strictly needed, and right now I don't think it is (and I'm fine to be
proven wrong in the future ;-)).

/test

/test

/test

/test

I'm going ahead and merging this one, as the changes will most likely help with the AMD / Intel CIs.


iPhone companion app with no streaming

Before writing I've searched everywhere; there are similar conditions but actually not really what I'm experiencing.
Premise:

Companion app: version 2023.4
go2rtc: version 1.6.2

Conditions:

Safari on macOS: Working "perfectly" (with MSE) inside and outside internal LAN.
Chrome on macOS: Working "perfectly" (with MSE)  inside and outside internal LAN.
iPhone companion app:
-- when I'm inside the internal LAN: everything works smoothly, in RTC MODE

-- when I'm outside, the card automatically tries to use HLS but NO streaming at all. Even forcing all the other modes.

What am I missing? Thank you!

Another info: if I use the companion app (iOS) but connected with a WiFi OUTSIDE my home (my company WiFi for ex.), everything is working (with HLS)...really strange

Have you added STUN server address to RTSPtoWebRTC integration?

You missing:

https://github.com/AlexxIT/go2rtc#codecs-madness
https://github.com/AlexxIT/WebRTC/issues/378