ImportError: cannot import name 'G'  - (python3.5)

I am trying out mecode for the first time and using python3.5 for this. I have followed the install instructions corresponding to source install in the readme file. After the install I tried the following line in the command line on Ubuntu 16.04, I get the following error.  I read an earlier issue where a python 3.4 compatibility was talked about, but has there been any update in this direction since? Or is mecode not compatible with python3 generally?
~$ python3
Python 3.5.2 (default, Nov 17 2016, 17:05:23)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from mecode import G
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: cannot import name 'G'

Did you name your python file mecode.py?

The above example is only command line script in a linux terminal. I have not written it in a separate file. Python 2 scripts work like a charm. For example, this is using python2
`
$ python2
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.

from mecode import G

`
It successfully imports G from mecode.
And this is using python3,
`
$ python3
Python 3.5.2 (default, Nov 17 2016, 17:05:23)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.

from mecode import G
Traceback (most recent call last):
File "", line 1, in
ImportError: cannot import name 'G'

`

Hmm, thats odd. I would guess there is some sort of install error. The tests are passing on python 3.4 and python 3.5:
https://travis-ci.org/jminardi/mecode
I am curious what is available under mecode when you import it. Can you let me know the output of the following?
import mecode
print(dir(mecode))

This is what it gives.
~$ python3
Python 3.5.2 (default, Nov 17 2016, 17:05:23)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import mecode
>>> print(dir(mecode))
['__doc__', '__loader__', '__name__', '__package__', '__path__', '__spec__']
>>>

Interesting that the python2 gives the following output.
$ python2
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import mecode
>>> print(dir(mecode))
['G', 'GMatrix', '__builtins__', '__doc__', '__file__', '__name__', '__package__', '__path__', 'decode2To3', 'is_str', 'main', 'matrix', 'printer']
>>>

Yeah the second one is what is expected. Were there any error messages when you installed in python 3?

It works now! I think I previously did not install mecode using python3 and pip3 in the following install steps,
$ pip install -r requirements.txt
$ python setup.py install

Thanks a ton!
Also, if I had some questions while using mecode, do I create a new issue or contact you some other way?

Great! If you have questions feel free to open another issue.

I only have Python 3 (I have 3.7.2) and I get the same readout as selvincephus does when using import mecode and print.
I installed mecode into my Downloads and extracted all, then places in the Lib folder of my python directory. I must be missing something in terms of the install

jminardi I am pretty new to this. where do I install?

@bioeng9 Sorry I never saw this as it is on a closed issue.
To install with python 3:
python3 -m pip install mecode
If you are still having trouble, please open another issue


Warning 32 ought to be respected

Attributes that disable warning 32 in sections of source code are used to silence the compiler's dead code warnings. It would be best if reanalyze were to also respect these warnings. For example, deriving compare in signatures is careful to disable warning 32 to prevent reports on generated code:
let compare_int = compare

module M : sig
  type t = int [@@deriving_inline compare]

  include sig
    [@@@ocaml.warning "-32"]

    val compare : t -> t -> int
  end
  [@@ocaml.doc "@inline"]

  [@@@end]
end = struct
  type t = int [@@deriving_inline compare]

  let _ = fun (_ : t) -> ()
  let compare = (compare_int : t -> t -> int)
  let _ = compare

  [@@@end]
end

This is used in generated code, but also in non-generated code, such as here.

Hey Josh. Unrelated but glad to see you again =)

I guess this falls in the category: it is dead, it should be considered dead, but should not be reported as dead.
Not sure I'd want to consider global options, only these local @@@ocaml.warning "-32". Which makes it an ad-hoc treatment. Also, not clear how to get hold of global options anyway. So one would observe a difference when disabling 32 in the command line, and in the code.

Yes: dead but shouldn't be reported. But it is not dead in the [@live] sense rather than the [@dead] one, since such code might be the only use of other code, and that other code should not be reported dead.
Considering only the attributes seems pragmatically sensible, since it isn't clear what the use case might be for compiling with -w -32 and expecting dead code analysis to not be noisy.
FWIW, I don't think that this really has anything crucially to do with ppx. Warning 32 predates ppx, and there are reasonable cases where e.g. a codebase's style mandates that each type comes with a common set of operations like compare and hash.
Also, it is probably a good idea to handle the other "unused" warnings similarly. Maybe 32-39, 60, 66.

edit: s/dead/not dead/

There’s also the automatic deletion of dead code. And for that one would like to delete even things kept alive only by this.
But it’s not a big issue. I’ll keep it simple.

Ok, but for automatic deletion of dead code it's a ppx that outputs a new parsetree right, so no need to change the source, just give it to the rest of the compiler.

Currently the analysis adds annotations, and the deletion  ppx runs afterwards. So there’s a limit anyway to what can be done with that setup, as you can’t annotate code generated by a ppx.
A fuller solution would require deep integration in the compiler, as typed ast to typed ast xform.

So for now it seems that indeed treating 42 as @live is the way forward.

Currently the analysis adds annotations

Does this work by adding attributes to the parsetree? If so, it would be rolled into one ppx pipeline and not have to worry about whether code was generated or not.

No it’s literally printing into the source file.
So it’s very easy to take a quick look at a project without going through the list of reported issues.

Btw the analysis operates on the typed tree. Does not fit a ppx pipeline.

Ah, yes, I knew but overlooked that. It would be a bit of work then to get back to the untyped tree, or construct a new one, or... Hmm. I suspect that you could reuse the implementation technique of ppx_import though, as it reads cmt files and generates code. That needs to use the staged_pps instead of pps flow in dune, but that should also be doable for reanalyze I think.

The analysis is global so strictly speaking does not fit in the ppx flow by that reason alone. However of course it could store the results somewhere then run a ppx that reads them.

Oh yeah, very true. I imagine that you can figure out something compositional to do... ;-)

I’d have first to make it slower, to give an excuse.

@jberdine this is done in https://github.com/cristianoc/reanalyze/pull/13.
Unsure about whether to support other suppressions beyond 32. What did you have in mind?

A relevant one is 34 Unused type declaration. Though that's a bit different from the sense in which variant/records parts are unused.
Merging this for now.

The following might also make sense:
37 Unused constructor.
38 Unused extension constructor.
60 Unused module declaration.

It seems that code like
let[@warning "-32"] foo x = print_int x

still gets flagged if foo is otherwise dead.

Let me check.

Tried to repro the example here:
https://github.com/cristianoc/reanalyze/pull/22
No cigar.
What was the example?

Oh wait warning not ocaml.warning?

Yes, they are synonyms.


update nbhd hovering text in "other resources" tab

hovering popup text in "other resources" tab shouldn't have information about reschool programs! also, make it clear that demographic information is for STUDENTS

add # parks, playgrounds, etc.


Added TOC and intro

Included Table of Contents (with click links for faster navigation)
Included main intro to Orbital plus hyperlink back to Orbital Mock main website for the full overview/how to's etc.
Preview on IDE:

Pushed a quick addition to include TOC title


APIs to be used

TVMaze API: TVmaze offers an API that allows you to retrieve information about TV shows, including information about popular and trending shows. No API key is needed.
Trakt API: Trakt provides an API that offers data related to TV shows, including trending shows. They have a public endpoint that can be used without requiring an API key to access trending data.


update for core api

Coverage increased (+0.009%) to 35.376% when pulling<PHONE_NUMBER>f4eb1ed91f8a53cce2f5d55057f304 on uat into fd42d22a6bd792510013c9c53758047a55bcd07e on master.


operator minio-directpv-operator-rhmp (4.0.7)

We no longer want to certify our operator in this catalog.


mmdetection integration does not consistently handle background class (k versus 0 position)

In MMDetection, the background class is k, not 0 (for k-1 classes).
Currently, show_results shows ground-truth with 0=background with predictions showing k=background and 0=first class.

The parser generates a class_map, but the dataloader needs to correct it (switching the background location) and doesn't have a way to return the corrected map.
A hack in eg show_preds - but this is very wrong!
for pred in preds:
pred['labels']=pred['labels']+1

A way of handling this is to do the conversion back in the predict method we implement for mmdet models.
what do you think?

I'm unsure of this. There is an architectural issue here about how the responsibility of the adaptors and what they apply to.
The parser hacks the class map and adds the background to '0' or 'k' as appropriate. If it adds it at 'k', then it also renumbers other classes (e.g. '1' becomes '0'). If it hacked the ground truth, shouldn't everything work out? I.e., I'm not sure why anyone should see a '1' as the first class.

A way of handling this is to do the conversion back in the predict method we implement for mmdet models.
what do you think?


62 fix omim

Quick fix to the load_multi_context function that appears to fix the issue with OMIM being dropped.
Added a context for Monarch KG in the etl_runner so Monarch can keep ownership of the file merged.monarch.csv to watch out for changes that might be breaking.

I had written  this longer summary earlier but decided this is better in the comments section.
This provides a quick fix to the load_multi_context function that appears to fix the issue with OMIM being dropped. I'm still unclear why this is happening.
Separately, I've added a context for Monarch KG in the etl_runner similar to merged.oak that is an exact copy of merged and merged.oak. These can now diverge if necessary and Monarch can keep ownership of the file merged.monarch.csv to watch out for changes that might be breaking.

I have elected not to run etl_runner for this PR so we don't add complexity. If you want me to do etl_runner and resubmit, I'm happy to.


PDF stamping: Is it possible with current merge functions?

Thank you for your awesome project! Hurray! I was waiting something like that for loooong time
Can you verify if I could stamp a pdf file with an another pdf file using the merging facility?
Cheers! Dimitris

It really depends on what you mean by stamping.
I do not understand what's stamping got to do with merging of two files.
When I hear PDF stamping I am thinking more about watermarking pages.
Please propose your exact requirements.
A content transformer for sweeping over pages and page content sections is being worked on.
Something like this can also be used for watermarking a file.

You are write, I mean watermarking pages. I used a term that is coming from the "Adobe Acrobat" world.
Again thanks for your time.

Dear Mr Hrutter,
Can you describe in general what is the procedure about  making an overlay to a pdf file?
Thank you in advance :)

There is no better source than the PDF specification document.

v0.1.15 comes with support for watermarking/stamping PDFs with text or images.

Excellent! Thank you spending time for this issue!

Is it possible to stamp/watermark a pdf file with another pdf file like adobe acrobat?

not yet but coming up

fixed with v0.1.18


Search is case senstive.

Package version
0.1.95
Error Message & Stack Trace
N/A
Relevant Information
Do I need to do something to get the search to not be case sensitive? Performs fine with partial matches, but fails to fetch records if there's a case mismatch, even on otherwise identical queries.
[
  Profile {
   ...
    '$attributes': {
      id: 1,
      userId: 1,
      firstName: 'Henry',
      lastName: 'Kessler',
      createdAt: [DateTime],
      updatedAt: [DateTime]
    },
  }
]

I have import { search } from 'adosearch' on my model as well as the following static method.
public static search = search(this, ['firstName'])
...
@column()
public firstName: string

This fails find the record.
const search = await Profile.query().withScopes((scopes) => scopes.search('henry'))
This finds the record.
const search = await Profile.query().withScopes((scopes) => scopes.search('Henry'))
I'm not sure what's causing this. Your examples made it seem like case didn't matter. Any help would be appreciated. Everything else seems great about it so far.

Hi @TDodgeCo,
Thanks for reporting this. It's an issue with how SQL's where like works out of the box. See this question on stack overflow. I'll modify the search query include the LOWER  inbuilt SQL function so that searches are no longer case sensitive

The same here I know it's not an issue but it will be good if used ilike instead or if possible to pass a parameter to the seach scope to accept or not Lowercases/uppercases search (like or ilike)


tensorboard gulp - analytics.js missing

installing frontend dependencies works fine from a clean ubuntu install:
npm install
bower install
tsd install
compiling all deps with gulp vulcanize throws an error however:
[~/tensorflow/tensorflow/tensorboard]$ gulp vulcanize
[07:29:48] Using gulpfile ~/tensorflow/tensorflow/tensorboard/gulpfile.js
[07:29:48] Starting 'compile.all'...
[07:29:48] Starting 'tslint-strict'...
[07:29:51] Finished 'tslint-strict' after 3.24 s
[07:29:53] Finished 'compile.all' after 5.34 s
[07:29:53] Starting 'vulcanize'...
[07:29:53] Finished 'vulcanize' after 6.08 ms
ERROR finding /home/ubuntu/tensorflow/tensorflow/tensorboard/lib/js/analytics.js

not a huge problem as the frontend works just fine with gulp server

Any update yet ??? I have same issue

For a temporary fix, remove the "lib/analytics.js" line from the top of app/index.html and gulp vulcanize will work fine. That said, I haven't actually tested the open-source build of the TensorBoard frontend so you might run into other problems. I'll test it out and comment back once I have a first class solution on the way.

Note by the way, you don't need to use gulp at all, unless you are recompiling the TensorBoard frontend from source.

@danmane ive been testing the tensorboard release quite a bit and it seems to be all working well. events, images, graph and histogram all good! :)

glad to hear it :)

Thanks @danmane that worked. :)

We've fixed this :)


[Feature Request] Remove sensitive personal information from info.json files

Checklist

[ x ] I'm reporting a feature request
[ x ] I've verified that I'm running yt-dlp version 2021.01.29
[ x ] I've searched the bugtracker for similar feature requests including closed ones

Description

Would it be possible to remove sensitive personal information such as IP adress and OS username from the info.json file? Since official YTDL doesn't seem to care.
more info on the problem:
https://github.com/ytdl-org/youtube-dl/issues/25576
https://github.com/ytdl-org/youtube-dl/issues/25681

The _filename entry can be easily stripped out, but the IP from the URL cannot; since the URL is useless without the IP. This will break the functionality of --load-info-json
I could, prevode a new switch to strip out identifying info, but again, this only really applied to youtube videos since I do not know what other identifying info could be present in other extractors' URLs. In my honest opinion, it is far better to create a small python script to be called using --exec that removes the information you deem troublesome.
Let me know your thoughts, and whether or not you think an additional script will satisfy your needs

a python script that can be used with  --exec should be sufficient. It would be nice if such a script could be added to the Readme with a disclaimer about it on the --write-info-json section.

Upon further investigation, it seems that your request is related to tubeup. On a quick read of tube-up source code, it seems that tubeup already post-processes the json. So it should be easy to strip the data from their end. Am I wrong?

oh didn't know about that. They say  that there isn't a fix yet but here is a pull request for it.

I have pushed a patch to remove _filename, but like I said above, the IP cannot be removed from my end without sacrificing existing functionality. So I am closing this issue.
I try to help the tubeup devs implement this on their side.

I'd personally like to keep this information. Without it, my project simply won't work with this fork. Would it be possible to add an arg like --keep-sensitive-info or the inverse?

@Tzahi12345 I only removed the _filename field which (by python naming convention) is for internal use only anyway. Could you explain how/why you are using this field?
Note that this filename maynot be the correct final filename depending on the options used

@pukkandan I use it in my project YoutubeDL-Material to locate the file after it's been downloaded, it all happens in the backend code and so it would be difficult to find the file otherwise (especially if users have a custom output like %(id)s)

Note that this filename maynot be the correct final filename depending on the options used

Yeah this is a pain to deal with but I basically just restrict the file type to mp3/4 and so I simply strip out the bad extension from _filename and replace it with the right one

@Tzahi12345 The correct way to get the filename after download is to either look at the infodict at the end (if using API) or to use --exec. Anyway, I'll see whether I can give an option to keep it in the infojson

@Tzahi12345 I have added an option --no-clean-infojson to prevent any fields from being filtered out of the infojson. Note that this doesn't just add back _filename, but also a few other fields like requested_formats, requested_subtitles etc that are normally removed when creating the json

Awesome thank you so much @pukkandan! I look forward to the next release so I can finally support yt-dlp :)

The other issue with stripping out IP is we can't test it or WHOIS the IP to see if theres unreported by the host georestrictions. So I'll have a bunch of kids dumping channels from other supported sites, supported site silently implements something to georestrict or throttle a region or country or IP, and I have no way of seeing whats going on because they'll refuse to post their IP - alternatively you could implement a flag that removes the redaction so I could force it's use for submitting issues for my project.
Or people who use Tubeup could just get a VPS and not rip from their desktops.

@vxbinaca See my earlier comments

I have pushed a patch to remove _filename, but like I said above, the IP cannot be removed from my end without sacrificing existing functionality. So I am closing this issue.

I have added an option --no-clean-infojson to prevent any fields from being filtered out of the infojson. Note that this doesn't just add back _filename, but also a few other fields like requested_formats, requested_subtitles etc that are normally removed when creating the json

Or people who use Tubeup could just get a VPS and not rip from their desktops.

and by "get" he meant "rent" a virtual ""private"" server, just so that the ip the program spits out is of your vps provider's rather your isp's
note to self: don't post when you are drunk or angry


Add LightGBM Tuner by Optuna links to README.md

Hi!
Optuna is a new hyperparameter optimization library, and we've written an tuner module for LightGBM to make it easy to use Optuna to search for good hyperparameter settings and prune unpromising trials. We're looking for ways to let users of LightGBM know this is available and thought a badge to show Optuna integration is available would be helpful along with a link to Optuna in the further readings section.
If there are other ways you would recommend reaching out to LightGBM users to let them know about Optuna, please let us know.
Thanks!

Thanks @Crissman , it is wonderful.
BTW, may be you can add optuna link to our documents, like https://github.com/microsoft/LightGBM/blob/master/docs/Parameters-Tuning.rst and https://github.com/microsoft/LightGBM/blob/master/docs/Parameters.rst

Thanks @Crissman , it is wonderful.
BTW, may be you can add optuna link to our documents, like https://github.com/microsoft/LightGBM/blob/master/docs/Parameters-Tuning.rst and https://github.com/microsoft/LightGBM/blob/master/docs/Parameters.rst

I think it would be enough to just add to Parameters-Tuning.rst. In my opinion we should try to keep Parameters.rst limited to just telling you what parameters exist and the impact of choosing different values for them.

thanks @Crissman !


Add support to some how attach a debug session to a remote instance of python process

E.g. attach debugger to remote docker container

That would be great. This is something, that I miss most when working in with docker-compose in my workflow. If there is a way to enable this working with docker quickly that would be super awesome!

This is possible today, this is achieved via Remote Debugging feature.

Option 1: Please have a look at the Remote debugging section in the documentation.
Option 2: You can use the experimental debugger to achieve this as follows:

pip install ptvsd --pre into your docker container
Start the program in debug mode (use the following as the value for CMD in your docker file: python -m ptvsd --server --port 5678 --file xyz.py
Where 5678 is the port to be exposed for debugging, and xyz.py is the file to be debugged.
Note: You'll need to use the experimental debugger.

E.g. attach debugger to remote docker container

Unfortunately there's no way to attach to a remove process. The only way to achieve something similar is:

Launch the remote process (docker container) and host the debugger libraries
Please see the two solutions provided in the previous section.

@brettcannon Not sure why we're flagging this as an enhancement?

@brettcannon When I created this issue, I don't quite remember the use case for this. Currently we can do this with the remote debugging options we have.
Once the experimental debugger is released, I believe we can close this issue.

I think you opened this to provide some UX for it to occur so it wasn't so manual.

This issue is outdated, we're looking at other ways of solving this


[Fix] Editing assistant without changing its name is possible again

Related thread

Merging, to fix the issue in production. cc @lasryaric and @spolu since you're the next eng runners
Post merge review welcome


Implement tag matching for tasks

Tasks may have a tag that designates them for certain dealbots.
Dealbots may be configured with 0 or more tags to request tasks by.
Tasks will to tag match any request
Requests that specify no tags can get any task

@mvdan Added comments to describe tag semantics.


Retina Bug

I'm pretty sure there is a bug with the retina, because you are calling handleSource to set the src, it calls the original dataSrc and not the split version.

I confirm this bug too, please fix it asap

+1

i believe it is related to this message from changelog:
Marked breakpoints as obsolete. Will be removed in upcoming version. Use srcset and/or the picture element instead.
p.s. here is a hotfix of problem (if you can't replace src with srcset immediately):
`    function handleSource(ele, attr, dataAttr) {
var dataSrc = ele.getAttribute(dataAttr);
    if (dataSrc) {

	        var dataSrcSplitted = dataSrc.split('|');
	        var dataSrc = dataSrcSplitted[isRetina && dataSrcSplitted.length > 1 ? 1 : 0];

        ele[attr] = dataSrc;
        ele.removeAttribute(dataAttr);
    }
}`

Thanks for the code, I took a look I think this is what the author is referring to for breakpoints
// Example
var bLazy = new Blazy({
    breakpoints: [{
          width: 420 // max-width
	, src: 'data-src-small'
     }
       , {
          width: 768 // max-width
        , src: 'data-src-medium'
}]
});

Hey guys.
You're right. I'll fix it when I'm home from the Beach tomorrow. You can use 1.5.x or @WPJobs's hotfix until

Back from the beach :) Fixed in v. 1.6.2


Listbox is calling onChange with the wrong selected value

It is basically sending the previous value.
The problem is here:
https://github.com/eBay/ebayui-core-react/blob/58ba1d65ce6f833cbe0af65aee978f5c9a88d67a/src/ebay-listbox-button/listbox-button.tsx#L113
It should be getSelectedValueByIndex(index) instead.
The solution might be easy, but it would be a good moment to refactor the state a bit.
Perhaps the 'selectedOption' could be removed and just use the index.
(I will try to fix the problem if I find some time, creating the issue so we do not forget)

Fixed in https://github.com/eBay/ebayui-core-react/commit/f229732a7f9627d6415079e6c589776c79f6f8ef


i have error does anyone know how to fix it?

Severity	Code	Description	Project	File	Line	Suppression State	Details
Error	C1047	The object or library file 'C:\Users\Administrator\Downloads\private-fivem-master\Realistic\ext\keyauth\library_x64.lib' was created by a different version of the compiler than other objects like 'x64\Release\imgui_impl_dx11.obj'; rebuild all objects and libraries with the same compiler	Realistic Thunder	C:\Users\Administrator\Downloads\private-fivem-master\Realistic\LINK	1

Remove keyauth

Sorry im not play game any more. Try google your porblem to find solutions

how do you fix this error :
[     29750] [b2699_DumpServ]               263520/   d3d10.dll+D66AD
[     29750] [b2699_DumpServ]               263520/   ucrtbase.dll!thread_start<unsigned int (__cdecl*)(void *),1> (0x41)
[     29750] [b2699_DumpServ]               263520/   KERNEL32.DLL!BaseThreadInitThunk (0x13)
[     29750] [b2699_DumpServ]               263520/   ntdll.dll!RtlUserThreadStart (0x20)


Support multiple requests for the same url

Summary
This PR addresses issue https://github.com/dstnbrkr/VCRURLConnection/issues/33.

Recordings are now saved in an array for each url+method key.
After a recording is replayed, it's removed from array. The last recording in the array is not removed, to maintain compatibility with current behavior.

If VCR is in both recording and replaying mode, the replayed recording is added back to the queue. This behavior doesn't make a difference when there is only one recording per url. But when there are multiple responses, this creates a "loop". For example, responses A and B are in the array, the replay sequence becomes A, B, A, B etc. If VCR is only in replay mode, then the replayed sequence is A, B, B, B...

Thanks @haitaoli, this looks interesting! I will review and get back to you.

The replaying:(BOOL) parameter makes the logic a little complex. What if instead there was no replaying parameter? Looks like that would be possible if recordingForRequestKey: used an index instead of mutating the recordings array. For instance, recordingForRequestKey could just increment an index and use index++ % [recordings count] to cycle through each recording for each request. Might make the test cases simpler too since you won't need to account for different behaviors depending on replaying/recording.

recordingForRequest is called at two places, one to check if there is recording available (canInit), the other to get recording. I thought about giving them different and more meaningful names such as hasRecording and replay, but that would require a lot of changes to rename it everywhere. If you don't mind a much bigger PR I can do that.

Got it - given that it's used for both recording and replaying, what you have makes sense! So then the only outstanding item is the extra __block declaration and this is good to go. Thanks again for this contribution!

Removed uncessary __block.

@dstnbrkr thanks for merging it. Do you have plan to release a new version?

@haitaoli all set! (just shipped 0.2.3)

Great! Thank you.


Unconsistent region for service tags API

Hello,
In the service tags API, the region attribute does not have a consistent value compared to the other Azure tools.
Here's a non exhaustive list:

France Central is centralfrance
France South is southfrance
BrazilSoutheast is brazilse
GermanyNorth is germanyn
...

The value is region should be eaither the full name (aka "France Central"), the sort name (aka frc) or the normalized name (aka francecentral). The last one would make sense in my opinion.
Here's a subset of the API response with the issue
  "changeNumber": "114",
  "cloud": "Public",
  "id": "/subscriptions/513f7e88-cb3c-4e80-a6bb-b3a4984cfa6a/providers/Microsoft.Network/serviceTags/Public",
  "name": "Public",
  "nextLink": "",
  "type": "Microsoft.Network/serviceTags",
  "values": [

   ...

    {
      "id": "ApiManagement.EastUS",
      "name": "ApiManagement.EastUS",
      "properties": {
        "addressPrefixes": [
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/28",
          "<IP_ADDRESS>/32",
          "2603:1030:210:402::140/124"
        ],
        "changeNumber": "3",
        "networkFeatures": [
          "API",
          "NSG",
          "UDR",
          "FW"
        ],
        "region": "eastus",
        "state": "GA",
        "systemService": "AzureApiManagement"
      },
      "serviceTagChangeNumber": "114"
    },
    {
      "id": "ApiManagement.EastUS2",
      "name": "ApiManagement.EastUS2",
      "properties": {
        "addressPrefixes": [
          "<IP_ADDRESS>/32",
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/28",
          "2603:1030:40c:402::140/124"
        ],
        "changeNumber": "3",
        "networkFeatures": [
          "API",
          "NSG",
          "UDR",
          "FW"
        ],
        "region": "eastus2",
        "state": "GA",
        "systemService": "AzureApiManagement"
      },
      "serviceTagChangeNumber": "114"
    },
    {
      "id": "ApiManagement.EastUS2EUAP",
      "name": "ApiManagement.EastUS2EUAP",
      "properties": {
        "addressPrefixes": [
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/28",
          "<IP_ADDRESS>/32",
          "2603:1030:40b:400::940/124"
        ],
        "changeNumber": "3",
        "networkFeatures": [
          "API",
          "NSG",
          "UDR",
          "FW"
        ],
        "region": "eastus2euap",
        "state": "GA",
        "systemService": "AzureApiManagement"
      },
      "serviceTagChangeNumber": "114"
    },
    {
      "id": "ApiManagement.FranceCentral",
      "name": "ApiManagement.FranceCentral",
      "properties": {
        "addressPrefixes": [
          "<IP_ADDRESS>/32",
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/28",
          "<IP_ADDRESS>/31",
          "2603:1020:805:402::140/124"
        ],
        "changeNumber": "3",
        "networkFeatures": [
          "API",
          "NSG",
          "UDR",
          "FW"
        ],
        "region": "centralfrance",
        "state": "GA",
        "systemService": "AzureApiManagement"
      },
      "serviceTagChangeNumber": "114"
    },
    {
      "id": "ApiManagement.FranceSouth",
      "name": "ApiManagement.FranceSouth",
      "properties": {
        "addressPrefixes": [
          "<IP_ADDRESS>/32",
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/28",
          "2603:1020:905:402::140/124"
        ],
        "changeNumber": "2",
        "networkFeatures": [
          "API",
          "NSG",
          "UDR",
          "FW"
        ],
        "region": "southfrance",
        "state": "GA",
        "systemService": "AzureApiManagement"
      },
      "serviceTagChangeNumber": "114"
    },
    {
      "id": "ApiManagement.GermanyNorth",
      "name": "ApiManagement.GermanyNorth",
      "properties": {
        "addressPrefixes": [
          "<IP_ADDRESS>/32",
          "<IP_ADDRESS>/28",
          "2603:1020:d04:402::140/124"
        ],
        "changeNumber": "2",
        "networkFeatures": [
          "API",
          "NSG",
          "UDR",
          "FW"
        ],
        "region": "germanyn",
        "state": "GA",
        "systemService": "AzureApiManagement"
      },
      "serviceTagChangeNumber": "114"
    },
    {
      "id": "ApiManagement.GermanyWestCentral",
      "name": "ApiManagement.GermanyWestCentral",
      "properties": {
        "addressPrefixes": [
          "<IP_ADDRESS>/31",
          "<IP_ADDRESS>/32",
          "<IP_ADDRESS>/28",
          "2603:1020:c04:402::140/124"
        ],
        "changeNumber": "3",
        "networkFeatures": [
          "API",
          "NSG",
          "UDR",
          "FW"
        ],
        "region": "germanywc",
        "state": "GA",
        "systemService": "AzureApiManagement"
      },
      "serviceTagChangeNumber": "114"
    },

   ...

  ]
}

@BzSpi Thank you for reaching out, we are looking into it.

More than a year and half after, is there any news ?


Let user change the status code in callback

In get/post/delete callbacks, there's currently no way to change the status code. It defaults to 200 OK for both get/post.

Implemented in 96f10b544d113f1a6b1fbf9c5f05a257bee006a4


fixProhibit deleting admin user

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Prohibit deleting admin user
Release note
Prohibit deleting admin user
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[x] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall


Update start point guidance to show more formats

Fixes https://github.com/alphagov/design-system-team-internal/issues/552
Why
Right now, we tell service teams it’s a set pattern that cannot be customised. That doesn’t reflect what we’ve been doing on GOV.UK for a while now.
We’re doing this for a few reasons:

Make start pages less “one size fits all” - reflect what we’ve learned about when it makes sense to start a service journey within a multipart guide, using custom button text etc.
Reflect fact that some service journeys start on Whitehall.
Helping reduce cognitive load of some start pages - eg by suggesting service teams consider asking more eligibility questions or moving some of the ‘documents you’ll need’ information to a dedicated page near the start of the service.

@StephenGill Raised this PR based on our latest resolved draft.
Be good to get your eye to see how well the revised page reads in preview.

Thanks @EoinShaughnessy.
@StephenGill: I've committed the minor looking suggestions, mostly around style.
For the rest, I've written out some suggested draft content. Can you let me know if you're happy for me to commit any of these?
I know you're having issues logging into GitHub, so feel free to grab me on Slack.

Okay, I got this through the automatic checks. Should be ready to review from a code and structure perspective @lfdebrux.
The filename change and archive page should now included in the PR.
However, I'll need to add the redirect link (on the archive page) as a further PR it was coming up as a broken link and breaking the tests.


feat: sets up 404 pages

Adds not-found.tsx and wires up page
Adds forceDark prop to Highlight component so I can use it

This is how it looks in light/dark mode:

Hey @JessChowdhury this looks great!
Only thing I changed is that instead of using a forceDark property in the Highlight component, I instead wrapped the CallToAction block in a <ThemeProvider theme="dark"> which tells the Highlight component that you used to render in dark mode. That's the right way to do this. Looks like Jake's CallToAction component was not properly set up for that. That's all!


identify fuzzing targets

Identified fuzzing targets:

Parse TdHob stage
VirtIO components (enumerate & start virtio devices)
other components (focus on finding bugs in code)

More details in the target prioritization document in the thesis repo


BUG: emit init code before body code

Copying comment here:
The crash in cdef_multiple_inheritance has to do with a change in the method assignment. In the old version, the code
cdef class Both(CBase, PyBase):
    cdef dict __dict__
    cdef c_method(self):
        return "Both"
    cpdef cp_method(self):
        return "Both"
    def call_c_method(self):
        return self.c_method()

would assign cp_method and call_c_method to a PyMethodDef struct. In the new code only call_c_method is assigned in the PyMethodDef struct, but both are assigned to the __pyx_ptype...Both->tp_dict inside __Pyx_Init_Globals which happens while __pyx_ptype...Both is still NULL. The assignment __pyx_ptype...Both = &__pyx_type...Both happens in the next fragment, a few lines down.
This PR reorders the fragments so that __pyx_type...Both is set up before the method assignments and does not seem to cause any new test failures.

This makes sense to me.

Yes, sounds right.
Since it makes a difference of live and death here, could you please also duplicate the specific test so that we have it explicitly as with both binding(True) and binding(False) ? Can be done in the same file with a class decorator, I think.

class decorator

I couldn't work out how to do that. A class decorator errors with Cdef functions/classes cannot take arbitrary decorators And this
for t in (True, False):
    with cython.binding(t):
        cdef class Both(CBase, PyBase):
            ...

fails with cdef statement not allowed here
I could convert these tests to a sourcetree?

class decorator errors with Cdef functions/classes cannot take arbitrary decorators

It shouldn't. That means that it somehow failed to detect that the decorator is a directive. Could you show the exact code that you used (or just commit it and let it fail in Jenkins?)

Pushed a commit, but CI is not set up to run in a PR against a branch

ping

Ah, sorry, thanks for the ping.
You need to cimport cython at the beginning of the test.

test passes locally with both cython.binding(True) and cython.binding(False)

tests pass on CI

Thanks. I'll merge it into the 'binding' branch to give it some testing.


DESENG-55

Should be the last PR for this ticket in this repo.
Completed file upload modal. Allow project to save logos. Clean up add-edit-project code
[-] Wrote a test or tests for code you added.
[x] Linted the files you touched.
[x] Wrote a doc block for each method you added.
[-] Add documentation to the README

Changing method names (i.e. selectFile -> handleSelect) in older code makes me incredibly nervous. Are you confident every instance has been updated?
The risk here is that some other old piece of code isn't updated to use the new method, and we are left in a worst state then we started in.

Thanks for the review @SteveMHoward @CR-gbc .
Getting nervous about changing function names is totally valid. I did make sure that every instance of this call was changed. FYI the selectFile method was added a couple of commits ago so it's not an old method.

Perfect :) I know how easy it is to miss one or two but looks like you fully tested it all.


Errors after installation, libraries not compressible

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Please inform with utmost urgency on the MUI website the libraries that are supported in this new version. I am of the opinion that some of them should be included in the documentation.
Expected behavior 🤔
No response
Steps to reproduce 🕹
Steps:

Context 🔦
No response
Your environment 🌎
TypeError: Cannot read properties of undefined (reading 'body1')

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Bulk Edit Sounds should be 2nd in the drop down list of right click on playlist options

If possible, inject it inside the list at 2nd position rather than at the bottom.

Defintely doable, will get this into the next feature release.


Completely unmap  from plugin

Is there a way to completely unmap <cr>? I don't want the plugin to do anything (no completion, confirm, etc) other than give me a new line when pressing <cr>.
I apologize in advance if the solution to this is already stated somewhere, but I could not find it and I have tried everything I could think of. Thanks!

You can specify mapping = {}

Thank you. I think I found the issue, was related to Packer not compiling and nothing to do with the cmp plugin, sorry for taking up your time. Keep up the great work, really enjoy it. 👍


Inconsistent Key Count after BGSAVE and IMPORT

I am trying to keep 2 RocksDB databases consistent (they are rarely updated). To do this, I was hoping to simply run BGSAVE on the first machine via ardb and then IMPORT that snapshot into the second machine using ardb.
Unfortunately, I get inconsistent key counts each time I run this.
Is there a better way to copy one to the other?

is the key count reported by 'info' command?

I was using dbsize, but the info command yields:
Keyspace
db0:keys=939859164
Keyspace
db0:keys=877488365

RocksDB provides only an estimated number of keys stored in the column family and it is affected by the number of pending compactions. See https://github.com/yinqiwen/ardb/issues/351

Is there any way to confirm the two sets are identical or is it best to just check a random set of the key=>value results as a test?

the only way is to save again to generate a new snapshot file and then compare the md5 checksum value.


Check domain and range hyperlinks to DCAT in specification

Some domain and range hyperlinks redirect browsers to the DCAT 2 specification.
For example:  https://ekgf.github.io/dprod/#inputdataset .  If you click on the Range link it takes you back to DCAT 2 .
Best regards,
Eric Stephan
Pacific Northwest National Laboratory

That is really an issue for DCAT since they decided to use the same namespace for DCAT 2 as was used for DCAT 3. We'll try to take it up with them.

FYI the DCAT team is taking action https://github.com/w3c/dxwg/pull/1618


How do you apply customized loss function?

Recently I am working on a tool utilization (mainly calculator) project. (Q: What is 32 times 13? A: The answer is [Calculator(32*13)]. )
In this task, I need the original loss to be timed by a coefficient number for punishing the model outputs if the API calculation results are wrong. What do I need to do with PEFT to apply customized loss function?
Thanks!

It really depends on how you train your model. PEFT is agnostic with regards to that, so it doesn't impose any kind of loss. It is up to you to define it.

It really depends on how you train your model. PEFT is agnostic with regards to that, so it doesn't impose any kind of loss. It is up to you to define it.

I am using a transformer trainer. I notice the loss is defined here (transformers.trainer_pt_utils)
@dataclass
class LabelSmoother:
"""
Adds label-smoothing on a pre-computed output from a Transformers model.
Args:
    epsilon (`float`, *optional*, defaults to 0.1):
        The label smoothing factor.
    ignore_index (`int`, *optional*, defaults to -100):
        The index in the labels to ignore when computing the loss.
"""

epsilon: float = 0.1
ignore_index: int = -100

def __call__(self, model_output, labels, shift_labels=False):
    logits = model_output["logits"] if isinstance(model_output, dict) else model_output[0]
    if shift_labels:
        logits = logits[..., :-1, :].contiguous()
        labels = labels[..., 1:].contiguous()

    log_probs = -nn.functional.log_softmax(logits, dim=-1)
    if labels.dim() == log_probs.dim() - 1:
        labels = labels.unsqueeze(-1)

    padding_mask = labels.eq(self.ignore_index)
    # In case the ignore_index is -100, the gather will fail, so we replace labels by 0. The padding_mask
    # will ignore them in any case.
    labels = torch.clamp(labels, min=0)
    nll_loss = log_probs.gather(dim=-1, index=labels)
    # works for fp16 input tensor too, by internally upcasting it to fp32
    smoothed_loss = log_probs.sum(dim=-1, keepdim=True, dtype=torch.float32)

    nll_loss.masked_fill_(padding_mask, 0.0)
    smoothed_loss.masked_fill_(padding_mask, 0.0)

    # Take the mean over the label dimensions, then divide by the number of active elements (i.e. not-padded):
    num_active_elements = padding_mask.numel() - padding_mask.long().sum()
    nll_loss = nll_loss.sum() / num_active_elements
    smoothed_loss = smoothed_loss.sum() / (num_active_elements * log_probs.shape[-1])
    return (1 - self.epsilon) * nll_loss + self.epsilon * smoothed_loss

I have no experience with the transformers trainer, but it appears it's not too difficult to provide a custom loss: https://huggingface.co/docs/transformers/main_classes/trainer
Anyway, that's out of scope for PEFT.

Many thanks mate


Ordered Groupby Keys

The current behavior of the xarray's Groupby.groups property provides a standard (unordered) dictionary.  This is fine for most cases but leads to odd orderings in use cases like this one where I am using xarray's FacetGrid plotting:
plot_kwargs = dict(col='season', vmin=15, vmax=35, levels=12, extend='both')

da_obs = ds_obs.SALT.isel(depth=0).groupby('time.season').mean('time')
da_obs.plot(**plot_kwargs)

Note that MAM and JJA are out of order.
I think this could be easily fixed by using an OrderedDict in xarray.core.Groupby.groups.

I agree this is annoying, but I don't think your diagnosis is correct here. The groups property isn't used by any internal routines AFAICT. The issue is that groups are sorted, but as text rather than ordered categorical -- notice that the labels are ordered alphabetically.

Hmmm, a mystery.  I'll look into this a bit more.

For what it's worth, I don't think we have any good solutions short of
adding our own array type do to Categorical in xarray. We could set
sort=False in some cases when we call pd.factorize but that's not a great
alternative.
On Wed, Feb 10, 2016 at 6:01 PM Joe Hamman<EMAIL_ADDRESS>wrote:

Hmmm, a mystery. I'll look into this a bit more.
—
Reply to this email directly or view it on GitHub
https://github.com/pydata/xarray/issues/757#issuecomment-182671718.

Ahh, so it's sorted, instead of keeping the original order.
I was expecting DataArray.groupby().reduce would work like np.apply_along_axis, and used the data of the result directly.

Just stumbled across this issue. Is there a recommended workaround?
I am usually doing this (specific to seasons):
import xarray as xr
ds = xr.tutorial.open_dataset('air_temperature')
airtemp_seasonal = ds.groupby('time.season').mean('time').sortby(xr.DataArray(['DJF','MAM','JJA', 'SON'],dims=['season']))

Thought this might help some folks who need to solve this problem.

I use reindex instead of sortby

Closing as stale, please reopen with an updated summary if relevant

Hehe, I'm so close to actually fixing this! :)


bug: Joins cannot be nested

1. Checklist

[X] I updated daisyUI (npm i -D daisyui@latest) but I still have this issue
[X] I looked up into FAQ but I couldn't find a solution
[X] I searched in existing issues and discussion forum but this bug has not addressed before
[X] This bug is dasiyUI issue and it's NOT specific to a JS framework or a build tool."

2. What version of daisyUI are you using?
v3.9.2
3. Which browsers are you seeing the problem on?
Chrome
4. Reproduction URL
https://play.tailwindcss.com/p2mVSoZSXy?layout=horizontal
5. Describe your issue
I have the necessity to use various join items which represent an entry, each of these entries should have a currency input (I used the currency-input class in the Tailwind Play snippet to find out them easily) but noticed that every :first-child and :last-child in the subtree will be styled as part of the root join group.
For example, other then currency-inputs I marked acceptance-indicators too which shouldn't be styled as part of the join but I fixed it out using .rounded on it (as shown in the snippet).
But the major issue appears with the currency-inputs which was formerly thought as an input-group and then opted to a join.
Right now I can fix it by opting on input-groups but, as shown in the documentation, they are going to be deprecated in the next major release so the issue is going to be a real a thing in the future.
I tried to solve the issue by using the > (child combinator), but nothing seem to work against the specificity of :first-child and :last-child, here is my code:
.join > :where(.join-item) {
    border-start-end-radius: 0;
    border-end-end-radius: 0;
    border-end-start-radius: 0;
    border-start-start-radius: 0;
}
.join > .join-item:not(:first-child):not(:last-child),
.join > *:not(:first-child):not(:last-child) > .join-item {
    border-start-end-radius: 0;
    border-end-end-radius: 0;
    border-end-start-radius: 0;
    border-start-start-radius: 0;
}
.join > .join-item:first-child:not(:last-child),
.join > *:first-child:not(:last-child) > .join-item {
    border-start-end-radius: 0;
    border-end-end-radius: 0;
}
.join > .dropdown > .join-item:first-child:not(:last-child),
.join > *:first-child:not(:last-child) > .dropdown > .join-item {
    border-start-end-radius: inherit;
    border-end-end-radius: inherit;
}
.join > :where(.join-item:first-child:not(:last-child)),
.join > :where(*:first-child:not(:last-child) > .join-item) {
    border-end-start-radius: inherit;
    border-start-start-radius: inherit;
}
.join > .join-item:last-child:not(:first-child),
.join > *:last-child:not(:first-child) > .join-item {
    border-end-start-radius: 0;
    border-start-start-radius: 0;
}
.join > :where(.join-item:last-child:not(:first-child)),
.join > :where(*:last-child:not(:first-child) > .join-item) {
    border-start-end-radius: inherit;
    border-end-end-radius: inherit;
}
:where(.join > *) {
    border-radius: inherit;
}

What is your expected output with nested join?
Because join makes the first and last item inside it rounded so I'm not sure what would be the reason to nest them? 🤔

The fact is that I wasn't directly nesting them for the need to nest them.
I had outer joins which were my entries and an input group inside of it, but since it is marked to be deprecated in the next version, I used joins for both the components and this happened:
Input group:

Outer join:

I kinda solved the issue on acceptance-indicator and currency-input here.

I have the same issue, but with accordions:

This triggers for the first nested element:

This is because it's a child of some  element that is the last child of element. Adding an empty div in two places per child fixes it for me (added text so you can see where):

The source of the problem is the *:last-child which catches way too much. I understand why it was made like this, but maybe someone can think of a better way?

Same problem while using with accordion which is joined with join-vertical. Then, I have some buttons inside it which are joined with join-horizontal. I fixed it with css.
.join-horizontal :not(:first-child):not(:last-child) {
  @apply !rounded-none;
}


Problem with the dollar symbol $

Bug description
 const findUser: User = await this.users.findFirst({
      where: { firstName: '$foo' },
    });

when i run  the above request i got a problem which is findUser is null , if i change  on the database the firstName too "foo"
i get the wanted result .
How to reproduce

Expected behavior
No response
Prisma information

const findUser: User = await this.users.findFirst({
where: { firstName: '$foo' },
});
Environment & setup

OS:
Database:
Node.js version:
Ubuntu
Mongodb
node v16.13.1

Prisma Version
3.12.0

What is the actual value of the Users.firstName in the database, foo or $foo?

i tried both  , the first one works fine but the second one doesn't work

If the database has an entry where firstName is foo that is very much expected - a where search returns back what you are searching for. That is the point of it.


shadow ioeventfd: add demo

This has already been reviewed at https://github.com/nutanix/libvfio-user/pull/714 but was erroneously merged into the intermediate branch, will merge into master once the base branch has been merged.

This is ready for review now.


Put node shebang in main.ts

This allows use of the CLI under windows under npm -g install. Otherwise trying to run every-ts from cmd just opens the entrypoint JS file in a code editor

Er, that's strange; the package manager should be creating a shim. I just now realized that I did it wrong and intended to have a dedicated bin dir like I do in hereby, so I can fix that quick (which I'd prefer over putting it in the TS).

Thanks for pointing this out! I guess I didn't test installing globally on Windows, just the local install...


fix: Update Irish bank holidays

Fixes #636

Some doubts about "Christmas Day"

@javicalle Thanks, updated the PR with new commits to fix the issue

@javicalle Could you please review it?

How can we get it through the finishing line?

Sorry, been not available for some weeks! On this now, will let you know 👍

Okay some fixes (tests, annotations, formatting) required, but then all done.
Released on beta branch, thank you both! 👍

Sweet, thanks a mil!

Any progress with this, I'm still getting the same error on 2022.5.3.
Thanks.

There hasn't been a release of holidays since 15-Feb-2022, so the merged changes are still only present in the beta branch.
A workaround for Home Assistant (assuming that is what "2022.5.3" is referring to) involves copying the core workday integration into custom_components (see here for more info) and pinning the holidays dependency to 0.12, the last working version, in manifest.json. You'll need to also add a version key (any string will do) to the manifest as it is required by custom components since 2021.6.

Thanks, I thought I was posting in HA Core Github, but obviously not! I'll try reverting holidays back to 0.12 as you suggest.


Add apache 2.0 license

@RackHD/corecommitters @heckj

👍

:+1:

@benbp it seems to needs a rebase to let travisci pass, it looks like the travis ci failure caused by 'curl' version could be resolved by @heckj 's fix
commit 8fdd07945410515c46add5d98cf87a04720b7127
Author: Joe Heck<EMAIL_ADDRESS>Date:   Wed Jul 20 12:09:29 2016 -0700
updating upstream version of curl

 - version 7.35.0-1ubuntu2.6 replaced with 7.35.0-1ubuntu2.7

@anhou done

:+1:


Revise the description of applyTransform for geometries

Fixes #10121
Revise the description of applyTransform and transformFn (loosely based on that used in ol/extent) to avoid any misunderstanding that the function is called inside a forEach method.

Thanks @mike-000.


IntegerNegation and RealNegation

Includes #75

Done, It's printing the exact size with the infix notation now.


How to train on custom dataset and how to train jpeg restoration model?

How to train on custom dataset and how to train jpeg restoration model?

You can follow the https://github.com/Janspiry/Palette-Image-to-Image-Diffusion-Models/blob/136b29f58d0af6e5db9f3655d2891f5a855fcdaa/data/dataset.py#L35 to rewrite your dataset and import it in the configure file.


Add indeterminate checkbox example to index.html

Closing, because we need more than an example. It's needs to work, first!


Attempt to fix missing files from the distributed package

This is an attempt to resolve both
ModuleNotFoundError: No module named 'khbr.KH2.KingdomHearts2'

when just running the local seed generator in a dev build, and
Unable to find "c:\users\[redacted]\source\repos\kh2randomizer\venv\lib\site-packages\khbr\KH2\data\ai_mods\*" when adding binary and data files.

when trying to build the seed generator .exe via pyinstaller.
Both these errors seem to center around the published khbr dependency missing files. Workarounds have been suggested such as just installing khbr from source, but this ideally needs to get fixed in this project to allow developers of the seed generator to just use the published dependency rather than needing to use the source.
I've done my best to verify that this does resolve both those errors, but I'm very much out of my element when it comes to Python library packaging and it's possible this is not the correct approach.

Sorry for the delay in responding, but this looks pretty good. I had forgotten to change the package data path during my major refactor. The second string referencing 'ai_mods/**' is not necessary, because ** will recursively grab any files in subdirectories. I will remove it after the merge when I bump the version.
Thanks.


⚠ Bofrost.es has degraded performance

In 82404d7, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1374 ms

Resolved: Bofrost.es performance has improved in 412e361 after 6 minutes.


Improved external components

Refactor bootable components and improve boot feature.
This is a substantial improvement, I managed to unify bootable/external components, so now we only have Components::Bootable and it doesn't matter if it's exernal or internal. Here's an overview of what this PR allows:
New boot syntax
We now have Container.boot and finalize was deprecated. It supports additional namespace option which is cool, because extra nesting is no longer needed:
# default behavior is registration w/o namespace
MyApp::Container.boot(:db) do
  init do
    require "some_gem/db"
  end

  start do
    register(:conn, SomeGem::Db.new)
  end
end

MyApp::Container["conn"]

# namespacing is simpler now
MyApp::Container.boot(:db, namespace: true) do
  init do
    require "some_gem/db"
  end

  start do
    register(:conn, SomeGem::Db.new)
  end
end

# you can provide a custom namespace too
MyApp::Container.boot(:db, namespace: "backend.persistence") do
  init do
    require "some_gem/db"
  end

  start do
    register(:conn, SomeGem::Db.new)
  end
end

MyApp::Container["backend.persistence.conn"]

External components are just bootable components
This means you register them using standard Container.boot API and provide :from option which specifies the component provider:
MyApp::Container.boot(:persistence, namespace: true, from: :some_provider) do
  after(:start) do |c|
    register(:conn, c[:database_connection])
  end
end

MyApp::Container['persistence.conn']

Configuration API is accessible via boot
Because there's no distinction between external/internal bootable components, the new configure API is now always available:
# if the provider sets up the component like this:
Dry::System.register_component(:persistence, provider: :some_provider) do
  start do
    register(:database_connection, SomeProvider::DB.new(config.database_url)
  end
end

# then this configuration in your app can be set like this
MyApp::Container.boot(:persistence, namespace: true, from: :some_provider) do
  configure do |config|
     config.database_url = "sqlite::memory"
  end

  # NOTICE that hook blocks receive external component instance where
  # objects are provided, so that you can register them under your identifiers
  after(:start) do |c|
    register(:conn, c[:database_connection])
  end
end

# and this works too using internal components
MyApp::Container.boot(:persistence, namespace: true) do
  configure do |config|
     config.database_url = "sqlite::memory"
  end

  start do
    register(:conn, MyDB.new(config.database_url)
  end
end

Bootable components are now accessible, standalone objects
This won't be super popular, but it can be helpful. Containers register their bootable components under Container.components.
Here's a sample pry session in an actual app, where we only require container file and then manually mess with a component:
[1] pry(main)> require App::Container.booter.path.join("persistence")
=> true
[2] pry(main)> p = App::Container.components[:persistence]
=> #<Dry::System::Components::Bootable:0x00007fdd95228dd0
 @identifier=:persistence,
 @namespace=true,
 @options={:container=>App::Container, :namespace=>true, :block=>#<Proc:0x00007fdd95228e98@/Users/solnic/Workspace/test-app/system/boot/persistence.rb:1>},
 @triggers={:before=>{}, :after=>{}}>
[3] pry(main)> p.init
=> #<Dry::System::Components::Bootable:0x00007fdd95228dd0
 @identifier=:persistence,
 @lifecycle=Kernel,
 @namespace=true,
 @options={:container=>App::Container, :namespace=>true, :block=>#<Proc:0x00007fdd95228e98@/Users/solnic/Workspace/test-app/system/boot/persistence.rb:1>},
 @triggers={:before=>{}, :after=>{:init=>[]}}>
[4] pry(main)> p.statuses
=> [:init]
[5] pry(main)> p.start
=> #<Dry::System::Components::Bootable:0x00007fdd95228dd0
 @identifier=:persistence,
 @lifecycle=Kernel,
 @namespace=true,
 @options={:container=>App::Container, :namespace=>true, :block=>#<Proc:0x00007fdd95228e98@/Users/solnic/Workspace/test-app/system/boot/persistence.rb:1>},
 @triggers={:before=>{}, :after=>{:init=>[], :start=>[]}}>
[6] pry(main)> p.statuses
=> [:init, :start]

We can add more convenient methods later on, what's important is that we now have nice access to these objects.
In-line booting is now supported
It is now possible to define bootable components w/o having special files. This means that you can just do this in your container class:
class MyApp::Container < Dry::System::Container
  # a local one (trivial example I know, just showing that it's possible :P)
  boot(:persistence, namespace: true) do
    start do
      register(:conn, Db.new)
    end
  end

  # an external local one
  boot(:persistence, namespace: true, from: :some_provider) do
    configure do |config|
       config.database_url = "sqlite::memory"
    end

    after(:start) do |c|
      register(:conn, c[:database_connection])
    end
  end
end

It will work just like with boot files, this is really cool as it could be used to set up lightweight applications w/o complex dir structures and many files, yet, it would benefit from isolated requires/booting process.
Lifecycle triggers / hooks and execution context
There are a couple things to know:

init, start and stop triggers are executed in the context of a lifecycle object. It has access to target container (as in, your application container), and it registers objects inside its temporary container, then we copy whatever was registered to the target container.
after and before hooks are executed in the context of your container so that you can easily call register which makes the syntax less verbose and you have access to common methods like root etc. In addition, these blocks receive the container that was used by lifecycle object, so that you can manually copy registered objects and register them under your keys.
boot block receives your container in case you want to refer to it, but it shouldn't really be needed so try to avoid it

Just wanted to mention that when I worked on this, I realized eventually we should end up with just bootable components, without special-casing "external" ones. After all an application is a component provider too, so there's no real difference between an app, and some external provider. Both have boot files under specified boot path, both register components to be finalized within a given container. For now I kept it separate as it'd be too much to change at once, but I'll definitely look into unifying this before 1.0.0.

This is merged. I'll add docs and maybe clean up some things a little bit in master. FYI this was tested in our apps at Icelab, so I'll release 0.8.0 as we'd like to start using it right away.


nixos/xserver: add option enableTearFree

Description of changes
Default modesetting driver now has the TearFree feature merged.
This PR enables it easily and makes newcomers aware of it.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Can you merge this @eclairevoyant ?


Passenger on High doesn't seem to work

title,
Got some reports that players with passenger on high prio wont give them the passenger spawn

this isn't a bug, head roles have higher prio. if they want passenger guaranteed they should have all other roles set to none,  everything set to none and the become passenger option picked, or latejoin


Fix VAE encode dtypes when running on CPU

What does this PR do?

Missed this during the diffusers 0.12 update.
Before submitting

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

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


Has (appears to) read data from disk

I've been experimenting a bit with the db.Has, adding a benchmark:
diff --git a/leveldb/bench_test.go b/leveldb/bench_test.go
index 435250c..33b2106 100644
--- a/leveldb/bench_test.go
+++ b/leveldb/bench_test.go
@@ -225,6 +225,20 @@ func (p *dbBench) gets() {
 	b.StopTimer()
 }

+func (p *dbBench) hass() {
+	b := p.b
+	db := p.db
+
+	b.ResetTimer()
+	for i := range p.keys {
+		_, err := db.Has(p.keys[i], p.ro)
+		if err != nil {
+			b.Error("got error: ", err)
+		}
+	}
+	b.StopTimer()
+}
+
 func (p *dbBench) seeks() {
 	b := p.b

@@ -252,6 +266,9 @@ func (p *dbBench) close() {
 	if bp, err := p.db.GetProperty("leveldb.blockpool"); err == nil {
 		p.b.Log("Block pool stats: ", bp)
 	}
+	if bp, err := p.db.GetProperty("leveldb.iostats"); err == nil {
+		p.b.Log("Block pool stats: ", bp)
+	}
 	p.db.Close()
 	p.stor.Close()
 	os.RemoveAll(benchDB)
@@ -453,6 +470,14 @@ func BenchmarkDBGet(b *testing.B) {
 	p.close()
 }

+func BenchmarkDBHas(b *testing.B) {
+	p := openDBBench(b, false)
+	p.populate(b.N)
+	p.fill()
+	p.hass()
+	p.close()
+}
+
 func BenchmarkDBGetRandom(b *testing.B) {
 	p := openDBBench(b, false)
 	p.populate(b.N)

Now, it appears that Has has the exact same disk IO footprint as Get:
pkg: github.com/syndtr/goleveldb/leveldb
BenchmarkDBHas-6   	 1000000	      4418 ns/op
--- BENCH: BenchmarkDBHas-6
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[0 0 0 0 0] Zm·[0 0 0 0 0] Zh·[0 0 0 0 0] G·0 P·0 H·0 <·0 =·0 >·0 M·0}
    bench_test.go:302: Block pool stats:  Read(MB):0.00000 Write(MB):0.00018
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[0 0 0 0 0] Zm·[0 0 0 0 0] Zh·[0 0 0 0 0] G·0 P·0 H·0 <·0 =·0 >·0 M·0}
    bench_test.go:302: Block pool stats:  Read(MB):0.00000 Write(MB):0.01142
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[0 0 0 0 0] Zm·[0 0 0 0 0] Zh·[0 0 0 0 0] G·0 P·0 H·0 <·0 =·0 >·0 M·0}
    bench_test.go:302: Block pool stats:  Read(MB):0.00000 Write(MB):1.13520
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[397 1144 4208 15971 30004] Zm·[17 4 0 15 0] Zh·[15 0 0 0 0] G·55176 P·53148 H·35 <·52765 =·93 >·34 M·2249}
    bench_test.go:302: Block pool stats:  Read(MB):60.57908 Write(MB):174.09719
PASS

Compare with BenchDBGet
BenchmarkDBGet-6   	 1000000	      4386 ns/op
--- BENCH: BenchmarkDBGet-6
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[0 0 0 0 0] Zm·[0 0 0 0 0] Zh·[0 0 0 0 0] G·0 P·0 H·0 <·0 =·0 >·0 M·0}
    bench_test.go:302: Block pool stats:  Read(MB):0.00000 Write(MB):0.00018
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[0 0 0 0 0] Zm·[0 0 0 0 0] Zh·[0 0 0 0 0] G·0 P·0 H·0 <·0 =·0 >·0 M·0}
    bench_test.go:302: Block pool stats:  Read(MB):0.00000 Write(MB):0.01142
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[0 0 0 0 0] Zm·[0 0 0 0 0] Zh·[0 0 0 0 0] G·0 P·0 H·0 <·0 =·0 >·0 M·0}
    bench_test.go:302: Block pool stats:  Read(MB):0.00000 Write(MB):1.13520
    bench_test.go:299: Block pool stats:  BufferPool{B·4101 Z·[795 1144 4208 15971 30004] Zm·[2 7 0 8 0] Zh·[19 0 0 0 0] G·55176 P·53148 H·39 <·52766 =·87 >·36 M·2248}
    bench_test.go:302: Block pool stats:  Read(MB):60.57908 Write(MB):174.09719
PASS

They both read the exact same amount. It appears that there are two different codepaths taken, depending of whether it should search for the key only and not the actual data, however, it seems that it does actually fetch the data from disk, even though it later discards it.
So I'm reporting it here, because I'm not sure if it's a known issue that just is not implemented.

Yes, that's to be expected. Has is only a convenience method with little optimization,  which is don't copy the value, other than that it's exactly the same as Get. It just not possible to (and not designed to) just pull the key from the disk without also pulling the value.


Error when using CUDA in Cycles

When I trying to use Cycles with CUDA (gpu+processor) active - it shows this error

and also don't render any lights(in any other versions of blender it works(on win 10-11, or those that earlier, that could work on win 7 too) (works only GPU compute without CUDA
When i tried to build it - error appears on trying to patch files apply bcompat7.patch

also even if i will not try to patch it, but just update and use make/make.bat i'll have this error

It could be great if any of those errors can be solved, because, with CUDA it really works faster(even if it with processor)

The CUDA error is because the binary i uploaded was compiled with CUDA 11.3 which doesn't have support for targetting cards with compute capability 3.0, so the kernel wasn't built. I will build it with CUDA 10 also.
I was unable to reproduce the patch error. Did you make any modifications to the files before applying the patch ?

no, any

I wasn't able to make a full build with CUDA 10 because nvcc crashes with an internal error on the sm7.5 kernel, however i managed to build the kernel for GPUs with compute capability 3.0. I've uploaded the binaries here: https://1drv.ms/u/s!AhpnXywMA4U1mTF-F-k19I17fE51.

Thanks for fix and for this project at all. :)


improvement(build-cli): Remove node-fetch dependency

node-fetch is not actually used in build-cli because Node 18+ supports fetch natively, and we weren't importing from node-fetch anyway.
I can't figure out how to get this to compile without "lib": ["DOM"] in the tsconfig, which seems wrong.

Hmm it compiles for me without touching the tsconfig... and the build for 061fcf4 before you tweaked the tsconfig is also green. What error are you getting?

Hmmm, I no longer get an error locally. It's possible my dev env was in a weird state since I've been doing a lot of dep updates. If the latest build passes, we can merge I think.

Hmmm, I no longer get an error locally.

Might also have been addressed by the latest node types.


Archetypes not available in Eclipse

Hello,
I wanted to start using the mvn archetypes for enRoute but I can't get Eclipse to find the archetypes.
It seems they are not visible in the archetypes catalogs.
I discovered the following link:
https://oss.sonatype.org/content/groups/osgi/archetype-catalog.xml
that seems to contain the archetypes. I can browse and download the file and it looks OK but when I point Eclipse to the catalog it says the catalog is empty. When I point Eclipse to the locally downloaded catalog file it does indeed display the archetypes...
The documentation / examples pages on enRoute seem really nice, but there is no indication on how to get Eclipse find the archetypes.
(from the command shell it seems to work fine ... I've also pointed my Eclipse to my local maven installation which is the latest available: Apache Maven 3.5.4).
What am I missing?
Regards,
Alex

Hi Alex,
Documentation issues really should be raised against the https://github.com/osgi/osgi.enroute.site repository, as that's where the website is maintained. I've created a PR osgi/osgi.enroute.site#191 which would add a section on Eclipse Archetype setup. It would be good if you could verify that this works for you.
In addition I would like to warn you that Eclipse M2E has quite a few bugs in its archetype generation (failure to correctly rename files, failure to match package folder structures to package names) so please be careful when using Eclipse to run the archetypes.

Hi Tim,
sorry for that, I thought I was doing something wrong with the archetypes.
I remember you insisting on the m2e issues during your sessions at the EclipseCon. Since I’m amongst the people that don’t like maven, that’s not going to help 😉 but I want to give it try since I believe you did an amazing job on the new enRoute for r7.
Thank you very much

Mat frëndleche Gréiss,
Mit freundlichen Grüßen,
Meilleures salutations,
Kind regards, Alex Weirig Responsable Technique
Ville de Luxembourg
Service Enseignement
Centre Technolink Tel +352 4796 - 6127 Fax +352 42 88 81 Email<EMAIL_ADDRESS>www.vdl.lu // www.technolink.lu Centre Technolink
2, rue Charles de Tornaco
L-2623 LUXEMBOURG
From: "Tim Ward"<EMAIL_ADDRESS>To: "osgi/osgi.enroute"<EMAIL_ADDRESS>Cc: "Alex Weirig"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Sent: Monday, October 29, 2018 10:38:58 AM
Subject: Re: [osgi/osgi.enroute] Archetypes not available in Eclipse (#85)
Hi Alex,
Documentation issues really should be raised against the https://github.com/osgi/osgi.enroute.site repository, as that's where the website is maintained. I've created a PR osgi/osgi.enroute.site#191 which would add a section on Eclipse Archetype setup. It would be good if you could verify that this works for you.
In addition I would like to warn you that Eclipse M2E has quite a few bugs in its archetype generation (failure to correctly rename files, failure to match package folder structures to package names) so please be careful when using Eclipse to run the archetypes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub , or mute the thread .
This email was Anti Virus checked by SOPHOS UTM

So we can close this issue then?

Hi BJ,
sure, you can close the ticket. Tim opened the ticket for the website and it doesn’t seem to be a problem with the archetypes, although I don’t understand why I can’t add the archetype file using the server URL while I can add it when I download it, but I guess once the documentation is updated I’ll figure that out.
Thanks

Mat frëndleche Gréiss,
Mit freundlichen Grüßen,
Meilleures salutations,
Kind regards, Alex Weirig Responsable Technique
Ville de Luxembourg
Service Enseignement
Centre Technolink Tel +352 4796 - 6127 Fax +352 42 88 81 Email<EMAIL_ADDRESS>www.vdl.lu // www.technolink.lu Centre Technolink
2, rue Charles de Tornaco
L-2623 LUXEMBOURG
From: "BJ Hargrave"<EMAIL_ADDRESS>To: "osgi/osgi.enroute"<EMAIL_ADDRESS>Cc: "Alex Weirig"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Sent: Monday, October 29, 2018 08:54:44 PM
Subject: Re: [osgi/osgi.enroute] Archetypes not available in Eclipse (#85)
So we can close this issue then?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub , or mute the thread .
This email was Anti Virus checked by SOPHOS UTM


General Availability Date for PU31

Hi, in this page, it is mentioned this will be generally available on January 2020, however the Service Update Availability page is showing Generally Available date for PU31 is Nov 29, 2019.
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/fin-ops/get-started/public-preview-releases
Can I know which one is true? And which page should be the single source of truth should this clash of information is happening again in the future?
Thank you.

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

ID: 0723686d-7b30-9831-8978-10077d700724
Version Independent ID: cbfeb85f-9ae0-8d93-bd8a-5a6bdd2fa41b
Content: Preview features in Platform update 31 for Finance and Operations apps (January 2020) - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-ops-core/dev-itpro/get-started/whats-new-platform-update-31.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @tonyafehr
Microsoft Alias: tfehr

Closing, as this is a duplicate of this issue: https://github.com/MicrosoftDocs/dynamics-365-unified-operations-public/issues/1831


Create InfluxDB database only if not exist

.. to avoid superficial warning.

Can one of the admins verify this patch?

ok to test

LGTM


PickPhotoAsync problem on Android 5.1.1

This is a

[x] Bug
[ ] Feature Request

Which plugin does this impact:

[ ] Battery
[ ] Connectivity
[ ] Contacts
[ ] DeviceInfo
[ ] ExternalMaps
[ ] Geolocator
[x] Media
[ ] Permissions
[ ] Settings
[ ] Text To Speech
[ ] Vibrate
Other:

Version Number of Plugin: 2.3.0, 2.4.0-beta2
Device Tested On: Android 5.1.1 (Nexus 4)
Simulator Tested On:
Expected Behavior
It should show Gallery image picker at least.
However it does nothing, because of the exception caught in MediaPickerActivity.cs file:

No Activity found to handle Intent act=android.intent.action.PICK typ=image

Looks like this issue is reproduced since Android Lollipop.
Actual Behavior
await CrossMedia.Current.PickPhotoAsync() returns null
How to fix
Please, replace Intent.ActionPick with Intent.ActionGetContent all over the code of the plugin (MediaImplementation.cs and MediaPickerActivity.cs files).
ACTION_GET_CONTENT provides more useful user interface.

I had a similar issue with TakePhotoAsync. Turns out the permissions wheren't set / asked for (in my case).
Using:
var status = await CrossPermissions.Current.CheckPermissionStatusAsync(Permission.Camera);
if (status != PermissionStatus.Granted)
{
	await CrossPermissions.Current.ShouldShowRequestPermissionRationaleAsync(Permission.Camera);

	var results = await CrossPermissions.Current.RequestPermissionsAsync(new[] {Permission.Camera});
	status = results[Permission.Camera];
}

if (status == PermissionStatus.Granted)
{
... use take photo here
}

Solved it on my part.

This issue was moved to jamesmontemagno/MediaPlugin#21


use http for submodules

I'm getting access denied errors when trying to install nim_zstd.
When i manually change the submodule url to http then it installs.

try https://github.com/bung87/nim_zstd if you have network conditions

I just commited the zstd source. Published v0.5.0


make the Schwartz @safe

At least make the schwartzSort() safe.
Partial fix for https://issues.dlang.org/show_bug.cgi?id=13054

LGTM

LGTM.

Auto-merge toggled on


Fixed #35941 -- Added composite GenericForeignKey support.

Trac ticket number
ticket-35941
Branch description
Add GenericForeignKey support to composite primary keys. Please check the ticket for a more detailed description.
Checklist

[X] This PR targets the main branch.
[X] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[x] I have added or updated relevant docs, including release notes if applicable.
[x] I have attached screenshots in both light and dark modes for any UI changes.

So I am thinking, we shouldn't make the "object_id" format dependent on the connection. So if we serialize a UUID into JSON it should always have hyphens, for example. It would be simpler this way, but we'll need to remove the hyphens when doing the JOINs. What do you think?

It looks like there's a failing test because MariaDB somehow isn't handling the Unicode characters. I'm looking into it.


rear cam filename fix

The rear camera files generated for my M3 on 2<IP_ADDRESS> end in -back.mp4

Yep, can confirm.

Also can confirm

Same.

Maybe not just replace the filename but check which one of the two is present? Better for backwards compatibility.


Test suite depends on specific libc version

The backlight test suite implicitly depends on implementation details of the libc version available in its environment. This needs to be corrected so the tests can pass when run in other environments.
But I want to think carefully about how to do this while also allowing the tests to give high confidence that backlight is functional while tracing libc library calls.
I'm certainly open to feedback on this. This feels like a problem that other tools of this nature would have had to solve before, but I'm not up to speed on the best practices here.

I've started working on this on a branch improve-test-suite-cross-compatiblity.


Multi selected files deletes wrong entity

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.57.1
OS Version: Win 10.0.19041

Steps to Reproduce:

Select a section of files/folders (say 5 listed adjacently in the explorer)
Ctrl Click to deselect a single file/folder within that selection
press delete
notice that only the file/folder you deselected was deleted

Possibly related to bug: 125464 [https://github.com/microsoft/vscode/issues/125464]

Duplicate detection noticed but report I had missed, good bot.


removes all non alphanumeric characters instead of just spaces

I found some problems when using this with other libraries since they used the “@“ character to represent linking between storyboards. This should fix the compile errors caused by this.

add swift20 branch


Stuck on "Starting continue server" on Windows 11

The extension is stuck on "Starting continue server" since around 10 minutes. I have python 3.10 as my python version.
Also tried launching vscode as Admin, didn't help

I'm in a similar situation. It eventually stopped showing that message, but the docs say that the extension should open on its own and bring up a Python file. That didn't happen, so I tried just working with one of my own files. None of the commands return anything. The README says to open dev tools and check the logs to see if the server is running, so I did that. There's nothing in the logs, so I assume it's not running, but there are no instructions on how to start it. I'm on Python 3.8.4 / MacOS.

Nevermind. I was on a VPN and I forgot that OpenAI blocks my VPN.
Getting a different error now, though. I might open a different issue for that.

@All-Ki are there any additional details you could provide about the situation? If anything it would greatly help to know whether you are on MacOS/Linux or Windows. Working on this and will get back to you with a fix

I can't start the server, it says "No module named 'continuedev'"

having issues as well. extension needs a lot of work still but looks promising

I'm on Ubuntu 22.04, python 3.10.6 and the extension stays at 'Loading Continue server...'. The server has started, but it responds with 404; (i also entered my API key)

Maybe it helps... can't wait to use it :)

I can't start the server, it says "No module named 'continuedev'"

i have the exact same problem.. any fixes?

Pretty much the same case, Ubuntu 22.04, Python 3.10.6: "Loading Continue server..." with the arrow slowly blinking but nothing seems to further happen.

@sestinj
Sorry, i'm on windows.
Any additionnal info you'd like ? And i don't know how to bring up the logs with VSCode, but if you tell me i can send those to you

Added a relevant ticket with slightly different error details here: https://github.com/continuedev/continue/issues/311

@TyDunn it seems like theres a lot of dependency issues. I dont know much about vs code extensions but maybe it'd help to package it as a binary using pyinstaller instead of installing packages locally?

Stuck on loading Continue Server

After (maybe) resolving my other issues, I'm stuck here again. I think some feedback in the UI would be helpful. I think the python packages are done downloading and it's hanging for some other reason, but there's nothing in the UI to tell you what's going on. What's weird is that I checked ~/.continue/continue.log and I see this:
INFO:     Started server process [91858]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:65432 (Press CTRL+C to quit)
INFO:     ('<IP_ADDRESS>', 54842) - "WebSocket /ide/ws?session_id=a7003805-15de-4cf6-a6ba-edb4690cc7a6" [accepted]
INFO:     connection open

But the "Starting server" message is up and no commands work...

For everyone who was experiencing this issue, there have been numerous updates that probably solve the problem, as well as improvements to logging which will quickly offer more information if something goes wrong. I'm going to wait to hear back from you all on whether it is fixed, but would like to close this issue and move to more specific issues once possible (we've already done this in a few cases). Thanks!

@dcstrange Thanks for sending over the console logs. This problem is one we know how to solve, and you can read a bit more here. I've also pushed changes that will address that TypeError you are seeing.

And for everyone in this thread still experiencing the issue, I've created much better troubleshooting documentation here, but MORE IMPORTANTLY and working on a permanent solution: we will soon ship with a packaged executable instead of trying to install python packages on your machine. You can track progress here, and I'll comment here as well once it is complete.

Closing this issue. We have since

Packaged the extension into a reliable binary with pyinstaller
Endured a week+ of testing without hearing these exact problems
Included testing as a part of CI to ensure that the server can start

Anyone can feel free to reopen if the issue is persisting and we'll address it right away


When selecting, ensure the last token is the option encoded

This prevents the case when we would have a token that represents a text that only ends with the option (ex: both " True" and "True" are in vocab).
Fixes https://github.com/microsoft/guidance/issues/67
This feels like the right solution, but I'm only trying to learn the codebase. The codebase seem to assume that options can only be 1 token long (for example, the call to the LLM has max_tokens=1) - any plans to support more tokens?

Thanks! Will take a look. The code does not assume a single token, but it does make single token generation calls to branch the decision tree that is implied by a select statement.

Closing as the issue can't be replicated in the latest release.


Autoconfigured data source DSN not compatible to MariaDB Connector/J v3

Is there an existing issue for this?

[X] I have searched the existing issues

Are you sure that this bug is related to this DDEV Integration Plugin?

[X] I am sure

Enter your error report ID (If available)
No response
Describe the bug
The autoconfigured data source can not establish a connection.
Error message: Incorrect driver and/or database URL specified.
Related documentation: jdbc:mysql scheme compatibility
Steps to reproduce
No response
Additional context
Data Source URL: jdbc:mysql://<IP_ADDRESS>:49156/db?user=db&password=db
MariaDB Connector/J: v3.0.7
PhpStorm: 2022.2.2 RC
DDEV: v1.21.1

Hey @dahaupt,
thank you for reporting this issue!
I've fixed this in https://github.com/php-perfect/ddev-intellij-plugin/commit/0534375af67771ccb7341f94b3f9f6db572dea79.
The fix will be included in the next release of the plugin.


Integrate dApp with eth-contract-metadata

Description
https://github.com/MetaMask/eth-contract-metadata/ contains a list of contract metadata (including addresses, decimal, name, symbol and logo) that includes a lot of popular ERC-20 tokens
Acceptance criteria

dApp shows the official logo for a token when right contract is detected.
dApp first checks the token metadata from the library and then tries to read the contract if it is not found.

Tasks

[ ]

This would be a great feature

There is a new option: https://uniswap.org/blog/token-lists/


MBL-807: ActivityFeed migrated to RXJava2

📲 What

Migration RXJava2 + ViewModel, take a look at the engineering plan if any doubt
android.defaults.buildfeatures.buildconfig (deprecated) moved  to gradle.build

|  |  |
📋 QA

Poke around the ActivityFeed

Story 📖
MBL-799

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (ac0203c) 74.09% compared to head (e1ee1cb) 74.06%.

Files
Patch %
Lines

...s/rx/transformers/IncrementalCountTransformerV2.kt
57.14%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1929      +/-   ##
============================================
- Coverage     74.09%   74.06%   -0.03%
  Complexity     1967     1967
============================================
  Files           336      337       +1
  Lines         19634    19642       +8
  Branches       2716     2716
============================================
+ Hits          14547    14548       +1
- Misses         3499     3506       +7
  Partials       1588     1588

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


Center viewport around playhead when zoomed in/out

When zooming in or out, the viewport should zoom vertically centered around the playhead.

It kinda works but is perfectible.


Fix: Enhanced logic for identifying burned native tokens.

Summary

Refactor getBurnedNativeTokens logic for identifying burned native tokens.
Update logic to determine remainder in convertTransactionOutputTypeToWrappedOutput
Update logic getNonRemainderBasicOutputsFromTransaction to use remainder property

resolves #7107
Testing
Platforms

Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[x] Linux
[ ] Windows

Checklist

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

Very nice work 💪🏼 Just a minor detail, I sent a native token to an NFT and it showed as burned 🙏🏼 https://explorer.shimmer.network/testnet/transaction/0xd348041e5134b5436bc7fe7df69da8e027c60eb5bce9d4d12775a47c698c6fce

Created a task for this issue https://github.com/iotaledger/firefly/issues/7617


Fix failure escalation detection

This is hacky.  I know it's hacky.  It works and we're short on time and it's a small change.   We need to do a redesign of recovery.  This approach is at the end of its rope.

@nickbp: We had reasonable unit test coverage.  Things went bad in the real world.  @NimaVaziri is adding a missing integration test.


⚠ Fortnite has degraded performance

In bf225bf, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1561 ms

Resolved: Fortnite performance has improved in 89a487c.


[SPARK-20148] [SQL] Extend the file commit API to allow subscribing to task commit messages

What changes were proposed in this pull request?
The internal FileCommitProtocol interface returns all task commit messages in bulk to the implementation when a job finishes. However, it is sometimes useful to access those messages before the job completes, so that the driver gets incremental progress updates before the job finishes.
This adds an onTaskCommit listener to the internal api.
How was this patch tested?
Unit tests.
cc @rxin

Test build #75374 has started for PR 17475 at commit a541fdd.

LGTM

Test build #75374 has finished for PR 17475 at commit a541fdd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75374/
Test PASSed.

Merging in master.


node panic: 'too many open files'

Describe the bug
chrysalis validator node crashed (too many open files panic)
To Reproduce
Steps to reproduce the behavior:

run chrysalis 1.0.1 node with default 1024 ulimit -n
PANIC   Tangle  unable to confirm milestone due to receipt backup failure: unable to persist receipt onto disk: open receipts/3724371.114361.json: too many open files

Expected behavior
no panic
Environment information:

OS:  Ubuntu 20.04]
RAM:  16 GB
Cores:  8 vCores
Type:  ESXi VM
HORNET version:  1.0.1

Additional context
while this is a validator node, and the error msg mentions a receipts backup failure, the open files error seems to be the cause of the crash (not receipt validation)
attaching console log at the time of the crash
too-many-files-panic.txt

@luca-yvr
https://muhammadtriwibowo.medium.com/set-permanently-ulimit-n-open-files-in-ubuntu-4d61064429a


Implement Api fns for arbitrary subresources and approval subresource for Api

Motivation
This PR tackles two tasks in #127.

Solution
Implement Api functions for arbitrary subresource and Api approve/deny.
One test case of Api  is added.
Should I try to come up with test cases for those Api subresource-related functions?
IMO,  testing the logic of Request.*_subresource should be sufficient.
Any suggestion is appreciated!

Codecov Report

Merging #773 (e17aff8) into master (6dfe170) will increase coverage by 0.24%.
The diff coverage is 60.78%.

:exclamation: Current head e17aff8 differs from pull request most recent head 61421f9. Consider uploading reports for the commit 61421f9 to get more accurate results

@@            Coverage Diff             @@
##           master     #773      +/-   ##
==========================================
+ Coverage   71.64%   71.89%   +0.24%
==========================================
  Files          54       54
  Lines        3562     3682     +120
==========================================
+ Hits         2552     2647      +95
- Misses       1010     1035      +25

Impacted Files
Coverage Δ

kube-client/src/api/subresource.rs
52.29% <0.00%> (-11.04%)
:arrow_down:

kube-client/src/api/util.rs
89.36% <94.44%> (+3.15%)
:arrow_up:

kube-client/src/lib.rs
93.52% <100.00%> (+0.72%)
:arrow_up:

kube-client/src/discovery/mod.rs
89.18% <0.00%> (ø)

kube-client/src/discovery/apigroup.rs
80.55% <0.00%> (ø)

kube-client/src/discovery/version.rs

kube-core/src/version.rs
94.40% <0.00%> (ø)

kube-client/src/client/mod.rs
70.10% <0.00%> (+1.03%)
: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 4017a3f...61421f9. Read the comment docs.

This looks great now. Thanks for doing all the minor tweaks here to get it into a great state! I'll merge this in tomorrow if you are happy with it :-)

I think this is ready for merge. Thanks for helping!