Open Organizations landing page

Fixes #22020

I tried to use many common css elements and js code between integrations and this.
Instead of going for the /open-organizations/research , I went for /open-organizations/#research ( hashtags to get current org type filter) since that felt easier to do. It works well here since this page is much simpler than the integrations page.

The Extra improvements section of #22020 is a separate issue, I will try to open a czo discussion of it tomorrow.

@alya this is ready for your review.

Cool! Is there an easy way to set up my dev environment with some sample organizations so that I can play around with this?

@terpimost could you take a look at this PR as well? Thanks!

Let's rename the page to "Zulip communities directory", and change the URL to "/communities". We put some thought into choosing that label for the setting and the help documentation, and I think it'll work well.
I'll play with other copy on the page when I get a chance, but it'll probably be a similar amount of content, in which case it shouldn't impact the design.

I probably do something wrong but on my end I don't see any organizations:

I guess I just want to check if there are 2 lines of description possible:

and what would be if there is no description at all
Also I hope there is a hover effect like in the design:

@terpimost you can set the default dev environment org to have web-public streams, and then it'll show up in the list. I'll leave it to @amanagr to comment on how best to see more orgs on the list.

Just run ./tools/provision --force after rebasing this PR locally.

I thought maximum 2 should be good

Done. (Ignore the CI failure, it is due to the last commit which will be dropped before merging)

Ah, nice, that works for seeing a few orgs in there!
Could we reorder "Unspecified" to be last in the left sidebar?

Hm, I noticed that clicking on "Categories", "Zulip communities directory", or the "Notable organizations..." text takes you to /communities/#undefined, which seems weird.
Testing /integrations (in production), the behavior is also weird, but differently so -- clicking in these places appends #undefined to your URL, but doesn't wipe out your currently selected category. I think that makes these links useless?
Probably the right solution is for none of these places to be links at all. You can always us the "All" link to clear your current category selection.

@alya fixed the issue in your last comment and pushed.

Cool, thanks! @timabbott I don't know if you want to give this an initial review first, but at some point we should sit down together to tweak the strings and merge.

There's a CSS issue at the bottom of the page, probably related to our recent footer reworking:

Another fun bug: Hovering over a different item in the left navigation causes "All" to become underlined:

@timabbott updated based on  your last review!

@timabbott marking this for "integration review", since the latest batch of feedback has been addressed.

OK; @amanagr can you do a quick rebase? Presumably this conflicts with the file moves.

@timabbott done!

With some consultation with @alya, I reworked the non-organization content to be this:

With those changes, this is ready to merge. We'll want to deploy it to Zulip Cloud and see how it looks with real data before advertising it.

Can we use the wording "opted to be listed" instead of "asked to be listed"?

Yeah, I think something that suggests more that it's a setting in the UI, not a "you have to email us" thing would be more clear.


WIP: refactor: rewrite @hap-toolkit/debugger with ESM

  packager: resolveSync('@hap-toolkit/packager/lib/webpack.post.js'),
  xvm<EMAIL_ADDRESS>

    const {
      ConcatSource,
      SourceMapSource,
      RawSource
    } = compiler.webpack.sources;


Reading waypoint with no MMSI fails

It seems that the position decoding subroutine doesn’t account for the fact that degrees of latitude only have two digits.
On the hx870 branch, given the following waypoint definition in the HX870 device:
00004300: ffff ffff ff11 0600 004e 0010 0060 0045  .........N...`.E
00004310: 3030 3157 5054 ffff ffff ffff ffff ff01  001WPT..........

Expected output:
001WPT (1, 4300)  11° 06.0000’ N 10° 00.6000’ E
Actual output:
001WPT (1, 4300) NaN° 06.0000’ N 10° 00.6000’ E

Thanks for the report!
It seems like the waypoint latitude, in the case of your device, is left-padded with a 0xF instead. My device seems to use 0x0 as left-padding. The fix should handle 0xF now.

The HX870 puts fffffffff0 into the DSC address field for waypoints manually created in the device. But the UI for editing waypoint coordinates on the device itself is a bit masochistic … most people would probably use YCE15 instead, which fills the DSC address with ffffffffff. The latter behaviour is consistent with how ff is generally used as padding and as marker for missing data on the device.
I did mention this briefly in https://johannessen.github.io/hx870/#wpt-def, but reading that again now, I agree it isn't very clear at all. 🙂
I propose a slightly different fix here. See the commit comment.

Thanks! I've addressed this in follow-ups 523115e and e1b520b


Only set _is_checked if the model has a selected column

This was being maskareted before when we assumed that the column idx 0
was the selected one, but now when we don't have a selected column,
selected_column_idx will be None.

The fail doesn't seem to be related to the code. It seems that mock there failed to import something from six. The tests pass locally

LGTM :+1:


fix issue #1352

Codecov Report

Merging #1354 into master will decrease coverage by <.01%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master    #1354      +/-   ##
==========================================
- Coverage    57.5%   57.49%   -0.01%
==========================================
  Files          44       44
  Lines        7833     7837       +4
==========================================
+ Hits         4504     4506       +2
- Misses       2773     2774       +1
- Partials      556      557       +1

Impacted Files
Coverage Δ

session_update.go
56.26% <0%> (-0.18%)
:arrow_down:

session_get.go
76% <100%> (+0.16%)
:arrow_up:

session_delete.go
58.49% <100%> (+0.26%)
:arrow_up:

session_find.go
68.01% <100%> (+0.09%)
:arrow_up:

xorm.go
66.66% <0%> (-1.52%)
:arrow_down:

Continue to review full report at Codecov.

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

It is unnecessary since queryRows will always invoke resetStatement.

It is unnecessary since queryRows will always invoke resetStatement.

Have you checked my example in issue #1352 ?
Let's take session.cacheGet() function as an example
func (session *Session) cacheGet(bean interface{}, sqlStr string, args ...interface{}) (has bool, err error) {
	// skip .........
	table := session.statement.RefTable
	ids, err := core.GetCacheSql(cacher, tableName, newsql, args)
	if err != nil {
		var res = make([]string, len(table.PrimaryKeys))
		rows, err := session.NoCache().queryRows(newsql, args...)
		// skip ........
	} else {
		session.engine.logger.Debug("[cacheGet] cache hit sql:", newsql, ids)
	}

The problem is, when cache hit sql, err will be nil, and queryRows won't be called !

@guangfnian I will send a PR to replace this one and fix that issue.

Fixed in #1375


V2 ion-toolbar in ion-header disappears on transition

When a page has a ion-toolbar in the ion-header, and you transition to a new page which also has a ion-toolbar in the ion-header, the 1st ion-toolbar gets blocked by the size of the toolbar on the next page.
The same happens when navigating back, the toolbar on the 1st page is blocked, and only becomes visible after the transition has ended.
Created with the latest ionic v2 beta10
![Ion-toolbar transition error video]

Hello, thanks for opening an issue with us! Would you be able to provide a plunker that demonstrates this issue? Thanks for using Ionic!

After creating a plunker, i could not reproduce the problem. I turns out that my problem was that i had set a background color on the ion-header, and after removing that everything was correct again.
Is it an idea to mention in the docs that the the main ion-header element should not have a background-color style set on it?
Greetings,
Kriz

Hello @krizroring sorry for the delay on this, i have been very busy with the beta.11 release. Since this issue is now solved i will be closing this issue. Also, thanks for your docs suggestion, but the docs are not the correct place to cover small errors in implementation like this unless we have a large number of issues open about the same user implementation error. Hope I explained that well!


want to get user data after submitting

Issue description
Don't know how to recuperate "fullname, email, password" after submitting in auth/register and login

i'am new in angularJS and i have same probleme.


TODO: Fix timeouts for publish etc

This got auto-fixed by removing timeout from the Push to broker. Closing for now.


(domain): Add @estudiante.edu.do

The MINERD (Ministry of Education of the Dominican Republic) has provided educational emails to students of Politécnico Madre Rafaela Ybarra, as well as to other educational institutions.

Politécnico Madre Rafaela Ybarra: pomaray.edu.do/

MINERD: ministeriodeeducacion.gob.do/

+1

+1

@eliasdanielr I am sorry, this domain seems to represent an umbrella institution, not a specific single university or a college.
Please advise all the students in the school to enroll to GitHub Student Developer Program at https://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with their GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).
I have to close this request without merging.
Thank you for your understanding!


Fix application of the scale argument for geoviews features

When creating a figure with geo=True, the scale argument for geoviews-features is currently ignored. According to the documentation and docstrings, this should be supported via setting the feature argument to a dictionary in the form of {feature: scale}.
With this change, the scale argument is forwarded to the opts function of the geoviews feature class.
Test with
import hvplot.pandas  # noqa
from bokeh.sampledata.airport_routes import airports

plot = airports.hvplot.points(
    "Longitude",
    "Latitude",
    geo=True,
    color="red",
    alpha=0.2,
    xlim=(-180, -30),
    ylim=(0, 72),
    tiles="ESRI",
    features={"states": "10m"},
)

hvplot.show(plot)

Good catch, thank you!


[DNM] Test CI

/test https://github.com/redhat-appstudio/infra-deployments/pull/2665

/test test-pipeline-service-upgrade-ocp-412


Make notable releasing optional

TBD.

No longer relevant after removing the concept of notable release notes #157


Fixed 404 error in docs links

Noticed two broken links at the bottom of https://dspy.ai/learn/programming/signatures/ so have updated the links accordingly. I've chosen the second broken link to redirect to https://dspy.ai/learn/optimization/optimizers/ rather than https://dspy.ai/learn/optimization/overview/ but this is equivocal.

Thanks @Dyadd !

ah actually needed a slight tweak to the optimizers doc path but I updated on main. thanks for raising again!

thanks for correcting me @arnavsinghvi11!


Handle non-unique json versions

Json doesn't implicitly support comments.
Since bump2version just does raw string comparisons, this can make unique matching difficult in npm package files etc.
It seems a bit grand in scope, but we could potentially use actual regex or actual json parsing to resolve json versions and do updating on the version_checker side.

this honestly might be reason enough to decouple from bump2version and just do it ourselves via regex...

looking at npm's versioning tool: https://docs.npmjs.com/cli/v7/commands/npm-version#configuration
maybe users could just tag a specific --preid and match based on that 🤔


Dubious performance of FlatList with only 500 items and 1  element in each

Environment
react-native -v:
% react-native -v
zsh: command not found: react-native
% npx react-native -v
error: unknown option `-v'

npm ls react-native-macos:
% npm ls react-native-macos
<EMAIL_ADDRESS>/Users/alex/dev/superpower
└──<EMAIL_ADDRESS>
node -v:
% node -v
v16.14.0

npm -v:
% npm -v
8.3.1

yarn --version:
% yarn --version
zsh: command not found: yarn

xcodebuild -version:
% xcodebuild -version
Xcode 13.2.1
Build version 13C100

Steps to reproduce the bug
const data = Array.from({length: 500}, (_, i) => ({
  id: i,
  text: `This is another item with index ${i}`,
}));

function App() {
  return (
    <FlatList
      ItemSeparatorComponent={() => <View style={{height: 2}}></View>}
      data={data}
      renderItem={({item}) => <Text>{item.text}</Text>}
    />
  );
}

Running this with npm run macos and scrolling through the list frequently stutters and shows blank temporarily while views are rendered. This is on a 2018 MacBook Pro. In my real application the item views are a fair bit more complex so this test case demonstrates the problem to a lesser extent than I am experiencing.
Expected Behavior
Smoother scrolling
Actual Behavior
Stuttered rendering while scrolling.
Reproducible Demo
I can provide this if need be, possibly I'm just doing something obviously wrong.
Additional context
I'm not aware of any way to run a react-native-macos app in production mode. AFAIUI React does a lot of sanity checking when running in dev mode, and maybe this is part of the problem...
Thanks!

I'm seeing the same problem in my app, in my case I have a row of emojis displayed in a flatlist, but scrolling is not smooth at all

Here is a simple test:
https://github.com/ospfranco/macosFlatlistTest

I could not reproduce the height issue, it seems only to be happening in my app
The initial num to render is not correctly calculated/respected though, you can see even if I set a low number the rendering still goes all the way to the first 100 something, for complex views, this might be a problem (I think this is my case, I only have 50 rows, rendering all of them at once is too slow)
Loading of more items when reaching the bottom is way too slow

Some more findings:

Adding a windowSize property helps a lot with the performance in my case, by setting it to 8 (default is 21) it only renders the visible cells
I'm still not able to figure out why the height prop is being ignored

Fwiw, this should be much faster in 0.71

w00p w00p! is 0.71 out?

We cut the stable branch, but are having troubles publishing it to NPM 😅. Working on it...
I'm not sure what a migration guide would have that's different from a standard RN upgrade? Which to be fair.. is already not fun..

I'll give it a try! :)

I just (somewhat laboriously, maybe I did it wrong :/) upgraded my react-native-macos app to 0.71.1. I ended up upgrading all dependencies (using npm-check-updates) because of a zoo of peer dependency issues. I see very little change, quite possibly worse performance on the FlatList browser in my app. It for some reason calls the render function on each item very slowly. I'm setting up a repro project now to check with a bit more a clean slate.

OK, I can see that even setting getItemLayout faithfully(?) it still has bad performance. There seems to be an issue where renderItem is called over and over again over the range of items in the data array that have already been rendered, even from wake up where there are only 14 items visible.
I've put the whole demo project here, and the log of a run on my system in this gist.
The pattern in the gist is quite predictable. It seems to decide it wants to increase the number of realized rows by 10 each time (even without any interaction), but then ends up calling the renderItem callback for each item, from the start, over and over again. I'm not sure if this is a red herring, but it seems odd that renderItem itself would be called repeatedly, right?
Thanks

The React DevTools Profiler doesn't seem to indicate anything too excessive - renders taking , though this is my first time using it so I have little intuition. The fact that the main thread is spinning 100% for tens of seconds makes me think that there is a performance problem with the inter-thread communication or something. (The heaviest render is 50ms in this profile where the app was frozen filtering the items in the list.)

@alexweej The fast that renderItem is called many times implies to me the flatlist items aren't getting properly memoized, and are re-rendering on every update. I'm also curious, if you do the same profiling on iOS, do you see similar hangs on the main thread? If you're using hooks in renderItem, you might want to try the prop listItemComponent instead (that's unhelpfully not well documented I think.. but is renderItem but with better support for hooks I think).

Also, this might be helpful for RN-macOS upgrades... https://react-native-community.github.io/upgrade-helper/?from=0.68.69&to=0.71.0&package=react-native-macos
They should be about the same as upgrading iOS to RN 0.71, which itself can be painful. There isn't much we can do about that unfortunately.

Sorry for the silence. Life got in the way. I'm still on react-native-macos 0.71.1 but I wanted to try and debug things a bit more before attempting an upgrade. I noticed that in Xcode there is a UI tree that clearly includes fully realized views and subviews for every single item in the FlatList.
(lldb) po [[((RCTScrollContentView*)0x7fb7de025000) subviews] count]
409

Not sure if I misunderstand something about how views are retained, but this should never be the case, right?

OK, I logged out the item index being passed to renderItem and loaded this into Google Sheets to see what was going on. It seems like some algorithm is going a bit wrong...

I can confirm that I see the exact same behaviour on iOS with upstream react-native 0.71.0. From a wakeup iPhone SE screen with the following code it just calls renderItem in the above pattern, starting from item 0 every time and eventually reaching index 366. When I scroll down a small amount less than the row height, it clearly knows it needs to render one more row, but it starts at 0 again.
const data = Array.from({length: 500}, (_, i) => ({
  id: i,
  text: `This is another item with index ${i}`,
}));

const ITEM_HEIGHT = 20;

function App() {
  return (
    <FlatList
      ItemSeparatorComponent={() => <View style={{height: 2}}></View>}
      data={data}
      getItemLayout={(data, index) => ({
        length: ITEM_HEIGHT,
        offset: ITEM_HEIGHT * index,
        index,
      })}
      renderItem={({ item, index }) => {
        console.log('rendering item', index);
        return <Text style={{height: ITEM_HEIGHT}}>hi {item.text}</Text>
      }}
    />
  );
}

export default App;

Curiously in the log I see:

LOG  rendering item 393
LOG  rendering item 394
LOG  VirtualizedList: You have a large list that is slow to update - make sure your renderItem function renders components that follow React performance best practices like PureComponent, shouldComponentUpdate, etc. {"contentLength": 7372.5, "dt": 654, "prevDt": 608}
LOG  rendering item 0
LOG  rendering item 1

Given that the returned element is a Text element, I don't see how there's anything I can do better or more optimally. Can anyone please confirm that this seems wrong to them?
Thanks

OK so it took me this long to discover that

windowSize is measured in 'number of screens'
windowSize default is 21

So that explains why it's trying to render so many rows... Doesn't explain why it's calling renderItem on rows that are already rendered to Views, though.

@alexweej The fact that renderItem is called many times implies to me the flatlist items aren't getting properly memoized, and are re-rendering on every update. I'm also curious, if you do the same profiling on iOS, do you see similar hangs on the main thread? If you're using hooks in renderItem, you might want to try the prop listItemComponent instead (that's unhelpfully not well documented I think.. but is renderItem but with better support for hooks I think).

Can we reopen an investigation into this issue? My component has been minimized to only <Text>{itemKey}</Text>, yet the scrolling performance on MacOS remains extremely poor. The complete component scrolls smoothly on iOS. I've implemented all optimizations outlined in the official React Native documentation, yet the performance remains terribly subpar, leading me to believe the issue lies at the native level.
 <AnimatedContainer testID={`listItem--${id}`}> // use React.memo()
            <Text>{itemKey}</Text>
 </AnimatedContainer>

Hi, I want to say I'm aware this is still an issue, seems to be something specific to macOS (like, at the Appkit vs UIKit level) given that the same code on iOS seems to work fine, and I don't really have any leads on what's causing the issue.


Question: Set the title to the top right or middle of a block.

I didn't find the relevant API in the documentation. Maybe I should implement a custom widget ?

This is not supported at the moment but should be fairly easy to implement, e.g adding a title_alignment to Block and change the logic here to handle this new property.
I might have the time to address it in a future release but in the meantime I'll leave it as a good first issue if anyone want to pick it up.

Hello, I am kinda newbie in Rust and would like to implement this.
This would be my first contribution to Open Source project, so I am very excited.
Thank you for pointing to the correct parts of code.

Here is my pull request https://github.com/fdehau/tui-rs/pull/462

Should we close this issue?


Document DEPCACHEAGE in env-vars.md

This PR adds DEPCACHEAGE env var documentation.
Related to #431

i think it's a bit too vague, as "source info" is ill-defined. Folks won't really know what's being stored in this cache.
Instead of expanding this definition further, could you also add a glossary item that describes exactly what the cache, and (crucially) that results of computation on immutably-keyed inputs is cached permanently? To that end, we need a name for this thing. Given that we already have local cache, maybe "metadata cache"?

i have no idea why this, and only this, PR seems to be failing on the hg clones.


[Improvement][Api] Interface and entity optimization to improve code scalability (#10466).

close #10466

Hi @hstdream , can you describe why did you want to do that? This may cause ui request error.

I think if some new content is added later, the restrictions here are relatively large. UI currently has a child partner to modify

I think if some new content is added later, the restrictions here are relatively large. UI currently has a child partner to modify

We suggest adding a feature to the same pr. It is not necessary to make meaningless changes in advance.

Hi @hstdream , can you describe why did you want to do that? This may cause ui request error.

I think if some new content is added later, the restrictions here are relatively large. UI currently has a child partner to modify

I think if some new content is added later, the restrictions here are relatively large. UI currently has a child partner to modify

We suggest adding a feature to the same pr. It is not necessary to make meaningless changes in advance.

How to operate?

Descirbed what feature you want to implement in the associate issue and develop it in the same branch. Submit this branch to a PR after completion. @hstdream

@hstdream please add some detail into pr description, includes Purpose of the pull request, Brief change log, Verify this pull request.
BTW, How does this relate to add license? Please use the right pr title.

@hstdream please add some detail into pr description, includes Purpose of the pull request, Brief change log, Verify this pull request. BTW, How does this relate to add license? Please use the right pr title.

ok

Descirbed what feature you want to implement in the associate issue and develop it in the same branch. Submit this branch to a PR after completion. @hstdream
@SbloodyS
Hi SbloodyS, with more and more new functions, there will be more incompatibilities in the API. We hope to fix the interface parameters to solve the problem of version compatibility and make fewer changes to interface parameters.
I think it's necessary to enhance the extensibility of the API.

Descirbed what feature you want to implement in the associate issue and develop it in the same branch. Submit this branch to a PR after completion. @hstdream
@SbloodyS
Hi SbloodyS, with more and more new functions, there will be more incompatibilities in the API. We hope to fix the interface parameters to solve the problem of version compatibility and make fewer changes to interface parameters.
I think it's necessary to enhance the extensibility of the API.

In this case, i think we should create a general issue to describe all APIs that need to be modified. Then create sub issues to associate them. And then create the pr link to the sub issues. WDYT? @lenboo

Descirbed what feature you want to implement in the associate issue and develop it in the same branch. Submit this branch to a PR after completion. @hstdream

purpose: to make the interface and entity easier to expand and improve the extensibility
interface: there are new interface parameters behind. You can use otherparamsjson in the code
entity: extend entity

Codecov Report

Merging #10468 (40d8be5) into dev (720a10a) will increase coverage by 0.01%.
The diff coverage is 57.89%.

@@             Coverage Diff              @@
##                dev   #10468      +/-   ##
============================================
+ Coverage     40.90%   40.91%   +0.01%
- Complexity     4852     4856       +4
============================================
  Files           883      885       +2
  Lines         35972    35982      +10
  Branches       3991     3991
============================================
+ Hits          14714    14722       +8
- Misses        19802    19805       +3
+ Partials       1456     1455       -1

Impacted Files
Coverage Δ

...uler/api/controller/ProcessInstanceController.java
72.09% <ø> (ø)

...dolphinscheduler/api/dto/ProcessDefinitionDto.java
0.00% <0.00%> (ø)

...e/dolphinscheduler/api/dto/ProcessInstanceDto.java
0.00% <0.00%> (ø)

...che/dolphinscheduler/api/python/PythonGateway.java
17.00% <ø> (ø)

...hinscheduler/api/service/impl/BaseServiceImpl.java
89.79% <ø> (ø)

...r/api/service/impl/ProcessInstanceServiceImpl.java
60.10% <ø> (ø)

...api/service/impl/ProcessDefinitionServiceImpl.java
31.14% <62.50%> (+0.38%)
:arrow_up:

...er/api/controller/ProcessDefinitionController.java
45.34% <100.00%> (ø)

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.77% <0.00%> (+0.69%)
:arrow_up:

Continue to review full report at Codecov.

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

close #10466

purpose: to make the interface and entity easier to expand and improve the extensibility
interface: there are new interface parameters behind. You can use otherparamsjson in the code
entity: extend entity

Descirbed what feature you want to implement in the associate issue and develop it in the same branch. Submit this branch to a PR after completion. @hstdream
@SbloodyS
Hi SbloodyS, with more and more new functions, there will be more incompatibilities in the API. We hope to fix the interface parameters to solve the problem of version compatibility and make fewer changes to interface parameters.
I think it's necessary to enhance the extensibility of the API.

In this case, i think we should create a general issue to describe all APIs that need to be modified. Then create sub issues to associate them. And then create the pr link to the sub issues. WDYT? @lenboo

Good idea!  But how can we let contributors know this when they modify the code?

Good idea! But how can we let contributors know this when they modify the code?

At present, the issue can only be associated when the committers review the code.


Dark mode

Is your feature request related to a problem? Please describe.
See user story #1809
Describe the solution you'd like

Comments
GUI presented by ShellSquare to Equinor 20.th April
Proposed enhancements for dark mode are accepted/approved by Equinor team

This issue implements user story #1809


Cannot open shader file

Visual Studio 2022
Os : Windows 11

After building everything,
========== Build: 57 succeeded, 0 failed, 16 up-to-date, 3 skipped ==========
But when running : \build\Chapter3\VK01_GLSLang\Debug>Ch3_SampleVK01_GLSLang_Debug.exe, the window closes immediately, and when executing it in command prompt, the following error occurs :
I/O error. Cannot open shader file 'data/shaders/chapter03/VK01.vert'
I/O error. Cannot open shader file 'data/shaders/chapter03/VK01.frag'
Both shaders files are present in the data/ directory.

After giving the full filepath of VK01.vert (""D:/3D-Graphics-Rendering-Cookbook/data/shaders/chapter03/VK01.vert") to testShaderCompilation in main.cpp , it seems to have compiIed before showing the following access denied.
The executable is generated but cannot be executed, nothing happens when running it in the cmd line.


Workspace Administration and Tenancy: Workspace minimal RBAC

Use Case: Workspace Administration and Tenancy:
Objectives:

[ ] An admin can prefill a workspace with a set of APIs that match a type
[ ] A user can be granted workspace access with minimal object/etcd fanout
[ ] A workspace can be typed as “organizational” and contain other workspaces.  Initially there will be a single org workspace, admin
[ ] Lack of workspace in org workspace gates user access

Details:
Proposed: RBAC interfaces within logical clusters are virtualized so that custom logic can union a basic set of permissions “does this user have access to all workspaces” and a specific set of permissions outside the workspace (via some data in the root workspace) and then anything inside the workspace (user A can grant access to user B in the workspace)
Steps:

wire two authorizers into kcp shard apiserver

[ ] taking roles and bindings from lcluster (those objects won't exist by default)
[ ] taking roles and bindings from some admin cluster (= inheritance)

[ ] wire authorization (logical AND with the other authorizers) checking that the user has access to the lcluster in the first place

Maybe restrict to read-only requests if only read access on the workspace object exists.
Non-goals for prototype 2:

add workspace/lcluster scope to Roles and SubjectAccessReviews
do any advanced modelling of inheritance, policies, …
do any multi-level authz in proxy

@s-urbaniak

figuring out the best place to wire additional static authorizers centrally. Currently this place seems like the right spot:
https://github.com/kcp-dev/kubernetes/blob/1f4695370a63cfc44e51573893ef97514c6c3ba6/pkg/genericcontrolplane/server.go#L402-L428

Initial work is available https://github.com/s-urbaniak/kcp/tree/multi-cluster-informers and https://github.com/s-urbaniak/kubernetes/tree/controllerz-config based on @ncdc's scope work.

done, demo will be available at https://www.youtube.com/channel/UCfP_yS5uYix0ppSbm2ltS5Q


YouTube Music Playlists do not show

What version of Music Assistant has the issue?
2022.7.0b1
The problem
Music Assistant does not retrieve YouTube Music playlists (both user defines playlists or YouTube mixed ones like Supermix).
How to reproduce
Configure YouTube Music integration.
Wait for sync to run
Open Music Assistant and select the Playlists tile. Observe "No items found".
In Music Assistant, Select Browse > YouTube Music > Playlists. Observe that operation does not complete and nothing loads.
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2022.7
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

Can you please look at your Home Assistant log ? Very good chance there will be an error in there. Can you share the error (or the logfile) here ? Thanks!

I think I have the same issue, here is my log:
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/mass/websockets.py:856
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 12:44:55 (9 occurrences)
Last logged: 12:49:34
[547095757056] Error handling message: Unknown error (unknown_error)
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 27, in _handle_async_response
await func(hass, connection, msg)
File "/config/custom_components/mass/websockets.py", line 106, in async_get_mass_func
await orig_func(hass, connection, msg, mass)
File "/config/custom_components/mass/websockets.py", line 856, in websocket_browse
result = await mass.music.browse(msg.get(URI))
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/init.py", line 209, in browse
return await prov.browse(path)
File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 274, in browse
return [x async for x in self.get_library_playlists()]
File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 274, in
return [x async for x in self.get_library_playlists()]
File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/ytmusic.py", line 234, in get_library_playlists
playlist.metadata.description = item["description"]
KeyError: 'description'
Maybe not every playlist has a description?

I also have the problem that albums wont load.
On Home Assistant restart:
Logger: music_assistant
Source: /usr/local/lib/python3.10/site-packages/music_assistant/mass.py:245
First occurred: 13:03:31 (1 occurrences)
Last logged: 13:03:31

Job [Library sync for provider Youtube Music] failed with error 'gridRenderer'.
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 315, in sync_library
    async for prov_item in self._get_library_gen(media_type)():
  File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/ytmusic.py", line 155, in get_library_albums
    response_albums = response["contents"]["singleColumnBrowseResultsRenderer"][
KeyError: 'gridRenderer'

If I try to open the albums:
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/mass/websockets.py:856
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 13:07:41 (1 occurrences)
Last logged: 13:07:41

[547812136592] Error handling message: Unknown error (unknown_error)
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 27, in _handle_async_response
    await func(hass, connection, msg)
  File "/config/custom_components/mass/websockets.py", line 106, in async_get_mass_func
    await orig_func(hass, connection, msg, mass)
  File "/config/custom_components/mass/websockets.py", line 856, in websocket_browse
    result = await mass.music.browse(msg.get(URI))
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/__init__.py", line 209, in browse
    return await prov.browse(path)
  File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 268, in browse
    return [x async for x in self.get_library_albums()]
  File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 268, in <listcomp>
    return [x async for x in self.get_library_albums()]
  File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/ytmusic.py", line 155, in get_library_albums
    response_albums = response["contents"]["singleColumnBrowseResultsRenderer"][
KeyError: 'gridRenderer'

same here I believe.. it just runs the graphic over the screen over and over again.
2022-07-08 23:33:00 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection] [140373425592816] Error handling message: Unknown error (unknown_error) Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 27, in _handle_async_response await func(hass, connection, msg) File "/config/custom_components/mass/websockets.py", line 106, in async_get_mass_func await orig_func(hass, connection, msg, mass) File "/config/custom_components/mass/websockets.py", line 856, in websocket_browse result = await mass.music.browse(msg.get(URI)) File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/__init__.py", line 209, in browse return await prov.browse(path) File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 274, in browse return [x async for x in self.get_library_playlists()] File "/usr/local/lib/python3.10/site-packages/music_assistant/models/music_provider.py", line 274, in <listcomp> return [x async for x in self.get_library_playlists()] File "/usr/local/lib/python3.10/site-packages/music_assistant/music_providers/ytmusic.py", line 216, in get_library_playlists response_playlists = response["contents"]["singleColumnBrowseResultsRenderer"][ IndexError: list index out of range

Fixed in beta 4, please report a new issue if you encounter this issue again.


Upgrade AzureRM to 3.41.0

What is being addressed
Monthly upgrade of Terraform AzureRM provider to 3.41.0.
Note that only the core modules and key templates are upgraded.
Change the log property since it has been deprecated.

/test-extended

/test-extended


Add missing repo types.

The existing list of repo types is incomplete, adding in those missing from:
https://www.jfrog.com/confluence/display/JFROG/Repository+Configuration+JSON
Fixes #71

I have read the CLA Document and I hereby sign the CLA


docs: broken links for edit page modified

PR Checklist

[x] Addresses an existing open issue: fixes #38
[x] That issue was marked as accepting prs
[ ] Steps in CONTRIBUTING.md were taken

Overview

Broken URL modified.

Thanks for sending this in!
Just a heads up: it's typical to check off the [ ] checkboxes in the description like [x] to indicate they're done. I edited it and did the first one. 🙂

https://github.com/LearningTypeScript/site/pull/47#issuecomment-1149229845, sorry!

@JoshuaKGoldberg It is completely OK. I had learnings. I wish to implement those learnings in the coming days. I appreciate your help and guidelines for the beginners like me. It will surely help me.
Thank You.


surprising precedence betwixt forall and ==>

First error in this recording, a type error expecting boolean, suggests verus is parsing this expression with the forall binding tighter than implication.


Add command to convert / modify Deflate streams to be more compliant / compatible

Some decoders may have issues decoding certain Deflate streams:

Some Deflate streams may use distance codes in a way that isn't strictly spec legal (#3)
Some Deflate streams may have dynamic headers which are spec legal, but cause problems for certain decoders (#2)

It would be nice to have a command that modifies the Deflate streams of files to be more compatible.

It would be nice to have the default use these more extreme/ edge case tweaks to squeeze every last byte.
Then provide a -pedantic mode that will output for the most fussy decoders.


Intermittent issue when running large jobs [SEVERE] [worker-123] [JM Agent] edu.iu.dsc.tws.master.worker.Pinger: Ping Response message requestID does not match.

Getting the "[SEVERE] [worker-123] [JM Agent] edu.iu.dsc.tws.master.worker.Pinger: Ping Response message requestID does not match." sometimes when running at 128 parallism with 128 workers

Pinger sends ping messages continuously and when it does, it replaces the
requedID with the new one. If  JM fails to send the pong in the meantime,
it could show this message, since it has already discarded the previous
ping request ID. This happens with larger no of workers, since they could
easily overload JM by sending ping messages with 10sec intervals.
What should we do? Should we delay the next ping message till it receive
the response from the previous one? Or we can create a queue of ping
request IDs.
On Thu, Apr 4, 2019 at 8:09 PM Pulasthi Supun<EMAIL_ADDRESS>wrote:

Getting the "[SEVERE] [worker-123] [JM Agent]
edu.iu.dsc.tws.master.worker.Pinger: Ping Response message requestID does
not match." sometimes when running at 128 parallism with 128 workers
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DSC-SPIDAL/twister2/issues/325, or mute the thread
https://github.com/notifications/unsubscribe-auth/AG-VCd3nbXiKX1x6IwobeGavsRYYfoQVks5vdpRLgaJpZM4cd7z8
.

Shall we make the ping interval configurable and make the next ping message wait until the first one finishes? Otherwise, it will anyway overload.

Does this become a sequential bottleneck?
One could use pub-sub?
On Thu, Apr 4, 2019 at 8:50 PM Chathura Widanage<EMAIL_ADDRESS>wrote:

Pinger sends ping messages continuously and when it does, it replaces the
requedID with the new one. If JM fails to send the pong in the meantime,
it could show this message, since it has already discarded the previous
ping request ID. This happens with larger no of workers, since they could
easily overload JM by sending ping messages with 10sec intervals.
What should we do? Should we delay the next ping message till it receive
the response from the previous one? Or we can create a queue of ping
request IDs.
On Thu, Apr 4, 2019 at 8:09 PM Pulasthi Supun<EMAIL_ADDRESS>wrote:

Getting the "[SEVERE] [worker-123] [JM Agent]
edu.iu.dsc.tws.master.worker.Pinger: Ping Response message requestID does
not match." sometimes when running at 128 parallism with 128 workers
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DSC-SPIDAL/twister2/issues/325, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AG-VCd3nbXiKX1x6IwobeGavsRYYfoQVks5vdpRLgaJpZM4cd7z8
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DSC-SPIDAL/twister2/issues/325#issuecomment-480112655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGPrFznoaAMoC2KTv3uWnxLowWPITftYks5vdp3BgaJpZM4cd7z8
.

--
:
: Geoffrey Fox<EMAIL_ADDRESS>Phones Cell<PHONE_NUMBER>  Lab<PHONE_NUMBER>
http://www.infomall.org

https://github.com/DSC-SPIDAL/twister2/pull/341


remove std::iterator from wrapped_compact_theta_sketch_alloc

Should fix #370

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4905

Relevant Lines:
5053

💛  - Coveralls

const uint64_t& operator*() const;
const uint64_t* operator->() const;

these should return "reference" and "pointer"

While we are at this, let's add operator->() to theta_update_sketch_base theta_iterator and theta_const_iterator


Fix StackOverflowError in PoolableConnection.isDisconnectionSqlException

If batch size is more than max call stack size, and all statements have nonfatal exception like data truncation, StackOverflowError will appear rather than the real SQLException.
This PR uses while loop instead of recursion to prevent it.

@newnewcoder
I pushed a cleaned-up version of the PR to git master, please verify your use-case and close this PR.
TY!

checked OK, thank you!


Update Boxcryptor 2.22.933

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Closed because of earlier PR ! #46142


Homepage tweaks for mobile

What does this PR do ?
it solves the below issues and some other issues
https://github.com/ArslanYM/StarterHive/issues/179

fix: below 323px navbar wraps now
fix: added a utility class to unset width as we go from sm to md devices
fix: margin removed from header logo
fix: header logo added
fix: hero section text width limited after md devices
fix: footer logo and title order changed

@rahilansari261 Some new changes were introduced to the Header , Can you resolve the conflicts and make sure the new changes are added are well in this PR. Also, I think we agreed on keeping the commits as less as possible. See to that next time.

@rahilansari261 Some new changes were introduced to the Header , Can you resolve the conflicts and make sure the new changes are added are well in this PR. Also, I think we agreed on keeping the commits as less as possible. See to that next time.

i get your point regarding commits i was just doing some experiment .

@rahilansari261 Some new changes were introduced to the Header , Can you resolve the conflicts and make sure the new changes are added are well in this PR. Also, I think we agreed on keeping the commits as less as possible. See to that next time.

i get your point regarding commits i was just doing some experiment .

Great Work! Looks good on mobile too. Well done @rahilansari261


Error while inference

Hello again, I faced this error when the input text exceeded certain number of characters (15987) to be exact, or about 3k words approximately
"    return torch.embedding(weight, input, padding_idx, scale_grad_by_freq, sparse)
IndexError: index out of range in self "
form my search I found that this error means that I got an index out of the embed-dings range how could his happen? I checked the token and it was normal letters so it should not be OOV or something? any idea about what is going on here and how to fix it? thanks in advance.

Hey again, I noticed the error is due to the absolute positional embeddings in Longformer which was 4098 in the allenai/longformer-base-4096 base, any solution to avoid this in longer sequences rather than just split the input into smaller segments?

Ah, unfortunately with Longformer that would likely require re-training the model as an LM. You can try with XLNet, it uses a relative positional embedding scheme and IIRC I implemented context propagation between chunks. I believe there may be new architectures that can deal with even longer inputs available now (BigBird is one that comes to mind), if Huggingface model hub has them, maybe it wouldn't be too much work to make this code working with that.

Okay thank you, I am closing this issue


Pylint

Pylint is bijna compleet. Ik moet dus nog wat refactoren/werkend maken in plot.py, dus ik laat het voor nu even hierbij. Pylint grade = 9.4

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is 31.25%.

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   64.95%   64.95%
=======================================
  Files          10       10
  Lines         428      428
=======================================
  Hits          278      278
  Misses        150      150


Handle custom types

It would be nice if there was a way to handle user-defined PostgreSQL types that map to Haskell ones such as enums into WalValue

If we have a table setup like this with a custom enumeration type:
create type foo as enum ('bar', 'baz', 'quz');

create table testing_types (
 id serial primary key,
 test foo not null
);

insert into testing_types (test) values ('bar');

We do receive the change with the Column type:
Change {changeNextLSN = LSN {filepart = 0, offset = 24033352}, changeDeltas = [WInsert (Insert {insertSchema = "public", insertTable = "testing_types", insertColumns = [Column {columnName = "id", columnType = "integer", columnValue = WalNumber 1.0},Column {columnName = "test", columnType = "foo", columnValue = WalString "bar"}]})]}

Which includes a string encoding of the name of the enumeration which would allow a user to parse the value.


Auto-add secrets file and slscrypt decryption helper to package.include

This makes serverless-crypt work with wildcard packaging rules like:
package:
  exclude:
    - '**/*'
  include:
    - handler.py

Coverage decreased (-5.8%) to 94.203% when pulling e9b2b9b390dd0f9806087344f0fffaf34ac98967 on dschep:master into 4bfb0162ff99827f4ac7b9dad727b6aaa989aedf on marcy-terui:master.

Coverage decreased (-5.8%) to 94.203% when pulling ef1b0797747d47a80b4a634da4e266591cc7d941 on dschep:master into 4bfb0162ff99827f4ac7b9dad727b6aaa989aedf on marcy-terui:master.

Coverage decreased (-5.8%) to 94.203% when pulling 294896dcede4b6f4b86558c3e0283aeca2bdeb53 on dschep:master into 4bfb0162ff99827f4ac7b9dad727b6aaa989aedf on marcy-terui:master.

@marcy-terui I see there's been no changes to this repo in 11 months. Is this plugin abandoned?

Closing in favor of #13 so that I can make more changes on my master branch.


S2S Authentication no longer working in 18.4 environments?

Describe the issue
I am using the Publish-PerTenantExtensionApps function to deploy my apps using S2S authentication as stated in Kauffman blogs and Micorosoft's ci/cd PDF document. However, just recently, I am now getting 401 Unauthorized errors.
Am I missing something or something changed in 18.4 release?
Scripts used to create container and cause the issue
This is a cloud environment.

Full output of scripts
Attempting authentication...
Authenticated as app ....
Invoke-RestMethod : The remote server returned an error: (401) Unauthorized.
BcSaas\Publish-PerTenantExtensionApps.ps1:74 char:22
...

Screenshots
N/A
Additional context

Yes, for >= 18.4 environments.
It works on my 18.3 environments.

Any help or pointers will really be appreciated.

Did you remember to add the clientId in the environment (allowed aad apps?)

Yes, it's there. The name of the option is "Azure Active Directory Services".
I created a new record with the client ID, granted consent and added the Automation and EXTN MGT permissions.
Is "Allowed AAD Apps" a different option than "Azure Active Directory Services".

No, nothing has changed there between 18.3 and 18.4 - we are probably looking for differences in your setup

Yes, you are right, I upgraded my 18.3 tenant to 18.4 and S2S still works.
I am following the steps carefully as stated in Kauffman blog and ci/cd document, but still not get through this.
Any pointers in troubleshooting would be helpful.

This issue has been resolved.
Cause: I was using Microsoft Azure's tenant Login ID while accepting the Consent Popup dialog.
Resolution:

I granted admin consent within Microsoft Azure using Application Administrator.
Then, I used the BC Cloud Tenant Login ID while accepting the Consent Popup dialog.


MFA Conditional Access Policies may block the Azure AD Connector Account from authenticating

There are scenarios where customers are enforcing user-level MFA to all users on AAD. In this scenario, the Azure AD Connect account is also targeted. It forces the AAD Connect Wizard to prompt for the SYNC_XXX password which is unknown and should not prompt. If there is a pre-existing CA rule in place for all users requiring MFA, the AAD Connector account will also be challenged for MFA. I suggest adding a note to this document to exclude the "Directory Synchronization Accounts" directory role from any Conditional Access rules targeting users. Otherwise, it will break the installation and/or scheduled cycles.
Suggest adding a note here:
https://docs.microsoft.com/en-us/azure/active-directory/hybrid/reference-connect-accounts-permissions#azure-ad-connector-account

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

ID: f2c5e335-4367-b1fe-2e14-311a6f2cf81d
Version Independent ID: a35eed80-eb37-e59c-07a4-43af4a985991
Content: Azure AD Connect: Accounts and permissions
Content Source: articles/active-directory/hybrid/reference-connect-accounts-permissions.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@josuvil
Thanks for your feedback! We will investigate and update as appropriate.

@josuvil
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thank for submitting this. I have created a work item for it and will send this to engineering so that they can review it and provide me with updated details. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


Pass exceptions to logging and output stacktrace on debug logging

Description
Fix passing exception with error log and log stacktrace in debug logging mode.
Motivation and Context
Fixes #4430
How Has This Been Tested?
Manual testing.
Type of Changes

Bug fix (non-breaking change which fixes an issue)

@raducoravu Are you able to test if this logging fix works for you?

@jelovirt thanks for this, I confirm that when running with "-d", errors logged like this from my ANT task now have the stack trace visible:
 getProject().log(mytask, ex.getMessage(), ex, Project.MSG_ERR);


Casting sol::protected_function_result to sol::error fails

#define SOL_CHECK_ARGUMENTS true
#include "sol.hpp"
#include <utility>
#include <cstdio>
#include <functional>

void Error(sol::error const & err)
{
    printf("%s\n", err.what());
}

template<typename T>
bool HandleError(T&& obj)
{
    if (!obj.valid())
    {
        Error(std::forward<T>(obj)); // <----- Here
        return true;
    }
    return false;
}

template<typename ...ARGS>
void PCall(sol::protected_function const & function, ARGS&&...args)
{
    HandleError(function(std::forward<ARGS>(args)...));
}

int main()
{
    sol::state state;
    state["f"] = [&](sol::table t) {
        int x = t["x"];
    };

    int x = 100;
    sol::table t = state.create_table();
    t["x"] = std::ref(x);
    PCall(state["f"], t);
}

I understand that int x = t["x"]; raises an error (at least when SOL_CHECK_ARGUMENTS is defined).
However, for some reason the casting of sol::protected_function_result to sol::error fails in the marked spot with following:
[sol2] An error occurred and panic has been invoked: stack index 1, expected string, received table
Any idea why this happens?
Isnt this the correct way to check the result of the call and get the error message?
Details: VisualStudio 2017 15.8.1, C++14, #define SOL_CHECK_ARGUMENTS true
Sol2: https://github.com/ThePhD/sol2/blob/f438284a47294dc0fc92fd1a8a8959e7cd84809d/single/sol/sol.hpp

Additionally to make things weird Visual Studio has a bug or a "feature" where running in Debug mode will ignore exceptions that are not catched and just continue executing the code on next line.
In Release compilation unhandled exceptions abort as I expect.
So debug causes behavior like this:

Debugging doesnt feel that reliable all of a sudden...

When we push stuff to the stack push_popper can be used, error occurs, we push error message, we throw.
Throwing will destruct all objects, so then the popping starts to happen for push_popper and the error message is popped along with the other values we pushed.

Oooh, I wasn't even considering the destructors.... thanks! I'll try to debug those specifically.

I have decided that this is a sufficiently weird case that I can't actually fix it.
The primary problem is that the only true fix I have basically means I need to mess with performance and add tons of checks for various ill-formed states of the program.
I would much rather that you are careful with your accesses and use the proxy methods on the lua["x"]; type or do sol::optional<int> x = lua["x"];. It sucks to have to explicitly annotate things but I don't have a better control mechanism for when things go to hell.


deno: 1.19.1 -> 1.20.5

Description of changes

Bump deno to 1.20.5
cc: @kidonng
Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

closes #167715

Result of nixpkgs-review pr 167500 run on x86_64-linux 1

1 package built:

deno

Built fine this morning

Built fine this morning

Good, because my Raspberry Pi crashes for unrelated reasons while building. I should probably swap out my power supply :thinking:

Result of nixpkgs-review pr 167500 run on x86_64-linux 1

1 package built:

deno


Add IconType enum to support more mime types in Icon

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
This change is for future compatibility. We should ensure that this is possible with JDA in the case that discord changes the behavior for this.

@DV8FromTheWorld I always put enum parameters first in methods. I feel like the sentiment that optional parameters are at the end of the argument list is subjective. On that note, I'm indifferent about the order.

@MinnDevelopment I think the way that we've been doing it thus far is optional parameters last, but with the changes made to the sendMessage method for v4, I'm not sure anymore.
I do think it is a bit more clear having them come at the end as it follows what the JDK does with its methods, but I'm only at like a 4 / 10 on the Care O' Meter


AST: Add a check to see if resolver is not null

Occasionally resolver is null and causes resolver->resolveDeclSignature(ctor); to crash SourceKitService. Checking to see if resolver is not null fixes this crash.

I remember considering adding this fix before (#7284), but it wasn't actually the right fix. If you have an initializer without an interface type but you're past type-checking, you're not going to get a good answer out of this function anyway. What's calling this function that causes it to crash?

(To be clear, #7284 is a workaround as well, but there's a much bigger architectural problem that we're not going to fight at the moment.)

Here is at least one place that is passing nullptr
https://github.com/apple/swift/blob/master/lib/AST/ASTContext.cpp#L1940-L1941
If #7284 fixes the side effect I have been observing (SourceKitService crashing non-stop while using Xcode, making it practically unusable.) I'm fine with that. I've tried to make a sample project that exhibits this this problem. so far only my very large project is the only place I can reproduce it.
Even if it doesn't fix the bigger problem it does prevent SourceKitService from crashing and allows Xcode to be usable again.

Yes, it's okay to pass nullptr at certain points in the program, because the initializer should already have an interface type by that point. It's unclear that simply skipping this step leads to a correct program.
We wouldn't take this without a test case anyway, so I'm afraid we'll have to figure out a way to reduce your large project. Even if you can't share it publicly on https://bugs.swift.org, is there any chance you can attach it to a Radar (https://bugreport.apple.com)?

I found the root cause of the problem, at least in SourceKit indexing: https://github.com/apple/swift/pull/8459
Note that most likely @jrose-apple's case that was hit in SIL passes is unrelated, but we haven't seen nearly as many external dupes of this as the SourceKit one.
@mcmurrym Thanks for taking the time to look at this originally -- the right fix was only slightly more elaborate than yours, basically instead of coping with a null resolver, we try harder to pass one in. I'm going to close this PR now, but looking forward to more contributions in the future ;-)


600ms startup time when loading dart file

Using the master branch no modifications, when I open an empty .dart file get 600ms startup time, when open .cpp file I get 250ms load time. In both instances the LazyDone + LazyStart times are very reasonable less than 100ms combined. But the UIEnter time is very slow. I am using running sway, foot,and neovim 0.9.0 from package manager. Sorry if dumb question but dont know where to start on this one.

you can run with --startuptime flag, and then see what the output says. Do it happen every time you open a dart file? it's possible that the dart LSP or something is blocking on startup

Thanks for the response! Yes it is only when opening dart files and happens every time. I did what you said and here is biggest time delay. I have done a TSInstall for dart.
091.739  001.216  000.047: require('luasnip.loaders.from_snipmate')
091.927  000.123  000.123: require('luasnip.util.jsonc')
091.932  000.180  000.057: require('luasnip.loaders.from_vscode')
587.558  000.093  000.093: require('vim.treesitter.highlighter')
587.925  000.048  000.048: require('vim.uri')
599.697  000.082  000.082: require('editorconfig')
601.156  512.024: opening buffers
602.523  001.367: BufEnter autocommands
602.528  000.005: editing files in windows


Node is not connected shown in case of a warning

Node is not connected shown on the info page, even if there is just a warning on the status page.
Example:

This is still an issue in the current version, using desktop 0.15.0-pre.14


Revert "#474 Only ":reload" file if it is equal to "intero-repl-last-loaded""

Reverts commercialhaskell/intero#498
If the module isn't successfully loaded, ghci just says:
 Ok, modules loaded: none.
 Ok, modules loaded: none.
 Ok, modules loaded: none.
 Ok, modules loaded: none.

Strange, when loading a file that doesn't compile I get output like:
λ [1 of 3] Compiling GhciTypes        ( /Users/kryoxide/programming/Haskell/intero/src/GhciTypes.hs, interpreted )
[2 of 3] Compiling GhciInfo         ( /Users/kryoxide/programming/Haskell/intero/src/GhciInfo.hs, interpreted )
[3 of 3] Compiling GhciFind         ( /Users/kryoxide/programming/Haskell/intero/src/GhciFind.hs, interpreted )
/Users/kryoxide/programming/Haskell/intero/src/GhciFind.hs:453:1:
    parse error (possibly incorrect indentation or mismatched brackets)
Failed, modules loaded: GhciInfo, GhciTypes.

And everything works just fine in my further workflow. If you have any idea what could cause the problem on your machine I will try to fix it.


How am I supposed to install it?

lit? npm? download it and ...?
thank you!

i haven't updated this to the new version of luvit using lit... it will happen

so its impossible to use it right now?

this will work with luvit:master. We have been using the tap module within luvit's luvi-up branch.

fixed it up... you can use<EMAIL_ADDRESS>with the luvi-up branch of luvit.

Thank you for the fast response :)

welcome!


Add .profile on README for Mac users

On MacOS the right file to include the source ~/.bash-powerline.sh should be .profile not .bashrc, so i included a hint for those users.

right on @arthurbarros  👍

I'm afraid this is incorrect. .profile can also be read by other shells like Zsh in sh-compatible mode. bash-powerline contains lots of Bash-only stuff and is therefore better to be sourced in .bashrc to avoid issues with other shell users.


Generate meaningful name for custom notebook image

Goal
Allow users to set a display name for the custom images they are importing and generate a meaningful k8s name
Dependency issue
Tracker: #1254
Itemized goals
When adding custom images, the image stream names are not intuitive - eg. 'byon-xxxx'. This makes it hard to differentiate across custom images. Would like the name to be more intuitive and not include 'byon' prefix.

Idea is to have an object (Kubernetes) name and display name for each custom NB image. The image stream would be "Custom: (Kubernetes name)".
Users should be able to access the Kubernetes name from the custom NB images admin UI, so they can find it in the list of image streams.
Kubernetes name must be unique (for custom NB images); display name does not need to be unique. Display name is what users will see in the UI for selecting images.
When adding a URL for image location field, remove preceding and trailing white spaces.

Aspects continued elsewhere
No response
Mocks
No mocks

@DaoDaoNoCode additionally, if not done so yet, a meaningful name should also be set for imagestream metadata.annotations opendatahub.io/notebook-image-name, like here

@shalberd We tend to use this annotation for internal resources in the dashboard, both of them are to add descriptions and display names to render in the dashboard. I wasn't aware of that annotation in the notebook, so it's up to @DaoDaoNoCode to see which one fits best for our logic

@shalberd Hmm, interesting, I didn't know we have this annotation before. I guess if we want to also show the OOTB images in the future, we should adopt the same annotations. So I agree that we should use opendatahub.io/notebook-image-name and opendatahub.io/notebook-image-desc for the display name and description. So the order of finding the display name could be opendatahub.io/notebook-image-name || openshift.io/display-name || metadata.name

@harshad16 do you have documentation behind these annotations?

We haven't clearly documented the annotation in the imagestream.
We will include it , tracker issue: https://github.com/opendatahub-io/notebooks/issues/138

Verified on RHODS 2.4:


App install banner #427

This should resolved #427
Opening PR for reviews and suggestions. Sorry still learning reactjs( and hating it so far ) so bear my bad ugly for now :)

Deployed to: http://staging.quran.com:32941

Hi,
i seem to be getting 2 banners on mobile safari

On 25 Nov 2016, at 1:10 PM, Naveed Ahmad<EMAIL_ADDRESS>wrote:
This should resolved #427
Opening PR for reviews and suggestions. Sorry still learning reactjs( and hating it so far ) so bear my bad ugly for now :)
You can view, comment on, or merge this pull request online at:
https://github.com/quran/quran.com-frontend/pull/508
Commit Summary
wip on app install banner #427
File Changes
A src/components/SmartBanner/index.js (193)
M src/config.js (4)
M src/containers/App/index.js (3)
A src/styles/components/SmartBanner.scss (299)
M src/styles/main.scss (1)
Patch Links:
https://github.com/quran/quran.com-frontend/pull/508.patch
https://github.com/quran/quran.com-frontend/pull/508.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@redhae thanks for reporting. Did you saw two banners simultaneously or closed the banner and saw again on page refresh ?
Second case is known issue, working on that. But if first case if true, can you send screenshot or something to help me debugging ?

Deployed to: http://staging.quran.com:32942

@naveed-ahmad amazing work bro. I don't have an android to test it on. Can you include some screenshots?
Happy for this to be merged in.

@sabeurthabti preview for different mobiles and tables:
https://drive.google.com/file/d/0B-_ddcseNUwLeHZKYU84V3lzRUk/view

Deployed to: http://staging.quran.com:32950

@naveed-ahmad https://egghead.io/lessons/react-react-fundamentals-development-environment-setup?course=react-fundamentals

Deployed to: http://staging.quran.com:32952

Deployed to: http://staging.quran.com:32953

Deployed to: http://staging.quran.com:32954

@arshdkhn1 removed the fonts, thanks for the course link.

@naveed-ahmad

This is how banner looks in my mobile browser.
Few suggestions -
1- Quran logo is too blurry to be used.
2- I see styling is different for different browsers/os. Did you change the colors of banner recently ?
3- IMO colors should be used that are complementary to our sites' color. They are looking a bit off the color scheme that is used in site.

Deployed to: http://staging.quran.com:32971

Android:

iOS:

strange, the build is failing because of the favicon for some odd reason...
 [859] ../static/images/favicon.ico 215 bytes {0} [optional] [built] [failed] [1 error]
 [860] ../static/images/titles ^\.\/.*\.svg$ 3.35 kB {0} [built]
[1577] ../static/images ^\.\/.*$ 8.2 kB {0} [built]
[1578] multi main 40 bytes {0} [built]
    + 1654 hidden modules

WARNING in ../static/images/favicon.ico
Module parse failed: /quran/static/images/favicon.ico Unexpected character '' (1:0)
You may need an appropriate loader to handle this file type.
(Source code omitted for this binary file)
 @ ../static/images ^\.\/.*$
 @ ./components/SmartBanner/index.js
 @ ./containers/App/index.js
 @ ./routes.js
 @ ./client.js
 @ multi main

rebuild

out of curiosity, does this use whatever banner stuff Apple and Google have (i.e. will it detect if the app is installed and either not show the banner or show it with 'open' instead?)

Deployed to: http://staging.quran.com:32972

@ahmedre it is using cookies to decide either to show banner or not. If user click on install then it won't show banner for 90 days, if user click close, then it will not show it for 15 days.

@naveed-ahmad imho, we should use whatever Apple and Google provide because they also know if the app is installed and don't bother the user. ex on iOS, i see double, and on Android, i have the app installed and see:

@ahmedre what's your browser version ? Already heard some devices are showing the banner twice, looks like first is native banner and second one is custom one.

Safari on iOS 10.1.1

not to mention for Android, the native one is a lot nicer because it also pulls in info from Google Play and looks platform native - https://developer.android.com/distribute/users/banners.html

For android, native banner is available only for Chrome 44 and  greater version:
https://developers.google.com/web/updates/2015/03/increasing-engagement-with-app-install-banners-in-chrome-for-android
And native iOS banners are available for safari 6 or greater.

can we do both then? native if we can, else fallback to this? sorry, i am not trying to be difficult!

Yep working on it.

@naveed-ahmad is this good to go?

@mmahalwy No, working on showing native banner for advance browsers and fallback to custom banner for older browsers. Ill try to finish this tonight.

Deployed to: http://staging.quran.com:32981

Deployed to: http://staging.quran.com:32982

good to go?

@mmahalwy Gooood to go!

Deployed to: http://staging.quran.com:32983

Deployed to: http://staging.quran.com:32984

Deployed to: http://staging.quran.com:32985

Deployed to: http://staging.quran.com:32986

good now? No more commits? :)

@mmahalwy lol yeah good to go now.


[Snyk] Upgrade @typescript-eslint/eslint-plugin from 5.59.11 to 5.60.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @typescript-eslint/eslint-plugin from 5.59.11 to 5.60.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 22 days ago, on 2023-06-19.

Release notes

Package name: @typescript-eslint/eslint-plugin

5.60.0 - 2023-06-195.60.0 (2023-06-19)
Features

eslint-plugin: [restrict-plus-operands] add allow* options (#6161) (def09f8)

5.59.12-alpha.0 - 2023-06-12

5.59.11 - 2023-06-125.59.11 (2023-06-12)
Note: Version bump only for package @ typescript-eslint/typescript-eslint

    from @typescript-eslint/eslint-plugin GitHub release notes

Commit messages

Package name: @typescript-eslint/eslint-plugin

c09b1c0 chore: publish v5.60.0
082c948 chore: update sponsors (#7094)
def09f8 feat(eslint-plugin): [restrict-plus-operands] add allow* options (#6161)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch and project coverage have no change.

Comparison is base (675ccf5) 90.34% compared to head (82096b8) 90.34%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #267   +/-   ##
=======================================
  Coverage   90.34%   90.34%
=======================================
  Files           9        9
  Lines         290      290
=======================================
  Hits          262      262
  Misses         28       28

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fix: Json::op_equal returns false when the keys of 2 maps have different order

#1188

Pull Request Test Coverage Report for Build 3785
Details

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

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

builtin/linked_hash_map.mbt
1
2
50.0%

Totals

Change from base Build 3783:
0.01%

Covered Lines:
4319

Relevant Lines:
5155

💛  - Coveralls


Crash on World Load

Going to be resurrecting #1 here for different reasons.
I was crashing with this stacktrace for a while in 1.18.5 when trying to update a world and resolved to just wait for 1.19.
Updating in 1.19, everything went smoothly, I switched between a few worlds including the one that was previously problematic, and had been playing on and off for hours before the problem started.
While I was in the singleplayer world, an (unrelated) crash occurred (iirc it was driver related rather than game), and since then I have not been able to open that world with ProspectTogether enabled.
Can provide configs/worldmaps (if you can tell me which one/how to find it, I haven't figured out how to get UID from a world) or a download for the world if needed for debugging.
Running on 64 bit Windows 10.0.19045.0 with 32640 MB RAM
Game Version: v1.19.1 (Stable)
1/24/2024 10:31:01 AM: Critical error occurred in the following mod<EMAIL_ADDRESS>Loaded Mods<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>System.NullReferenceException: Object reference not set to an instance of an object.
   at ProspectTogether.Client.ProspectorOverlayLayer.GuiDialogWorldMapOpenPatch.Postfix(EnumDialogType type) in D:\a\ProspectTogether\ProspectTogether\src\Client\ProspectorOverlayLayer.cs:line 430
   at Vintagestory.GameContent.GuiDialogWorldMap.Open_Patch2(GuiDialogWorldMap this, EnumDialogType type)
   at Vintagestory.GameContent.WorldMapManager.ToggleMap(EnumDialogType asType) in VSEssentials\Systems\WorldMap\WorldMapManager.cs:line 306
   at Vintagestory.GameContent.WorldMapManager.OnLvlFinalize() in VSEssentials\Systems\WorldMap\WorldMapManager.cs:line 220
   at Vintagestory.Client.NoObf.ClientEventAPI.TriggerLevelFinalize() in VintagestoryLib\Client\API\ClientEventAPI.cs:line 372
   at Vintagestory.Client.NoObf.ClientSystemStartup.HandleLevelFinalize(Packet_Server packet) in VintagestoryLib\Client\Systems\Startup.cs:line 1040
   at Vintagestory.Client.NoObf.ProcessPacketTask.ProcessPacket(Packet_Server packet) in VintagestoryLib\Client\Systems\NetworkProcess.cs:line 446
   at Vintagestory.Client.NoObf.ProcessPacketTask.Run() in VintagestoryLib\Client\Systems\NetworkProcess.cs:line 440
   at Vintagestory.Client.NoObf.ClientMain.ExecuteMainThreadTasks(Single deltaTime) in VintagestoryLib\Client\ClientMain.cs:line 769
   at Vintagestory.Client.GuiScreenConnectingToServer.RenderToDefaultFramebuffer(Single dt) in VintagestoryLib\Client\MainMenu\Screens\GuiScreenConnectingToServer.cs:line 313
   at Vintagestory.Client.ScreenManager.Render(Single dt) in VintagestoryLib\Client\ScreenManager.cs:line 675
   at Vintagestory.Client.ScreenManager.OnNewFrame(Single dt) in VintagestoryLib\Client\ScreenManager.cs:line 650
   at Vintagestory.Client.NoObf.ClientPlatformWindows.window_RenderFrame(FrameEventArgs e) in VintagestoryLib\Client\ClientPlatform\GameWindow.cs:line 78
   at OpenTK.Windowing.Desktop.GameWindow.Run()
   at Vintagestory.Client.ClientProgram.Start(ClientProgramArgs args, String[] rawArgs) in VintagestoryLib\Client\ClientProgram.cs:line 318
   at Vintagestory.Client.ClientProgram.<>c__DisplayClass9_0.<.ctor>b__1() in VintagestoryLib\Client\ClientProgram.cs:line 128
   at Vintagestory.ClientNative.CrashReporter.Start(ThreadStart start) in VintagestoryLib\Client\ClientPlatform\ClientNative\CrashReporter.cs:line 93

Hi @caucow
I just did some testing. I was able to reproduce this error after making the config for the current world invalid.
You should have a log entry in Logs/client-main.txt telling you what went wrong. In my case I put invalid json data in the config file (ProspectTogetherClient-3e7dde45-b410-431d-b447-625d07b048a6.json) and got the following error in the log:
24.1.2024 21:49:19 [Error] ProspectTogetherClient-3e7dde45-b410-431d-b447-625d07b048a6: Failed loading modconfig file at 'ModConfig/ProspectTogetherClient-3e7dde45-b410-431d-b447-625d07b048a6.json', with an error of 'Newtonsoft.Json.JsonReaderException: Error parsing comment. Expected: *, got d. Path '', line 1, position 2.
   at Newtonsoft.Json.JsonTextReader.ParseComment(Boolean setToken)
   at Newtonsoft.Json.JsonTextReader.ParseObject()
   at Newtonsoft.Json.JsonReader.ReadAndAssert()
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
   at Vintagestory.Common.APIBase.LoadModConfig[T](String filename) in C:\Users\Tyron\Documents\vintagestory\game\VintagestoryLib\Common\API\APIBase.cs:line 102
   at Foundation.ModConfig.ApiExtensions.LoadOrCreateConfig[TModConfig](ICoreAPI api, String filename, Boolean required) in D:\a\ProspectTogether\ProspectTogether\src\Foundation\ModConfig\ApiExtensions.cs:line 32'! Stopping...

So from there you should be able to identify the corrupt config file and either fix the error by hand or delete the file, in which case the mod will create a new default config. I hope this solves your problem.

So it seems the error was not in the config, but in the prospecting map data. I'm guessing the first crash somehow resulted in the map data being deleted, so the old prospectorInfo file was copied in place of the missing PT file and failed while attempting to migrate (no "Values" array, only a "Message" string). Then due to the error it never registered the PT map layer and... NPE
The file at fault:
bad_prospectTogetherClient.json
All logs relevant to PT from client-main:
24.1.2024 16:12:39 [Error] Failed to migrate prospecting data file at 'C:\Users\caucow\AppData\Roaming\VintagestoryData\ModData\3b8a8084-d50f-4b0e-857c-611f5c4fd66f\prospectTogetherClient.json', with an error of 'System.NullReferenceException: Object reference not set to an instance of an object.
   at ProspectTogether.ProspectTogetherModSystem.MigrateSerializationFormatToV1(String filename, ICoreAPI api) in D:\a\ProspectTogether\ProspectTogether\src\ProspectorTogetherModSystem.cs:line 128'! Either delete that file or check what is causing the problem.
24.1.2024 16:12:39 [Error] [prospecttogether] An exception was thrown when trying to start the mod:
24.1.2024 16:12:39 [Error] [prospecttogether] Exception: Object reference not set to an instance of an object.
   at ProspectTogether.ProspectTogetherModSystem.MigrateSerializationFormatToV1(String filename, ICoreAPI api) in D:\a\ProspectTogether\ProspectTogether\src\ProspectorTogetherModSystem.cs:line 128
   at ProspectTogether.ProspectTogetherModSystem.StartClientSide(ICoreClientAPI api) in D:\a\ProspectTogether\ProspectTogether\src\ProspectorTogetherModSystem.cs:line 46
   at Vintagestory.Common.ModLoader.TryRunModPhase(Mod mod, ModSystem system, ICoreAPI api, ModRunPhase phase) in VintagestoryLib\Common\API\ModLoader.cs:line 654
24.1.2024 16:12:39 [Error] Failed to start system ProspectTogether.ProspectTogetherModSystem

...

1/24/2024 4:12:41 PM: Critical error occurred in the following mod<EMAIL_ADDRESS>Loaded Mods<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>System.NullReferenceException: Object reference not set to an instance of an object.
   at ProspectTogether.Client.ProspectorOverlayLayer.GuiDialogWorldMapOpenPatch.Postfix(EnumDialogType type) in D:\a\ProspectTogether\ProspectTogether\src\Client\ProspectorOverlayLayer.cs:line 430
   at Vintagestory.GameContent.GuiDialogWorldMap.Open_Patch2(GuiDialogWorldMap this, EnumDialogType type)
   at Vintagestory.GameContent.WorldMapManager.ToggleMap(EnumDialogType asType) in VSEssentials\Systems\WorldMap\WorldMapManager.cs:line 306
   at Vintagestory.GameContent.WorldMapManager.OnLvlFinalize() in VSEssentials\Systems\WorldMap\WorldMapManager.cs:line 220
   at Vintagestory.Client.NoObf.ClientEventAPI.TriggerLevelFinalize() in VintagestoryLib\Client\API\ClientEventAPI.cs:line 372
   at Vintagestory.Client.NoObf.ClientSystemStartup.HandleLevelFinalize(Packet_Server packet) in VintagestoryLib\Client\Systems\Startup.cs:line 1040
   at Vintagestory.Client.NoObf.ProcessPacketTask.ProcessPacket(Packet_Server packet) in VintagestoryLib\Client\Systems\NetworkProcess.cs:line 446
   at Vintagestory.Client.NoObf.ProcessPacketTask.Run() in VintagestoryLib\Client\Systems\NetworkProcess.cs:line 440
   at Vintagestory.Client.NoObf.ClientMain.ExecuteMainThreadTasks(Single deltaTime) in VintagestoryLib\Client\ClientMain.cs:line 769
   at Vintagestory.Client.GuiScreenConnectingToServer.RenderToDefaultFramebuffer(Single dt) in VintagestoryLib\Client\MainMenu\Screens\GuiScreenConnectingToServer.cs:line 313
   at Vintagestory.Client.ScreenManager.Render(Single dt) in VintagestoryLib\Client\ScreenManager.cs:line 675
   at Vintagestory.Client.ScreenManager.OnNewFrame(Single dt) in VintagestoryLib\Client\ScreenManager.cs:line 650
   at Vintagestory.Client.NoObf.ClientPlatformWindows.window_RenderFrame(FrameEventArgs e) in VintagestoryLib\Client\ClientPlatform\GameWindow.cs:line 78
   at OpenTK.Windowing.Desktop.GameWindow.Run()
   at Vintagestory.Client.ClientProgram.Start(ClientProgramArgs args, String[] rawArgs) in VintagestoryLib\Client\ClientProgram.cs:line 318
   at Vintagestory.Client.ClientProgram.<>c__DisplayClass9_0.<.ctor>b__1() in VintagestoryLib\Client\ClientProgram.cs:line 128
   at Vintagestory.ClientNative.CrashReporter.Start(ThreadStart start) in VintagestoryLib\Client\ClientPlatform\ClientNative\CrashReporter.cs:line 93
Crash written to file at "C:\Users\caucow\AppData\Roaming\VintagestoryData\Logs\client-crash.txt"

You are right. It seems that even older version of the ProspectorInfo files did not even have a Values attribute. I have adjusted the import logic to handle this case and uploaded a new version.


fix(environment): change bundler to vite and package manager to pnpm

:tada: This PR is included in version 0.7.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[xmlsec] add pkgconfig files

Describe the pull request

What does your PR fix?
Fixes #16830
Also fixes the library filename, which should actually be libxmlsec1. This follows the upstream naming scheme.
I believe this port in general should be renamed to xmlsec1, which is how vast majority of repositories call it: https://repology.org/project/xmlsec/versions . I did not, however, make that change as part of this PR.

Which triplets are supported/not supported? Have you updated the CI baseline?
No change

Does your PR follow the maintainer guide?
Yes

I believe this port should be renamed to xmlsec1, which is how vast majority of repositories call it: https://repology.org/project/xmlsec/versions

I see those osgearth errors, but I can't reproduce them locally. Once it build successfully, the other time (after rm -rf installed packages) it fails with:
CMake Error at src/osgEarth/CMakeLists.txt:791 (PROTOBUF_GENERATE_CPP):
  Unknown CMake command "PROTOBUF_GENERATE_CPP".`

I don't see how these errors are related to this PR. I especially don't understand why that many ports are being rebuilt as part of CI/CD run, since none of them depend on xmlsec. In fact none of the ports in the repository depend on it. I suspect this must have to do with your CI/CD and that osgearth was recently modified, together with dependencies, and has not cached properly? Before you ask, the branch is up to date with master.

/azp run

This looks great! Thanks @wrobelda :)


sqs: Use unix_time in place of time.time()

unix_time() from moto.core.utils is used as the time source through moto,
and it is identical to time.time() in output. Hence, using unix_time()
since it makes mocking easier during testing (when time is mocked out).

Coverage decreased (-0.07%) to 93.91% when pulling af1cb5194655dc6e7ac18ebe39b933f65ea8c1c6 on ronin13:unix_time_in_models into a20906ff155494c4994db9a3faeacb141079363c on spulec:master.

Thanks!


Ionic cordova Run Android - Failed to apply plugin [class 'com.google.gms.googleservices.GoogleServicesPlugin']

Expected Behaviour
when running command:  "ionic cordova run android", the build should succeed and run on an AVD or physical mobile device.
Actual Behaviour
The build fails when running the command:  "ionic cordova run android"
Error:

Where:
Script 'C:\Mobile APPS\Iyeza Client APP\platforms\android\phonegap-plugin-push\patient-push.gradle' line: 35

What went wrong:
A problem occurred evaluating project ':app'.

 Android 5.0 or iOS 9.2.1)
Cordova Platforms     : android 7.0.0
Ionic Framework    : ionic-angular 3.9.2
(Android) What device vendor (e.g. Samsung, HTC, Sony...)

Nokia 3 (Physical mobile device)
AVD (Android 8.0 (Google APIs))

Cordova CLI version and cordova platform version
cordova --version                                    8.0.0
cordova platform version android           7.0.0

Plugin version
cordova plugin version | grep phonegap-plugin-push   ^4.16.0

Sample Push Data Payload
N/A
Sample Code that illustrates the problem
N/A
Logs taken while reproducing problem
Please see the CMD output in the following link:
https://gist.github.com/shaunsideclick/c08d9812360bd5651629cb7ae30d5672

the plugin version you provided is not valid, looks like the ionic-native/push version and not the phonegap-plugin-push version.
Anyway, if you are using cordova-android 7.0.0, you won't get latest version of the plugin as it requires cordova-android 7.1.x, so update your platform to latest

Hi
Thank you for your response. I upgraded cordova-android 7.0.0 about 15 min
ago and the build still failed.
I then commented out the "apply plugin:
com.google.gms.googleservices.GoogleServicesPlugin" in the
cordova-support-google-services build.gradle file and could then do ionic
cordova run android.
Busy testing it now and implementing.
I will keep you updated.
Thank you so much for your feedback.
Shaun
On Mon, Oct 22, 2018 at 2:46 PM jcesarmobile<EMAIL_ADDRESS>wrote:

Closed #2593
https://github.com/phonegap/phonegap-plugin-push/issues/2593.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phonegap/phonegap-plugin-push/issues/2593#event-1917443926,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AqT5r5FBhGGpBMbSeil2yTphGGEFldN2ks5unb4pgaJpZM4XzRzm
.

Latest is 7.1.1


Breakpoints: When using mouse scroll wheel to click on one of the Breakpoints, have it remove clicked Breakpoints, similar to closing tabs

This feature request is about accessibility. This is only for removing 1 or 2 Breakpoints from a list of, say 30 breakpoints across multiple files in the Breakpoints List.
Right now, there is no Close Button in the Breakpoints List when debugging a program in Visual Studio Code. The only way to remove 1 or 2 breakpoints right now is to:

Click on the Breakpoint from the Breakpoints List.
Untick the red dot on the line highlighted by VS Code.
Observe the Breakpoint is now removed from the Breakpoints List.

I would like to request that we can simplify this process by using the middle scroll wheel click on the mouse, so when you click on one of the Breakpoints with the scroll wheel button, it does all of the 3 steps mentioned above.
And, it can act as a Remove Single Breakpoint mouse click action. Just like the middle scroll wheel click on tabs to close 1 tab out of many other tabs.

@tommai78101 you can already use "delete" or "cmd + backspace" on the mac to remove the breakpoint. That way you can easily remove multiple.
I do not understand what this has to do with accessibility since all these actions are already accessible.
A middle click to remove a breakpoint would be consistent with our middle click to remove a tab.
Thus I accept prs that fix this
Code pointer https://github.com/Microsoft/vscode/blob/4fa62b246c4305b4ca00507551f77c58135ce928/src/vs/workbench/parts/debug/browser/breakpointsView.ts#L90
Simply add a listener in the list for the mouse click, check if it is the middle one and if yes remove the breakpoint.
If nobody volunteres I can look into this in the future

@isidorn I am not aware of using "delete" keypress to remove the breakpoint.
How is this triggered? If I press "delete", it would remove a character in my source code.

@tommai78101 focus on the breakpoints view in the debug viewlet
F1 > Debug: focus breakpoints

@ris58h It's more about middle scroll wheel click to remove breakpoints, and I think that may be a separate issue.


Customize Histogram for TASKS_RUNTIME and LATENCY

The default histogram buckets are on the order of miliseconds and seconds. For celery if you have long running tasks like re-encoding a video and then uploading gigabyte sized files it puts all the tasks in the INF bucket. A solution could be to add a config.py which lets you customize the histogram. The super nice part is that it'd be pretty easy to put the file as a ConfigMap in Kubernetes or build the container with the extra file(basically as an overlay).
I'm happy to put up a pull request for this would be useful for the community.

This would be super useful for me as well :+1:

A solution could be to add a config.py which lets you customize the histogram.

I think specifying the values as a comma-separated list in the command options would suffice. A .py would only have the advantage to dynamically choose bucket values, which is probably hard to do sensibly while maintaining stable time series.
An alternative would be to use summary metrics instead or additionally to the histogram metrics.

This would be super helpful, so far there seems to be no celery exporter with such a feature.
#17 would solve this, is there a way to help getting that merged?


Does keras have any plans to support multi-dimensional LSTM？

You mean stacked LSTMs? Yes, it currently does. Also, you might want to ask this type of question on the keras-users google group.

@jmhessel  No, I meant the LSTM described in this paper: http://arxiv.org/pdf/0705.2011.pdf

Oh, I see. I can't think of how that could be implemented at the moment, no.

No, if you are doing funky things like you are almost certainly better off working directly in Theano. There are assumptions in Keras that make it unsuitable for certain types of research, and multi-dimensional RNNs in one of these cases.

What do you think about modifying DeepRNN such that?

Hidden layer matrix vector instead will be +1 dimension with matrix with shape ((shape_hidden_layer_units), N - number of dimensions in the sequence)
Instead of accessing up to -depth steps, there will be only access the -size per each dimension on each step and taking only [(n_hidden_layer_units), nth dimension]

Does that sound feasible?
Sorry if it is not that clear and not well formulated, I can explain in depth if there is a need, I hope you can get the idea.

＠davidbuniat  that sounds very interesting! Could you elaborate it a little more?
Thanks!

@davidbuniat

@shampool Suppose you have an image (2D, with 300x400px). Reshape it into 1D vector (with 120,000px) on each rnn scan step access h-1 element and (h-400) element. Number 400 is based on how the reshape is done on your image. And you can expand this into multidimensional case.
I might be wrong, but seems scan supports multiple access to previous ones. I need MDRNN as well and I think to implement Grid LSTM which seems more powerful. In case of success in implementation let me know, I will as well.

＠davidbuniat  That sounds doable. If we make the taps fixed, things might be easier. But in certain types of traversal of the 2D grid. We may need taps to be able to chang over time. I am not sure if scan supports that.
Will let you know, if I finally make it work.
Thanks!

@davidbuniat

@davidbuniat
Wondering if you had positive results  on this :
(2D, with 300x400px). Reshape it into 1D vector (with 120,000px)

No ,@arita297 as I said before I moved to Torch and I got positive results on MNIST dataset with 28x28 reshaping to 784px which I will agree is very far from 120,000px. Reshaping into 1D is just an idea how you could overcome Theano limitations. Furthermore, you might also consider to split the 120,000px into chunks and feed iteratively (should be very careful).

"No, if you are doing funky things like you are almost certainly better off working directly in Theano. There are assumptions in Keras that make it unsuitable for certain types of research, and multi-dimensional RNNs in one of these cases."
@fchollet : Is it thus pointless that I try to make my own layer of MD-LSTM, from existing implementation, fit the "build-your-own-layer" canvas ?

so , does keras support multi-dimensional LSTM now ?

+1

+1

I have a 2D implementation of several RNN, but they are pretty old and for pre-functional keras api. Will put it somewhere when I got time.

+1

:+1:

+1

+1

I really need to 2_dimentional LSTM in kers for my project and i have a little time,
Please give me this code
Thanks alot


Mantis Bug Tracker for tracking system and project management tool

Describe the issue
We propose using Mantis Bug Tracker as a tracking tool for clients and internally.
It is an open source web-based application tool. It is used for Bug Tracking throughout Defect Lifecycle. Along with bug tracking, Mantis supports Release Features to manage various releases of a project or a software.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context
Some features of Mantis Bt are:
• Email notifications: It sends out emails of updates, comments, resolutions to the concerned stakeholders.
• Access Control: You can control user access at a project level
• Customize: You can easily customize Mantis as per your requirements.
• Mobile Support: Mantis supports iPhone, Android, and Windows Phone Platforms.
• Plugins: An ever-expanding library of plugins to add custom functionality to Mantis Issue Tracker.
• Reporting and graphing built
Definition of done

Look at our current processes and document painpoints if any with the current tool.

Not doing