TypeScript doesn't set selection range properly

open document symbols on a test-files
:bug: the selection range is always the (full) range

@mjbvz fyi - I have pushed a commit that fixes most of these cases but there is still an issue with TS and I believe it's a tsserver bug. With the snippet below, the IFoo2 will have two spans but only one nameSpan and therefore only is contained. Do you know if that's a known bug to them?
interface IFoo2 {
    foo: any;
}

interface IFoo2 {
    foo2: any;
}
``

Thanks for the fix @jrieken
Opened https://github.com/microsoft/TypeScript/issues/35068 to track root cause upstream


refactor: HighlightedSql into functional component

SUMMARY
Changed HighlightSql.jsx from a class component to a functional component.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

This is what highlight text looks like now.
TEST PLAN
This passed all four tests in HighlightedSql_spec.jsx
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Yes, I would be happy to do that. I was running into that with the other
component that I was working on as well.
On Tue, Feb 9, 2021 at 6:51 PM Elizabeth Thompson<EMAIL_ADDRESS>wrote:

@eschutho commented on this pull request.
In superset-frontend/src/SqlLab/components/HighlightedSql.jsx
https://github.com/apache/superset/pull/13032#discussion_r573336259:

render() {
return (

const [modalTrigger, setModalTrigger] = useState(

can we put this up under line 45 just by convention? We can add a lint
rule for this in the future.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/superset/pull/13032#pullrequestreview-587125806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALVKTWE4F5B4C2M3VGKSYKDS6HDAJANCNFSM4XLMX6FA
.

--
Arash Afghahi

Codecov Report

Merging #13032 (fde9000) into master (2ce7982) will increase coverage by 8.79%.
The diff coverage is 86.36%.

@@            Coverage Diff             @@
##           master   #13032      +/-   ##
==========================================
+ Coverage   53.06%   61.85%   +8.79%
==========================================
  Files         489      546      +57
  Lines       17314    20152    +2838
  Branches     4482     5271     +789
==========================================
+ Hits         9187    12465    +3278
+ Misses       8127     7474     -653
- Partials        0      213     +213

Flag
Coverage Δ

cypress
?

javascript
61.85% <86.36%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...perset-frontend/src/views/CRUD/chart/ChartList.tsx
71.65% <ø> (-2.66%)
:arrow_down:

...rontend/src/views/CRUD/dashboard/DashboardList.tsx
73.87% <0.00%> (+7.56%)
:arrow_up:

...-frontend/src/SqlLab/components/HighlightedSql.jsx
86.20% <90.47%> (+69.54%)
:arrow_up:

superset-frontend/src/views/App.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/views/menu.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/views/index.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/SqlLab/index.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 456 more

Continue to review full report at Codecov.

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

I'll do another manual qa when you're ready.

Hello, I think I am ready. Would it be ok if I kept the ternary? Or is it better with the logical and?

I'll do another manual qa when you're ready.

Hello, I think I am ready. Would it be ok if I kept the ternary? Or is it better with the logical and?

Is this working? {(rawSql && rawSql !== sql) && ( It would make more sense than returning null. If you don't have a second value, then a ternary is a little strange.

Thank you @AAfghahi, this looks great! I manually tested and didn't find any regressions:

🏷 ready-to-merge


process.stdin.setRawMode(true);

I received this error from running the example found on npm. I am using node v0.10.35.
/node_modules/terminal-menu/index.js:62
process.stdin.setRawMode(true);
TypeError: Object # has no method 'setRawMode'
at new Menu(/terminal-menu/index.js:62:19)
at module.exports (node_modules/terminal-menu/index.js:9:12)
at Object. (test.js:3:45)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Function.Module.runMain (module.js:497:10)
at startup (node.js:119:16)
at node.js:929:3

Yeah, unfortunately I have the exact same problem

me too

Possible to upgrade your node? I mean the exception is clear, setRawMode  doesn't exist on process.stdin. The only way is using the newer version of node

Yep, that solved it for me!


[wip] bamoe-issues#41

This PR closes https://github.ibm.com/dba/bamoe-issues/issues/41

Is this still in draft?

@LightGuard yes, this is still a draft, will keep you informed.

Put it in and I’ll make sure it works.
On Tue, Jun 20, 2023 at 01:23 Jozef Marko @.***> wrote:

@.**** commented on this pull request.
In runtime-services-modeling/decisions-with-dmn.adoc
https://github.com/IBM/bamoe-docs/pull/28#discussion_r1234846728:

@@ -704,6 +701,7 @@ Some boxed expressions are more complex with greater detail and calculation, suc
.Boxed function expression for passenger reassignment
image::dmn-reassign-passenger.png[]

+// replace with https://raw.githubusercontent.com/kiegroup/drools/8.39.0.Final/kie-dmn/kie-dmn-core/src/test/resources/org/kie/dmn/core/0019-flight-rebooking.dmn ?
running asciidoctor -vw -R . -D output */.adoc
getting:
[image: image]
https://user-images.githubusercontent.com/8044780/247037611-853aafa2-68bc-4bf1-8f32-15a15da4895b.png
is that ok, and it wil be correct in official BAMOE docs?
—
Reply to this email directly, view it on GitHub
https://github.com/IBM/bamoe-docs/pull/28#discussion_r1234846728, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHKE6GT4IEXSW3KNVFNATXMFFY5ANCNFSM6AAAAAAZJHXTXY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Sent from Gmail Mobile

@LightGuard not sure about your last comment. From my site it is ready for a review.


Fix a multithreading violation introduced in RKEntityByAttributeCache

The last pull request I submitted introduced a multithreading violation, which is fixed now

Awesome, thanks!


Many bug fixes and optimizations for the two approval flows

@wendybujalski I went ahead and fixed the linting errors, please squash them in prior to merging.

bors merge


Text output XSLTs are not rendering correctly in the stepper

If you have an XSLT pipeline element that outputs text rather than XML. The text is not being displayed correctly in the stepper output pane.  Text that should be tab delimited appears something like:
<?xml version "1.1" encoding="UTF-8">value1<?javax.xml.transform.disable-output-escaping?><?javax.xml.transform.disable-output-escaping?>value2<?javax.xml.transform.disable-output-escaping?>...

Duplicate of #188


Webdriver v0.6.0 stalls installation via composer

Webdriver v0.6.0 stalls entire installation via composer.
Seems like there is a corrupted .zip archive. Same thing happens when I tried to extract the file manually.
This is part of console output from "composer -vvv update":
- Installing facebook/webdriver (v0.6.0)
Reading /vagrant/.composer/files/facebook/webdriver/2c5b305ea91b00ebbc433ad1663b7f16c1b31ec5.zip from cache
  Loading from cache
  Extracting archive
Executing command (CWD): unzip '/vagrant/web/libs/vendor/facebook/webdriver/95ab95adf32422d1886107cc620208a0' -d '/vagrant/web/libs/vendor/composer/8bf18198' && chmod -R u+w '/vagrant/web/libs/vendor/composer/8bf18198'

Your consoleoutput looks ab bit weiter but it says clearly installing from cache. The composer cache is lokaly on your machine. Habe you tried to perform a "composer clear-cache"? This will drop your cache and forces composer to load files fresh form remote repo.

Hi, thanks for answering. Yes. I've tried clearing the cache both manually and by composer clear-cache.
Output after running composer clear-cache and subsequent composer -vvv update:
- Installing facebook/webdriver (v0.6.0)
Downloading https://api.github.com/repos/facebook/php-webdriver/zipball/2c5b305ea91b00ebbc433ad1663b7f16c1b31ec5
    Downloading: 100%
Writing /vagrant/.composer/files/facebook/webdriver/2c5b305ea91b00ebbc433ad1663b7f16c1b31ec5.zip into cache
    Extracting archive
Executing command (CWD): unzip '/vagrant/web/libs/vendor/facebook/webdriver/b5b503fbc6aa2403f5df6b51ecd417b5' -d '/vagrant/web/libs/vendor/composer/1645bbe0' && chmod -R u+w '/vagrant/web/libs/vendor/composer/1645bbe0'

Hi, thanks for answering. Yes. I've tried clearing the cache both manually and by composer clear-cache.
Output after running composer clear-cache and subsequent composer -vvv update:
- Installing facebook/webdriver (v0.6.0)
Downloading https://api.github.com/repos/facebook/php-webdriver/zipball/2c5b305ea91b00ebbc433ad1663b7f16c1b31ec5
    Downloading: 100%
Writing /vagrant/.composer/files/facebook/webdriver/2c5b305ea91b00ebbc433ad1663b7f16c1b31ec5.zip into cache
    Extracting archive
Executing command (CWD): unzip '/vagrant/web/libs/vendor/facebook/webdriver/b5b503fbc6aa2403f5df6b51ecd417b5' -d '/vagrant/web/libs/vendor/composer/1645bbe0' && chmod -R u+w '/vagrant/web/libs/vendor/composer/1645bbe0'

I'm not able to reproduce this, it installs correctly for me through composer... I think this must be something on your end.  Please try from a different machine and/or network.


Make the library source Deno first.

Deno is a very close match for the browser.
Replace axios with shim around fetch api directly.
Fetch is in the box for Node 18.x, and can be shimmed with a global declaration global.fetch = require('node-fetch'); for the node build.

Deno requires a .ts as part of the import statement, by doing that change and adding a /mod.ts in the root of the project containing export * from "./src/index.ts it would be easy enough for deno users to reference from github raw content directly (in the near term).
From there, easy enough to make a clone of the src/ directory, and remove the .ts in the import lines... would only need to change the existing build configuration to make that copy, with the changes, then use tmp-src for tests and builds for browser and node usage.
Note: this would make testing slightly more complex... but could probably replace the test harness with Deno's built in testing framework.  Which would simplify dependencies and output.
After that, could do a publish to https://deno.land/x to make the package more easy to discover.  I know it may seem backward to suggest targeting the least used platform first, but it would be the easiest path towards the other two.

If there's interrest, I'd be happy to help on such a transition.

Hi,
Because of issues of bundling commonjs modules (axios, qs) and trying to make them work on every environment, I've rewritten the core and have removed the 3rd party dependency in v0.2.0.
Now the library is using only fetch() which should be supported by Deno (haven't got the time to test, but the documentation says that it is since Deno v1.0.0).
As a bonus the size was also reduced, from ~75kb to ~23kb (or ~5kb gzipped!).
Please give it a try and share your feedback.

I'll close the issue for now, because I'm not planning to make the library Deno first, because it will complicate the build process and will introduce some maintenance burden since I'm not very uptodate with Deno.
The recent fetch rewrite should simplify loading the SDK, but if anyone is still having difficulties, feel free to let me know and I'll research it more thoroughly.

The build process won’t be complicated if you use deno2node or dnt.

Here’s a Deno-first library we’ve been working on for years, that supports CJS Node.js and browsers as well: https://github.com/grammyjs/grammy

Yo. How things currently stand with Deno & Pocketbase?
@ganigeorgiev you're still working on Pocketbase?
If i know more about the issue i may be able to chip in;


Preprocess the target branch

Failure: https://dev.azure.com/azure-sdk/public/_build/results?buildId=1356766&view=logs&j=bc67675d-56bf-581f-e0a2-208848ba68ca&t=a6c890d8-b83f-504d-afc5-f7ec178ffaaf
Reason: The targetbranch returns refs/heads/main instead of main.
Fix: Trim the 'refs/heads' and make the target branch as parameters.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Rector PHP Downgrade Dry Run Github Action

Just a github action that run Rector on pull requests and point out where need to be downgraded to PHP 7.3 (the minimum PHP version Laravel 8 supports) and how.
It is a dry run output with the --dry-run flag. It does not attempt to make any commits.
This way if an external contributor accidentally uses later PHP versions syntax and features, the suggested fixes will be pointed out.
Credits to: https://getrector.org/blog/how-all-frameworks-can-bump-to-php-81-and-you-can-use-older-php
What i did in this PR not really the same as that link however. the article link described a more advanced version.

Thanks @ziming - we're currently deep into improving the onboarding, but we'll take a look shortly.
Cheers!

A new inspection was created.

The inspection completed: 678 Issues, 100 Patches


inconsistent library name completion behavior, varies by provider

Functional impact
Inconsistent experience.  Behavior for CDNJS is much more usable than with other providers.  This also applies in the Add Client Side Library wizard, though less severely.
Minimal repro steps

In the libman.json file, add a new library with the CDNJS provider
Type a library name, then tab to complete (Note: it completes the name@latest_version)
Add a library with jsdelivr or unpkg as the provider
Type a library name, then tab to complete

Expected result
Same behavior as CDNJS - complete with name and version. Latest version seems good; arguably it could open completion for the version, but I think latest is more often the right call.
Actual result
Completes the name, and moves the caret outside of the value so the user has to move it back in before typing @ and a version.

Showing completion list may be better for technical reasons (getting latest version number is a separate async call and may not be ready when name is completed). We might pop up completion with "loading" initially and then replace that with the actual version list (as we do for package.json for npm).
I think this should be P1 as it's very confusing to use unpkg and jsdeliver without it.

Also see https://github.com/aspnet/LibraryManager/issues/383 - another example of why we should fix this soon.

Another one to add. Some of the libraries have an "@" in their names so something like "@progress/kendo-ui" is a valid package, but then if you type "@" at the end of it, you don't get the intellisense for the version.

@bsimser can you open a separate issue to track your problem?  Please include which provider(s) you're using in that scenario.

This issue was fixed with #507


Podman: "Layer not known" error

Hi,
on vSphere in our company and on my homelab I've had a problem recently where podman reports "Layer not known" after showing the images with "sudo podman images".
This is not only a "visual" bug but also crio can't access the images and therefore can't start containers. No containers are running anymore: crictl ps -a shows an empty list.
[core@chmu3w0 ~]$ sudo podman images
ERRO[0000] error checking if image is a parent "70a981a7a6de09f373771eb639ad600dd30cb9bb1de96cee2b92f6940807d44b": layer not known
ERRO[0000] error checking if image is a parent "659694877391ed9f53dcefdeed267c5f8dfd24154ef6abe3c1f1be9c54a34088": layer not known
ERRO[0000] error checking if image is a parent "4665c12a232572c6f18d7594fdd9c68b8b8c05a95f4c81ab3f62c74469f4ca78": layer not known
ERRO[0000] error checking if image is a parent "e6de8ebc835401112ba0657e672ad187acaf77754456f9fec7ba04401f80f314": layer not known
REPOSITORY                                                                                TAG                            IMAGE ID       CREATED         SIZE

docker.io/minio/minio                                                                       latest                         4665c12a2325   3 weeks ago     unable to determine size: error locating layer with ID "4bb81fe5da12b12e168842c264247bfb520c43a19374d720f3c3f8c2ec574b2b": layer not known
quay.io/openshift/okd-content                                                               <none>                         256bc3e8da88   3 weeks ago     unable to determine size: error locating layer with ID "2aadc2ab4ea6a3598b45efba862a5d521e9554a2e73893c7f76fa34f9b46c457": layer not known
quay.io/openshift/okd-content                                                               <none>                         b278aa4f2719   4 weeks ago     unable to determine size: error locating layer with ID "0f003470a3208f897ceb401ab7df52c0e1a0eb8bf8e4b2ac6f6b540da450cea8": layer not known
quay.io/openshift/okd-content                                                               <none>                         ebb673575b8b   4 weeks ago

This seems to be an old problem with podman. Any ideas how we can fix this ? This seems to happen randomly if a node restarts. But we have also seen that without having nodes restarted.
Temprorary solution:
Delete the directory: /var/lib/containers/storage/overlay-images/
After restarting the node, the images get pulled again and the node gets ready afterwards.
Greetings,
Josef

Looks very similar to https://bugzilla.redhat.com/show_bug.cgi?id=1857224 - so next cri-o update might fix that. It appears that next FCOS stable update (scheduled ~Aug 25) would resolve blocking NFS bug and new CRI-O update would be pulled in

Ok. That is worth a try!

FCOS bump didn't resolve the NFS blocking bug, not sure why :(
Its certainly fixed in kernel 5.8, so we're waiting for another FCOS stable update

Fixed in latest OKD 4.5 nighlies (with CRI-O 1.18.4) and OKD 4.6 (using CRI-O 1.19)


Initial proposal of PSE001 rule

@JTurbakiewicz if this rule is also checking that psets have only the standard properties, I would make it explicit as a third "then" statement

@JTurbakiewicz if this rule is also checking that psets have only the standard properties, I would make it explicit as a third "then" statement
https://github.com/buildingSMART/ifc-gherkin-rules/pull/96#issuecomment-1752915660

No problem, added.

Looking good, have a look at the inline comments. As said in the meetings, next steps could be:

Check for Pset association to subtypes of IfcTypeObject
Check for correct template type usage (PSET_TYPEDRIVENOVERRIDE, PSET_OCCURENCEDRIVEN, PSET_TYPEDRIVENONLY)
Check for properties inside property set (should be of correct type, standardized psets cannot contain other props than in the spec).
Check for subtypes of IfcExtendedProperties


We need a way to protect the trading station from destruction

Description

Syndicates can easily bomb it, as seen during round #44294.
Screenshots

maybe force pacifism on trade station?

maybe force pacifism on trade station?

or taser turrets that tase people with weapons in-hand

many many ways to bypass both of those ideas

permaban no appeal anyone who bombs trade station then

Need a way to remind people the trading station is as sacred as the arrivals terminal

I was simply thinking make it indestructible

are you not allowed to kill people on the trade station?


RFC: Orbit.js model layer: The Adapter

Start Date: 2018-09-07
RFC PR: (leave this empty)
Orbit Issue: (leave this empty)

An ~Adapter~ Middleware for the model layer
Summary
The model layer by itself won't be of much use, if users are not able to let the data travel from the current APIs to the newly defined ones and vice versa.
The main task of the proposed Adapter (and the included Serializer) class is to provide that missing link.
Motivation
I believe a properly designed Adapter/Serializer interface will be critical to this model layers long term success, as it will be the key that allows us, to change parts of the orbit ecosystem without interference in other parts. -- This RFC will be key to stable APIs for the user.
Detailed design
Note: Because of our goals specified above, we might need a complete overhaul of the interfaces defined below.
Middleware-Adapter Interface
// @orbit/model/contracts/Adapter.ts
import Model from './Model';
import { Record } from "@orbit/data";

export default interface Adapter {
  createFromRecord: <T extends Model>(
      record: Record,
      setter?: (attr: string, value: any, model: T) => void
  ) => T;
  updateModel: <T extends Model>(
      record: Record,
      model: T,
      getter?: (attr: string, model?: T) => any,    // do we need a getter?
      setter?: (attr: string, value: any, model?: T) => void
  ) => void;
  save: <T extends Model>(model: T, getter?: (attr: string, model?: T) => any) => Promise<void>;
  destroy: <T extends Model>(model: T, getter?: (attr: string, model?: T) => any) => Promise<void>;
}

createFromRecord(record, setter?): T
Searches for the model class of record.type in the dependency injection system, instantiates a new model and sets the attributes on it. The optional setter can be used to set attributes on an attributes object inside the model (or to work around private properties).
updateModel(Record, model, getter?, setter?): void
This method allows refreshing the models data, e.g. after a merge.
save(model, getter?): Promise<void>
Saves the model's attributes into the store on which the model lives.
This is a high level explanation of what will be happening in here:

convert the attributes into a Transform
get the store form the DI Container
push the serialized data into the store
subscribe to the branches "merged" event, to know when the server has received the changes
query the store's cache for the record pushed in 3
update any meta data (e.g. orbits autogenerated UUID)
on "merged": update any properties that have been changed by the server (e.g. set a remoteId for a newly created record)
The promise indicates that the store finished processing the Transform, but has nothing to do with the data being persisted.

destroy(model, getter?): Promise<void>
Deletes the model. Again, the promise does not indicate whether or not the change has reached the server.
Relationship Adapter
I'm still not sure, if it's a better idea to keep all the methods in one class or to split it up. For now the adapter is split into the two parts: the "MainAdapter" (the one above) and the "RelationshipAdapter".
This adapter will hold all of the logic described for the relationship classes in the RFC for the model class #529
// @orbit/model/contracts/RelationshipAdapter.ts
import Model from './Model';

export default interface RelationshipAdapter {
  // hasOne relationship:
  getRelatedModel<T extends Model, R extends Model>(model: T, relationship: string): Promise<R>;

  setRelatedModel<T extends Model, R extends Model>(model: T, value: R, relationship?: string): Promise<void>;

  // hasMany relationship:
  getAllRelatedModels<T extends Model, R extends Model>(model: T, relationship: string): Promise<R[]>;

  addRelatedModel<T extends Model, R extends Model>(model: T, value: R, relationship?: string): Promise<void>;

  removeRelatedModel<T extends Model, R extends Model>(model: T, value: R, relationship?: string): Promise<void>;

  replaceRelatedModels<T extends Model, R extends Model>(model: T, value: R[], relationship?: string): Promise<void>;

  syncRelatedModels<T extends Model, R extends Model>(model: T, value: R[], relationship?: string): Promise<void>;
}

Replace vs. Sync
First of all, if you don't do forking, the result will be the same - it won't matter which one you choose.
Now consider this scenario:

You have a model "sun" with many "planets".
And you have a "root" branch, which you fork now to create a "forked" branch
First you add a planet "pluto" to the "planets" relationship.
Now on "forked" you edit the list of planets and set it again on the "sun" model (there is no pluto in the list of planets).

Let's see what would happen if you used Sync or Replace:
Sync: If you now merge "forked" into "root" again, "pluto" would still be there.
Replace: If you now merge "forked" into "root" again, "pluto" would be gone.
Now you have to decide, how you want your app to behave.
Middleware-Serializer interface
This interface is not final and will be adapted as needed during the first implementation.
The serializer interface is supposed to allow switching out or moving certain parts (like the attributes) of a model without the need to change anything except the model itself and the serializer.
// @orbit/model/contracts/ModelSerializer.ts
import { RecordIdentity } from '@orbit/data';
import { Dict } from '@orbit/utils';
import OrbitReflection from './OrbitReflection';

export default interface ModelSerializer<T /* extends Model */> {
  getOrbitReflection: (klass: { new(): any }) => OrbitReflection;
  setOrbitReflection: (klass: { new(): any }, reflection: OrbitReflection) => void;

  getHiddenOrbit: (model: T, getter?: () => any) => HiddenOrbit;
  setHiddenOrbit:
      (model: T, value: HiddenOrbit, setter?: (value: HiddenOrbit) => void) => void;

  getAttributeValues:
      (model: T, getter?: (attr: string) => any) => Dict<any>;
  setAttributeValues:
      (model: T, Dict<any>, setter?: (attr: string, value: any) => void) => void;

  getIdentity: (model: T) => RecordIdentity;

  // ...
}

// @orbit/model/contracts/RecordSerializer.ts
import { Record, RecordIdentity } from '@orbit/data';
import { Dict } from '@orbit/utils';

export default interface RecordSerializer {
  getAttributeValues: (record: Record) => Dict<any>;
  getIdentity: (record: Record) => RecordIdentity;

  // ...
}

Hidden Orbit Model Data
This defines the contract for the data stored on planet.__orbit property.
Note: The actual name of the property will be defined by the serializer implementation.
// @orbit/model/contracts/HiddenOrbit.ts
import Branch from './Branch';

export default interface HiddenOrbit {
  klass: { new(): any };    // just to get easy access to the static reflection property
  identifier?: HiddenIDs;
  branch?: Branch
  [key: string]: any;
}

export interface HiddenIDs {
  id: string;    // probably a autogenerated UUID from orbit's internals
  keys: {
    remoteId?: string;
    [key: string]: string;
  };
}

Middleware Services
The idea here is to create an additional layer which, in a picture, would be drawn between the Adapter and the Serializer. Very much like the Serializers, the service layer interfaces will be considered public API, but is geared towards power users and add-on authors.
The goal here is, to allow

splitting up the big junk of code into oversee able pieces and
to add a simple but powerful extension point

So what is a service now? How does it work?
A middleware service will simply be a class, implementing one or more methods (hooks) which will then be called by the adapter or serializer. This will work similarly to an event handler system, but instead of the given parameters being read only, you may actually modify them and/or return new values. The services will get called in the order they got registered.
TODO: actually define interfaces
How we teach this

What names and terminology work best for these concepts and why? How is this
idea best presented? As a continuation of existing Ember patterns, or as a
wholly new one?

Would the acceptance of this proposal mean the Ember guides must be
re-organized or altered? Does it change how Ember is taught to new users
at any level?

How should this feature be introduced and taught to existing Ember
users?

The Adapter is an important part of the core APIs, which might find heavy usage by a user and should thus be taught in the "Getting Started" part of the guides.
All the other interfaces defined in this RFC on the other hand aren't less important, but will be much less interesting or useful to any beginners. These need of course be documented, but it doesn't need to be added to the guides.
Drawbacks

Why should we not do this? Please consider the impact on teaching Ember,
on the integration of this feature with other existing and planned features,
on the impact of the API churn on existing apps, etc.

There are tradeoffs to choosing any path, please attempt to identify them here.

Alternatives

What other designs have been considered? What is the impact of not doing this?

This section could also include prior art, that is, how other frameworks in the same domain have solved this problem.

We could always just implement stuff where we need it instead of this central place.

Unresolved questions

Should we call these things something else to avoid confusing people into thinking the adapter/serializer has something to do with preparing a network request (like they do in the ember.js world)?
Edit: I started the process of renaming stuff to be the "middleware" layer - It's obviously not a classic middleware, but thats the best name I could come up with.

Related RFCs:

#529 The Model
#532 Orbit Reflection System
#533 The ~Adapter~ Middleware
#534 Branching
#531 Add a dependency injection system
#530 Add transforms to Models
QueryBuilder (coming soon)

credits:
emberjs/rfcs
ember-data
Laravel Eloquent

I'm going ahead and close this, as there is already a viable implementation in this module:
https://github.com/orbit-model/main/tree/develop/%40orbit-model/middleware


RCT-Folly conflict in react native 0.76.1

having this RCT-Folly Version conflict in my latest react native (0.76.1) project i already implemented this module in one of my app which is on react native (0.71.2) and its working perfectly
but know having this issue while pod install
[!] CocoaPods could not find compatible versions for pod "RCT-Folly":
In snapshot (Podfile.lock):
RCT-Folly (from ../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec)
In Podfile:
RCT-Folly (from ../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec)
React-RCTImage (from `../node_modules/react-native/Libraries/Image`) was resolved to 0.76.1, which depends on
  RCT-Folly (= 2<IP_ADDRESS>)

zego-uikit-rn (from `../node_modules/@zegocloud/zego-uikit-rn`) was resolved to 2.16.0, which depends on
  RCT-Folly (= 2<IP_ADDRESS>-v2)

[!] [Codegen] warn: using experimental new codegen integration

I'm very sorry for the poor experience you've had. I can't reproduce this issue here, and there is no specified version of React Native in the dependencies. Could you try running pod deintegrate and then pod install?

Yes tried this

After executing pod deintegrate, also need to manually delete the Podfile.lock file, and then you can pod install.

no its issue is same its saying zego-uikit-rn depends upon RCT-Folly (2<IP_ADDRESS>-v2) while new latest react native (0.76.1) uses RCT-Folly (2<IP_ADDRESS>)

Thanks, this issue has been fixed in version 6.2.2 of zego-uikit-prebuilt-call-rn.


refactoring vocab to its own file

Description
This PR puts everything that has to do with vocab into its own module, but from the user perspective they can still use it like cript.API.get_vocab() and it will work correctly.
Changes
Tests
Known Issues
Notes
Checklist

[ ] My name is on the list of contributors (CONTRIBUTORS.md) in the pull request source branch.
[ ] I have updated the documentation to reflect my changes.

Brad said to not do any refactoring for now until later when the system is more mature


Can the app add 5.0 support?

Very good app! But can it add 5.0 support?

Sorry, but there is no quick settings tile API on Android 5.0.

You can use version 2.4.5 if you want to run WADB itself on Android 5.0 without quick settings tile support.
fec387dcee62be0e61d8fe58ae476f23914f3611


Create pluggable socket system

Resolve https://github.com/erlanglab/erlangpl-ui/issues/8
@arkgil you can find example how to create handlers in src/index.js. You have to import on function from src/sockets.js and return it as handlers for given plugin. Then you have to add it in combineSockets.

This looks perfect, thanks! 👍

I'm polishing internals and then I'll merge


[c++] Fix short-circuiting in ValidateTwoWay

Due to a typo in ValidateTwoWay, if Validate(s1, s2) returns
false, Validate(s2, s1) will be called unnecessarily. This
change enables short-circuiting to avoid the unnecessary right hand side
computation.

Since some tests failed, this could be by design. Can you check which test cases fail and why?

Investigating, they're related to schema validation

These are the tests that are failing on both appveyor and travis:
/root/bond/cpp/test/core/validate_tests.cpp(146): error: in "Validation tests/0x20010000_Validate_tests": check (Check<type1, type1_upgrade_fields>(False, Error, Error)) has failed
/root/bond/cpp/test/core/validate_tests.cpp(147): error: in "Validation tests/0x20010000_Validate_tests": check (Check<type1, type1_removed_required_field>(False, Error, Error)) has failed
/root/bond/cpp/test/core/validate_tests.cpp(151): error: in "Validation tests/0x20010000_Validate_tests": check (Check<type4, type4_compatible>(False, Error, Error)) has failed

I think these caught a legitimate regression that would be introduced by this change. Take the first one for example: Check<type1, type1_upgrade_fields>(False, Error, Error).
struct type1
{
    1: required int16 field1;
    2: int32 field2;
};

struct type1_upgrade_fields
{
    1: required int32 field1;
    2: int64 field2;
};

type1_upgrade_fields can deserialize payload generated from type1, but type1 cannot deserialize payloads from type1_upgrade_fields (it would cause truncation). By short-circuiting, this function now returns false and no longer detects that type1 cannot deserialize type1_upgrade_fields. It does seem like the correct behavior is to call both Validate(s1, s2) and Validate(s2, s1).
@lalo, would you like me to document this behavior in ValidateTwoWay, split this out into separate lines, or just abandon the PR?
doc:
// It is necessary to call both Validate(s1, s2) and Validate(s2, s1), such as when promoting an
// integer field from a smaller size to a larger size. See type1 and type1_upgrade_fields
// in the validation unit tests.

separate lines:
const bool forwardDeserializable = Validate<Protocols>(s1, s2);
const bool backwardDeserializable = Validate<Protocols>(s2, s1);
return forwardDeserializable && backwardDeserializable;

Also, when I run make check locally on Ubuntu (Windows Subsystem for Linux), I also don't see the validation test running, though the README suggests that that command should run all tests.

Seems that the best course of action is to let it as is :+1:
Thanks for your work and investigation.

Will do, thanks @lalo


[Bitstamp] Some Currency Metadata and CurrencyPair Metadata are null

When calling the functions .getCurrencies and .getCurrencyPairs which are defined in org.knowm.xchange.dto.meta.ExchangeMetaData.
Some of the metadata were null. Basically everything that was not defined in /resources/bitstamp.json was returning null for the metadata.
After inspection, this is because the remoteInit() function of org.knowm.xchange.bitstamp.BitstampExchange is not doing much, more specifically the BitstampAdapters.adaptMetaData(Arrays.asList(bitstampPairInfos), exchangeMetaData) function is just adding currencies and pairs from API Call that it cannot find in bitstamp.json but not parsing the metadata from the Api call and just adding the currencies and currencypairs with "null" as meta data.
I have updated the bitstamp.json with new currencies infos (taken from : https://www.bitstamp.net/fee-schedule/)
And I have updated the "adaptMetaData" so that it will fill the infos it can find from the Api Call.
I will submit a PR that fixes this.

I forgot that my current develop branch is already out of sync, so the PR #4023 contains both the fixes for #4021 and #4021

Closing as merged


NOTICE: Renaming planned for similar GitHub organization

This issue serves as a notice that the similarly named open-cluster-management is planning to move to a new GitHub organization in order to remove naming confusion with open-cluster-management-io. The owners of open-cluster-management have agreed to move and have started preparations. We expect the move to be completed by mid 2022 and will keep this issue up to date with any new developments.

We have renamed out GitHub organization and are now https://github.com/stolostron.


[Task]: Create a doc on how to update the pub-pvt keypair used in the signature verification of CS APIs

https://www.notion.so/appsmith/Procedure-to-rotate-keys-bb87c366f0f744839d3613398a12f385


Merge repositories.tag and repositories.sha fields

Commit d306437a045ae878a85d249282dbc838ce2ada77 made the handling of these two fields 100% equivalent, so they should be deprecated and combined into a repositories.version field, or similar.

Mind taking a look @0xDEC0DE


[WIP] : Nuxt preparing for Storybook testing

Related issue
no
Scope of work
Preparing nuxt branch for testing SSR before release.
Screenshots of visual changes
no
Checklist

[x] No commented blocks of code left
[x] Run tests and docs
[x] Self code-reviewed
[x] Changes documented

If applicable:

[x] I followed composition rules for my component
[x] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: AdamPawlinski:x: pspaczekYou have signed the CLA already but the status is still pending? Let us recheck it.

this PR is wrong addressed, should be merged to nuxt branch


AdminLTE removes active class on sibling sidebar item

I'm using ng-class to toggle "active" state on sidebar menu items.
Though, when clicking a "multilevel" button (another button), it removes "active" class on all other menu items. Even worse, when we close the menu (reclicking on multilevel), it doesn't set's .active back on previously activated menu item, loosing the information in the process.
Is there a way to disable the dynamic behavior of .active class in multilevel ?

Unfortunately, you will have to edit app.js itself to do that. Specifically, lines 405-422-423.
Changes on our would be difficult since it would break backwards compatibility. This issue has been addressed in version 3.0.0-alpha-1, which has not been released yet. We apologize for the inconvenience.
Thanks!


GCC ASan compatibility

I looks like ASan support in aws-c-common only works for clang.
I have faced this surprising issue when trying to use ASan with gcc: https://github.com/awslabs/aws-c-http/issues/362
The fix is quite easy. Instead of giving a blacklist as a compilation flag (which is not supported by gcc), blacklist the function directly in code.
I found the macro AWS_SUPPRESS_ASAN here: https://github.com/awslabs/aws-c-common/blob/main/include/aws/common/macros.h#L78
It doesn't work with gcc. The recommanded way to define this macro is:
#if defined(__clang__) || defined (__GNUC__)
# define AWS_SUPPRESS_ASAN __attribute__((no_sanitize_address))
#else
# define AWS_SUPPRESS_ASAN
#endif

See https://github.com/google/sanitizers/wiki/AddressSanitizer#turning-off-instrumentation
Then you can apply this macro to hashlittle2() function.

resolved by https://github.com/awslabs/aws-c-common/commit/e86e1b6f4622ff8f09d8219d00d79bf8025deaae


fixed description for Devise::Generators::ControllersGenerator

Hello. I found one little mistake in controllers generator's description. When I run command rails generate devise:controllers  I see
For example:

  rails generate devise:controllers users -c=sessions

This will create a controller class at app/controllers/users/sessions_controller.rb like this:

  class Users::ConfirmationsController < Devise::ConfirmationsController
    content...
  end

You show how to overwrite sessions controller, but use the code of ConfirmationsController 😅

Wow, nice catch!
Thanks, @kopylovvlad!


Some questions about attention model!

Hi, @codekansas, thanks for sharing the project. I have read the main file about attention model to question answering and the paper you suggested. However, I have some questions as following:
Take insuranceqa.py for example.
1.Is the input of your data the word frequence index? Would you give me an actual example of your input data?
q_data, ag_data, ab_data, targets = get_data(data_sets[0])
2. What is the "mrr" in your code? A evaluation measure?
def get_mrr(model, questions, all_answers, n_good, n_eval=-1):
3. Why are the targets all zeros?
def get_data(f_name):
....
        targets += [0] * len(bad_answers)

4   I find you load the weights of test model, as following:
test_model.load_weights(os.path.join(models_path, 'iqa_model_for_training_iter_900.h5'))
Where are the weights of test model? The weights of trained model can transfer to test model?
5  Similar to my task (figuring out how similar two tweets), I only need to get the similarity of each tweet pair. But I have to set a threshold value, which is used to determine whether two tweets are from the same person or not (If the similarity greater than threshold value, the pair is from the same person, vice versa). Therefore, do you think what the threshold value should be set? Or other method to leverage similarity to determine whether two tweets are from the same person or not?

I should warn you that a lot of the code is pretty messy since I've just been trying different things out.

The input is a sentence, encoded as indices. So a sentence "the cat in the hat" might be encoded as [1, 2, 3, 1, 4]. This is done by constructing a dictionary of words and taking the index of the word in the dictionary (with a dummy index if the word doesn't exist).
MRR means Mean Reciprocal Rank, it is a common metric in question answering. If you have a selection of good answers out of all your answers, the reciprocal rank is 1 over the highest rank of a good answer. Then you take the mean of all of those to get MRR.
In my model, I was minimizing hinge loss between a good and a bad answer, using something like merge([good,  bad], mode=lambda x: 0.2 - x[0] + x[1]). So you want this to go to 0, so that good (the cosine similarity between the question encoding and the good answer encoding) is maximized and bad is minimized, up to a margin. This is done in the two papers.
That was just an artifact of messing around. I didn't want to retrain a model to evaluate it, but some stuff is out of order because I commented different things.
I'd say you could figure that out from your training set, like run a bunch of training sets through and determine how similar tweets from the same person usually are. I'd imagine there would be a lot of noise in that kind of task though. I'd be interested to hear if it actually works.

I don't think this is really Keras related though, so I'd suggest closing this issue and messaging me directly if you have more questions. My email is on my Github page. Also, I started writing some of this stuff up here but it isn't complete yet.


Hotflix/issue 4154 min steps override early stopping

What does this PR do?

Fixes # (issue)
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @tchaton! Thanks for opening this PR.

In the file tests/callbacks/test_early_stopping.py:

Line 224:5: E124 closing bracket does not match visual indentation
Line 283:58: E201 whitespace after '('

Do see the Hitchhiker's guide to code style


Sync fixes

This PR fixes two issues related to sync (specifically push):

account for how record updates work.

updating a record actually creates another RecordsWrite with the same recordId. only the first and most recent RecordsWrite messages are kept for a given recordId. any in between are outright nuked from everywhere. Prior to this PR, sync would attempt to push these non-existent records which would end up resulting in Sync failed due to error:  Error: TODO: message not found

account for scenario where a record is written and deleted before a push occurs.

RecordsDelete messages keep the initial RecordsWrite and simply delete the data associated to that message. Prior to this PR, sync would attempt to fetch data for a deleted record which would end up resulting in Sync failed due to error:  Error: (404) Failed to read data associated with record <record_id>

💡  Note that https://github.com/TBD54566975/dwn-server/pull/31 needs to be merged in order for the 2nd issue to be fixed

fix can be tested by doing the following:

add the following to packages/web5/examples:

<!DOCTYPE html>
<html lang="en">

<head>
  <meta charset="UTF-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <title>Document</title>
</head>

<body>
  <script type="module">
    import { Web5 } from<EMAIL_ADDRESS>
    const { web5, did: aliceDid } = await Web5.connect({
      techPreview: {
        dwnUrls: ['http://localhost:3000']
      }
    });
    console.log(aliceDid);

    const { record, status } = await web5.dwn.records.create({
      data: "Hello Web5",
      message: {
        dataFormat: 'text/plain',
      },
    });

    console.log('WRITE RESULT', status);

    const readResult = await record.data.text();
    console.log('READ RESULT', readResult);

    const updateResult = await record.update({ data: "Hello, I'm updated!"


Question : Incorrect docker image used in test ?

The Rust client on the master branch uses ghcr.io/eventstore/eventstore/eventstore:[tag]
shouldn't that be ghcr.io/eventstore/eventstore:[tag] ?

It's the correct value.


Make the plugin work with AWS SSO

We are using AWS SSO login feature to authenticate our S3 instance.
This plugin just doesn't work with SSO by setting lookupDefaultAwsCredentials = true, forcing clients to configure the plugin with awsAccessKeyId, awsSecretKey and sessionToken.
We found out that just by adding the AWS SSO dependency will make the plugin work with no extra effort.
This StackOverflow answer helped us.
As a workaround we have now added the same dependencies (BOM and SSO) to our settings.gradle as buildscript.dependencies.classpath, but it would be nice if it was supported by this plugin out of the box.
I took the opportunity to update AWS but feel free to revert that change.

Thank you for the fix. I've pushed 1.4, enjoy: https://plugins.gradle.org/plugin/com.github.burrunan.s3-build-cache


Allow alternate means of retrieving content

At the moment the SchemaStore will, if it finds a schema that hasn't yet been resolved, call the getContent method to retrieve it:
https://github.com/jimblackler/jsonschemafriend/blob/master/library/src/main/java/net/jimblackler/jsonschemafriend/SchemaStore.java#L144
That getContent method will then use the static CacheLoader.load method.
I see you can also preload the SchemaStore using the public store or register methods.
What I'm asking for though is the ability to intercept that getContent loading method (in my case I may want to store schemas in some persistence layer), via say an interface ContentLoader ...
  public SchemaStore(UrlRewriter urlRewriter, boolean cacheSchema, ContentLoader contentLoader) {
     ...
     this.contentLoader = contentLoader;
  }

  private String getContent(URI documentUri) throws IOException {
    Uri uriToLoad = urlRewriter == null : documentUri : urlRewriter.rewrite(documentUri);
    if (contentLoader != null) {
      return contentLoader.load(uriToLoad, cacheSchema);
    }
    return load(uriToLoad, cacheSchema);
  }

Maybe that would also help with developers who have issues like https://github.com/jimblackler/jsonschemafriend/issues/3 and need to provide their own means of resolving a schema?

Hi, I was checking out this library, and did notice it attempts to download content from https://json-schema.org/ -- in my use case, I would need application to work offline.. so probably embed the schema files would work. Any advice to avoid the external download attempt?

They can be cached on the first instance if you ever do have internet
access.
It's not strictly necessary to have the metaschema, it just can't validate
the schemas themselves.
I can't recall currently what happens if the fetch can't complete.
Certainly it could be configured to continue without the schema validation
against the metaschema.
On Tue, 14 Dec 2021 at 17:15, Pedro Henriques dos Santos Teixeira <
@.***> wrote:

Hi, I was checking out this library, and did notice it attempts to
download content from https://json-schema.org/ -- in my use case, I would
need application to work offline.. so probably embed the schema files would
work. Any advice to avoid the external download attempt?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jimblackler/jsonschemafriend/issues/14#issuecomment-993799800,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFH6F4CVSWQSTXAD5WU32LUQ53UNANCNFSM5IWSKY4A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

At the moment there's a network exception, so the library wouldn't really work in offline mode.

Hi are there any plans to release this change? In my use case, I would need the application to work offline, is there a way it can pick up the schema from resource folder if specified there?

Sorry I thought I had made a release since then. I'll do a release when the current outstanding PR is in.

Hi are there any plans on releasing the latest changes?
Thanks!

Thank you for the reminder. I just pushed 0.11.3, it should show up on Jitpack shortly.


Fix LDAP auth in 1.0.x

Backport of #1962 from master branch

Codecov Report

Merging #1963 into 1.0.x will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##            1.0.x    #1963      +/-   ##
==========================================
+ Coverage   66.28%   66.28%   +<.01%
==========================================
  Files          56       56
  Lines        6605     6606       +1
  Branches     1319     1319
==========================================
+ Hits         4378     4379       +1
  Misses       2004     2004
  Partials      223      223

Impacted Files
Coverage Δ

webapp/graphite/settings.py
75.3% <100%> (+0.15%)
: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 3c90fe2...5a327ce. Read the comment docs.


Realsense D435I error: FrameSync not supported

Required Info

Camera Model
D435I

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Platform
PC

SDK Version
<IP_ADDRESS>0

Issue Description:
I faced an error when running Elsticfuion.exe which was generated by VS2015:
[RS2] ERROR at FILE D:\Elastic\librealsense\wrappers\openni2\src\Rs2Driver.cpp LINE 258 FUNC oni::driver::Rs2Driver::enableFrameSync
FrameSync not supported.
This error is the same as (#5001 (comment)) mentioned.
How to fix it ?

I have explored your problem very carefully.  There is not much information available on this subject, so I had to make some theories about the cause of frame sync being unavailable.
As a first step, could you text please whether metadata support is enabled in Windows.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_windows.md#enabling-metadata-on-windows

I have explored your problem very carefully. There is not much information available on this subject, so I had to make some theories about the cause of frame sync being unavailable.
As a first step, could you text please whether metadata support is enabled in Windows.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_windows.md#enabling-metadata-on-windows

Hello, Thank you so much for your support.
I checked as you mentioned, it seems metadata support is enabled in Windows because I checked the item ENFORCE_METADATA in the Cmake configuration. Configuring done, and Generating done.

What can I do next step ?

I had another look at the problem.  It is difficult to diagnose though because there are few references about the FrameSync Not Supported issue, and it was originally developed by a non-Intel team called the Palo Alto Group.
https://github.com/PaloAlto-Group/RealSense-OpenNI2-Wrapper
I wonder if it would be possible to try Palo Alto's older version of the OpenNI2 wrapper in the link above to see if the frame sync error still occurs with that version.

I tried, but this error was still happened.
Let me clarified my case as below:

Complied and generated "Librealsense" by VS2017. Copied "rs2drverd.dll and realsense2d.dll" to OPENNI2 folder. Operteing "NiViewer", D435I can be detected ;
But after generated Elasticfusion acc. the link:
https://github.com/mp3guy/ElasticFusion
This error "FrameSync Not Supported" will be happened;

I doubt if it's problem from Librealsense ? or a problem from Elasticfusion?

Given that ElasticFusion's last code update was in July 2018, before the D435i model was even launched in late 2018, it is probable that the program was not tested with D435i.  This opens the possibility of an incompatibility with D435i that is not present in the D435 model.
If you need to do Visual SLAM (VSLAM) with a D435i, I wonder if an option for you may be to switch to the modern OpenVSLAM software, which seems to be very flexible with the cameras that it will work with.
https://support.intelrealsense.com/hc/en-us/community/posts/360038549493-Highlighting-the-OpenVSLAM-open-source-Visual-SLAM-platform

It's bad news for me. :(
But you mentioned OpenVSLAM, then it means D435i can be compatibility with this OpenVSLAM software, right ? It's totally new for me again.

I do not know of anybody who has used OpenVSLAM with RealSense, though the instructions say that " the system can deal with various type of camera models, such as perspective, fisheye, and equirectangular.  If needed, users can implement extra camera models with ease".
If you want a SLAM system proven to work with D435i, Intel have this guide that uses ROS:
https://github.com/IntelRealSense/realsense-ros/wiki/SLAM-with-D435i

It looks another way. Let me have a try.
In parallel, I also can try again to compile ElasticFusion in Ubuntu system.
Anyway, Thank you so much for your support and suggestion.


Fix scripts/switch-to-preproduction

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

@bganapa #7354 @leonardbf #7660 Please rebase or cherry pick this fix and try again for prettier fix


Run functional tests (web tests) without disabling shadow copying

We really like being able to compile while running our tests in the background. However, if we disable shadow copying in our tests we run into https://github.com/aspnet/Razor/issues/1212 . The workaround presented in that issue works, however, it includes disabling shadow copying which is a real downer.
According to https://github.com/aspnet/Mvc/issues/6233 there should be a fix in 2.x, but as far as I can see (e.g., here: https://github.com/aspnet/Mvc/commit/35152d5933ca885aaa3633301d344e7a58b20cd7) shadow copying still needs to be disabled.
Can anyone elaborate WHICH assemblies have to be loaded in our web tests so that Razor can find the necessary dependencies?
If it is of any help: we are targeting ASP.NET Core 1.1 and .NET Framework(!) 4.7.

I've found that the following hackaround (code must run before web application is started):
  var webClientAssembly = typeof (Startup).Assembly;

  // The path where the shadow copy of the assembly resides
  var webClientAssemblyLocationDirectory = Path.GetDirectoryName(webClientAssembly.Location).AssertNotNull();

  // The original path from where the assembly has been loaded
  var webClientAssemblyCodeBaseDirectory = Path.GetDirectoryName(new Uri(webClientAssembly.CodeBase).AbsolutePath).AssertNotNull();

  var depsFileName = $"{webClientAssembly.GetName().Name}.deps.json";
  var depsFile = new FileInfo(Path.Combine(webClientAssemblyCodeBaseDirectory, depsFileName));
  if (!depsFile.Exists)
  {
    throw new InvalidOperationException(
      $"Can't find '{depsFile}'. This file is required for Razor to know the compilation context. There should"
      + "be a copy of the file in your web client\'s bin directory.");
  }

  // Copy deps file to shadow copy location
  File.Copy(depsFile.FullName, Path.Combine(webClientAssemblyLocationDirectory, depsFile.Name));

Is there any catch with this solution? Any counter arguments, or hints as to why we would run into troubles later on?

The reason to disable shadow copying is that ASP.NET Core needs access to static files, views, and the .deps.json file to function correctly. What you have here basically looks the same to me as the MSBuild snippet from this example https://github.com/aspnet/Razor/issues/1212#issuecomment-297885722
<!--
    Work around https://github.com/NuGet/Home/issues/4412. MVC uses DependencyContext.Load() which looks next to a .dll
    for a .deps.json. Information isn't available elsewhere. Need the .deps.json file for all web site applications.
  -->
  <Target Name="CopyDepsFiles" AfterTargets="Build" Condition="'$(TargetFramework)'!=''">
    <ItemGroup>
      <DepsFilePaths Include="$([System.IO.Path]::ChangeExtension('%(_ResolvedProjectReferencePaths.FullPath)', '.deps.json'))" />
    </ItemGroup>

    <Copy SourceFiles="%(DepsFilePaths.FullPath)" DestinationFolder="$(OutputPath)" Condition="Exists('%(DepsFilePaths.FullPath)')" />
  </Target>

Can you be specific about what problems you are having with shadow copying off?

Static files and views should be accessible by setting the appropriate content root path, so this should not affect shadow copying.
The reference assemblies should be - if loaded correctly - shadow copied as well, so I don't really see why this should be a blocker for shadow copying. Indeed, manually searching for assemblies must be done relative to the original assemlby's CodeBase not its Location (maybe that's the bug?).
The MSBuild snippet shown by you is not enough, we have it in place as well. It ensures that the deps.json file is being copied to the build output of the integration test project (normally it would only be in the build output of the SUT (the web project), and would not be copied to the integration test project although the integr. test project depends on the web project). So this manual MSBuild task copies the deps.json file by hand to the integration test project's output folder. However, that's not enough, at the momennt I still have to copy the deps.json file to the shadow copy location, which should normally not be necessary.

OK, well based on the points here it sounds like you understand what's required to make this work for you.

Yes, as I stated in my initial post I've found a workaround to my problem. Still I have these two open questions:

Is this actually a problem in the Razor engine and should be fixed, so that such a workaround is no longer necessary?

Is my workaround feasible in the eyes of you insiders. Does the approach really work, or is it just enough to run my first tests and I will run into troubles later on?

Is this actually a problem in the Razor engine and should be fixed, so that such a workaround is no longer necessary?

This is a fundamental requirement. The deps file needs to be present so that we can find the compilation settings and reference assemblies to use. Our test package that we're developing will include this snippet of msbuild code by default so that you don't have to put it everywhere, but the deps file always needs to be present.

Is my workaround feasible in the eyes of you insiders. Does the approach really work, or is it just enough to run my first tests and I will run into troubles later on?

It sounds to me like you've covered everything. This approach should work fine. Everything here is just files on disk, so as long as you're willing to make sure that those files are also copied to the right places, that should be fine.

Ad 1) I'm afraid you missed my point. Is it a bug, that the assembly is trying to locate the deps file / its references at the assembly's Location instead of the assembly's CodeBase (which seems to be the case, as the deps file is found only if copied to the shadow copy location manually).
Ad 2) OK, let's hope so...

Ah yes I was missing the point on that. Let me check on that and see what I can find out

Looping in @eerhardt since the deps file discovery is in DependencyModel. Should https://github.com/dotnet/core-setup/blob/master/src/managed/Microsoft.Extensions.DependencyModel/DependencyContextLoader.cs#L118 use Assembly.CodeBase instead of Assembly.Location to support shadow copy scenarios?

@pranavkm what further action is required from us here? Do we need to file bugs elsewhere? Any code changes or investigations left for us to do?

Ping @eerhardt . Should https://github.com/dotnet/core-setup/blob/master/src/managed/Microsoft.Extensions.DependencyModel/DependencyContextLoader.cs#L118 use Assembly.Codebase to support shadow copied binaries?

I'm not sure, it's been like that since the beginning:  https://github.com/dotnet/core-setup/commit/7ce1e267f7d922e014c668d901a2e5a16f3fddf3.
Can you open a bug in core-setup for it?  We can evaluate it there.

Done - https://github.com/dotnet/core-setup/issues/3308. Closing this since there's nothing further to do from our end.


clipman: init at 1.0.1

Motivation for this change
Simple clipboard manager for wayland: https://github.com/yory8/clipman
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build clipman

2019/09/25 18:17:42 Error running wl-copy: exec: "wl-copy": executable file not found in $PATH

Would it make sense to wrap this and add wl-clipboard to the path?

Yeah indeed. Missed that because I have wl-copy in my local profile, sorry!

@alexarice fixed. Would you mind taking another look at this? :)

I'm slightly confused, I've managed to store some text but it doesn't seem to interact with the clipboard. How do I use this?

This is mainly used to persist stuff that also goes to a clipboard. I use it with sway as recommended in the readme: https://github.com/yory8/clipman#usage


test-branch/execute-job/wrong-commit-sha-on-pr/d6295f0e-b002-4d68-b80c-97618678d1ea

PR for testing b2f7ea6300ee6ea3ef316dc54b23f51645247bd0

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


CI will ignore push to non-master

This contributes to rancher/rke2#4248
We want CI to exclude push events from dev or automated branches

PR was cancelled, restarting


Add top level makefile

Add top level makefile that

builds docker-images for composition and all expanders
release manifests for composition and all expanders
load docker images and deploy manifests to local kind cluster for composition and all expanders

Tests failing. Fixes in https://github.com/cloud-native-compositions/compositions/pull/18
That needs to be merged first

Closing in favor of https://github.com/cloud-native-compositions/compositions/pull/20


Updated: spotify <IP_ADDRESS>2

${}

ae8d1d5d-187e-4425-9cf9-2c83971571ed

Newer version of spotify (<IP_ADDRESS>8) is now available in master. 972b8035dc22d9a6548d7300c3ea14a4fc3dbcce


Multi-Column Layout of Completion Candidates #2846

I'm missing the option to display completion candidates in multi-column layout like Vanilla Emacs does. Is this planned?
C-p/C-n navigation could be like Zshell performs its completion; up/down the current column with C-n going to next right column after at the end. Along with arrow keys for navigating between rows and columns.
Alternatively C-p/C-n could navigate along current line like vanilla Emacs presents it's completion candidates in a row-major order.

There are indeed plans to add a grid view at some point see #470, use that issue for further discussion.

Thanks.


Feature to change user name

Is your feature request related to a problem? Please describe.
The problem is i created account with github and so my email coming up in name place, i want to change my name
Describe the solution you'd like
A feature to change user name

can you describe more about this in detail ?

Do you mean in the Nhost Dashboard or in your own app built with Nhost?

Do you mean in the Nhost Dashboard or in your own app built with Nhost?

In dashboard, if you are workspace member list you can see your email address as name when you create nhost account with Github

can you describe more about this in detail ?

When i create my account with github, it only request read email address permission for github not other eg. Username
And after that nhost didn't ask for entering name and put my name as email

i signup from github and nhost  put my email as name instead of github name
there  is no change option there


fix(FrontendFields): Fix calling getCMSFields on for frontend fields.

Calling getCMSFields on the frontend should not attempt to write a 'ElementArea'/'Page' if the page doesn't exist.
Setup UnsavedRelationList for cases where ElementArea()->Elements() is empty. (ie. frontend fields / empty data object)

Thanks :) Your project using frontend editing? Might be worth adding unit tests in the future for your use case to ensure any future changes don't break this.


Enable Typescript strict opts

Enable remaining strict options in Typescript with the exception of noUnusedParameters, which should be taken care of by eslint

If we agree to enable them I'll start fixing the issues

So is there a consensus on turning some of these rules on?

Review after merging https://github.com/ChainSafe/lodestar/pull/1924 should fix the type issues breaking CI

dang, one more file conflict


Make port-tclsh a relative symlink, not absolute

This is a step toward allowing the MacPorts installer package to install to any prefix.
Using Perl is the most straightforward and most compatible way I found to compute a relative path from two absolute paths.
I wrote the Perl command inline because this is the only place where it's needed right now. But it could be broken out into a custom Makefile function to make it reusable.
Is it ok to just use perl, since it is so widely available, or do we need to do autoconf stuff to find it and set a variable with its path? We already use sed in the same Makefile without checking for it.

That should work and would be simpler.

Are you going to update the PR with the shell-based solution?

Yes I should do that.

how about using the ${foo#bar} prefix pattern expansion syntax to get the relative path?

The problem with this is that we are perpetuating the mistake of making assumptions about the "bin" and "libexec" directory names/paths and their locations relative to one another. If we fix that mistake, then we need to compute the relative path from the one to the other, e.g. using the Perl solution I proposed.

If avoiding Perl is strongly desired, the following should work:
bindir="${INSTALLDIR}/bin" && shopt -s extglob && rel="$${bindir//+([^\/])/..}" && $(LN_S) -f "$${rel:1}${TCLSH}" "${DESTDIR}${INSTALLDIR}/bin/port-tclsh"

The relative path this constructs is not minimal/optimal, but it should be correct regardless of how different bindir and libexecdir are from one another.
(Obviously this still incorrectly hardcodes the bin directory, but at least it makes it obvious where ${INSTALLDIR}/bin could be replaced by the right thing.)


fix pitch shifter sox error

@oreillyp  any chance you can tackle this?

Opened a PR to address this: https://github.com/audacitorch/pyharp/pull/1


Transaction list keeps falling behind

I have noticed (in my chain with 180+ validators) that the transaction tab is lagging. Sometimes, it is behind by 1-2 days or just hours, but it often cannot "keep up" with what is going on right now.
Any idea?
I am using the 0.38.1 branch.

Big Dipper indexes all blocks and transactions. You may not have enough resources to index the blocks in time.

Indeed that is possible but I am not sure what I need to do then to
resolve this.
I specifically setup an instance on AWS with the following config:
32 GB RAM, 8 vCPUs, 640 GB SSD
I expected this would be overkill, but I am still getting this problem. Is
there something I need to do here to properly use these resources? I am
running a production instance.
On Sat, Jun 27, 2020 at 10:01 PM Kwun Yeung<EMAIL_ADDRESS>wrote:

Big Dipper indexes all blocks and transactions. You may not have enough
resources to index the blocks in time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/forbole/big-dipper/issues/366#issuecomment-650696726,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADTCQRQSZJHJFU4SGEIYULRY3FCPANCNFSM4OKFCCSQ
.

Wow, you should have enough resources to run.

Yes... which is why I do not know what I can do about it now...
On Tue, Jun 30, 2020 at 11:49 AM Kwun Yeung<EMAIL_ADDRESS>wrote:

Wow, you should have enough resources to run.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/forbole/big-dipper/issues/366#issuecomment-651977806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADTCQQ3OTZHX6E742DO6ILRZIXTBANCNFSM4OKFCCSQ
.

Any thoughts on why the transactions are falling behind? Is it related to
the other issue regarding high # of validators?
On Tue, Jun 30, 2020 at 11:49 AM Kwun Yeung<EMAIL_ADDRESS>wrote:

Wow, you should have enough resources to run.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/forbole/big-dipper/issues/366#issuecomment-651977806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADTCQQ3OTZHX6E742DO6ILRZIXTBANCNFSM4OKFCCSQ
.

@njmurarka are the blocks being processed up-to-date? How many transactions are in the blocks? We had over 230 validators during Game of Stakes and Big Dipper ran properly.

There are at most 1-2 transactions in each block.
I am not sure what you mean by "are the blocks processed up-to-date"? That is the actual problem -- when I go and look at the transactions screen, it is lagging behind by hours and sometimes longer.
http://explorer.testnet.public.bluzelle.com/transactions
I am not saying it is the # of validators -- there is indeed a different issue I filed where BD is not reporting the right info on validators, that you are already familiar with.
But this is the issue that TX's are lagging way behind.
We are on Cosmos 0.38.5 and so we are using BD's 0.38.x branch.
Please help ASAP. I have tried everything I possibly can to resolve this including ensuring the BD node is sitting in the same data center as the node it syncs with and throwing an obscene amount of resources (CPU, etc) at the BD machine, but it still keeps falling behind like this.
Please tell me what info you need so we can resolve this. Thanks.

It's pretty strange. Your BD is running pretty slow. Can you restart it and see if it helps? Also try restarting the node and the rest server.

I do restart it every few days.
If I do that, it takes 24-48 hrs and then BD "catches up". Then, after a few days, it falls behind again, until I again restart it.
Can't easily restart the node and rest server in my current setup as BD is connecting to a round-robin DNS of many nodes and REST servers for the purpose of redundancy and load balancing. And all these nodes are up and operational.
That having been said, I don't know what to do at this point. Not sure where the problem lies. Not saying the problem is in BD, of course.

I don't get it. You mentioned that you are running production build, why it takes 2 days to catch up? How do you restart BD?

When I say restart, I do not mean that I stop and reset the DB state of BD and then start it again. I mean that I stop the process and start it again. The "catching up" refers to BD displaying TX's in the TX tab that are current. As mentioned, it "falls behind" regularly and the only way to get it to catch up is to stop and start it again (not reset and restart) and then wait a day or so.
Right now, for example, it is showing TX's that happened 20 hrs ago. If I stop BD now and immediately start it again, it will slowly "catchup" and then eventually fall behind again.
I honestly have no idea what to do at this point.
Would appreciate any help. I am on Cosmos 0.38.5 now, but this problem was occurring with 0.38.x in general.

I really don't have any idea. BD of Akash, Kava, Desmos are running with this branch and I don't see a problem like this. We had a lag of processing in Akash once as there were 5000 txs in nearly every block.

Is there any logs I can email to you? Diagnostics? Could I give you access to one of our boxes to login yourself to look at it? I have no idea either of course, but I think that if you directly looked at it, you might get insight.

Also I noted a new branch cherry-pick-v0.38-fixes but also that the 0.38.x branch is gone (maybe I missed it). Any changes there?

The v0.38.x branch is merged to master. We are trying to cherry pick some commits we have done for other projects back in master.


Search 11.3.0 GA for python

We are targeting Azure Search SDK GA next month. Here is the scope for upcoming GA:
Public surface area of last GA version
AAD support including multi cloud support
Any bug fixes since then
GA release version:
Release azure-search-documents GA version 11.3.0 with the features listed above
Right after releasing GA 11.3.0, release 11.4.0b1 with all the preview features released after 11.3.0, so that if users try to use the latest version, they will get all GA as well as preview features.
Follow the below steps for the GA release:

[ ] Create a feature branch using tag of the last released version
[ ] Add below features
- [ ] AAD support including multi cloud support
- [ ] Any bug fixes since then
[ ] Update the changelog by adding the release information about all the preview versions released after last 11.4.0 GA.
[ ] Update the changelog to add GA release features for 11.3.0 version.
[ ] Create PR
[ ] Create API view and add link [issue to be created] here for architects to review.
[ ] Update PR as per the feedback and merge it.
[ ] Release azure-search-documents - 11.3.0 GA version using feature branch.
[ ] Update changelog in the main branch to add 11.3.0 and 11.4.0b1 release information.
[ ] Release azure-search-documents - 11.4.0b1 beta version using main branch.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Search:0.48668343,SQL:0.09990058,Storage:0.03749073'

Done.


how to properly satisfy pylds dependency

Hi,
How do I properly satisfy the pylds dependency?
If I do it using 'pip install pylds' and run demo.py I get:
AttributeError: type object 'LDSStates' has no attribute '_info_extra_loglike_terms'
If I checkout the current version from the GIT repository I end up with:
ImportError: /home/someuser/code/pylds/pylds/lds_messages.so: undefined symbol: PyFPE_jbuf
Thanks!

Looks like a build error. Can you try cloning the master pylds branch from github, like this?
cd ~/code
git clone https://github.com/mattjj/pylds.git
cd pylds
pip install -e .

You might also want to update cython, numpy, and scipy.

Hmmm, no luck.
I did the pip install -e . from the github master branch and it finally states:
Successfully installed pylds-0.0.2

But when running examples/meanfield.py or examples/demo.py I still get:
File "/home/someuser/code/pylds/pylds/states.py", line 7, in <module>
from lds_messages_interface import kalman_filter, filter_and_sample, E_step, \
  File "/home/someuser/code/pylds/pylds/lds_messages_interface.py", line 10, in <module>
from lds_messages import \
ImportError: /home/someuser/code/pylds/pylds/lds_messages.so: undefined symbol: PyFPE_jbuf

I have numpy version 1.11.1, cython 0.24.1, scipy 0.17.1.

Alright, sorry my mistake. I forgot to explicitly remove all the old files from the previous build.
Now, both examples work.

Great! Glad you got it sorted out, and thanks for updating here.


updated district office info

A few updates to the district office info. Here are the sources:
notes:

A000367: https://amash.house.gov/
B000490: https://bishop.house.gov/
B000575: https://www.blunt.senate.gov/public/
B001230: https://www.baldwin.senate.gov/
B001243: https://blackburn.house.gov/contact/
B001261: https://www.barrasso.senate.gov/public/
B001267: https://www.bennet.senate.gov/?p=contact
B001275: https://bucshon.house.gov/
B001285: https://juliabrownley.house.gov/
B001291: https://babin.house.gov/contact/officeinformation.htm
B001294: https://blum.house.gov/
B001296: https://boyle.house.gov/
B001303: https://bluntrochester.house.gov/
C000141: https://www.cardin.senate.gov/
C000880: https://www.crapo.senate.gov/contact/office-locations

Thank you for including sources. I double-checked all these changes and they look good to me.


Adding MudRenderQueue (#7210)

I added MudRenderQueue.razor and MudRenderQueueItem.razor
This will help for lazy render I talked about in #7210
Just additional component if you you want to understand the idea
Description
Just a fix for issue #7210
And how to use it like this
<MudRenderQueue Delay=300> @* Default delay is 100 *@
    <MudRenderQueueItem>
        @* What inside here will be render after 300 milliseconds *@
    </MudRenderQueueItem>
    <MudRenderQueueItem>
        @* What inside here will be rendered after 600 milliseconds *@
    </MudRenderQueueItem>
</MudRenderQueue>

How Has This Been Tested?
In Empty Blazor App
Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.

I followed the code style you made MudRender.razor

[ ] I've added relevant tests.

Does this yes or what do I need to pass codecov/patch successfully

Do it yourself! I don't have time to do unit tests sorry

There is a bug in MudRenderQueue I made, It is when two renders caught in the same time it lagged also I think it would not help in MudBlazor as library with fast rendering

@wisamidris7 just FYI issues and pull request are, like GIT, to maintain history so there is no need to completely remove the description.
Cancelling the PR is enough to indicate that.


W&B sweeps giving OOM

I'm trying to run W&B sweeps for transformer. It was able to finish 5 sweeps successfully and failed on rest of 150 sweeps. Here is the error that I got after 5 sweeps -
Run x7n6tsz8 errored:
Traceback (most recent call last):
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/wandb/agents/pyagent.py", line 308, in _run_job
self._function()
File "/home/aru-sarthak/yoyodyne_041123/yoyodyne/examples/wandb_sweeps/./train_wandb_sweep.py", line 43, in train_sweep
best_checkpoint = train.train(trainer, model, datamodule, args.train_from)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/yoyodyne/train.py", line 253, in train
trainer.fit(model, datamodule, ckpt_path=train_from)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 608, in fit
call._call_and_handle_interrupt(
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/pytorch_lightning/trainer/call.py", line 38, in _call_and_handle_interrupt
return trainer_fn(*args, **kwargs)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 650, in _fit_impl
self._run(model, ckpt_path=self.ckpt_path)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 1093, in _run
self.strategy.setup(self)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/pytorch_lightning/strategies/single_device.py", line 73, in setup
self.model_to_device()
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/pytorch_lightning/strategies/single_device.py", line 70, in model_to_device
self.model.to(self.root_device)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/lightning_fabric/utilities/device_dtype_mixin.py", line 54, in to
return super().to(*args, **kwargs)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1152, in to
return self._apply(convert)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/torch/nn/modules/module.py", line 802, in _apply
module._apply(fn)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/torch/nn/modules/module.py", line 802, in _apply
module._apply(fn)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/torch/nn/modules/module.py", line 802, in _apply
module._apply(fn)
[Previous line repeated 2 more times]
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/torch/nn/modules/module.py", line 825, in _apply
param_applied = fn(param)
File "/home/aru-sarthak/anaconda3/envs/python_10/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1150, in convert
return t.to(device, dtype if t.is_floating_point() or t.is_complex() else None, non_blocking)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 2.00 MiB. GPU 0 has a total capacity of 15.69 GiB of which 4.38 MiB is free. Process 2063 has 211.51 MiB memory in use. Including non-PyTorch memory, this process has 15.23 GiB memory in use. Of the allocated memory 14.92 GiB is allocated by PyTorch, and 60.01 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True to avoid fragmentation.  See documentation for Memory Management  (https://pytorch.org/docs/stable/notes/cuda.html#environment-variables)
Gist to reproduce the error -
https://gist.github.com/Othergreengrasses/4b950d336a112dd799b4120bcbeb60e7

Happy to take a look. Can you provide the hyperparameter YAML file and the command you used? (I didn't see that in the Gist, maybe I missed it though.)

Your device ran out of memory. You need to lower the max batch size and
accumulate gradients to simulate the requested batch size.
On Thu, Apr 11, 2024 at 5:39 PM Kyle Gorman @.***>
wrote:

Happy to take a look. Can you provide the hyperparameter YAML file and the
command you used? (I didn't see that in the Gist, maybe I missed it though.)
—
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/yoyodyne/issues/178#issuecomment-2050721560,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIQBSW5ASH6Y5FTLDL2GNDY44NM7AVCNFSM6AAAAABGDFINAOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJQG4ZDCNJWGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@kylebgorman
Github doesn't support YAML file type so I am just pasting whatever is in the file (btw I got the YAML file from you)
method: bayes
metric:
  name: val_accuracy
  goal: maximize
parameters:
  # Constants.
  arch:
    value: transformer
  max_epochs:
    value: 200
  patience:
    value: 40
  reduceonplateau_mode:
    value: accuracy
  gradient_clip_val:
    value: 3
  # Hyperparameters.
  attention_heads:
    values: [4, 6, 8]
  encoder_layers:
    values: [4, 6, 8]
  decoder_layers:
    values: [4, 6, 8]
  embedding_size:
    distribution: q_uniform
    q: 16
    min: 16
    max: 512
  hidden_size:
    distribution: q_uniform
    q: 64
    min: 64
    max: 1024
  dropout:
    distribution: uniform
    min: 0
    max: 0.5
  label_smoothing:
    distribution: uniform
    min: 0.0
    max: 0.2
  batch_size:
    distribution: q_uniform
    q: 128
    min: 128
    max: 2048
  learning_rate:
    distribution: log_uniform_values
    min: 0.00001
    max: 0.01
  scheduler:
    values: [null, reduceonplateau, warmupinvsqrt]
  reduceonplateau_factor:
    distribution: uniform
    min: 0.1
    max: 0.9
  reduceonplateau_patience:
    distribution: q_uniform
    q: 1
    min: 1
    max: 5
  min_lr:
    distribution: log_uniform_values
    min: 0.000001
    max: 0.001
  warmup_samples:
    distribution: q_uniform
    q: 100
    min: 100
    max: 5000000

command that I used -
wandb sweep --entity arundhati-sengupta81 --project Google_ben transformer_broader_config.yaml
./train_wandb_sweep.py --entity arundhati-sengupta81 --project Google_ben --sweep_id sz5urvxk --model_dir /home/aru-sarthak/yoyodyne/yoyodyne/models --experiment Google_transformer --train /home/aru-sarthak/multisource_g2p/g2p-Google-train.tsv --val /home/aru-sarthak/multisource_g2p/g2p-Google-dev.tsv --arch transformer --patience 10 --max_time 00:06:00:00 --count 200 --accelerator gpu --seed 1818 --source_sep ' ' --target_sep ' '

@Adamits this is a pretty straightforward grid for a transformer, I think, so it's a surprise to me this would OOM on the vast majority of runs on a... I believe @Othergreengrasses is on a 4th generation Nvidia card. (That said I don't have one at home so I can only replicate with a 1st gen card.)

Yes, Kyle you are right. I am on a 4th generation Nvidia card.
Also, the error is saying that I have 8.19 MB is free space then I am not able to understand why it is shooting OOM.
I'm running the experiment on a fresh environment (python 3.10) installing yoyodyne from source.

You could try the suggestion to set PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True (prepend it to whatever command you're running) if you haven't. I haven't tried this yet.

There's a panel in W&B for "GPU Memory Allocated (%)" (under "System"). If there is a memory leak across the multiple runs of a sweep, you'd expect that this would creep up as the number of sweeps increased. I just checked an old sweep (pre the supposed fix we put in place for this) and I don't see this pattern at all. Not sure what to make of that.

You could try the suggestion to set PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True (prepend it to whatever command you're running) if you haven't. I haven't tried this yet.

This didn't work either.

After replicating some of these things on our lab machines (with 1080 GTXs, which have 8 GB of VRAM) I think that the transformer models we are working with are just too large for the combination of batch sizes, number of layers, and hidden layer dimensionalities. I am not seeing evidence of a leak. Automagical batch sizing (à la #148, but let's also stipulate that having found the max batch size it then picks the right size for mini-batches and the right number of mini-batches per batch) ought to handle this for good.

I'm going to close this for now. We can return to it later. #148 is the path forward...


Malformed string in theme.toml causes problem with Hugo Themes site

https://github.com/guangzhengli/hugo-theme-ladder/blob/8ea84e0ef81998de520e517187baf5a544e20882/theme.toml#L9
Missing quotation at end of string.

Thanks for your reminder.


Update Meta “authors”

Automatically generated by Netlify CMS

:construction_worker: Deploy Preview for wizardly-edison-391c51 processing.
:hammer: Explore the source changes: 70a57dc89a75edaec00ff88f5aacb513a1515c06
:mag: Inspect the deploy log: https://app.netlify.com/sites/wizardly-edison-391c51/deploys/60ddf6ce43f9cb0008f5a392


Greenkeeper/eslint plugin import 2.18.2

Closes #18

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CFE allows extending Function

There are new test failures on Enforce current library restrictions..
The tests
language/nonfunction_type_aliases/generic_usage_function_error_test MissingCompileTimeError (expected Pass)
language/nonfunction_type_aliases/usage_function_error_test MissingCompileTimeError (expected Pass)

are failing (at least) on configurations
cfe-linux
cfe-strong-linux
cfe-weak-linux

A change of behavior was added, making Function be a final class, which should prevent it from being used as a superclass, interface or mixin in language version 3.0 code (but not below, where it should still be ignored like now).
The error is reported by the analyzer, probably because the analyzer used the final class Function declaration as basis for its type, and is not reported by the CFE, presumably because it replaces the declaration entirely with its own internal definition.
The CFE should make it an error to use the Function class as a supertype in class-modifier-enabled code.
(Maybe it's as easy as setting the final flag on the internal type definition. One can hope.)

Implemented with https://github.com/dart-lang/sdk/commit/1ce23bbd0bbb419537dc9d136aad774c9b96f4ac.


Proper way to remove user from group?

Hi there, I'm just wondering is there a Roles.deleteUsersFromRoles( function to delete a user from the admin role. I tried using setUserRoles to [] but then when I tried adding a user to admin again + a group it gave me an error "Error: Roles error: Can't mix grouped and non-grouped roles for same user".

You have to decide if you are using groups or not. Can you share more of the code you are using?

I'm just running tests from meteor shell
> Roles.addUsersToRoles("jNX4BenrzyEgsMqTY", "admin");
> Roles.setUserRoles("jNX4BenrzyEgsMqTY", []);
> Roles.addUsersToRoles("jNX4BenrzyEgsMqTY", "admin", "default-group");
Error: Roles error: Can't mix grouped and non-grouped roles for same user

I think I've worked out what's happening anyway. When I added my user to the admin role with no group it created the group "admin" doc in db.groups. I'm guessing to use the "admin" role along with groups I would now have to go and delete the "admin" doc from the database and start again.

You are mixing groups and no groups. In the first command you have run it without groups. So now you are stuck with no groups.

Ahh yes I ran into that. Fixed it by deleting the roles: [] key from my user. Thanks for your help. It might not be too difficult to simply force using a "default-group' when no group is specified. I might fork and see if I can make a patch if I have time down the track.
Cheers.

This is fixed in the v2.0 branch, BTW. There you do not have such issues anymore.


refactor grafana proxy and add helper for grateful shutdowns

First of all, I added a new helper to our internal pkg for easily setting up graceful shutdowns. This should be more robust and will reduce code duplication by quite a bit, once implemented into all other services. Check out golang/internal/gracefulShutdown_test.go for an example.
Aside from that, I refactored some of the code related to the grafana proxy and fixed some issues that were present, especially in the main.go file. So let's begin with the issue, before this PR right here, the main function ended with:
select {} // block forever

As the comment implies, this blocks until the program is explicitly terminated with an os.Exit(0), which should be normally avoided if possible, reason being it immediately exits the program, so any other tasks will abruptly end and defer calls will not run. This last point takes me to the block of code located at the top of the main function (current code in main branch):
logLevel, _ := env.GetAsString("LOGGING_LEVEL", false, "PRODUCTION") //nolint:errcheck
log := logger.New(logLevel)
defer func(logger *zap.SugaredLogger) {
	err := logger.Sync()
	if err != nil {
		panic(err)
	}
}(log)

So because of the select trick above to block forever, and the shutting down of the service with os.Exit(0), this defer call never runs, but with the new graceful shutdown helper implementation it works as it should. Once I tested my changes and triggered a shutdown, the actual deferred function ran, great! But... the logger.Sync() call returned an error and therefore panicked, and this last thing is the reason for this PR.
The rest of the changes are of refactoring nature to clean up the code. For example I turned the CORS handling of the main REST API to a middleware and introduced some dependency injection where it made sense to do so.
NOTE: CI will complain about the logger.Sync call because it's part of this. And it also seems to complain about errors.Join being undefined, the explanation for that is simple, the setup-go action seems to use Go version 1.19 and the Join method was added to Go 1.20. So I guess we also should update that.

Looks great! I have updated the go version in the CI and bumped umh-utils to 0.2.2 in this PR. You can wait for it to be merged (might take a while) and rebase, or you can merge that branch here to see if the checks are passing

Awesome. Yeah, whenever your PR is merged, I'll just rebase and push again, so no worries.

Looks good

Squash and merge?


Print out warning if statistics are nonsensical

... like F3 with homozygous pop C, but also see FST.

Done in v<IP_ADDRESS>


double click result to closing the parent modal

I use ionic datepicker inside a ionic modal.
If I single-click to ionic-datepicker, it appears properly, but if I double click to ionic-datepicker, it appears but close the parent modal at the same time.
How could I fix this ?

May i know what is the exact need to double click it. You can only click it once. If you don't want to allow users to click multiple times, then you can disable the button on first click.

Thank you for your reply!
It 's not a usual use case when user double click the datepicker. But if they do, the issue happen, so I just want to prevent them from doing that. Is it possible to disable the button in the end of the ng-click function ?

Yes, you can disable the button. Check this link. Once the user clicks on a button you can make it disable.